Re: [PATCH 11/12] nvme: use the changed namespaces list log to clear ns data changed AENs

2018-06-04 Thread Hannes Reinecke
On Wed, 30 May 2018 18:45:59 +0200
Christoph Hellwig  wrote:

> Per section 5.2 we need to issue the corresponding log page to clear
> an AEN, so for a namespace data changed AEN we need to read the
> changed namespace list log.  And once we read that log anyway we
> might as well use it to optimize the rescan.
> 
> Signed-off-by: Christoph Hellwig 
> ---
>  drivers/nvme/host/core.c | 51
>  drivers/nvme/host/nvme.h |
> 2 ++ 2 files changed, 49 insertions(+), 4 deletions(-)
> 
Reviewed-by: Hannes Reinecke 

Cheers,

Hannes



Re: [PATCH 11/12] nvme: use the changed namespaces list log to clear ns data changed AENs

2018-05-30 Thread Sagi Grimberg

Looks good,

Reviewed-by: Sagi Grimberg