mbox series

[v2,0/6] nfit/ars: Improve polling and short-ARS execution

Message ID 155025982726.1043225.15436410616433285475.stgit@dwillia2-desk3.amr.corp.intel.com (mailing list archive)
Headers show
Series nfit/ars: Improve polling and short-ARS execution | expand

Message

Dan Williams Feb. 15, 2019, 7:43 p.m. UTC
Changes since v1: [1]
* Fix the root poll interval support to avoid a infinite loop condition
  when the polling is faster than the ARS completion.
* Move the introduction of scrub_flags earlier in the series and
  introduce ARS_POLL to fix the above issue.

[1]: https://lists.01.org/pipermail/linux-nvdimm/2019-February/019964.html

---

Here is a small pile of updates to better coordinate the Linux ARS state
machine with platform-BIOS implementations. Specifically, take advantage
of opportunities to run short-ARS whenever the ARS interface is found to
be idle at init, always run short-ARS even if no_init_ars is specified,
allow root to reset the exponential backoff polling interval for ARS
completion, and protect the kernel against the consumption of stale ARS
results.

---

Dan Williams (6):
      nfit/ars: Attempt a short-ARS whenever the ARS state is idle at boot
      nfit/ars: Attempt short-ARS even in the no_init_ars case
      nfit/ars: Remove ars_start_flags
      nfit/ars: Introduce scrub_flags
      nfit/ars: Allow root to busy-poll the ARS state machine
      nfit/ars: Avoid stale ARS results


 drivers/acpi/nfit/core.c |   70 ++++++++++++++++++++++++++++++++--------------
 drivers/acpi/nfit/nfit.h |   11 +++++--
 2 files changed, 57 insertions(+), 24 deletions(-)

Comments

Kani, Toshi Feb. 20, 2019, 12:06 a.m. UTC | #1
On Fri, 2019-02-15 at 11:43 -0800, Dan Williams wrote:
> Changes since v1: [1]
> * Fix the root poll interval support to avoid a infinite loop condition
>   when the polling is faster than the ARS completion.
> * Move the introduction of scrub_flags earlier in the series and
>   introduce ARS_POLL to fix the above issue.
> 
> [1]: https://lists.01.org/pipermail/linux-nvdimm/2019-February/019964.html
> 
> ---
> 
> Here is a small pile of updates to better coordinate the Linux ARS state
> machine with platform-BIOS implementations. Specifically, take advantage
> of opportunities to run short-ARS whenever the ARS interface is found to
> be idle at init, always run short-ARS even if no_init_ars is specified,
> allow root to reset the exponential backoff polling interval for ARS
> completion, and protect the kernel against the consumption of stale ARS
> results.
> 
> ---
> 
> Dan Williams (6):
>       nfit/ars: Attempt a short-ARS whenever the ARS state is idle at boot
>       nfit/ars: Attempt short-ARS even in the no_init_ars case
>       nfit/ars: Remove ars_start_flags
>       nfit/ars: Introduce scrub_flags
>       nfit/ars: Allow root to busy-poll the ARS state machine
>       nfit/ars: Avoid stale ARS results
> 
> 
>  drivers/acpi/nfit/core.c |   70 ++++++++++++++++++++++++++++++++--------------
>  drivers/acpi/nfit/nfit.h |   11 +++++--
>  2 files changed, 57 insertions(+), 24 deletions(-)

The changes look good to me.  For the series:

Reviewed-by: Toshi Kani <toshi.kani@hpe.com>

Thanks,
-Toshi