Re: [edk2] [PATCH] MdeModulePkg/PciBus: Fix system hang when no PCI Option ROM exists

2018-12-14 Thread Philippe Mathieu-Daudé
On 12/12/18 4:10 PM, Ruiyu Ni wrote: > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1394 > > When there is no PCI option ROM exists, today's logic still creates > virtual BAR for option ROM using Length = 0, Alignment = (-1). > It causes the final MEM32 alignment requirement is as big as >

Re: [edk2] [PATCH] MdeModulePkg/PciBus: Fix system hang when no PCI Option ROM exists

2018-12-13 Thread Wu, Hao A
> -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Ruiyu Ni > Sent: Wednesday, December 12, 2018 11:10 PM > To: edk2-devel@lists.01.org > Cc: Wu, Hao A > Subject: [edk2] [PATCH] MdeModulePkg/PciBus: Fix system hang when no > PCI Option ROM

Re: [edk2] [PATCH] MdeModulePkg/PciBus: Fix system hang when no PCI Option ROM exists

2018-12-12 Thread Wang, Jian J
Reviewed-by: Jian J Wang > -Original Message- > From: Ni, Ruiyu > Sent: Wednesday, December 12, 2018 11:10 PM > To: edk2-devel@lists.01.org > Cc: Chiu, Chasel ; Wu, Hao A ; > Wang, Jian J > Subject: [PATCH] MdeModulePkg/PciBus: Fix system hang when no PCI Option > ROM exists > >

Re: [edk2] [PATCH] MdeModulePkg/PciBus: Fix system hang when no PCI Option ROM exists

2018-12-12 Thread Ard Biesheuvel
On Wed, 12 Dec 2018 at 16:08, Ruiyu Ni wrote: > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1394 > > When there is no PCI option ROM exists, today's logic still creates > virtual BAR for option ROM using Length = 0, Alignment = (-1). > It causes the final MEM32 alignment requirement is