Re: [PATCH v2] nvme: Fix nvme reset command timeout handling

2017-08-15 Thread Sagi Grimberg
Looks good, Reviewed-by: Sagi Grimberg p.s. we'll need also to update the configure timestamp configuration.

Re: [PATCH v2] nvme: Fix nvme reset command timeout handling

2017-08-15 Thread Sagi Grimberg
Looks good, Reviewed-by: Sagi Grimberg p.s. we'll need also to update the configure timestamp configuration.

Re: [PATCH v2] nvme: Fix nvme reset command timeout handling

2017-08-10 Thread Keith Busch
On Thu, Aug 10, 2017 at 11:23:31AM +0200, Johannes Thumshirn wrote: > From: Keith Busch > > We need to return an error if a timeout occurs on any NVMe command during > initialization. Without this, the nvme reset work will be stuck. A timeout > will have a negative error

Re: [PATCH v2] nvme: Fix nvme reset command timeout handling

2017-08-10 Thread Keith Busch
On Thu, Aug 10, 2017 at 11:23:31AM +0200, Johannes Thumshirn wrote: > From: Keith Busch > > We need to return an error if a timeout occurs on any NVMe command during > initialization. Without this, the nvme reset work will be stuck. A timeout > will have a negative error code, meaning we need to

Re: [PATCH v2] nvme: Fix nvme reset command timeout handling

2017-08-10 Thread Christoph Hellwig
Looks good, Reviewed-by: Christoph Hellwig

Re: [PATCH v2] nvme: Fix nvme reset command timeout handling

2017-08-10 Thread Christoph Hellwig
Looks good, Reviewed-by: Christoph Hellwig

[PATCH v2] nvme: Fix nvme reset command timeout handling

2017-08-10 Thread Johannes Thumshirn
From: Keith Busch We need to return an error if a timeout occurs on any NVMe command during initialization. Without this, the nvme reset work will be stuck. A timeout will have a negative error code, meaning we need to stop initializing the controller. All postitive

[PATCH v2] nvme: Fix nvme reset command timeout handling

2017-08-10 Thread Johannes Thumshirn
From: Keith Busch We need to return an error if a timeout occurs on any NVMe command during initialization. Without this, the nvme reset work will be stuck. A timeout will have a negative error code, meaning we need to stop initializing the controller. All postitive returns mean the controller