On 2017-02-07 21:55:52, Dandan Bi wrote: > Cc: Jordan Justen <jordan.l.jus...@intel.com> > Cc: Laszlo Ersek <ler...@redhat.com> > Cc: Liming Gao <liming....@intel.com> > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Dandan Bi <dandan...@intel.com> > --- > OvmfPkg/Library/QemuBootOrderLib/ExtraRootBusMap.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/OvmfPkg/Library/QemuBootOrderLib/ExtraRootBusMap.c > b/OvmfPkg/Library/QemuBootOrderLib/ExtraRootBusMap.c > index ec42214..1bb89d4 100644 > --- a/OvmfPkg/Library/QemuBootOrderLib/ExtraRootBusMap.c > +++ b/OvmfPkg/Library/QemuBootOrderLib/ExtraRootBusMap.c > @@ -306,8 +306,8 @@ MapRootBusPosToBusNr ( > return EFI_INVALID_PARAMETER; > } > if (RootBusPos > ExtraRootBusMap->Count) { > return EFI_NOT_FOUND; > } > - *RootBusNr = ExtraRootBusMap->BusNumbers[RootBusPos - 1]; > + *RootBusNr = ExtraRootBusMap->BusNumbers[(UINT32)RootBusPos - 1];
Is this failing on IA32? If so, think UINTN would be better. You might also want to add the impacted toolchain and arch to the commit message. -Jordan _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel