Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/121417/?format=api
http://patches.dpdk.org/api/patches/121417/?format=api", "web_url": "http://patches.dpdk.org/project/dts/patch/20221227102131.529684-4-ke1.xu@intel.com/", "project": { "id": 3, "url": "http://patches.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": "<20221227102131.529684-4-ke1.xu@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dts/20221227102131.529684-4-ke1.xu@intel.com", "date": "2022-12-27T10:21:33", "name": "[V1,3/5] tests/vf_offload: Use modified checksum_validate to replace checksum_validate_tunnel.", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": false, "hash": "f6976b86c2e64a1113217b5566ea616b21ffb370", "submitter": { "id": 2810, "url": "http://patches.dpdk.org/api/people/2810/?format=api", "name": "Ke Xu", "email": "ke1.xu@intel.com" }, "delegate": null, "mbox": "http://patches.dpdk.org/project/dts/patch/20221227102131.529684-4-ke1.xu@intel.com/mbox/", "series": [ { "id": 26297, "url": "http://patches.dpdk.org/api/series/26297/?format=api", "web_url": "http://patches.dpdk.org/project/dts/list/?series=26297", "date": "2022-12-27T10:21:27", "name": "Improve checksum validate and tso packet counting.", "version": 1, "mbox": "http://patches.dpdk.org/series/26297/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/121417/comments/", "check": "pending", "checks": "http://patches.dpdk.org/api/patches/121417/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 9C029A04FD;\n\tTue, 27 Dec 2022 11:25:11 +0100 (CET)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 968C741143;\n\tTue, 27 Dec 2022 11:25:11 +0100 (CET)", "from mga03.intel.com (mga03.intel.com [134.134.136.65])\n by mails.dpdk.org (Postfix) with ESMTP id 0D31740E2D\n for <dts@dpdk.org>; Tue, 27 Dec 2022 11:25:09 +0100 (CET)", "from fmsmga002.fm.intel.com ([10.253.24.26])\n by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 27 Dec 2022 02:25:08 -0800", "from dpdk-xuke-lab.sh.intel.com ([10.67.119.8])\n by fmsmga002.fm.intel.com with ESMTP; 27 Dec 2022 02:25:06 -0800" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple;\n d=intel.com; i=@intel.com; q=dns/txt; s=Intel;\n t=1672136710; x=1703672710;\n h=from:to:cc:subject:date:message-id:in-reply-to:\n references:mime-version:content-transfer-encoding;\n bh=ax8d1oOAJTnOexRBv0RMJgSkzPwyzjrYSDEbZorn7nQ=;\n b=GdIQEZn3GR0oK/G2es5GrJ24KRIovBy/RXM/bYyyhpWVb5E7x4hR+7MP\n 9DqnH3+q0zC4h9iSMu5LtVqM6Oc9uRdRoa1h+T9sHQ3MBzwy9x/PlPePE\n DPbRop+HfD4gY2EYz5ecy5pFWE/4UcwiCmGgmg+vakf6e4VlMoBcuVBGf\n JQzRRnx7gjOsPt7IuM9NusTae+uYvrX+LMV3yAl/ZOHsw1MA67AJO6knz\n AWyIFHokTDch+phEPVkiCjcCoem9DT0DO6u+y6Z3BpHY9elZm+UG8TO2j\n APgNMJxRCoKlA7vE/NaP6eMc/py87b9CwmV47ezrsijWIC4CaVaM0N+5q g==;", "X-IronPort-AV": [ "E=McAfee;i=\"6500,9779,10572\"; a=\"322656462\"", "E=Sophos;i=\"5.96,278,1665471600\"; d=\"scan'208\";a=\"322656462\"", "E=McAfee;i=\"6500,9779,10572\"; a=\"760292185\"", "E=Sophos;i=\"5.96,278,1665471600\"; d=\"scan'208\";a=\"760292185\"" ], "X-ExtLoop1": "1", "From": "Ke Xu <ke1.xu@intel.com>", "To": "dts@dpdk.org", "Cc": "qi.fu@intel.com, weiyuanx.li@intel.com, lijuan.tu@intel.com,\n ke1.xu@intel.com", "Subject": "[DTS][PATCH V1 3/5] tests/vf_offload: Use modified checksum_validate\n to replace checksum_validate_tunnel.", "Date": "Tue, 27 Dec 2022 18:21:33 +0800", "Message-Id": "<20221227102131.529684-4-ke1.xu@intel.com>", "X-Mailer": "git-send-email 2.25.1", "In-Reply-To": "<20221227102131.529684-1-ke1.xu@intel.com>", "References": "<20221227102131.529684-1-ke1.xu@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": "Use checksum_validate method to validate and remove the tunnel one. The\n checksum_validate is updated to cover tunnel packet validation, the\n checksum_validate_tunnel is no longer necessary.\n\nSigned-off-by: Ke Xu <ke1.xu@intel.com>\n---\n tests/TestSuite_vf_offload.py | 62 ++---------------------------------\n 1 file changed, 3 insertions(+), 59 deletions(-)", "diff": "diff --git a/tests/TestSuite_vf_offload.py b/tests/TestSuite_vf_offload.py\nindex fa56578b..d6f2a134 100644\n--- a/tests/TestSuite_vf_offload.py\n+++ b/tests/TestSuite_vf_offload.py\n@@ -394,64 +394,6 @@ class TestVfOffload(TestCase):\n \n self.verify(len(result) == 0, \",\".join(list(result.values())))\n \n- def checksum_validate_tunnel(self, packets_sent, packets_expected):\n- \"\"\"\n- Validate the checksum.\n- \"\"\"\n- tx_interface = self.tester.get_interface(\n- self.tester.get_local_port(self.dut_ports[0])\n- )\n- rx_interface = self.tester.get_interface(\n- self.tester.get_local_port(self.dut_ports[1])\n- )\n- sniff_src = self.vm0_testpmd.get_port_mac(0)\n- checksum_pattern = re.compile(\"chksum.*=.*(0x[0-9a-z]+)\")\n- sniff_src = \"52:00:00:00:00:00\"\n- chksum = dict()\n- # self.tester.send_expect(\"scapy\", \">>> \")\n-\n- for packet_type in list(packets_expected.keys()):\n- self.tester.send_expect(\"scapy\", \">>> \")\n- self.tester.send_expect(\"p = %s\" % packets_expected[packet_type], \">>>\")\n- out = self.tester.send_expect(\"p.show2()\", \">>>\")\n- chksums = checksum_pattern.findall(out)\n- expected_chksum = chksums\n- chksum[packet_type] = chksums\n- print(packet_type, \": \", chksums)\n-\n- self.tester.send_expect(\"exit()\", \"#\")\n-\n- self.tester.scapy_background()\n- inst = self.tester.tcpdump_sniff_packets(\n- intf=rx_interface,\n- count=len(packets_sent),\n- filters=[{\"layer\": \"ether\", \"config\": {\"src\": sniff_src}}],\n- )\n-\n- # Send packet.\n- self.tester.scapy_foreground()\n-\n- self.tester.scapy_append(\n- 'sendp([%s], iface=\"%s\")' % (packets_sent[packet_type], tx_interface)\n- )\n- self.tester.scapy_execute()\n- out = self.tester.scapy_get_result()\n- p = self.tester.load_tcpdump_sniff_packets(inst)\n- nr_packets = len(p)\n- print(p)\n- chksums = checksum_pattern.findall(p[0].show2(dump=True))\n- packets_received = chksums\n- self.logger.debug(f\"packets_received: {packets_received}\")\n- self.logger.debug(f\"expected_chksum: {expected_chksum}\")\n- self.verify(\n- len(expected_chksum) == len(packets_received),\n- f\"The chksum type {packet_type} length of the actual result is inconsistent with the expected length!\",\n- )\n- self.verify(\n- packets_received == expected_chksum,\n- f\"The actually received chksum {packet_type} is inconsistent with the expectation\",\n- )\n-\n @check_supported_nic(\n [\"ICE_100G-E810C_QSFP\", \"ICE_25G-E810C_SFP\", \"ICE_25G-E810_XXV_SFP\"]\n )\n@@ -570,7 +512,7 @@ class TestVfOffload(TestCase):\n self.vm0_testpmd.execute_cmd(\"start\")\n self.vm0_testpmd.wait_link_status_up(0)\n self.vm0_testpmd.wait_link_status_up(1)\n- self.checksum_validate_tunnel(pkts, pkts_ref)\n+ result = self.checksum_validate(pkts, pkts_ref)\n # Validate checksum on the receive packet\n out = self.vm0_testpmd.execute_cmd(\"stop\")\n bad_outer_ipcsum = self.vm0_testpmd.get_pmd_value(\"Bad-outer-ipcsum:\", out)\n@@ -588,6 +530,8 @@ class TestVfOffload(TestCase):\n self.verify(bad_inner_ipcsum == 36, \"Bad-ipcsum check error\")\n self.verify(bad_inner_l4csum == 72, \"Bad-l4csum check error\")\n \n+ self.verify(len(result) == 0, \",\".join(list(result.values())))\n+\n def test_checksum_offload_disable(self):\n \"\"\"\n Enable SW checksum offload.\n", "prefixes": [ "V1", "3/5" ] }{ "id": 121417, "url": "