[dts] [PATCH V1] framework qemu_kvm.py: Add "server" and "multi_queue" parameter for more qemu launch option and adjust some timeout setting to improve the test stability
Yao, Lei A
lei.a.yao at intel.com
Mon Dec 19 04:19:56 CET 2016
Hi, Yong
Only part test cases will use this parameter, so I don't want add it to the .cfg file but keep them in the tests file. Then I can use common .cfg file to different vhost/virtio test cases.
For the comment, I can add most info in the commit log in the V2 patch. Thanks
BRs
Lei
> -----Original Message-----
> From: Liu, Yong
> Sent: Monday, December 19, 2016 11:00 AM
> To: Yao, Lei A <lei.a.yao at intel.com>; dts at dpdk.org
> Cc: Yao, Lei A <lei.a.yao at intel.com>
> Subject: RE: [dts] [PATCH V1] framework qemu_kvm.py: Add "server" and
> "multi_queue" parameter for more qemu launch option and adjust some
> timeout setting to improve the test stability
>
> Hi Lei,
> Both "server" and "multi_queue" are new parameters for qemu hypervisor
> library.
> You'd better add some reference in configuration file like sriov_kvm.cfg.
>
> Please also give some basic descriptions for these two parameters in commit
> log.
>
> Thanks,
> Marvin
>
> > -----Original Message-----
> > From: dts [mailto:dts-bounces at dpdk.org] On Behalf Of Yao Lei
> > Sent: Friday, December 16, 2016 3:55 PM
> > To: dts at dpdk.org
> > Cc: Yao, Lei A
> > Subject: [dts] [PATCH V1] framework qemu_kvm.py: Add "server" and
> > "multi_queue" parameter for more qemu launch option and adjust some
> > timeout setting to improve the test stability
> >
> > From: lei yao <lei.a.yao at intel.com>
> >
> > Signed-off-by: lei yao <lei.a.yao at intel.com>
> > ---
> > framework/qemu_kvm.py | 24 +++++++++++++++++-------
> > 1 file changed, 17 insertions(+), 7 deletions(-)
> >
> > diff --git a/framework/qemu_kvm.py b/framework/qemu_kvm.py
> > index 6ced54a..e9d29cc 100644
> > --- a/framework/qemu_kvm.py
> > +++ b/framework/qemu_kvm.py
> > @@ -606,21 +606,31 @@ class QEMUKvm(VirtBase):
> > if 'opt_path' in options.keys() and options['opt_path']:
> > dev_boot_line = '-chardev socket'
> > char_id = 'char%d' % self.char_idx
> > - dev_boot_line += separator + 'id=%s' % char_id + separator +
> > 'path=%s' % options['opt_path']
> > - self.char_idx += 1
> > - self.__add_boot_line(dev_boot_line)
> > + if 'opt_server' in options.keys() and options['opt_server']:
> > + dev_boot_line += separator + 'id=%s' % char_id +
> > separator + 'path=%s' %options['opt_path'] + separator + '%s' %
> > options['opt_server']
> > + self.char_idx += 1
> > + self.__add_boot_line(dev_boot_line)
> > + else:
> > + dev_boot_line += separator + 'id=%s' % char_id +
> > separator + 'path=%s' %options['opt_path']
> > + self.char_idx += 1
> > + self.__add_boot_line(dev_boot_line)
> > # netdev parameter
> > netdev_id = 'netdev%d' % self.netdev_idx
> > self.netdev_idx += 1
> > - dev_boot_line = '-netdev type=vhost-
> > user,id=%s,chardev=%s,vhostforce' % (netdev_id, char_id)
> > + if 'opt_queue' in options.keys() and options['opt_queue']:
> > + queue_num=options['opt_queue']
> > + dev_boot_line = '-netdev type=vhost-
> > user,id=%s,chardev=%s,vhostforce,queues=%s' % (netdev_id,
> > char_id,queue_num)
> > + else:
> > + dev_boot_line = '-netdev type=vhost-
> > user,id=%s,chardev=%s,vhostforce' % (netdev_id, char_id)
> > self.__add_boot_line(dev_boot_line)
> > # device parameter
> > opts = {'opt_netdev': '%s' % netdev_id}
> > if 'opt_mac' in options.keys() and \
> > options['opt_mac']:
> > opts['opt_mac'] = options['opt_mac']
> > -
> > - self.__add_vm_virtio_net_pci(**opts)
> > + if 'opt_settings' in options.keys() and
> > options['opt_settings']:
> > + opts['opt_settings'] = options['opt_settings']
> > + self.__add_vm_virtio_net_pci(**opts)
> >
> > def __add_vm_virtio_cuse_pci(self, **options):
> > """
> > @@ -939,7 +949,7 @@ class QEMUKvm(VirtBase):
> > wait for 120 seconds for vm net ready
> > 10.0.2.* is the default ip address allocated by qemu
> > """
> > - count = 20
> > + count = 40
> > while count:
> > out = self.__control_session('ifconfig')
> > if "10.0.2" in out:
> > --
> > 2.7.4
More information about the dts
mailing list