From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by dpdk.org (Postfix) with ESMTP id 417752B94 for ; Mon, 19 Dec 2016 05:46:28 +0100 (CET) Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga102.fm.intel.com with ESMTP; 18 Dec 2016 20:46:27 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,372,1477983600"; d="scan'208";a="20037516" Received: from fmsmsx104.amr.corp.intel.com ([10.18.124.202]) by orsmga002.jf.intel.com with ESMTP; 18 Dec 2016 20:46:26 -0800 Received: from fmsmsx154.amr.corp.intel.com (10.18.116.70) by fmsmsx104.amr.corp.intel.com (10.18.124.202) with Microsoft SMTP Server (TLS) id 14.3.248.2; Sun, 18 Dec 2016 20:46:26 -0800 Received: from shsmsx103.ccr.corp.intel.com (10.239.4.69) by FMSMSX154.amr.corp.intel.com (10.18.116.70) with Microsoft SMTP Server (TLS) id 14.3.248.2; Sun, 18 Dec 2016 20:46:25 -0800 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.54]) by SHSMSX103.ccr.corp.intel.com ([169.254.4.11]) with mapi id 14.03.0248.002; Mon, 19 Dec 2016 12:46:23 +0800 From: "Yao, Lei A" To: "Liu, Yong" , "dts@dpdk.org" Thread-Topic: [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 Thread-Index: AQHSWaPv28EROBSphUKgUrUVzsD3JqEOmV8g//9+jQCAAJox4A== Date: Mon, 19 Dec 2016 04:46:23 +0000 Message-ID: <2DBBFF226F7CF64BAFCA79B681719D953A135FA5@shsmsx102.ccr.corp.intel.com> References: <1481874899-4510-1-git-send-email-lei.a.yao@intel.com> <86228AFD5BCD8E4EBFD2B90117B5E81E62D1A500@SHSMSX103.ccr.corp.intel.com> <2DBBFF226F7CF64BAFCA79B681719D953A135F42@shsmsx102.ccr.corp.intel.com> <86228AFD5BCD8E4EBFD2B90117B5E81E62D1A5A7@SHSMSX103.ccr.corp.intel.com> In-Reply-To: <86228AFD5BCD8E4EBFD2B90117B5E81E62D1A5A7@SHSMSX103.ccr.corp.intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 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 X-BeenThere: dts@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: test suite reviews and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 19 Dec 2016 04:46:29 -0000 Understand. I will add more introduction into the vhost_sample.cfg in the V= 2 patch. Thanks. BRs Lei > -----Original Message----- > From: Liu, Yong > Sent: Monday, December 19, 2016 11:33 AM > To: Yao, Lei A ; dts@dpdk.org > 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 >=20 > Lei, I known only few cases use these parameters. What I mean is that to = add > some comments for these parameters. > There's one document doc/dts_gsg/virtualization.rst which is the referenc= e > for qemu configure parameters. > But modified this doc need more effort, so which I recommend is that just > add the comments in one configuration file which contain all parameters. > It can help other users for understand how to use these parameters. >=20 > Thanks, > Marvin >=20 > > -----Original Message----- > > From: Yao, Lei A > > Sent: Monday, December 19, 2016 11:20 AM > > To: Liu, Yong; dts@dpdk.org > > 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, 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 .c= fg > > 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 ; dts@dpdk.org > > > Cc: Yao, Lei A > > > 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 > > >