Am 01.07.2014 um 13:13 hat reza.jel...@tuhh.de geschrieben:
> From: Reza Jelveh <reza.jel...@tuhh.de>
> 
> The data byte count(DBC) read from the description information is defined for
> bits 21:00. Bits 30:22 are reserved and bit 31 is the Interrupt on Completion
> (I) flag.
> 
> Completion interrupts are triggered after every transaction instead of on
> I-flag in QEMU. tbl_entry_size is a signed integer and improperly reading the
> DBC leads to a negative offset that causes sglist allocation to fail.
> 
> Signed-off-by: Reza Jelveh <reza.jel...@tuhh.de>
> ---
> This requires a custom ovmf image with sata controller for testing:
> 
> http://reza.jelveh.me/assets/OVMF.fd.bz2
> 
> Signed-off-by: Reza Jelveh <reza.jel...@tuhh.de>

Reviewed-by: Kevin Wolf <kw...@redhat.com>

The spec also seems to require an even byte count, which we don't seem
to check. Do we want to add this? (In a separate patch, of course.)

We'll also want a qtest case to verify the fix and for regression
testing. John?

And finally, please don't forget to CC the block maintainers (Stefan and
me) for any AHCI patches.

Kevin

Reply via email to