[Xen-changelog] [xen-unstable] xl: error if vif backend!=0 is used with run_hotplug_scripts

[Xen patchbot-unstable <patchbot@xxxxxxx>]

# HG changeset patch
# User Roger Pau Monne <roger.pau@xxxxxxxxxx>
# Date 1347613369 -3600
# Node ID ff6d9494403908d43bf61ce11bc1a5daa21133c7
# Parent  259c4e9d8adf0ba8731d09c496ebec85b0e2c1aa
xl: error if vif backend!=0 is used with run_hotplug_scripts

Print an error and exit if backend!=0 is used in conjunction with
run_hotplug_scripts. Currently libxl can only execute hotplug scripts
from the toolstack domain (the same domain xl is running from).

Added a description and workaround of this issue on
xl-network-configuration.

Signed-off-by: Roger Pau Monne <roger.pau@xxxxxxxxxx>
Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
Committed-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
---


diff -r 259c4e9d8adf -r ff6d94944039 docs/misc/xl-network-configuration.markdown
--- a/docs/misc/xl-network-configuration.markdown       Fri Sep 14 10:02:48 
2012 +0100
+++ b/docs/misc/xl-network-configuration.markdown       Fri Sep 14 10:02:49 
2012 +0100
@@ -122,8 +122,10 @@ specified IP address to be used by the g
 ### backend
 
 Specifies the backend domain which this device should attach to. This
-defaults to domain 0. Specifying another domain requires setting up a
-driver domain which is outside the scope of this document.
+defaults to domain 0. This option does not work if `run_hotplug_scripts`
+is not disabled in xl.conf (see xl.conf(5) man page for more information
+on this option). Specifying another domain requires setting up a driver
+domain which is outside the scope of this document.
 
 ### rate
 
diff -r 259c4e9d8adf -r ff6d94944039 tools/libxl/xl_cmdimpl.c
--- a/tools/libxl/xl_cmdimpl.c  Fri Sep 14 10:02:48 2012 +0100
+++ b/tools/libxl/xl_cmdimpl.c  Fri Sep 14 10:02:49 2012 +0100
@@ -1096,6 +1096,13 @@ static void parse_config_data(const char
                         fprintf(stderr, "Specified backend domain does not 
exist, defaulting to Dom0\n");
                         nic->backend_domid = 0;
                     }
+                    if (nic->backend_domid != 0 && run_hotplug_scripts) {
+                        fprintf(stderr, "ERROR: the vif 'backend=' option "
+                                "cannot be used in conjunction with "
+                                "run_hotplug_scripts, please set "
+                                "run_hotplug_scripts to 0 in xl.conf\n");
+                        exit(EXIT_FAILURE);
+                    }
                 } else if (!strcmp(p, "rate")) {
                     parse_vif_rate(&config, (p2 + 1), nic);
                 } else if (!strcmp(p, "accel")) {

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog