Re: [PATCH] 2.5.18 IDE 73

Gerald Champagne (gerald@io.com)
29 May 2002 09:26:42 -0500


>
> Dear Gerald please look closer. The hdparm -i is executing the
> drive id command directly and does *not* rely on the internally
> permanently dragged around id structure. So the change I did
> is entierly fine. Just go ahead and check whatever hdparm -i /dev/hdx
> reports the proper thing after changing some dma setting.
> It does - I did check it :-).

Ah, sorry about that. I missed that in your patch. The previous
version of the ioctl just returned copied the values from the id
structure. Doing the id on the fly is much more accurate and will catch
any other fields that happen to change over time.

> BTW> The next thing to be gone is simple the fact that we drag
> around the id information permanently, where infact only
> some capabilitie fields are sucked out of it and the
> device identification string is only needed for reporting
> during boot-up.

That sounds good. That would make it easier to see what values from the
id are actually used.

Thanks.

Gerald

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