Re: [bisected] Re: todays git: WARNING: atdrivers/ata/libata-sff.c:1017 ata_sff_hsm_move+0x45e/0x750()

From: Alan Cox
Date: Sat Jan 10 2009 - 19:12:20 EST


> >> data_xfer transfers 18 bytes + 2 padding (correctly) -> 20 bytes
>
> Correctly indeed? I'm not at all sure it's correct to read an extra
> 16-bit word off the device when it thinks it's already done with the
> data transfer.

The VLB controller docs explicitly require you do entirely 32bit. The AMD
ones seem to imply you should.

Its easy enough to not do that but for the VLB case we have to so the
WARN needs fixing which ever path we go down.
--
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/