libxl: Remove redundant setting of phyical-device
authorGeorge Dunlap <george.dunlap@citrix.com>
Thu, 24 Mar 2016 17:18:32 +0000 (17:18 +0000)
committerIan Jackson <Ian.Jackson@eu.citrix.com>
Mon, 4 Apr 2016 16:48:06 +0000 (17:48 +0100)
commite885362ccbec19321874abb74e2d317193f45ce5
tree7ca664f2190bdf51fd86f6dee759872210fcabf3
parent5806acf503887e2ffe82aba4465c2e9183d19055
libxl: Remove redundant setting of phyical-device

Regardless of whether we're running a custom hotplug script or using
normal phy: or file:, the "block" script will be run, which will set
all the necessary xenstore nodes.

In fact, writing this value here prevents the block script from
accomplishing its only purpose: to detect duplicate physical block
devices used in different virtual devices.  The first thing the block
script does is check to see if this node is written; and if it is, it
silently exits.

Remove this, and let the block script perform its duplicate checking
function.

NOTE: It's likely that the duplicate checking for physical devices has
never been run under libxl (at least since this bug was introduced);
this may shake out some issues.

Signed-off-by: George Dunlap <george.dunlap@citrix.com>
Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
Tested-by: George Dunlap <george.dunlap@citrix.com>
tools/libxl/libxl.c