[PATCH 09/12] nvme: properly free resources for cancelled command

Keith Busch keith.busch at intel.com
Mon Nov 9 12:12:33 PST 2015


On Mon, Nov 09, 2015 at 08:25:19PM +0100, Christoph Hellwig wrote:
> On Mon, Nov 09, 2015 at 06:57:31PM +0000, Keith Busch wrote:
> > Whoa now, it's neither Intel nor userpace that needs this. It's to know
> > if the controller is unresponsive or returned an error. The difference
> > matters to the driver for initialization.
> 
> Haha, so we at least can root cause this now.  Can you point me
> to the caller that cares?  I'd really like to get rid of the special
> case of passing a negative errno here, so I'd like to figure out how
> we could pass this information on instead.

The "set_queue_count()" was the function that cared, but looks like
patch 8/12 makes this think a timeout is an aborted status.



More information about the Linux-nvme mailing list