On Mon, Dec 4, 2017 at 3:40 AM, m...@ark-net.org <m...@ark-net.org> wrote:
> So I figured out that I don't need a running instance of xenstored on the
> driver domain, since xenstore-read works for the driver domains own trees in
> xenstore. Are the xenstore-read errors in vif-openvswitch because that
> script is written to be run on dom0 only? Do I need to create a new vif
> script that is specific to driver domains?
It sounds like that's a bug in the vif-openvswitch script -- it
shouldn't rely on reading data from the domain's "private" xenstore
area (available to domain 0 but not to a driver domain).
Can you try the attached patch and see if it helps?
-George
From 4c2194428bda881541760df489b4d68d3da5da88 Mon Sep 17 00:00:00 2001
From: George Dunlap <george.dun...@citrix.com>
Date: Mon, 4 Dec 2017 14:44:48 +0000
Subject: [PATCH] hotplug: Fix vif-openvswitch to run inside a driver domain
Driver domains can't read VM-specific xenstore paths.
Signed-off-by: George Dunlap <george.dun...@citrix.com>
---
tools/hotplug/Linux/vif-openvswitch | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/tools/hotplug/Linux/vif-openvswitch b/tools/hotplug/Linux/vif-openvswitch
index 18bfb6cefb..e850bc1d8e 100644
--- a/tools/hotplug/Linux/vif-openvswitch
+++ b/tools/hotplug/Linux/vif-openvswitch
@@ -47,11 +47,7 @@ openvswitch_external_id() {
openvswitch_external_id_all() {
local dev=$1
local frontend_id=$(xenstore_read "$XENBUS_PATH/frontend-id")
- local vm_path=$(xenstore_read "/local/domain/${frontend_id}/vm")
- local name=$(xenstore_read "${vm_path}/name")
- openvswitch_external_id $dev "xen-vm-name" "$name"
- local uuid=$(xenstore_read "${vm_path}/uuid")
- openvswitch_external_id $dev "xen-vm-uuid" "$uuid"
+ openvswitch_external_id $dev "xen-vm-domid" "$frontend_id"
local mac=$(xenstore_read "$XENBUS_PATH/mac")
openvswitch_external_id $dev "attached-mac" "$mac"
}
--
2.15.1
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
