Re: [edk2] [PATCH 0/2] MdeModulePkg/.../IdeMode: restore handling of "short reads"

2016-01-19 Thread Laszlo Ersek
On 01/19/16 01:34, Laszlo Ersek wrote: > The series > > [edk2] [patch 0/2] Fix a DVD device compatbility issue > http://thread.gmane.org/gmane.comp.bios.edk2.devel/3681 > > fixed an IDE timeout masking (misreporting) bug in the AtaAtapiPassThru > driver. (And, separately, it increased SCSI

Re: [edk2] [PATCH 0/2] MdeModulePkg/.../IdeMode: restore handling of "short reads"

2016-01-19 Thread Laszlo Ersek
On 01/19/16 06:40, Tian, Feng wrote: > Laszlo & John > > Thanks for your detail analysis. It's very helpful for me. > > Reviewed-by: Feng Tian Thank you! I'll soon commit the patches with your R-b. > I am thinking shall we update > EFI_SCSI_INQUIRY_DATA/ATAPI_INQUIRY_DATA

[edk2] [PATCH 0/2] MdeModulePkg/.../IdeMode: restore handling of "short reads"

2016-01-18 Thread Laszlo Ersek
The series [edk2] [patch 0/2] Fix a DVD device compatbility issue http://thread.gmane.org/gmane.comp.bios.edk2.devel/3681 fixed an IDE timeout masking (misreporting) bug in the AtaAtapiPassThru driver. (And, separately, it increased SCSI disk timeouts to 30s.) However, the IDE change that

Re: [edk2] [PATCH 0/2] MdeModulePkg/.../IdeMode: restore handling of "short reads"

2016-01-18 Thread Tian, Feng
Laszlo & John Thanks for your detail analysis. It's very helpful for me. Reviewed-by: Feng Tian I am thinking shall we update EFI_SCSI_INQUIRY_DATA/ATAPI_INQUIRY_DATA data structure as well to only contain the first 36 bytes? According to ATAPI SFF 8020i & SPC4 spec,