On 01/25/16 03:45, Tian, Feng wrote: > I am OK with the fix. > > Reviewed-by: Feng Tian <feng.t...@intel.com> > > PS: The "should" words is my thought on the write case (not tested as > well:-)). I thought the first patch should also work on short write case.
Patch committed to SVN as r19737. Thanks guys! Laszlo > > Thanks > Feng > > -----Original Message----- > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Saturday, January 23, 2016 00:59 > To: Paolo Bonzini; Tian, Feng > Cc: edk2-de...@ml01.01.org; js...@redhat.com > Subject: Re: [edk2] [PATCH] MdeModulePkg/.../IdeMode: correctly report length > of returned data > > On 01/22/16 15:23, Paolo Bonzini wrote: >> >> >> On 21/01/2016 02:27, Tian, Feng wrote: >>> Paolo, >>> >>> I think for short write case it means the data length to be written >>> in AtaPacketReadWrite, that is ByteCount, is less than the one >>> shipped in ATA cmd, for example, CDB (READ10.byte7&8). >>> >>> For such case, it should jump out the while loop in >>> AtaPacketReadWrite and send EFI_DEVICE_ERROR as DRQ is not clear. >>> >>> IMHO, Laszo & your patch all looks ok to me. of course, your patch >>> did an enhancement on how to reflect the real transfer data length >>> :) >> >> Thanks! Laszlo, can you commit it? > > Absolutely; I haven't done it already only because Feng didn't send a formal > R-b. That fact, combined with "it should ..." in the second paragraph, told > me that there were further details to iron out. > > If that's not the case: Feng, can you please post your Reviewed-by as usual? > Then I'll add my own T-b as well (already posted) and commit Paolo's patch. > > Thanks > Laszlo > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.01.org > https://lists.01.org/mailman/listinfo/edk2-devel > _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel