From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 12A355A31 for ; Tue, 10 Feb 2015 17:54:37 +0100 (CET) Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by fmsmga103.fm.intel.com with ESMTP; 10 Feb 2015 08:47:34 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.09,551,1418112000"; d="scan'208";a="675829113" Received: from irsmsx102.ger.corp.intel.com ([163.33.3.155]) by fmsmga002.fm.intel.com with ESMTP; 10 Feb 2015 08:54:34 -0800 Received: from irsmsx108.ger.corp.intel.com ([169.254.11.64]) by IRSMSX102.ger.corp.intel.com ([169.254.2.69]) with mapi id 14.03.0195.001; Tue, 10 Feb 2015 16:54:34 +0000 From: "Iremonger, Bernard" To: "Qiu, Michael" , Tetsuya Mukawa , "dts@dpdk.org" Thread-Topic: [PATCH] test_plan: add hotplug_test_plan Thread-Index: AQHQROESSPmZsjvrL02oMBr5A1mZQ5zqGkPg Date: Tue, 10 Feb 2015 16:54:33 +0000 Message-ID: <8CEF83825BEC744B83065625E567D7C2049DF61E@IRSMSX108.ger.corp.intel.com> References: <1422866813-23729-1-git-send-email-michael.qiu@intel.com> <533710CFB86FA344BFBF2D6802E60286CE7206@SHSMSX101.ccr.corp.intel.com> <54D979EA.6010403@igel.co.jp> <533710CFB86FA344BFBF2D6802E60286CE7941@SHSMSX101.ccr.corp.intel.com> In-Reply-To: <533710CFB86FA344BFBF2D6802E60286CE7941@SHSMSX101.ccr.corp.intel.com> Accept-Language: en-GB, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [163.33.239.180] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-Mailman-Approved-At: Sun, 15 Feb 2015 08:10:27 +0100 Cc: "Iremonger, Bernard" Subject: Re: [dts] [PATCH] test_plan: add hotplug_test_plan 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: Tue, 10 Feb 2015 16:54:38 -0000 > -----Original Message----- > From: Qiu, Michael > Sent: Tuesday, February 10, 2015 8:55 AM > To: Tetsuya Mukawa; dts@dpdk.org > Cc: Iremonger, Bernard > Subject: Re: [PATCH] test_plan: add hotplug_test_plan >=20 > On 2/10/2015 11:24 AM, Tetsuya Mukawa wrote: > > Hi Michael, > > > > On 2015/02/09 22:13, Qiu, Michael wrote: > >> [+cc] Bernard > >> > >> Hi, Bernard and Tetsuya > >> > >> Would you have time to review this test plan? > >> Your comments are very important. > >> > >> Thanks, > >> Michael > >> On 2/2/2015 4:47 PM, Qiu, Michael wrote: >=20 > [...] >=20 > >> + > >> +Prerequisites > >> +------------- > >> +Assume DPDK managerd at least one device for physical or none for vir= tual. > > "managerd" may be typo. > > > >>> +This feature only supports igb_uio now, for uio_pci_generic is on > >>> +the way, will test it after enabled. > >>> + > >>> +To run the testpmd application in linuxapp environment with 4 > >>> +lcores, > >>> +4 channels with other default parameters in interactive mode. > >>> + > >>> + $ ./testpmd -c 0xf -n 4 -- -i > >>> + > >>> +Test ENV: > >>> + > >>> +All test case can be run in 32bit and 64bit platform. > >>> + > >>> +All os can be support, like fedora, ubuntu, rhel, suse, but freebsd > >>> +will not include as hotplug has no plan to support that platform > >>> + > >>> +All kernel version(start from 2.6) can be support, for vfio need ker= nel > >>> + version greater than 3.6. > >>> + > >>> +Virtulization support: KVM/VMware/Xen, container is in the roadmap > > "Virtulization" may be typo. > > > >>> + > >>> +------------------------------------------------------------------- > >>> +------------ Test Case 1: port dettach & attach for physical > >>> +devices with igb_uio > >>> +------------------------------------------------------------------- > >>> +------------ > >>> +1. run "port attach 0000:02:00.0", > >>> + assume the physical devices is BDF is 0000:02:00.0 > > Above sentence may be typo. >=20 > [...] >=20 > > Usually before submitting hotplug patches, I test almost same as > > above, and additionally below cases. > > > > --------------------------- > > Test Re-attach > > --------------------------- > > - testpmd> attach > > - testpmd> port start > > - testpmd> start > > - testpmd> stop > > - testpmd> port stop > > - testpmd> port close > > - testpmd> port detach > > - testpmd> port attach > > - testpmd> port start > > - testpmd> start > > - testpmd> stop > > (To check if re-attached port works fine.) >=20 > This will be added for every test case >=20 > > --------------------------- > > start testpmd with "--vdev" option, and detach. > > --------------------------- > > > > - ./testpmd -c 0xf -n 4 --vdev "eth_pcap0,iface=3D" -- -i > > - testpmd> start > > - testpmd> stop > > - testpmd> port stop > > - testpmd> port close > > - testpmd> port detach > > (To make sure if we can detach the port) > > > > --------------------------- > > start testpmd with an attached physical NIC port, and detach. > > --------------------------- > > - ./dpdk_nic_bind -b igb_uio > > - ./testpmd -c 0xf -n 4 -- -i > > - testpmd> start > > - testpmd> stop > > - testpmd> port stop > > - testpmd> port close > > - testpmd> port detach > > (To make sure if we can detach the port) >=20 > I will add both. >=20 > Thanks, > Michael > > How about adding such cases? > > > > Thanks, > > Tetsuya > > > > Hi Michael, Tetsuya, Should there be test cases added using pmd_null (eth_null0) ? Regards, Bernard.