Mark Lord wrote:
Sergei Shtylyov wrote:
The fussy bit is that these commands require a non-standard
sector size, usually 520 bytes instead of 512.
Which requires from the drivers to be able to turn off IDE prefetch
(and maybe posting too). I don't see that in this patch (or are you
expecting them to just "snoop' the commands and do it automagically?).
Most controllers seem to be behave well enough
and just not muck up. And if it fails on any "too clever" controllers,
then no big deal -- we just switch to a controller that works instead.
This is not an essential command for normal operation.
What it is for, is injecting errors so that we can experience
firsthand the (excellent or crappy) error handling in the kernel
and in applications. And then work out and test better strategies.
So 100% working everywhere, for an obsolete opcode, isn't really
essential to me. But it should work most places, and it does.
Again, ata_data_xfer() doesn't seem capable of performing ECC
read/writes
-- the ECC bytes must be transferred in 8-bit mode, AFAIR.
ata_data_xfer()
can oinly do that for optionally trailing odd byte.
I have no idea what that was all about. Care to explain again?
RWLONG transfer the ECC info 8-bits at a time, using 16-bit words
to do so, no different from normal. ???
Here it is again with the whitespace not so messed up.
Thanks.
The READ/WRITE LONG commands are theoretically obsolete,
but the majority of drives in existance still implement them.
The WRITE_LONG and WRITE_LONG_ONCE commands are of particular
interest for fault injection testing -- eg. creating "media errors"
at specific locations on a disk.
The fussy bit is that these commands require a non-standard
sector size, usually 520 bytes instead of 512.
This patch adds support to libata for READ/WRITE LONG commands
issued via SG_IO/ATA_16.
This patch was generated against a 2.6.21-rc3-git7 base:
drivers/ata/libata-core.c | 20 ++++++++++----------
drivers/ata/libata-scsi.c | 12 ++++++++++++
include/linux/ata.h | 6 ++++++
include/linux/libata.h | 2 ++
4 files changed, 30 insertions(+), 10 deletions(-)
Signed-off-by: Mark Lord <mlord@xxxxxxxxx>
applied
-
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
