diff options
author | Olaf Hering <olaf@aepfle.de> | 2012-11-13 18:00:42 +0000 |
---|---|---|
committer | Olaf Hering <olaf@aepfle.de> | 2012-11-13 18:00:42 +0000 |
commit | 2eb37e5be47597435a017bf3513c0a1c322c44f2 (patch) | |
tree | 327d01f8ee2571ea0e2a9816bc6ad7d4adb8bb15 | |
parent | ec82f9fe73ab06600fcd02dafc89d497dba5f176 (diff) | |
download | xen-2eb37e5be47597435a017bf3513c0a1c322c44f2.tar.gz xen-2eb37e5be47597435a017bf3513c0a1c322c44f2.tar.bz2 xen-2eb37e5be47597435a017bf3513c0a1c322c44f2.zip |
xend/pvscsi: fix passing of SCSI control LUNs
Currently pvscsi can not pass SCSI devices that have just a scsi_generic node.
In the following example sg3 is a control LUN for the disk sdd.
But vscsi=['4:0:2:0,0:0:0:0'] does not work because the internal 'devname'
variable remains None. Later writing p-devname to xenstore fails because None
is not a valid string variable.
Since devname is used for just informational purpose use sg also as devname.
carron:~ $ lsscsi -g
[0:0:0:0] disk ATA FK0032CAAZP HPF2 /dev/sda /dev/sg0
[4:0:0:0] disk HP P2000G3 FC/iSCSI T100 /dev/sdb /dev/sg1
[4:0:1:0] disk HP P2000G3 FC/iSCSI T100 /dev/sdc /dev/sg2
[4:0:2:0] storage HP HSV400 0950 - /dev/sg3
[4:0:2:1] disk HP HSV400 0950 /dev/sdd /dev/sg4
[4:0:3:0] storage HP HSV400 0950 - /dev/sg5
[4:0:3:1] disk HP HSV400 0950 /dev/sde /dev/sg6
Signed-off-by: Olaf Hering <olaf@aepfle.de>
Acked-by: Ian Campbell <ian.campbell@citrix.com>
Committed-by: Ian Campbell <ian.campbell@citrix.com>
xen-unstable changeset: 26008:eecb528583d7
Backport-requested-by: Olaf Hering <olaf@aepfle.de>
Committed-by: Ian Jackson <ian.jackson@eu.citrix.com>
-rw-r--r-- | tools/python/xen/util/vscsi_util.py | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/tools/python/xen/util/vscsi_util.py b/tools/python/xen/util/vscsi_util.py index b73fc5834b..da09d46727 100644 --- a/tools/python/xen/util/vscsi_util.py +++ b/tools/python/xen/util/vscsi_util.py @@ -105,6 +105,8 @@ def _vscsi_get_scsidevices_by_lsscsi(option = ""): devname = None try: sg = s[-1].split('/dev/')[1] + if devname is None: + devname = sg scsi_id = _vscsi_get_scsiid(sg) except IndexError: sg = None |