Change in vdsm[master]: introducing virtio-scsi support

danken at redhat.com danken at redhat.com
Thu Jun 13 14:42:11 UTC 2013


Dan Kenigsberg has submitted this change and it was merged.

Change subject: introducing virtio-scsi support
......................................................................


introducing virtio-scsi support

Drive class:
* makeName - map 'scsi' to 'sd'
* getXML:
- added 'sgio' flag [1]
- device='lun' is used instead of device='disk' for supporting
  LUN passthrough (which can be used with virtio-scsi).
  Hence, adding 'driver' tag similarly to 'disk' device.

LibvirtVm class:
* getUnderlyingDriveInfo - get/set 'sgio' attribute.

TestLibvirtvm class:
* testControllerXML - test 'virtio-scsi' controller
* testDriveXML:
- test virtio-scsi image disk
- test virtio-scsi LUN passthrough

API Schema:
* Updated:
- VmDiskDeviceInterface
- VmDiskDeviceType
- VmControllerDevice

[1] 'sgio': SCSI Generic IO - filtered/unfiltered
(indicates whether the kernel will filter unprivileged
SG_IO commands for the disk).

Change-Id: If624064269328253dbc3cb32a72e20f01cdb0958
Signed-off-by: Daniel Erez <derez at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/14681
Reviewed-by: Federico Simoncelli <fsimonce at redhat.com>
---
M tests/libvirtvmTests.py
M vdsm/vm.py
M vdsm_api/vdsmapi-schema.json
3 files changed, 58 insertions(+), 9 deletions(-)

Approvals:
  Federico Simoncelli: Looks good to me, approved
  Daniel Erez: Verified
  Dan Kenigsberg: 


-- 
To view, visit http://gerrit.ovirt.org/14681
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If624064269328253dbc3cb32a72e20f01cdb0958
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Mark Wu <wudxw at linux.vnet.ibm.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list