Cover Detail
Show a cover letter.
GET /api/covers/129512/?format=api
{ "id": 129512, "url": "http://patchwork.dpdk.org/api/covers/129512/?format=api", "web_url": "http://patchwork.dpdk.org/project/dts/cover/20230712203334.2032445-1-ke1.xu@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": "<20230712203334.2032445-1-ke1.xu@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dts/20230712203334.2032445-1-ke1.xu@intel.com", "date": "2023-07-12T20:33:32", "name": "[V1,0/2] Rewrite test suite for checksum_offload", "submitter": { "id": 2810, "url": "http://patchwork.dpdk.org/api/people/2810/?format=api", "name": "Ke Xu", "email": "ke1.xu@intel.com" }, "mbox": "http://patchwork.dpdk.org/project/dts/cover/20230712203334.2032445-1-ke1.xu@intel.com/mbox/", "series": [ { "id": 28926, "url": "http://patchwork.dpdk.org/api/series/28926/?format=api", "web_url": "http://patchwork.dpdk.org/project/dts/list/?series=28926", "date": "2023-07-12T20:33:32", "name": "Rewrite test suite for checksum_offload", "version": 1, "mbox": "http://patchwork.dpdk.org/series/28926/mbox/" } ], "comments": "http://patchwork.dpdk.org/api/covers/129512/comments/", "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 B462B42E58;\n\tWed, 12 Jul 2023 22:33:30 +0200 (CEST)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id AE05F406BA;\n\tWed, 12 Jul 2023 22:33:30 +0200 (CEST)", "from mga11.intel.com (mga11.intel.com [192.55.52.93])\n by mails.dpdk.org (Postfix) with ESMTP id 0EB8D400D5\n for <dts@dpdk.org>; Wed, 12 Jul 2023 22:33:29 +0200 (CEST)", "from orsmga007.jf.intel.com ([10.7.209.58])\n by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 12 Jul 2023 13:33:28 -0700", "from dpdk-xuke-host.sh.intel.com ([10.67.114.220])\n by orsmga007.jf.intel.com with ESMTP; 12 Jul 2023 13:33:26 -0700" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/simple;\n d=intel.com; i=@intel.com; q=dns/txt; s=Intel;\n t=1689194010; x=1720730010;\n h=from:to:cc:subject:date:message-id:mime-version:\n content-transfer-encoding;\n bh=iGX7iG+p9Wm//+6gn8F1QY+qtWt3DJWDfGAtLMkShUc=;\n b=aahnK/Q8Pyh+6HpVynoOCGvgxnklTYgP6w89oaHrea/QSCovrH01fKEi\n PC/tGv5E8pMQghOqBLYyCdwRKhjHsJxq9J+aKWKXJJHnWhRS+vSI6uUW3\n 1H0VluAdhpCIj3rZ5eN0HJnbczpwPnWHSGcIczqPgPmqhevkXXmaa8H4Q\n WH6b2sTvjUB0N1oZZ/ADeiW7PTQ+T0idHv4krWN1ngxH/01ADurMFePnD\n 1R1nX4/VClCavhrvYJy/+0p2ezLLL1T7VeAOYvdtSfPz9CzMRPsjEK0Kd\n 5nYwg1ua3uTpX1Heegzmx+rnuNpD25ZuBCyDfdPaGR6BWp2Kp+2Kw1klK w==;", "X-IronPort-AV": [ "E=McAfee;i=\"6600,9927,10769\"; a=\"362467038\"", "E=Sophos;i=\"6.01,200,1684825200\"; d=\"scan'208\";a=\"362467038\"", "E=McAfee;i=\"6600,9927,10769\"; a=\"715696949\"", "E=Sophos;i=\"6.01,200,1684825200\"; d=\"scan'208\";a=\"715696949\"" ], "X-ExtLoop1": "1", "From": "Ke Xu <ke1.xu@intel.com>", "To": "dts@dpdk.org", "Cc": "ke1.xu@intel.com,\n\ttarcadia@qq.com", "Subject": "[DTS][Patch V1 0/2] Rewrite test suite for checksum_offload", "Date": "Wed, 12 Jul 2023 20:33:32 +0000", "Message-Id": "<20230712203334.2032445-1-ke1.xu@intel.com>", "X-Mailer": "git-send-email 2.34.1", "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": "Checksum offload cases were in bad situation. Cases are not well organized\n in structure. Cases were added one by one along with the development of\n features.\n\nThe very beginning cases are designed to verifying RX Checksum Offload\n enabling / disabling. The followed cases are to verify the TX Offload, and\n found a gap when TX introduces differences to IP and L4. Then IP and L4 RX\n cases are added again. There were also a gap found on the flag verifying\n and a seperate ol_flag case was added.\n\nTo better organizing the checksum offload cases. I introduced the scenario\n based common cases and specific cases named with sufix to cover specific\n scenarios of special requirements. And we placed all available test scopes\n introduced in any existing cases to each of our common test execution to\n ensure no leakage to our test scopes.\n\nFor scenarios, we introduce the very commonly used configurations for\n checksum as our test scenarios. These scenarios cover combined usage of HW\n checksum Offload, SW checksum and differently configured tunneling scenario,\n w/wo VLAN. These cases formed a organized web for future cases that if a new\n configuration is introduced, cases of new configuration combinations are to\n be added but existing configurations for different scopes are not to be added.\n\nFor test scopes, we introduce verifying ol_flag, fwd stats, other verbose lines,\n and forwarded packet capturing and analyzing. All these scopes are recorder for\n verifying in each common checksum offload execution. This ensures no scope would\n be missed in this rewrite and in future case updating.\n\nThe newly organized cases also helps better covering our existing test\n requirements. A common packet sending and recording helps cut down the repeating\n time cost of different executions for scope based redundant cases.\n\nThis rewrite also helps formating a data-process-seperated suite structure. The\n test parameters and required packets are listed in front and cases are referring\n to the configured parameters. This helps better reading the suite code.\n\nIn the test plan, new cases are added and old ones are merged with the new cases.\n Packets, ol_flags, stats and related test scopes are listed in table to show where\n the expected result is from.\n\nGenerally speaking, this case rewrite helps formating the case structure and helps\n better maintaining the cases in future.\n\n\nKe Xu (2):\n test_plans/checksum_offload: Refactor checksum_offload case.\n tests/checksum_offload: Refactor checksum_offload case.\n\n test_plans/checksum_offload_test_plan.rst | 1478 +++++++----------\n tests/TestSuite_checksum_offload.py | 1766 +++++++++++----------\n 2 files changed, 1549 insertions(+), 1695 deletions(-)" }