Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/104684/?format=api
http://patchwork.dpdk.org/api/patches/104684/?format=api", "web_url": "http://patchwork.dpdk.org/project/dts/patch/20211125131311.134679-2-linglix.chen@intel.com/", "project": { "id": 3, "url": "http://patchwork.dpdk.org/api/projects/3/?format=api", "name": "DTS", "link_name": "dts", "list_id": "dts.dpdk.org", "list_email": "dts@dpdk.org", "web_url": "", "scm_url": "git://dpdk.org/tools/dts", "webscm_url": "http://git.dpdk.org/tools/dts/", "list_archive_url": "https://inbox.dpdk.org/dts", "list_archive_url_format": "https://inbox.dpdk.org/dts/{}", "commit_url_format": "" }, "msgid": "<20211125131311.134679-2-linglix.chen@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dts/20211125131311.134679-2-linglix.chen@intel.com", "date": "2021-11-25T13:13:10", "name": "[V1,1/2] tests/*: Move Intel IOAT devices from Misc to DMA devices", "commit_ref": null, "pull_url": null, "state": "accepted", "archived": false, "hash": "68376b34b895904f829b2241a0421d23be5cd65f", "submitter": { "id": 1843, "url": "http://patchwork.dpdk.org/api/people/1843/?format=api", "name": "Lingli Chen", "email": "linglix.chen@intel.com" }, "delegate": null, "mbox": "http://patchwork.dpdk.org/project/dts/patch/20211125131311.134679-2-linglix.chen@intel.com/mbox/", "series": [ { "id": 20763, "url": "http://patchwork.dpdk.org/api/series/20763/?format=api", "web_url": "http://patchwork.dpdk.org/project/dts/list/?series=20763", "date": "2021-11-25T13:13:09", "name": "move ioat device IDs to DMA class: change misc to dma.", "version": 1, "mbox": "http://patchwork.dpdk.org/series/20763/mbox/" } ], "comments": "http://patchwork.dpdk.org/api/patches/104684/comments/", "check": "pending", "checks": "http://patchwork.dpdk.org/api/patches/104684/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dts-bounces@dpdk.org>", "X-Original-To": "patchwork@inbox.dpdk.org", "Delivered-To": "patchwork@inbox.dpdk.org", "Received": [ "from mails.dpdk.org (mails.dpdk.org [217.70.189.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id 47636A0C4B;\n\tThu, 25 Nov 2021 06:12:38 +0100 (CET)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 3C2D0410F7;\n\tThu, 25 Nov 2021 06:12:38 +0100 (CET)", "from mga05.intel.com (mga05.intel.com [192.55.52.43])\n by mails.dpdk.org (Postfix) with ESMTP id CBA5140140\n for <dts@dpdk.org>; Thu, 25 Nov 2021 06:12:33 +0100 (CET)", "from fmsmga005.fm.intel.com ([10.253.24.32])\n by fmsmga105.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 24 Nov 2021 21:12:33 -0800", "from unknown (HELO dpdk.lan) ([10.240.183.77])\n by fmsmga005-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 24 Nov 2021 21:12:32 -0800" ], "X-IronPort-AV": [ "E=McAfee;i=\"6200,9189,10178\"; a=\"321679156\"", "E=Sophos;i=\"5.87,262,1631602800\"; d=\"scan'208\";a=\"321679156\"", "E=Sophos;i=\"5.87,262,1631602800\"; d=\"scan'208\";a=\"741061172\"" ], "From": "Lingli Chen <linglix.chen@intel.com>", "To": "dts@dpdk.org", "Cc": "Lingli Chen <linglix.chen@intel.com>", "Subject": "[dts][PATCH V1 1/2] tests/*: Move Intel IOAT devices from Misc to DMA\n devices", "Date": "Thu, 25 Nov 2021 13:13:10 +0000", "Message-Id": "<20211125131311.134679-2-linglix.chen@intel.com>", "X-Mailer": "git-send-email 2.33.1", "In-Reply-To": "<20211125131311.134679-1-linglix.chen@intel.com>", "References": "<20211125131311.134679-1-linglix.chen@intel.com>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "X-BeenThere": "dts@dpdk.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "test suite reviews and discussions <dts.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dts>,\n <mailto:dts-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://mails.dpdk.org/archives/dts/>", "List-Post": "<mailto:dts@dpdk.org>", "List-Help": "<mailto:dts-request@dpdk.org?subject=help>", "List-Subscribe": "<https://mails.dpdk.org/listinfo/dts>,\n <mailto:dts-request@dpdk.org?subject=subscribe>", "Errors-To": "dts-bounces@dpdk.org" }, "content": "1. According to dpdk commit e6f6dda60a5f(\"usertools/devbind: move ioat device IDs to DMA class\") change misc to dma.\n2. Modify re.search('\\s*(0000:\\S*:\\d*.\\d*)', device) for some DMA\ndevices_info contains letter.\n3. Cbdma only tests vfio-pci from 21.11, so remove igb_uio.\n\nSigned-off-by: Lingli Chen <linglix.chen@intel.com>\n---\n tests/TestSuite_cbdma.py | 4 +---\n tests/TestSuite_dpdk_gro_lib.py | 8 ++------\n tests/TestSuite_vhost_cbdma.py | 7 ++-----\n tests/TestSuite_vhost_event_idx_interrupt.py | 7 ++-----\n tests/TestSuite_vhost_user_interrupt.py | 8 ++------\n tests/TestSuite_vhost_virtio_pmd_interrupt.py | 7 ++-----\n tests/TestSuite_vhost_virtio_user_interrupt.py | 8 ++------\n tests/TestSuite_virtio_event_idx_interrupt.py | 7 ++-----\n tests/TestSuite_vm2vm_virtio_net_perf.py | 14 +++++---------\n tests/TestSuite_vm2vm_virtio_pmd.py | 6 +-----\n tests/TestSuite_vm2vm_virtio_user.py | 8 ++------\n tests/TestSuite_vswitch_sample_cbdma.py | 6 +-----\n 12 files changed, 24 insertions(+), 66 deletions(-)", "diff": "diff --git a/tests/TestSuite_cbdma.py b/tests/TestSuite_cbdma.py\nindex b33290f7..08dc48af 100644\n--- a/tests/TestSuite_cbdma.py\n+++ b/tests/TestSuite_cbdma.py\n@@ -91,12 +91,10 @@ class TestCBDMA(TestCase):\n \"\"\"\n get all cbdma ports\n \"\"\"\n- # check driver name in execution.cfg\n- str_info = 'Misc (rawdev) devices using kernel driver'\n out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev dma', '# ', 30)\n device_info = out.split('\\n')\n for device in device_info:\n- pci_info = re.search('\\s*(0000:\\d*:\\d*.\\d*)', device)\n+ pci_info = re.search('\\s*(0000:\\S*:\\d*.\\d*)', device)\n if pci_info is not None:\n dev_info = pci_info.group(1)\n # the numa id of dma dev, only add the device which\ndiff --git a/tests/TestSuite_dpdk_gro_lib.py b/tests/TestSuite_dpdk_gro_lib.py\nindex e11bfd32..c1bfd936 100644\n--- a/tests/TestSuite_dpdk_gro_lib.py\n+++ b/tests/TestSuite_dpdk_gro_lib.py\n@@ -114,14 +114,10 @@ class TestDPDKGROLib(TestCase):\n \"\"\"\n get all cbdma ports\n \"\"\"\n- # check driver name in execution.cfg\n- self.verify(self.drivername == 'igb_uio',\n- \"CBDMA test case only use igb_uio driver, need config drivername=igb_uio in execution.cfg\")\n- str_info = 'Misc (rawdev) devices using kernel driver'\n- out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev misc', '# ', 30)\n+ out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev dma', '# ', 30)\n device_info = out.split('\\n')\n for device in device_info:\n- pci_info = re.search('\\s*(0000:\\d*:\\d*.\\d*)', device)\n+ pci_info = re.search('\\s*(0000:\\S*:\\d*.\\d*)', device)\n if pci_info is not None:\n dev_info = pci_info.group(1)\n # the numa id of ioat dev, only add the device which\ndiff --git a/tests/TestSuite_vhost_cbdma.py b/tests/TestSuite_vhost_cbdma.py\nindex 5ebd734f..c0cc772d 100644\n--- a/tests/TestSuite_vhost_cbdma.py\n+++ b/tests/TestSuite_vhost_cbdma.py\n@@ -114,13 +114,10 @@ class TestVirTioVhostCbdma(TestCase):\n \"\"\"\n get all cbdma ports\n \"\"\"\n- # check driver name in execution.cfg\n- self.verify(self.drivername == 'igb_uio',\n- \"CBDMA test case only use igb_uio driver, need config drivername=igb_uio in execution.cfg\")\n- out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev misc', '# ', 30)\n+ out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev dma', '# ', 30)\n device_info = out.split('\\n')\n for device in device_info:\n- pci_info = re.search('\\s*(0000:\\d*:\\d*.\\d*)', device)\n+ pci_info = re.search('\\s*(0000:\\S*:\\d*.\\d*)', device)\n if pci_info is not None:\n dev_info = pci_info.group(1)\n # the numa id of ioat dev, only add the device which\ndiff --git a/tests/TestSuite_vhost_event_idx_interrupt.py b/tests/TestSuite_vhost_event_idx_interrupt.py\nindex 2d79f6b7..ef666b8a 100644\n--- a/tests/TestSuite_vhost_event_idx_interrupt.py\n+++ b/tests/TestSuite_vhost_event_idx_interrupt.py\n@@ -281,14 +281,11 @@ class TestVhostEventIdxInterrupt(TestCase):\n \"\"\"\n get all cbdma ports\n \"\"\"\n- # check driver name in execution.cfg\n- self.verify(self.drivername == 'igb_uio',\n- \"CBDMA test case only use igb_uio driver, need config drivername=igb_uio in execution.cfg\")\n self.cbdma_dev_infos = []\n- out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev misc', '# ', 30)\n+ out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev dma', '# ', 30)\n device_info = out.split('\\n')\n for device in device_info:\n- pci_info = re.search('\\s*(0000:\\d*:\\d*.\\d*)', device)\n+ pci_info = re.search('\\s*(0000:\\S*:\\d*.\\d*)', device)\n if pci_info is not None:\n # dev_info = pci_info.group(1)\n # the numa id of ioat dev, only add the device which\ndiff --git a/tests/TestSuite_vhost_user_interrupt.py b/tests/TestSuite_vhost_user_interrupt.py\nindex c0d1b4f2..d8c7b043 100644\n--- a/tests/TestSuite_vhost_user_interrupt.py\n+++ b/tests/TestSuite_vhost_user_interrupt.py\n@@ -112,14 +112,10 @@ class TestVhostUserInterrupt(TestCase):\n \"\"\"\n get all cbdma ports\n \"\"\"\n- # check driver name in execution.cfg\n- self.verify(self.drivername == 'igb_uio',\n- \"CBDMA test case only use igb_uio driver, need config drivername=igb_uio in execution.cfg\")\n- str_info = 'Misc (rawdev) devices using kernel driver'\n- out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev misc', '# ', 30)\n+ out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev dma', '# ', 30)\n device_info = out.split('\\n')\n for device in device_info:\n- pci_info = re.search('\\s*(0000:\\d*:\\d*.\\d*)', device)\n+ pci_info = re.search('\\s*(0000:\\S*:\\d*.\\d*)', device)\n if pci_info is not None:\n dev_info = pci_info.group(1)\n # the numa id of ioat dev, only add the device which\ndiff --git a/tests/TestSuite_vhost_virtio_pmd_interrupt.py b/tests/TestSuite_vhost_virtio_pmd_interrupt.py\nindex 504266f8..b9798393 100644\n--- a/tests/TestSuite_vhost_virtio_pmd_interrupt.py\n+++ b/tests/TestSuite_vhost_virtio_pmd_interrupt.py\n@@ -268,11 +268,8 @@ class TestVhostVirtioPmdInterrupt(TestCase):\n \"\"\"\n get all cbdma ports\n \"\"\"\n- # check driver name in execution.cfg\n- self.verify(self.drivername == 'igb_uio',\n- \"CBDMA test case only use igb_uio driver, need config drivername=igb_uio in execution.cfg\")\n- out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev misc', '# ', 30)\n- cbdma_dev_infos = re.findall('\\s*(0000:\\d+:\\d+.\\d+)', out)\n+ out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev dma', '# ', 30)\n+ cbdma_dev_infos = re.findall('\\s*(0000:\\S+:\\d+.\\d+)', out)\n self.verify(len(cbdma_dev_infos) >= cbdma_num, 'There no enough cbdma device to run this suite')\n \n used_cbdma = cbdma_dev_infos[0:cbdma_num]\ndiff --git a/tests/TestSuite_vhost_virtio_user_interrupt.py b/tests/TestSuite_vhost_virtio_user_interrupt.py\nindex 4385681b..8c6e263f 100644\n--- a/tests/TestSuite_vhost_virtio_user_interrupt.py\n+++ b/tests/TestSuite_vhost_virtio_user_interrupt.py\n@@ -185,14 +185,10 @@ class TestVirtioUserInterrupt(TestCase):\n \"\"\"\n get all cbdma ports\n \"\"\"\n- # check driver name in execution.cfg\n- self.verify(self.drivername == 'igb_uio',\n- \"CBDMA test case only use igb_uio driver, need config drivername=igb_uio in execution.cfg\")\n- str_info = 'Misc (rawdev) devices using kernel driver'\n- out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev misc', '# ', 30)\n+ out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev dma', '# ', 30)\n device_info = out.split('\\n')\n for device in device_info:\n- pci_info = re.search('\\s*(0000:\\d*:\\d*.\\d*)', device)\n+ pci_info = re.search('\\s*(0000:\\S*:\\d*.\\d*)', device)\n if pci_info is not None:\n dev_info = pci_info.group(1)\n # the numa id of ioat dev, only add the device which\ndiff --git a/tests/TestSuite_virtio_event_idx_interrupt.py b/tests/TestSuite_virtio_event_idx_interrupt.py\nindex 2a1f7b26..40e8bb1e 100644\n--- a/tests/TestSuite_virtio_event_idx_interrupt.py\n+++ b/tests/TestSuite_virtio_event_idx_interrupt.py\n@@ -89,11 +89,8 @@ class TestVirtioIdxInterrupt(TestCase):\n \"\"\"\n get all cbdma ports\n \"\"\"\n- # check driver name in execution.cfg\n- self.verify(self.drivername == 'igb_uio',\n- \"CBDMA test case only use igb_uio driver, need config drivername=igb_uio in execution.cfg\")\n- out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev misc', '# ', 30)\n- cbdma_dev_infos = re.findall('\\s*(0000:\\d+:\\d+.\\d+)', out)\n+ out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev dma', '# ', 30)\n+ cbdma_dev_infos = re.findall('\\s*(0000:\\S+:\\d+.\\d+)', out)\n self.verify(len(cbdma_dev_infos) >= cbdma_num, 'There no enough cbdma device to run this suite')\n used_cbdma = cbdma_dev_infos[0:cbdma_num]\n dmas_info = ''\ndiff --git a/tests/TestSuite_vm2vm_virtio_net_perf.py b/tests/TestSuite_vm2vm_virtio_net_perf.py\nindex 8c4b4ff2..dc8eaed5 100644\n--- a/tests/TestSuite_vm2vm_virtio_net_perf.py\n+++ b/tests/TestSuite_vm2vm_virtio_net_perf.py\n@@ -87,14 +87,10 @@ class TestVM2VMVirtioNetPerf(TestCase):\n \"\"\"\n get all cbdma ports\n \"\"\"\n- # check driver name in execution.cfg\n- self.verify(self.drivername == 'igb_uio',\n- \"CBDMA test case only use igb_uio driver, need config drivername=igb_uio in execution.cfg\")\n- str_info = 'Misc (rawdev) devices using kernel driver'\n- out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev misc', '# ', 30)\n+ out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev dma', '# ', 30)\n device_info = out.split('\\n')\n for device in device_info:\n- pci_info = re.search('\\s*(0000:\\d*:\\d*.\\d*)', device)\n+ pci_info = re.search('\\s*(0000:\\S*:\\d*.\\d*)', device)\n if pci_info is not None:\n dev_info = pci_info.group(1)\n # the numa id of ioat dev, only add the device which on same socket with nic dev\n@@ -113,11 +109,11 @@ class TestVM2VMVirtioNetPerf(TestCase):\n dmas_info = ''\n for dmas in used_cbdma[0:int(cbdma_num/2)]:\n number = used_cbdma[0:int(cbdma_num/2)].index(dmas)\n- dmas = 'txq{}@{},'.format(number, dmas.replace('0000:', ''))\n+ dmas = 'txq{}@{},'.format(number, dmas)\n dmas_info += dmas\n for dmas in used_cbdma[int(cbdma_num/2):]:\n number = used_cbdma[int(cbdma_num/2):].index(dmas)\n- dmas = 'txq{}@{},'.format(number, dmas.replace('0000:', ''))\n+ dmas = 'txq{}@{},'.format(number, dmas)\n dmas_info += dmas\n self.dmas_info = dmas_info[:-1]\n self.device_str = ' '.join(used_cbdma)\n@@ -476,7 +472,7 @@ class TestVM2VMVirtioNetPerf(TestCase):\n self.vm_args = \"disable-modern=false,mrg_rxbuf=on,csum=on,guest_csum=on,host_tso4=on,guest_tso4=on,guest_ecn=on,packed=on\"\n self.prepare_test_env(cbdma=True, no_pci=False, client_mode=False, enable_queues=1, nb_cores=2,\n server_mode=False, opt_queue=None, combined=False, rxq_txq=None)\n- self.start_iperf_and_verify_vhost_xstats_info(iperf_mode='ufo')\n+ self.start_iperf_and_verify_vhost_xstats_info(iperf_mode='tso')\n \n def test_vm2vm_packed_ring_iperf_with_ufo(self):\n \"\"\"\ndiff --git a/tests/TestSuite_vm2vm_virtio_pmd.py b/tests/TestSuite_vm2vm_virtio_pmd.py\nindex cbb0321c..1ddf571b 100644\n--- a/tests/TestSuite_vm2vm_virtio_pmd.py\n+++ b/tests/TestSuite_vm2vm_virtio_pmd.py\n@@ -676,11 +676,7 @@ class TestVM2VMVirtioPMD(TestCase):\n \"\"\"\n get all cbdma ports\n \"\"\"\n- # check driver name in execution.cfg\n- self.verify(self.drivername == 'igb_uio',\n- \"this case use igb_uio driver, need config drivername=igb_uio in execution.cfg\")\n- str_info = 'Misc (rawdev) devices using kernel driver'\n- out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev misc', '# ', 30)\n+ out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev dma', '# ', 30)\n device_info = out.split('\\n')\n for device in device_info:\n pci_info = re.search('\\s*(0000:\\S*:\\d*.\\d*)', device)\ndiff --git a/tests/TestSuite_vm2vm_virtio_user.py b/tests/TestSuite_vm2vm_virtio_user.py\nindex 4b868b32..2b66af77 100644\n--- a/tests/TestSuite_vm2vm_virtio_user.py\n+++ b/tests/TestSuite_vm2vm_virtio_user.py\n@@ -709,14 +709,10 @@ class TestVM2VMVirtioUser(TestCase):\n \"\"\"\n get all cbdma ports\n \"\"\"\n- # check driver name in execution.cfg\n- self.verify(self.drivername == 'igb_uio',\n- \"CBDMA test case only use igb_uio driver, need config drivername=igb_uio in execution.cfg\")\n- str_info = 'Misc (rawdev) devices using kernel driver'\n- out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev misc', '# ', 30)\n+ out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev dma', '# ', 30)\n device_info = out.split('\\n')\n for device in device_info:\n- pci_info = re.search('\\s*(0000:\\d*:\\d*.\\d*)', device)\n+ pci_info = re.search('\\s*(0000:\\S*:\\d*.\\d*)', device)\n if pci_info is not None:\n dev_info = pci_info.group(1)\n # the numa id of ioat dev, only add the device which\ndiff --git a/tests/TestSuite_vswitch_sample_cbdma.py b/tests/TestSuite_vswitch_sample_cbdma.py\nindex 81aede81..bd41135b 100644\n--- a/tests/TestSuite_vswitch_sample_cbdma.py\n+++ b/tests/TestSuite_vswitch_sample_cbdma.py\n@@ -235,11 +235,7 @@ class TestVswitchSampleCBDMA(TestCase):\n \"\"\"\n get all cbdma ports\n \"\"\"\n- # check driver name in execution.cfg\n- self.verify(self.drivername == 'igb_uio',\n- \"CBDMA test case only use igb_uio driver, need config drivername=igb_uio in execution.cfg\")\n- str_info = 'Misc (rawdev) devices using kernel driver'\n- out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev misc', '# ', 30)\n+ out = self.dut.send_expect('./usertools/dpdk-devbind.py --status-dev dma', '# ', 30)\n device_info = out.split('\\n')\n for device in device_info:\n pci_info = re.search('\\s*(0000:\\S*:\\d*.\\d*)', device)\n", "prefixes": [ "V1", "1/2" ] }{ "id": 104684, "url": "