On Fri, Jun 1, 2018 at 4:28 PM, Vishal Verma <[email protected]> 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 <[email protected]> > --- > ndctl/lib/ars.c | 11 +---------- > 1 file changed, 1 insertion(+), 10 deletions(-)
Looks good, Reviewed-by: Dan Williams <[email protected]> _______________________________________________ Linux-nvdimm mailing list [email protected] https://lists.01.org/mailman/listinfo/linux-nvdimm
