Re: Off-by-one in both LIBATA and IDE drivers

[Norman Diamond]

Jim Paris wrote:
> Norman Diamond wrote:
> > It looks like both LIBATA and the old IDE drivers have an off-by-one 
> > error in deciding whether to use READ SECTOR(S) instead of READ SECTOR(S) 
> > EXT.
>
> This was fixed here:
> http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=97b697a11b07e2ebfa69c488132596cc5eb24119

I'm still having trouble after applying the same patch to 2.6.24.3 (applying 
it three times in order to build Slax).  But now I wonder if it's no longer 
the fault of drivers.

Does hdparm construct its own taskfiles for ATA and SATA in order to produce 
an error trying to read sector number 0x0fffffff even after I patched the 
kernel?

If dd works then did I adequately patch the kernel?

Meanwhile I think the kernel needs more patches than ata.h.
(1)  libata-core.c contains a suspicious expression 1UL << 28.
(2)  sata_inic162x.c contains a suspicious expression 1 << 28. 

--------------------------------------
Power up the Internet with Yahoo! Toolbar.
http://pr.mail.yahoo.co.jp/toolbar/
--
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/