Laszlo Ersek [mailto:ler...@redhat.com]  wrote:

]Scott,
]
]On 11/13/14 18:27, Anthony PERARD wrote:
]> Hi,
]> 
]> I've took ovmf-ia32-msft-xen.patch from Scott Duplichan and made a few 
changes.
]> 
]> So the first two patchs of this series changes the type of some
]> variables/arguments/fields in order to avoid to have to cast them to a
]> different type.
]> 
]> Then, the last two patchs are what's left from Scott patch and the many cast
]> which I think are correct and cannot really be avoided.
]> 
]> Thanks,
]> 
]> Anthony PERARD (2):
]>   OvmfPkg/XenBusDxe: In XenStore, replace type of Len from UINTN to
]>     UINT32.
]>   OvmfPkg/XenBusDxe: Fix some types.
]> 
]> Scott Duplichan (2):
]>   OvmfPkg/XenBusDxe: fix VS2010 build failures
]>   OvmfPkg/XenPvBlkDxe: fix VS2010 build failures
]> 
]>  OvmfPkg/XenBusDxe/EventChannel.c |  6 +++---
]>  OvmfPkg/XenBusDxe/GrantTable.c   |  8 ++++----
]>  OvmfPkg/XenBusDxe/XenBus.c       |  6 +++---
]>  OvmfPkg/XenBusDxe/XenHypercall.c |  8 ++++----
]>  OvmfPkg/XenBusDxe/XenHypercall.h |  2 +-
]>  OvmfPkg/XenBusDxe/XenStore.c     | 42 
++++++++++++++++++++--------------------
]>  OvmfPkg/XenPvBlkDxe/BlockFront.c | 16 +++++++--------
]>  OvmfPkg/XenPvBlkDxe/BlockIo.c    |  2 +-
]>  8 files changed, 45 insertions(+), 45 deletions(-)
]> 
]
]can you please build-test this as well?

Build-tested-by: Scott Duplichan <sc...@notabs.org>

Passed Windows build test using these commands:
build.exe -p OvmfPkg\OvmfPkgX64.dsc -b DEBUG -t VS2010x86 -n 16 -a X64 
-DSECURE_BOOT_ENABLE -DFD_SIZE_2MB
build.exe -p OvmfPkg\OvmfPkgX64.dsc -b RELEASE -t VS2010x86 -n 16 -a X64 
-DSECURE_BOOT_ENABLE -DFD_SIZE_2MB
build.exe -p OvmfPkg\OvmfPkgX64.dsc -b NOOPT -t VS2010x86 -n 16 -a X64 
-DSECURE_BOOT_ENABLE -DFD_SIZE_2MB
build.exe -p OvmfPkg\OvmfPkgIA32.dsc -b DEBUG -t VS2010x86 -n 16 -a IA32 
-DSECURE_BOOT_ENABLE -DFD_SIZE_2MB
build.exe -p OvmfPkg\OvmfPkgIA32.dsc -b RELEASE -t VS2010x86 -n 16 -a IA32 
-DSECURE_BOOT_ENABLE -DFD_SIZE_2MB
build.exe -p OvmfPkg\OvmfPkgIA32.dsc -b NOOPT -t VS2010x86 -n 16 -a IA32 
-DSECURE_BOOT_ENABLE -DFD_SIZE_2MB
build.exe -p OvmfPkg\OvmfPkgIA32X64.dsc -b DEBUG -t VS2010x86 -n 16 -a IA32 -a 
X64 -DSECURE_BOOT_ENABLE -DFD_SIZE_2MB
build.exe -p OvmfPkg\OvmfPkgIA32X64.dsc -b RELEASE -t VS2010x86 -n 16 -a IA32 
-a X64 -DSECURE_BOOT_ENABLE -DFD_SIZE_2MB
build.exe -p OvmfPkg\OvmfPkgIA32X64.dsc -b NOOPT -t VS2010x86 -n 16 -a IA32 -a 
X64 -DSECURE_BOOT_ENABLE -DFD_SIZE_2MB
build.exe -p OvmfPkg\OvmfPkgX64.dsc -b DEBUG -t GCC49 -n 16 -a X64 
-DSECURE_BOOT_ENABLE -DFD_SIZE_2MB
build.exe -p OvmfPkg\OvmfPkgX64.dsc -b RELEASE -t GCC49 -n 16 -a X64 
-DSECURE_BOOT_ENABLE -DFD_SIZE_2MB
build.exe -p OvmfPkg\OvmfPkgX64.dsc -b NOOPT -t GCC49 -n 16 -a X64 
-DSECURE_BOOT_ENABLE -DFD_SIZE_2MB
build.exe -p OvmfPkg\OvmfPkgIA32.dsc -b DEBUG -t GCC49 -n 16 -a IA32 
-DSECURE_BOOT_ENABLE -DFD_SIZE_2MB
build.exe -p OvmfPkg\OvmfPkgIA32.dsc -b RELEASE -t GCC49 -n 16 -a IA32 
-DSECURE_BOOT_ENABLE -DFD_SIZE_2MB
build.exe -p OvmfPkg\OvmfPkgIA32.dsc -b NOOPT -t GCC49 -n 16 -a IA32 
-DSECURE_BOOT_ENABLE -DFD_SIZE_2MB
build.exe -p OvmfPkg\OvmfPkgIA32X64.dsc -b DEBUG -t GCC49 -n 16 -a IA32 -a X64 
-DSECURE_BOOT_ENABLE -DFD_SIZE_2MB
build.exe -p OvmfPkg\OvmfPkgIA32X64.dsc -b RELEASE -t GCC49 -n 16 -a IA32 -a 
X64 -DSECURE_BOOT_ENABLE -DFD_SIZE_2MB
build.exe -p OvmfPkg\OvmfPkgIA32X64.dsc -b NOOPT -t GCC49 -n 16 -a IA32 -a X64 
-DSECURE_BOOT_ENABLE -DFD_SIZE_2MB

Thanks,
Scott

](I pushed the non-xen patches.)
]
]Thanks
]Laszlo


------------------------------------------------------------------------------
Comprehensive Server Monitoring with Site24x7.
Monitor 10 servers for $9/Month.
Get alerted through email, SMS, voice calls or mobile push notifications.
Take corrective actions from your mobile device.
http://pubads.g.doubleclick.net/gampad/clk?id=154624111&iu=/4140/ostg.clktrk
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to