From mboxrd@z Thu Jan 1 00:00:00 1970 From: Roger Pau Monne Subject: [PATCH 4/5] libxl: add "downscript=no" to Qemu call Date: Mon, 16 Apr 2012 16:06:43 +0100 Message-ID: <1334588804-7755-5-git-send-email-roger.pau@citrix.com> References: <1334588804-7755-1-git-send-email-roger.pau@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1334588804-7755-1-git-send-email-roger.pau@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: xen-devel@lists.xen.org Cc: Roger Pau Monne List-Id: xen-devel@lists.xenproject.org Currently we only pass script=no to Qemu, to avoid calling any scripts when attaching a tap interface, but we should also pass downscript=no to avoid Qemu trying to execute a script when disconnecting the interface. This prevents the following harmless error message: /etc/qemu-ifdown: could not launch network script Signed-off-by: Roger Pau Monne --- tools/libxl/libxl_dm.c | 17 +++++++++++------ 1 files changed, 11 insertions(+), 6 deletions(-) diff --git a/tools/libxl/libxl_dm.c b/tools/libxl/libxl_dm.c index ba5bef7..8dc588d 100644 --- a/tools/libxl/libxl_dm.c +++ b/tools/libxl/libxl_dm.c @@ -216,11 +216,14 @@ static char ** libxl__build_device_model_args_old(libxl__gc *gc, else ifname = vifs[i].ifname; flexarray_vappend(dm_args, - "-net", libxl__sprintf(gc, "nic,vlan=%d,macaddr=%s,model=%s", - vifs[i].devid, smac, vifs[i].model), - "-net", libxl__sprintf(gc, "tap,vlan=%d,ifname=%s,bridge=%s,script=%s", - vifs[i].devid, ifname, vifs[i].bridge, libxl_tapif_script(gc)), - NULL); + "-net", libxl__sprintf(gc, "nic,vlan=%d,macaddr=%s,model=%s", + vifs[i].devid, smac, vifs[i].model), + "-net", libxl__sprintf(gc, + "tap,vlan=%d,ifname=%s,bridge=%s,script=%s," + "downscript=%s", + vifs[i].devid, ifname, vifs[i].bridge, + libxl_tapif_script(gc), libxl_tapif_script(gc)), + NULL); ioemu_vifs++; } } @@ -462,8 +465,10 @@ static char ** libxl__build_device_model_args_new(libxl__gc *gc, vifs[i].devid, smac)); flexarray_append(dm_args, "-netdev"); flexarray_append(dm_args, - libxl__sprintf(gc, "type=tap,id=net%d,ifname=%s,script=%s", + libxl__sprintf(gc, "type=tap,id=net%d,ifname=%s,script=%s," + "downscript=%s", vifs[i].devid, ifname, + libxl_tapif_script(gc), libxl_tapif_script(gc))); ioemu_vifs++; } -- 1.7.7.5 (Apple Git-26)