Re: [PATCH] barrier patch set

[Jens Axboe <axboe@suse.de>]

On Sat, Mar 20 2004, Bartlomiej Zolnierkiewicz wrote:
> > >>All drives that support flush-cache list the relevant bits in
> > >>identify-device, even on pre-ATA-6 devices.  Whether the feature was
> > >>optional or mandantory, we can check the feature bits.
> > >
> > > Hm. so this is undocumented in the spec?
> >
> > ?  When it was optional, there was a feature bit to test.  When it
> > became mandantory, the feature bit to test stayed in there.  The feature
> > bit is zero, otherwise.  Makes it possible to use "just test the bit"
> > and have things Just Work(tm).  :)
> 
> I wish it was so simple.  Here is an example to make it clear:
> 
> model: WDC WD800JB-00CRA1 firmware: 17.07W77
> word 0x83 is 4b01, word 0x86 is 0x0801
> 
> and drive of course supports CACHE FLUSH command.

I agree with Bart, it's usually never that clear. Quit harping the
stupid LG issue, they did something brain dead in the firmware and I
almost have to say that they got what they deserved for doing something
as _stupid_ as that.

Jeff, it's wonderful to think that you can always rely on checking spec
bits, but in reality it never really 'just works out' for any given set
of hardware.

-- 
Jens Axboe