From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by dpdk.org (Postfix) with ESMTP id 7A52B58FE for ; Mon, 17 Apr 2017 10:03:22 +0200 (CEST) Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by orsmga105.jf.intel.com with ESMTP; 17 Apr 2017 01:03:20 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.37,213,1488873600"; d="scan'208";a="1156969542" Received: from fmsmsx103.amr.corp.intel.com ([10.18.124.201]) by fmsmga002.fm.intel.com with ESMTP; 17 Apr 2017 01:03:19 -0700 Received: from fmsmsx125.amr.corp.intel.com (10.18.125.40) by FMSMSX103.amr.corp.intel.com (10.18.124.201) with Microsoft SMTP Server (TLS) id 14.3.319.2; Mon, 17 Apr 2017 01:03:19 -0700 Received: from shsmsx104.ccr.corp.intel.com (10.239.4.70) by FMSMSX125.amr.corp.intel.com (10.18.125.40) with Microsoft SMTP Server (TLS) id 14.3.319.2; Mon, 17 Apr 2017 01:03:19 -0700 Received: from shsmsx103.ccr.corp.intel.com ([169.254.4.117]) by SHSMSX104.ccr.corp.intel.com ([10.239.4.70]) with mapi id 14.03.0319.002; Mon, 17 Apr 2017 16:02:07 +0800 From: "Liu, Yong" To: Angela Czubak , "dts@dpdk.org" Thread-Topic: [dts] [PATCH v2 00/28] DTS support for Cavium NICs Thread-Index: AQHSsh1O9Kou4I1Cc0mjy2mpU2Ma4qHJOqnA Date: Mon, 17 Apr 2017 08:02:06 +0000 Message-ID: <86228AFD5BCD8E4EBFD2B90117B5E81E62DBE0F5@SHSMSX103.ccr.corp.intel.com> References: <1491501092-14278-1-git-send-email-aczubak@caviumnetworks.com> <1491844162-16300-1-git-send-email-aczubak@caviumnetworks.com> In-Reply-To: <1491844162-16300-1-git-send-email-aczubak@caviumnetworks.com> Accept-Language: zh-CN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 10.0.102.7 dlp-reaction: no-action 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 v2 00/28] DTS support for Cavium NICs 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, 17 Apr 2017 08:03:23 -0000 Angela, I would like to merge your patches. There're two patches look like the latt= er one will replace the previous one. I can merge others first or waiting for next version with these two patches= combined to one. Which choice is your preferred? [dts] [PATCH v2 02/28] framework dut and project_dpdk: using correct availa= ble binding scripts [dts] [PATCH v2 08/28] framework: getting DPDK binding script moved to a me= thod in dut Thanks, Marvin > -----Original Message----- > From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of Angela Czubak > Sent: Tuesday, April 11, 2017 1:09 AM > To: dts@dpdk.org > Cc: Angela Czubak > Subject: [dts] [PATCH v2 00/28] DTS support for Cavium NICs >=20 > Changes done to run DTS for Cavium NICs >=20 > 1) Cavium devices with 177d:a034 and 177d:0011 PCI IDs are added to NICs > list, both of which are supported by thunder-nicvf driver > 2) When appending pci devices info, the only Cavium NICs appended are the > ones with 10Gb/s linkspeed > 3) For arm64 architecture the hugepage size is detected and if it is > 524288, then lesser amount of hugepages is acquired > 4) Created a method in Dut class for getting the right binding script (if > someone chooses to use older DPDK version) > 5) Checking link is done via IPv4 ping alternately, so that DUT can be a > system without IPv6 (previously link was detected solely by checking if a= n > interface obtained an IPv6 address and then by using ping6). This also > makes force binding necessary, as those interfaces are detected as active= . > 6) Checking whether vfio-pci was loaded using lsmod can be deceptive. If > someone decides to use DPDK on a system with modules built into the kerne= l, > then this approach will fail. Instead, DPDK binding script can be used an= d > if it shows that vfio-pci can be used, DTS proceeds, otherwise it send > with an error due to a failed assertion. > 7) Modifying tests and config for Cavium NICs > 8) Added support for Cavium NICs to choose only cards with specified > linkspeed > 9) Skipping recompling DPDK in TestSuite_pmd.py if the driver is not mean= t > for FVL >=20 > Changes since initial version: > - Added support for Cavium NICs to choose only cards with specified > linkspeed > - Skipping recompling DPDK in TestSuite_pmd.py if the driver is not meant > for FVL > - Setting max-pkt-len before setting vlan stripping off in > TestSuite_shutdown_api.py when testing jumbo frames >=20 >=20 > Angela Czubak (28): > framework settings: Adding cavium PCI IDs and drivers > framework dut and project_dpdk: using correct available binding > scripts > framework dut and project_dpdk: binding smaller number of hugepages > for arm64 > framework crb: Appending only 10G devices for cavium > framework project_dpdk: asserting loading vfio-pci by using DPDK > binding script > framework: checking link with IPv4 ping > tests: ipv4_reassembly - added splitting result in tcpdump_command > framework: getting DPDK binding script moved to a method in dut > tests: Modified tests to take cavium nics into account > tests: checksum_offload - fixes > tests: dynamic_config - taking into account cavium NICs > tests: fdir - skipping fdir tests for Cavium NICs > tests: ieee1588 - skipping ieee1588_enable test for Cavium NICs > tests: jumboframes - skipping jumboframes tests testing bigger than > allowed frames for Cavium NICs > tests: l2fwd - fixes to run on two cards (same NIC) and expecting > "L2FWD: entering main loop" instead of "memory mapped" > tests: link_status_interrupt - skipping > test_link_status_interrupt_port_available for Cavium NICs > tests: link_status_interrupt - skipping tests verifying adding MAC > addresses for Cavium NICs > tests: pmdpcap - modifying the right config file + importing utils > after scapy > tests: pmdrssreta - skipping tests for Cavium NICs > tests: queue_start_stop - taking into account cavium NICs > tests: quota_watermark - skipping tests for Cavium NICs > tests: short_live - taking into account cavium NICs + fixes > tests: tso - taking into account cavium NICs > tests: uni_pkt - taking into account cavium NICs > tests: unit_tests_dump - taking into account cavium NICs > tests: unit_tests_lpm - asking for lesser amount of hugepages if they > are 524288kB > framework: Adding linkspeed parameter for Cavium NICs > tests: pmd - Recompiling DPDK only if that is relevant >=20 > conf/dpdk_support_test_case.xls | Bin 25600 -> 8704 bytes > conf/dpdk_test_case_checklist.xls | Bin 31232 -> 21504 bytes > doc/dts_gsg/config.rst | 4 + > framework/crb.py | 12 +- > framework/dts.py | 5 + > framework/dut.py | 52 ++++++--- > framework/pmd_output.py | 4 + > framework/project_dpdk.py | 18 ++- > framework/settings.py | 5 + > framework/test_capabilities.py | 9 ++ > framework/test_case.py | 7 +- > framework/tester.py | 11 ++ > nics/net_device.py | 8 +- > tests/TestSuite_checksum_offload.py | 38 +++---- > tests/TestSuite_dynamic_config.py | 10 +- > tests/TestSuite_ipv4_reassembly.py | 2 +- > tests/TestSuite_l2fwd.py | 4 +- > tests/TestSuite_pmd.py | 5 +- > tests/TestSuite_pmdpcap.py | 13 ++- > tests/TestSuite_queue_start_stop.py | 5 +- > tests/TestSuite_short_live.py | 44 ++++++-- > tests/TestSuite_shutdown_api.py | 31 ++--- > tests/TestSuite_tso.py | 10 +- > tests/TestSuite_uni_pkt.py | 219 ++++++++++++++++++++++++------= - > ----- > tests/TestSuite_unit_tests_dump.py | 17 ++- > tests/TestSuite_unit_tests_lpm.py | 9 +- > 26 files changed, 383 insertions(+), 159 deletions(-) > create mode 100644 framework/test_capabilities.py >=20 > -- > 2.7.4