All of lore.kernel.org
 help / color / mirror / Atom feed
* [patch] xm: Add missing AUTO_PHP_SLOT
@ 2009-04-05 22:50 Simon Horman
  0 siblings, 0 replies; only message in thread
From: Simon Horman @ 2009-04-05 22:50 UTC (permalink / raw)
  To: xen-devel; +Cc: Ian Jackson, Keir Fraser, Cui, Dexuan

This patch changes over a few sections of code that
I missed when adding AUTO_PHP_SLOT to signify that
qemu-xen should pick a slot - previously 0 was used.

I have not experienced any breakage as a result of these
changes being missing, nor am I even sure that is possible.
But it seems worthwhile to clean things up in this way for 3.4.
And possibly remove the (spurious?) checks afterwards.

Signed-off-by: Simon Horman <horms@vereg.net.au>

Index: xen-unstable.hg/tools/python/xen/xend/XendDomainInfo.py
===================================================================
--- xen-unstable.hg.orig/tools/python/xen/xend/XendDomainInfo.py	2009-04-06 08:20:31.000000000 +1000
+++ xen-unstable.hg/tools/python/xen/xend/XendDomainInfo.py	2009-04-06 08:42:01.000000000 +1000
@@ -619,7 +619,7 @@ class XendDomainInfo:
             pci_devs = pci_conf['devs']
             for x in pci_devs:
                 if (int(x['vslt'], 16) == int(new_dev['vslt'], 16) and
-                   int(x['vslt'], 16) != 0 ):
+                   int(x['vslt'], 16) != AUTO_PHP_SLOT):
                     raise VmError("vslot %s already have a device." % (new_dev['vslt']))
 
                 if (int(x['domain'], 16) == int(new_dev['domain'], 16) and
@@ -1045,7 +1045,7 @@ class XendDomainInfo:
         if devnum >= pci_len:
             raise VmError("Device @ vslot 0x%x doesn't exist." % (vslot))
 
-        if vslot == 0:
+        if vslot == AUTO_PHP_SLOT:
             raise VmError("Device @ vslot 0x%x do not support hotplug." % (vslot))
 
         # Check the co-assignment.

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2009-04-05 22:50 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-04-05 22:50 [patch] xm: Add missing AUTO_PHP_SLOT Simon Horman

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.