[lkml]   [2006]   [Jan]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] libata queue updated
Hi Tejun,

On Tuesday 31 January 2006 01:04, Tejun wrote:
> Ingo Oeser wrote:
> > What about putting the information directly into "ap->device[INDEX].class"
> > in the sole caller (ata_drive_probe_reset) so far?
> >
> Not altering ->class directly in lldd driver is one major point of this
> whole patchset such that higher level driving logic has a say on whether
> a device is online or not, not the low level driver. Primarily this is
> useful for sharing low-level codes with hot plugging / EH but it's also
> possible to retry some of the operations during probing in limited cases.

Ok, with this argument, I finally get it. Now I know why you do it this
way. You let the lld driver suggest a class for it's devices and verify
these suggestions by high level code.

The only way to get to this classification data is by resetting the ATA

It might be technically possible to set ->class directly and
fix it up in high level logic, as needed.

Your explicit design decision was NOT to do this but to put this
suggestions from low level driver into a temporary on stack structure
from the higher level API.

And since the maintainer is happy already, I couldn't care less.

Thanks for your patience :-)


Ingo Oeser

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2006-01-31 03:37    [W:0.032 / U:2.748 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site