Re: [PATCH v2 4/4] ata: add ata port runtime PM callbacks

From: Tejun Heo
Date: Thu Nov 10 2011 - 10:46:03 EST


On Thu, Nov 10, 2011 at 10:40:21AM -0500, Alan Stern wrote:
> Are you really sure you want to keep this flag set throughout the
> lifetime of the port?
>
> What happens if a command fails, then the port suspends, and then the
> error handler is invoked to take care of the failed command?

Runtime PM doesn't happen if the device is in use in any way, so
there's no request flying in from upper layers. While the port is
suspended, qc's which which requested EH are short-circuited. AFAICS,
nothing is really wrong.

Thanks.

--
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/