On Fri, Jun 1, 2018 at 4:28 PM, Vishal Verma <vishal.l.verma(a)intel.com> wrote:
In ndctl_cmd_ars_in_progress, which expects a successfully completed
ars_status command, we used to invalidate the command by setting its
status to '1', so that the user has to provide a fresh ars_status
command the next time this check is performed.
In hindsight, this is needless and violates the principle of least
surprise. We shouldn't be touching the user's data (command), so remove
this invalidation. If the user uses the same ars_status command again,
we will simply report that ARS is still in progress.
Signed-off-by: Vishal Verma <vishal.l.verma(a)intel.com>
ndctl/lib/ars.c | 11 +----------
1 file changed, 1 insertion(+), 10 deletions(-)
Reviewed-by: Dan Williams <dan.j.williams(a)intel.com>