diff mbox

[V8,RESEND,2/8] configure: Introduce --enable-xen-pci-passthrough.

Message ID 1331916862-20504-3-git-send-email-anthony.perard@citrix.com
State New
Headers show

Commit Message

Anthony PERARD March 16, 2012, 4:54 p.m. UTC
Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
---
 configure |   25 +++++++++++++++++++++++++
 1 files changed, 25 insertions(+), 0 deletions(-)

Comments

Stefano Stabellini March 19, 2012, 11:51 a.m. UTC | #1
On Fri, 16 Mar 2012, Anthony PERARD wrote:
> Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>

ack

>  configure |   25 +++++++++++++++++++++++++
>  1 files changed, 25 insertions(+), 0 deletions(-)
> 
> diff --git a/configure b/configure
> index afe7395..1d1204a 100755
> --- a/configure
> +++ b/configure
> @@ -136,6 +136,7 @@ vnc_png=""
>  vnc_thread="no"
>  xen=""
>  xen_ctrl_version=""
> +xen_pci_passthrough=""
>  linux_aio=""
>  cap_ng=""
>  attr=""
> @@ -682,6 +683,10 @@ for opt do
>    ;;
>    --enable-xen) xen="yes"
>    ;;
> +  --disable-xen-pci-passthrough) xen_pci_passthrough="no"
> +  ;;
> +  --enable-xen-pci-passthrough) xen_pci_passthrough="yes"
> +  ;;
>    --disable-brlapi) brlapi="no"
>    ;;
>    --enable-brlapi) brlapi="yes"
> @@ -1034,6 +1039,8 @@ echo "                           (affects only QEMU, not qemu-img)"
>  echo "  --enable-mixemu          enable mixer emulation"
>  echo "  --disable-xen            disable xen backend driver support"
>  echo "  --enable-xen             enable xen backend driver support"
> +echo "  --disable-xen-pci-passthrough"
> +echo "  --enable-xen-pci-passthrough"
>  echo "  --disable-brlapi         disable BrlAPI"
>  echo "  --enable-brlapi          enable BrlAPI"
>  echo "  --disable-vnc-tls        disable TLS encryption for VNC server"
> @@ -1478,6 +1485,21 @@ EOF
>    fi
>  fi
>  
> +if test "$xen_pci_passthrough" != "no"; then
> +  if test "$xen" = "yes" && test "$linux" = "yes"; then
> +    xen_pci_passthrough=yes
> +  else
> +    if test "$xen_pci_passthrough" = "yes"; then
> +      echo "ERROR"
> +      echo "ERROR: User requested feature Xen PCI Passthrough"
> +      echo "ERROR: but this feature require /sys from Linux"
> +      echo "ERROR"
> +      exit 1;
> +    fi
> +    xen_pci_passthrough=no
> +  fi
> +fi
> +
>  ##########################################
>  # pkg-config probe
>  
> @@ -3628,6 +3650,9 @@ case "$target_arch2" in
>      if test "$xen" = "yes" -a "$target_softmmu" = "yes" ; then
>        target_phys_bits=64
>        echo "CONFIG_XEN=y" >> $config_target_mak
> +      if test "$xen_pci_passthrough" = yes; then
> +        echo "CONFIG_XEN_PCI_PASSTHROUGH=y" >> "$config_target_mak"
> +      fi
>      else
>        echo "CONFIG_NO_XEN=y" >> $config_target_mak
>      fi
> -- 
> Anthony PERARD
>
Konrad Rzeszutek Wilk May 12, 2012, 1:42 a.m. UTC | #2
On Fri, Mar 16, 2012 at 12:54 PM, Anthony PERARD
<anthony.perard@citrix.com> wrote:
> Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>

I thought I reviewed this last time? Is there a reason for not
attaching 'Reviewed-by: Konrad Rzeszutek Wilk
<konrad.wilk@oracle.com>' on this patch?
Anthony PERARD May 14, 2012, 10:49 a.m. UTC | #3
On Sat, May 12, 2012 at 2:42 AM, Konrad Rzeszutek Wilk
<konrad@darnok.org> wrote:
>
> I thought I reviewed this last time? Is there a reason for not
> attaching 'Reviewed-by: Konrad Rzeszutek Wilk
> <konrad.wilk@oracle.com>' on this patch?

Yes, one: you reviewed a later patch :-)

Your reviewed-by is in the last version of this series, V11.
Konrad Rzeszutek Wilk May 16, 2012, 11:15 a.m. UTC | #4
On Mon, May 14, 2012 at 11:49:46AM +0100, Anthony PERARD wrote:
> On Sat, May 12, 2012 at 2:42 AM, Konrad Rzeszutek Wilk
> <konrad@darnok.org> wrote:
> >
> > I thought I reviewed this last time? Is there a reason for not
> > attaching 'Reviewed-by: Konrad Rzeszutek Wilk
> > <konrad.wilk@oracle.com>' on this patch?
> 
> Yes, one: you reviewed a later patch :-)

Duh!
> 
> Your reviewed-by is in the last version of this series, V11.
> 
> -- 
> Anthony PERARD
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel
diff mbox

Patch

diff --git a/configure b/configure
index afe7395..1d1204a 100755
--- a/configure
+++ b/configure
@@ -136,6 +136,7 @@  vnc_png=""
 vnc_thread="no"
 xen=""
 xen_ctrl_version=""
+xen_pci_passthrough=""
 linux_aio=""
 cap_ng=""
 attr=""
@@ -682,6 +683,10 @@  for opt do
   ;;
   --enable-xen) xen="yes"
   ;;
+  --disable-xen-pci-passthrough) xen_pci_passthrough="no"
+  ;;
+  --enable-xen-pci-passthrough) xen_pci_passthrough="yes"
+  ;;
   --disable-brlapi) brlapi="no"
   ;;
   --enable-brlapi) brlapi="yes"
@@ -1034,6 +1039,8 @@  echo "                           (affects only QEMU, not qemu-img)"
 echo "  --enable-mixemu          enable mixer emulation"
 echo "  --disable-xen            disable xen backend driver support"
 echo "  --enable-xen             enable xen backend driver support"
+echo "  --disable-xen-pci-passthrough"
+echo "  --enable-xen-pci-passthrough"
 echo "  --disable-brlapi         disable BrlAPI"
 echo "  --enable-brlapi          enable BrlAPI"
 echo "  --disable-vnc-tls        disable TLS encryption for VNC server"
@@ -1478,6 +1485,21 @@  EOF
   fi
 fi
 
+if test "$xen_pci_passthrough" != "no"; then
+  if test "$xen" = "yes" && test "$linux" = "yes"; then
+    xen_pci_passthrough=yes
+  else
+    if test "$xen_pci_passthrough" = "yes"; then
+      echo "ERROR"
+      echo "ERROR: User requested feature Xen PCI Passthrough"
+      echo "ERROR: but this feature require /sys from Linux"
+      echo "ERROR"
+      exit 1;
+    fi
+    xen_pci_passthrough=no
+  fi
+fi
+
 ##########################################
 # pkg-config probe
 
@@ -3628,6 +3650,9 @@  case "$target_arch2" in
     if test "$xen" = "yes" -a "$target_softmmu" = "yes" ; then
       target_phys_bits=64
       echo "CONFIG_XEN=y" >> $config_target_mak
+      if test "$xen_pci_passthrough" = yes; then
+        echo "CONFIG_XEN_PCI_PASSTHROUGH=y" >> "$config_target_mak"
+      fi
     else
       echo "CONFIG_NO_XEN=y" >> $config_target_mak
     fi