On Mon, Nov 09, 2015 at 06:57:31PM +0000, Keith Busch wrote:
> On Sat, Nov 07, 2015 at 09:45:03AM +0100, Christoph Hellwig wrote:
> > + if (unlikely(req->errors)) {
> > + /*
> > + * Some silly Intel userspace code breaks if it doesn't get a
> > + * negative errno back for driver returns values.
> > + */
>
> 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.
--
To unsubscribe from this list: send the line "unsubscribe linux-block" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
