On 09/16/15 16:47, Gabriel L. Somlo wrote:
> whole series:
> Tested-by: Gabriel Somlo <[email protected]>

Thanks for the testing. Here's a small but important update re: QEMU:

>> NOTE: You will also need the following QEMU bugfix for this to work:
>> <http://thread.gmane.org/gmane.comp.emulators.qemu.block/4327>.

Turns out Hannes already reported this issue, and John Snow (IDE / AHCI
/ Floppy maintainer in QEMU) already has a patch (more comprehensive
than mine) queued for it. So for testing this edk2 series, people should
rather pick John's patch (which he designated in a later message in the
thread linked above). I tested that QEMU patch and it works.

Thanks!
Laszlo


>>
>> Testing with various guest OSes is welcome. I tested Fedora (installer
>> CD-ROM and installed disk). I also did some light-weight regression
>> testing with i440fx IDE devices.
>>
>> "Enjoy", I guess. :)
>>
>> Cc: Alexander Graf <[email protected]>
>> Cc: Reza Jelveh <[email protected]>
>> Cc: Jordan Justen <[email protected]>
>> Cc: Ruiyu Ni <[email protected]>
>> Cc: Hannes Reinecke <[email protected]>
>> Cc: Gabriel L. Somlo <[email protected]>
>> Cc: Feng Tian <[email protected]>
>>
>> Thanks
>> Laszlo
>>
>> Laszlo Ersek (7):
>>   DuetPkg: SataControllerDxe: fix typo in "DisqulifiedModes"
>>   DuetPkg: SataControllerDxe: fix private array subscripting
>>   OvmfPkg: SataControllerDxe: add cascading error handling to Start()
>>   OvmfPkg: SataControllerDxe: enable IO / mem access and DMA when
>>     binding
>>   OvmfPkg: SataControllerDxe: enable AHCI mode if IS_PCI_SATADPA()
>>   PcAtChipsetPkg: IdeControllerDxe: fix protocol usage hints in the INF
>>     file
>>   OvmfPkg: QemuBootOrderLib: recognize Q35 SATA disks / CD-ROMs
>>
>> Reza Jelveh (3):
>>   OvmfPkg: copy SataControllerDxe from DuetPkg
>>   MdeModulePkg: AtaAtapiPassThru: select master/slave around DIAG
>>     command
>>   OvmfPkg: enable SATA controller
>>
>>  {DuetPkg => OvmfPkg}/SataControllerDxe/SataControllerDxe.inf |   2 +-
>>  PcAtChipsetPkg/Bus/Pci/IdeControllerDxe/IdeControllerDxe.inf |   4 +-
>>  DuetPkg/SataControllerDxe/SataController.h                   |   2 +-
>>  {DuetPkg => OvmfPkg}/SataControllerDxe/SataController.h      |   9 +-
>>  DuetPkg/SataControllerDxe/SataController.c                   |  74 +++++++--
>>  MdeModulePkg/Bus/Ata/AtaAtapiPassThru/IdeMode.c              |   5 +
>>  OvmfPkg/Library/QemuBootOrderLib/QemuBootOrderLib.c          |  43 +++++
>>  {DuetPkg => OvmfPkg}/SataControllerDxe/ComponentName.c       |   0
>>  {DuetPkg => OvmfPkg}/SataControllerDxe/SataController.c      | 172 
>> +++++++++++++++-----
>>  OvmfPkg/OvmfPkgIa32.dsc                                      |   5 +-
>>  OvmfPkg/OvmfPkgIa32.fdf                                      |   5 +-
>>  OvmfPkg/OvmfPkgIa32X64.dsc                                   |   5 +-
>>  OvmfPkg/OvmfPkgIa32X64.fdf                                   |   5 +-
>>  OvmfPkg/OvmfPkgX64.dsc                                       |   5 +-
>>  OvmfPkg/OvmfPkgX64.fdf                                       |   5 +-
>>  15 files changed, 267 insertions(+), 74 deletions(-)
>>  copy {DuetPkg => OvmfPkg}/SataControllerDxe/SataControllerDxe.inf (91%)
>>  copy {DuetPkg => OvmfPkg}/SataControllerDxe/SataController.h (96%)
>>  copy {DuetPkg => OvmfPkg}/SataControllerDxe/ComponentName.c (100%)
>>  copy {DuetPkg => OvmfPkg}/SataControllerDxe/SataController.c (85%)
>>
>> -- 
>> 1.8.3.1
>>

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

Reply via email to