[Xen-devel] Fwd: [PATCH] xen: handle TapdiskException when add device failed

[æä <yilikernel@xxxxxxxxx>]

From 2098e71261cf6a8b65524033f321d80a4f70c29d Mon Sep 17 00:00:00 2001
From: Yi Li <peteryili@xxxxxxxxxxx>
Date: Thu, 20 Mar 2014 19:55:10 +0800

Subject: [PATCH] xen: handle TapdiskException when add device failed

handle the TapdiskException when add device
and check the args when using xm block-attach
Signed-off-by: Yi Li <peteryili@xxxxxxxxxxx>
---
Âtools/python/xen/xend/XendDomainInfo.py | Â Â4 ++--
Âtools/python/xen/xm/main.py       |  12 ++++++++++++
Â2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/tools/python/xen/xend/XendDomainInfo.py b/tools/python/xen/xend/XendDomainInfo.py
index 8d4ff5c..2270ab1 100644
--- a/tools/python/xen/xend/XendDomainInfo.py
+++ b/tools/python/xen/xend/XendDomainInfo.py
@@ -65,7 +65,7 @@ from xen.xend.XendConstants import *
Âfrom xen.xend.XendAPIConstants import *
Âfrom xen.xend.XendCPUPool import XendCPUPool
Âfrom xen.xend.server.DevConstants import xenbusState
-from xen.xend.server.BlktapController import TapdiskController
+from xen.xend.server.BlktapController import TapdiskController, TapdiskException
Â

Âfrom xen.xend.XendVMMetrics import XendVMMetrics
Â
@@ -861,7 +861,7 @@ class XendDomainInfo:
          Â# blktap1
          Âdev_type = self.getBlockDeviceClass(devid)
        Âself._waitForDevice(dev_type, devid)
- Â Â Â Â Â Âexcept VmError, ex:
+ Â Â Â Â Â Âexcept (VmError, TapdiskException), ex:
        Âdel self.info['devices'][dev_uuid]
        Âif dev_type == 'pci':
          Âfor dev in dev_config_dict['devs']:
diff --git a/tools/python/xen/xm/main.py b/tools/python/xen/xm/main.py

index 5ed781c..ce5d9a2 100644

--- a/tools/python/xen/xm/main.py
+++ b/tools/python/xen/xm/main.py
@@ -2650,6 +2650,18 @@ def parse_block_configuration(args):
Âdef xm_block_attach(args):
  Âarg_check(args, 'block-attach', 4, 5)
Â

+ Â Âuname = args[1]
+ Â Âuname_list = uname.split(":")
+ Â Âback_dev = uname_list[len(uname_list)-1]
+ Â Âfront_dev = args[2]

+ Â Âif not os.path.exists(back_dev):
+ Â Â Â Âprint "Error: %s not exist " % back_dev
+ Â Â Â Âsys.exit(1)
+
+ Â Âif os.path.exists(front_dev):
+ Â Â Â Âprint "Error: %s already exists " % front_dev
+ Â Â Â Âsys.exit(1)
+
  Âif serverType == SERVER_XEN_API:
    Âdom  = args[0]
    Âuname = args[1]
--
1.7.1

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel