Hyper-V is available on 64-bit versions of Windows,
do not try to build its support on 32-bit versions.
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
---
Since v1: Handle MinGW automatic detection (Thomas, Stefan)
---
configure | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/configure b/configure
index 4231d56bcc0..91c5d781845 100755
--- a/configure
+++ b/configure
@@ -2980,7 +2980,10 @@ fi
##########################################
# Windows Hypervisor Platform accelerator (WHPX) check
-if test "$whpx" != "no" ; then
+if test "$whpx" = "yes" && test "$ARCH" != "x86_64"; then
+ error_exit "WHPX requires 64-bit host"
+fi
+if test "$whpx" != "no" && test "$ARCH" = "x86_64"; then
if check_include "WinHvPlatform.h" && check_include "WinHvEmulation.h"; then
whpx="yes"
else
--
2.26.2
Am 10.09.20 um 07:45 schrieb Philippe Mathieu-Daudé: > Hyper-V is available on 64-bit versions of Windows, > do not try to build its support on 32-bit versions. > > Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org> > --- > Since v1: Handle MinGW automatic detection (Thomas, Stefan) > --- > configure | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/configure b/configure > index 4231d56bcc0..91c5d781845 100755 > --- a/configure > +++ b/configure > @@ -2980,7 +2980,10 @@ fi > > ########################################## > # Windows Hypervisor Platform accelerator (WHPX) check > -if test "$whpx" != "no" ; then > +if test "$whpx" = "yes" && test "$ARCH" != "x86_64"; then > + error_exit "WHPX requires 64-bit host" > +fi > +if test "$whpx" != "no" && test "$ARCH" = "x86_64"; then > if check_include "WinHvPlatform.h" && check_include "WinHvEmulation.h"; then > whpx="yes" > else Reviewed-by: Stefan Weil <sw@weilnetz.de> Thanks!
On 10/09/20 07:45, Philippe Mathieu-Daudé wrote: > Hyper-V is available on 64-bit versions of Windows, > do not try to build its support on 32-bit versions. > > Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org> > --- > Since v1: Handle MinGW automatic detection (Thomas, Stefan) > --- > configure | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/configure b/configure > index 4231d56bcc0..91c5d781845 100755 > --- a/configure > +++ b/configure > @@ -2980,7 +2980,10 @@ fi > > ########################################## > # Windows Hypervisor Platform accelerator (WHPX) check > -if test "$whpx" != "no" ; then > +if test "$whpx" = "yes" && test "$ARCH" != "x86_64"; then > + error_exit "WHPX requires 64-bit host" > +fi > +if test "$whpx" != "no" && test "$ARCH" = "x86_64"; then > if check_include "WinHvPlatform.h" && check_include "WinHvEmulation.h"; then > whpx="yes" > else > Queued this one instead. Paolo
On 9/10/20 12:45 AM, Philippe Mathieu-Daudé wrote: In the subject, s/intent/attempt/ > Hyper-V is available on 64-bit versions of Windows, > do not try to build its support on 32-bit versions. > > Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org> > --- > Since v1: Handle MinGW automatic detection (Thomas, Stefan) > --- > configure | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3226 Virtualization: qemu.org | libvirt.org
On 9/11/20 7:18 PM, Eric Blake wrote: > On 9/10/20 12:45 AM, Philippe Mathieu-Daudé wrote: > > In the subject, s/intent/attempt/ Oops thanks... Paolo can you fix that directly, or should I repost? > >> Hyper-V is available on 64-bit versions of Windows, >> do not try to build its support on 32-bit versions. >> >> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org> >> --- >> Since v1: Handle MinGW automatic detection (Thomas, Stefan) >> --- >> configure | 5 ++++- >> 1 file changed, 4 insertions(+), 1 deletion(-) >> -- > Eric Blake, Principal Software Engineer > Red Hat, Inc. +1-919-301-3226 > Virtualization: qemu.org | libvirt.org > >
© 2016 - 2024 Red Hat, Inc.