Re: [libvirt] [PATCH v2 10/10] tests: tests for scsi hostdev

2013-04-05 Thread Han Cheng



On 04/03/2013 06:26 PM, Osier Yang wrote:

On 01/04/13 20:01, Han Cheng wrote:

diff --git a/tests/qemuhelpdata/qemu-1.0-device
b/tests/qemuhelpdata/qemu-1.0-device
index 0bdfbbd..d557f0e 100644
--- a/tests/qemuhelpdata/qemu-1.0-device
+++ b/tests/qemuhelpdata/qemu-1.0-device
@@ -136,3 +136,13 @@ virtio-net-pci.romfile=string
virtio-net-pci.rombar=uint32
virtio-net-pci.multifunction=on/off
virtio-net-pci.command_serr_enable=on/off
+scsi-generic.drive=drive
+scsi-generic.logical_block_size=uint16
+scsi-generic.physical_block_size=uint16
+scsi-generic.min_io_size=uint16
+scsi-generic.opt_io_size=uint32
+scsi-generic.bootindex=int32


I might be wrong, but what I got is bootindex showed up since
qemu 1.2.


I checked again. Compile from qemu v1.0 
(1c8a881daaca6fe0646a425b0970fb3ad25f6732). It is there.

Could you check it again?

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list


Re: [libvirt] [PATCH v2 10/10] tests: tests for scsi hostdev

2013-04-03 Thread Osier Yang

On 01/04/13 20:01, Han Cheng wrote:

This patch add tests for scsi hostdev.


s/add/adds/



Signed-off-by: Han Cheng hanc.f...@cn.fujitsu.com
---
  tests/qemuhelpdata/qemu-1.0-device |   10 ++
  tests/qemuhelpdata/qemu-1.1.0-device   |   10 ++
  tests/qemuhelpdata/qemu-1.2.0-device   |5 +++
  tests/qemuhelpdata/qemu-kvm-1.2.0-device   |5 +++
  tests/qemuhelptest.c   |   19 ---
  .../qemuxml2argv-hostdev-scsi-boot.args|   10 ++
  .../qemuxml2argv-hostdev-scsi-boot.xml |   34 +++
  .../qemuxml2argv-hostdev-scsi-readonly.args|   10 ++
  .../qemuxml2argv-hostdev-scsi-readonly.xml |   35 
  .../qemuxml2argv-hostdev-scsi.args |   10 ++
  .../qemuxml2argvdata/qemuxml2argv-hostdev-scsi.xml |   34 +++
  tests/qemuxml2argvtest.c   |   12 +++
  tests/qemuxml2xmltest.c|4 ++
  13 files changed, 193 insertions(+), 5 deletions(-)
  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-hostdev-scsi-boot.args
  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-hostdev-scsi-boot.xml
  create mode 100644 
tests/qemuxml2argvdata/qemuxml2argv-hostdev-scsi-readonly.args
  create mode 100644 
tests/qemuxml2argvdata/qemuxml2argv-hostdev-scsi-readonly.xml
  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-hostdev-scsi.args
  create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-hostdev-scsi.xml

diff --git a/tests/qemuhelpdata/qemu-1.0-device 
b/tests/qemuhelpdata/qemu-1.0-device
index 0bdfbbd..d557f0e 100644
--- a/tests/qemuhelpdata/qemu-1.0-device
+++ b/tests/qemuhelpdata/qemu-1.0-device
@@ -136,3 +136,13 @@ virtio-net-pci.romfile=string
  virtio-net-pci.rombar=uint32
  virtio-net-pci.multifunction=on/off
  virtio-net-pci.command_serr_enable=on/off
+scsi-generic.drive=drive
+scsi-generic.logical_block_size=uint16
+scsi-generic.physical_block_size=uint16
+scsi-generic.min_io_size=uint16
+scsi-generic.opt_io_size=uint32
+scsi-generic.bootindex=int32


I might be wrong, but what I got is bootindex showed up since
qemu 1.2.


+scsi-generic.discard_granularity=uint32
+scsi-generic.channel=uint32
+scsi-generic.scsi-id=uint32
+scsi-generic.lun=uint32
diff --git a/tests/qemuhelpdata/qemu-1.1.0-device 
b/tests/qemuhelpdata/qemu-1.1.0-device
index abbf850..7313a34 100644
--- a/tests/qemuhelpdata/qemu-1.1.0-device
+++ b/tests/qemuhelpdata/qemu-1.1.0-device
@@ -158,3 +158,13 @@ scsi-disk.dpofua=on/off
  scsi-disk.channel=uint32
  scsi-disk.scsi-id=uint32
  scsi-disk.lun=uint32
+scsi-generic.drive=drive
+scsi-generic.logical_block_size=blocksize
+scsi-generic.physical_block_size=blocksize
+scsi-generic.min_io_size=uint16
+scsi-generic.opt_io_size=uint32
+scsi-generic.bootindex=int32
+scsi-generic.discard_granularity=uint32
+scsi-generic.channel=uint32
+scsi-generic.scsi-id=uint32
+scsi-generic.lun=uint32
diff --git a/tests/qemuhelpdata/qemu-1.2.0-device 
b/tests/qemuhelpdata/qemu-1.2.0-device
index 5613e00..40845e4 100644
--- a/tests/qemuhelpdata/qemu-1.2.0-device
+++ b/tests/qemuhelpdata/qemu-1.2.0-device
@@ -208,3 +208,8 @@ usb-host.bootindex=int32
  usb-host.pipeline=on/off
  usb-host.port=string
  usb-host.full-path=on/off
+scsi-generic.drive=drive
+scsi-generic.bootindex=int32
+scsi-generic.channel=uint32
+scsi-generic.scsi-id=uint32
+scsi-generic.lun=uint32
diff --git a/tests/qemuhelpdata/qemu-kvm-1.2.0-device 
b/tests/qemuhelpdata/qemu-kvm-1.2.0-device
index 879a049..09e3ef7 100644
--- a/tests/qemuhelpdata/qemu-kvm-1.2.0-device
+++ b/tests/qemuhelpdata/qemu-kvm-1.2.0-device
@@ -220,3 +220,8 @@ usb-host.bootindex=int32
  usb-host.pipeline=on/off
  usb-host.port=string
  usb-host.full-path=on/off
+scsi-generic.drive=drive
+scsi-generic.bootindex=int32
+scsi-generic.channel=uint32
+scsi-generic.scsi-id=uint32
+scsi-generic.lun=uint32
diff --git a/tests/qemuhelptest.c b/tests/qemuhelptest.c
index a28109a..156be16 100644
--- a/tests/qemuhelptest.c
+++ b/tests/qemuhelptest.c
@@ -506,7 +506,8 @@ mymain(void)
  QEMU_CAPS_DEVICE_CIRRUS_VGA,
  QEMU_CAPS_DEVICE_VMWARE_SVGA,
  QEMU_CAPS_DEVICE_USB_SERIAL,
-QEMU_CAPS_DEVICE_USB_NET);
+QEMU_CAPS_DEVICE_USB_NET,
+QEMU_CAPS_SCSI_GENERIC);
  DO_TEST(qemu-kvm-0.12.1.2-rhel61, 12001, 1, 0,
  QEMU_CAPS_VNC_COLON,
  QEMU_CAPS_NO_REBOOT,
@@ -723,7 +724,9 @@ mymain(void)
  QEMU_CAPS_DEVICE_CIRRUS_VGA,
  QEMU_CAPS_DEVICE_VMWARE_SVGA,
  QEMU_CAPS_DEVICE_USB_SERIAL,
-QEMU_CAPS_DEVICE_USB_NET);
+QEMU_CAPS_DEVICE_USB_NET,
+QEMU_CAPS_SCSI_GENERIC,
+QEMU_CAPS_SCSI_GENERIC_BOOTINDEX);
  DO_TEST(qemu-1.1.0, 1001000, 0, 0,
  QEMU_CAPS_VNC_COLON,
  QEMU_CAPS_NO_REBOOT,
@@ -812,7 +815,9 @@ mymain(void)
  

[libvirt] [PATCH v2 10/10] tests: tests for scsi hostdev

2013-04-01 Thread Han Cheng
This patch add tests for scsi hostdev.

Signed-off-by: Han Cheng hanc.f...@cn.fujitsu.com
---
 tests/qemuhelpdata/qemu-1.0-device |   10 ++
 tests/qemuhelpdata/qemu-1.1.0-device   |   10 ++
 tests/qemuhelpdata/qemu-1.2.0-device   |5 +++
 tests/qemuhelpdata/qemu-kvm-1.2.0-device   |5 +++
 tests/qemuhelptest.c   |   19 ---
 .../qemuxml2argv-hostdev-scsi-boot.args|   10 ++
 .../qemuxml2argv-hostdev-scsi-boot.xml |   34 +++
 .../qemuxml2argv-hostdev-scsi-readonly.args|   10 ++
 .../qemuxml2argv-hostdev-scsi-readonly.xml |   35 
 .../qemuxml2argv-hostdev-scsi.args |   10 ++
 .../qemuxml2argvdata/qemuxml2argv-hostdev-scsi.xml |   34 +++
 tests/qemuxml2argvtest.c   |   12 +++
 tests/qemuxml2xmltest.c|4 ++
 13 files changed, 193 insertions(+), 5 deletions(-)
 create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-hostdev-scsi-boot.args
 create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-hostdev-scsi-boot.xml
 create mode 100644 
tests/qemuxml2argvdata/qemuxml2argv-hostdev-scsi-readonly.args
 create mode 100644 
tests/qemuxml2argvdata/qemuxml2argv-hostdev-scsi-readonly.xml
 create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-hostdev-scsi.args
 create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-hostdev-scsi.xml

diff --git a/tests/qemuhelpdata/qemu-1.0-device 
b/tests/qemuhelpdata/qemu-1.0-device
index 0bdfbbd..d557f0e 100644
--- a/tests/qemuhelpdata/qemu-1.0-device
+++ b/tests/qemuhelpdata/qemu-1.0-device
@@ -136,3 +136,13 @@ virtio-net-pci.romfile=string
 virtio-net-pci.rombar=uint32
 virtio-net-pci.multifunction=on/off
 virtio-net-pci.command_serr_enable=on/off
+scsi-generic.drive=drive
+scsi-generic.logical_block_size=uint16
+scsi-generic.physical_block_size=uint16
+scsi-generic.min_io_size=uint16
+scsi-generic.opt_io_size=uint32
+scsi-generic.bootindex=int32
+scsi-generic.discard_granularity=uint32
+scsi-generic.channel=uint32
+scsi-generic.scsi-id=uint32
+scsi-generic.lun=uint32
diff --git a/tests/qemuhelpdata/qemu-1.1.0-device 
b/tests/qemuhelpdata/qemu-1.1.0-device
index abbf850..7313a34 100644
--- a/tests/qemuhelpdata/qemu-1.1.0-device
+++ b/tests/qemuhelpdata/qemu-1.1.0-device
@@ -158,3 +158,13 @@ scsi-disk.dpofua=on/off
 scsi-disk.channel=uint32
 scsi-disk.scsi-id=uint32
 scsi-disk.lun=uint32
+scsi-generic.drive=drive
+scsi-generic.logical_block_size=blocksize
+scsi-generic.physical_block_size=blocksize
+scsi-generic.min_io_size=uint16
+scsi-generic.opt_io_size=uint32
+scsi-generic.bootindex=int32
+scsi-generic.discard_granularity=uint32
+scsi-generic.channel=uint32
+scsi-generic.scsi-id=uint32
+scsi-generic.lun=uint32
diff --git a/tests/qemuhelpdata/qemu-1.2.0-device 
b/tests/qemuhelpdata/qemu-1.2.0-device
index 5613e00..40845e4 100644
--- a/tests/qemuhelpdata/qemu-1.2.0-device
+++ b/tests/qemuhelpdata/qemu-1.2.0-device
@@ -208,3 +208,8 @@ usb-host.bootindex=int32
 usb-host.pipeline=on/off
 usb-host.port=string
 usb-host.full-path=on/off
+scsi-generic.drive=drive
+scsi-generic.bootindex=int32
+scsi-generic.channel=uint32
+scsi-generic.scsi-id=uint32
+scsi-generic.lun=uint32
diff --git a/tests/qemuhelpdata/qemu-kvm-1.2.0-device 
b/tests/qemuhelpdata/qemu-kvm-1.2.0-device
index 879a049..09e3ef7 100644
--- a/tests/qemuhelpdata/qemu-kvm-1.2.0-device
+++ b/tests/qemuhelpdata/qemu-kvm-1.2.0-device
@@ -220,3 +220,8 @@ usb-host.bootindex=int32
 usb-host.pipeline=on/off
 usb-host.port=string
 usb-host.full-path=on/off
+scsi-generic.drive=drive
+scsi-generic.bootindex=int32
+scsi-generic.channel=uint32
+scsi-generic.scsi-id=uint32
+scsi-generic.lun=uint32
diff --git a/tests/qemuhelptest.c b/tests/qemuhelptest.c
index a28109a..156be16 100644
--- a/tests/qemuhelptest.c
+++ b/tests/qemuhelptest.c
@@ -506,7 +506,8 @@ mymain(void)
 QEMU_CAPS_DEVICE_CIRRUS_VGA,
 QEMU_CAPS_DEVICE_VMWARE_SVGA,
 QEMU_CAPS_DEVICE_USB_SERIAL,
-QEMU_CAPS_DEVICE_USB_NET);
+QEMU_CAPS_DEVICE_USB_NET,
+QEMU_CAPS_SCSI_GENERIC);
 DO_TEST(qemu-kvm-0.12.1.2-rhel61, 12001, 1, 0,
 QEMU_CAPS_VNC_COLON,
 QEMU_CAPS_NO_REBOOT,
@@ -723,7 +724,9 @@ mymain(void)
 QEMU_CAPS_DEVICE_CIRRUS_VGA,
 QEMU_CAPS_DEVICE_VMWARE_SVGA,
 QEMU_CAPS_DEVICE_USB_SERIAL,
-QEMU_CAPS_DEVICE_USB_NET);
+QEMU_CAPS_DEVICE_USB_NET,
+QEMU_CAPS_SCSI_GENERIC,
+QEMU_CAPS_SCSI_GENERIC_BOOTINDEX);
 DO_TEST(qemu-1.1.0, 1001000, 0, 0,
 QEMU_CAPS_VNC_COLON,
 QEMU_CAPS_NO_REBOOT,
@@ -812,7 +815,9 @@ mymain(void)
 QEMU_CAPS_DEVICE_VMWARE_SVGA,
 QEMU_CAPS_DEVICE_USB_SERIAL,
 QEMU_CAPS_DEVICE_USB_NET,
-QEMU_CAPS_DTB);
+QEMU_CAPS_DTB,
+