On 01/19/16 21:34, Paolo Bonzini wrote:
> 
> 
> On 19/01/2016 00:29, Laszlo Ersek wrote:
>>> INQUIRY works slightly different from other commands. It has a maximum
>>> length instead of a deterministic length; once HCYL/LCYL are read back,
>>> should RequiredBytes not be adjusted in this case? (This way we don't
>>> short the loop, we just finish gracefully.)
> 
> I like John's suggestion, and it can be applied to _any_ command, not
> just INQUIRY.
> 
> However, ByteCount should be changed to a UINT32* and updated by
> AtaPacketReadWrite.  Then the ActualWordCount can be reflected into the
> InTransferLength or OutTransferLength of struct
> EFI_EXT_SCSI_PASS_THRU_SCSI_REQUEST_PACKET.

Thanks!

Can you contribute a patch? :)

Laszlo
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to