Mark Lord wrote: > In an ideal world, we would use the existing ATA_12 opcode > to issue 12-byte ATA passthrough commands for libata ATAPI drives > from userspace. > > But ATA_12 happens to have the same SCSI opcode value as the older > CD/RW "BLANK" command, widely used by cdrecord and friends. > > So, to achieve ATA passthru capability for libata ATAPI, > we have to instead use the ATA_16 opcode: a 16-byte command. > > SCSI normally disallows issuing 16-byte commands to 12-byte devices, > so special support has to be added for this. > > Introduce an "allow_ata_16" boolean to the scsi_host struct. > This provides a means for libata to signal that 16-byte ATA_16 > commands should be permitted even for 12-byte ATAPI devices. > > There are companion patches submitted separately > to add ATAPI ATA_16 capability to libata. > > Signed-off-by: Mark Lord <mlord@xxxxxxxxx> I might have missed the discussion but can't we just set host->max_cmd_len to 16 unconditionally? libata is emulating a SCSI device anyway and, from SCSI's POV, the situation is that any libata ATAPI device can do both 12 and 16 byte commands while some of them only allow allow ATA_16 for 16 byte command. Also, it's not like host->max_cmd_len gives any guaranteed protection against CDB length. Being a 'host' limit, it's set to the highest number in the host. In that respect, it should be set to 16 too. All ATA hosts can do 16 byte CDBs. Thanks. -- tejun - 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
