Cover Detail
Show a cover letter.
GET /api/covers/91303/?format=api
{ "id": 91303, "url": "http://patches.dpdk.org/api/covers/91303/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/cover/1618344896-2090-1-git-send-email-timothy.mcdaniel@intel.com/", "project": { "id": 1, "url": "http://patches.dpdk.org/api/projects/1/?format=api", "name": "DPDK", "link_name": "dpdk", "list_id": "dev.dpdk.org", "list_email": "dev@dpdk.org", "web_url": "http://core.dpdk.org", "scm_url": "git://dpdk.org/dpdk", "webscm_url": "http://git.dpdk.org/dpdk", "list_archive_url": "https://inbox.dpdk.org/dev", "list_archive_url_format": "https://inbox.dpdk.org/dev/{}", "commit_url_format": "" }, "msgid": "<1618344896-2090-1-git-send-email-timothy.mcdaniel@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/1618344896-2090-1-git-send-email-timothy.mcdaniel@intel.com", "date": "2021-04-13T20:14:30", "name": "[v3,00/26] Add DLB V2.5", "submitter": { "id": 826, "url": "http://patches.dpdk.org/api/people/826/?format=api", "name": "Timothy McDaniel", "email": "timothy.mcdaniel@intel.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/cover/1618344896-2090-1-git-send-email-timothy.mcdaniel@intel.com/mbox/", "series": [ { "id": 16345, "url": "http://patches.dpdk.org/api/series/16345/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=16345", "date": "2021-04-13T20:14:31", "name": "Add DLB V2.5", "version": 3, "mbox": "http://patches.dpdk.org/series/16345/mbox/" } ], "comments": "http://patches.dpdk.org/api/covers/91303/comments/", "headers": { "Return-Path": "<dev-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 25F98A0524;\n\tTue, 13 Apr 2021 22:16:48 +0200 (CEST)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 003D61612E2;\n\tTue, 13 Apr 2021 22:16:21 +0200 (CEST)", "from mga03.intel.com (mga03.intel.com [134.134.136.65])\n by mails.dpdk.org (Postfix) with ESMTP id 8B3461612B3\n for <dev@dpdk.org>; Tue, 13 Apr 2021 22:16:14 +0200 (CEST)", "from orsmga008.jf.intel.com ([10.7.209.65])\n by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 13 Apr 2021 13:16:09 -0700", "from txasoft-yocto.an.intel.com ([10.123.72.192])\n by orsmga008.jf.intel.com with ESMTP; 13 Apr 2021 13:16:08 -0700" ], "IronPort-SDR": [ "\n eNKm0a7FeoAqb35vPznvjaPLwPhleAIu+zg07RXPJSeKmFi/7ev+4WhrrB7uLDeNuXuak3IUhB\n m+H2Eyci18qA==", "\n WucfP7QmGQA0ixGKRTrCAGWnNifRD81jbtK2iqex01TE3fA8hQ++TTc4Y+XSlBSSNqL05VwyAU\n R20Bq9DDV1Cg==" ], "X-IronPort-AV": [ "E=McAfee;i=\"6200,9189,9953\"; a=\"194519679\"", "E=Sophos;i=\"5.82,220,1613462400\"; d=\"scan'208\";a=\"194519679\"", "E=Sophos;i=\"5.82,220,1613462400\"; d=\"scan'208\";a=\"424406456\"" ], "X-ExtLoop1": "1", "From": "Timothy McDaniel <timothy.mcdaniel@intel.com>", "To": "", "Cc": "dev@dpdk.org, erik.g.carrillo@intel.com, gage.eads@intel.com,\n harry.van.haaren@intel.com, jerinj@marvell.com, thomas@monjalon.net", "Date": "Tue, 13 Apr 2021 15:14:30 -0500", "Message-Id": "<1618344896-2090-1-git-send-email-timothy.mcdaniel@intel.com>", "X-Mailer": "git-send-email 1.7.10", "In-Reply-To": "<20210316221857.2254-2-timothy.mcdaniel@intel.com>", "References": "<20210316221857.2254-2-timothy.mcdaniel@intel.com>", "Subject": "[dpdk-dev] [PATCH v3 00/26] Add DLB V2.5", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dev>,\n <mailto:dev-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://mails.dpdk.org/archives/dev/>", "List-Post": "<mailto:dev@dpdk.org>", "List-Help": "<mailto:dev-request@dpdk.org?subject=help>", "List-Subscribe": "<https://mails.dpdk.org/listinfo/dev>,\n <mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "This patch series adds support for DLB v2.5 to\nthe current DLB V2.0 PMD. The resulting PMD supports\nboth hardware versions.\n\nThe main differences between the DLB v2.5 and v2.0 hardware\nare:\n- Number of queues/ports\n- DLB v2.5 uses a combined credit pool, whereas DLB v2.0\n splits credits into 2 pools, a directed credit pool and a\n load balanced credit pool.\n- Different register maps, with different bit names and offsets\n\nIn order to support both hardware versions with the same PMD,\nand avoid code duplication, the file dlb2_resource.c required a\ncomplete rewrite. This required some creative staging of the changes\nin order to keep the individual patches relatively small, while\nalso meeting the requirement that all individual patches in the set\ncompile cleanly.\n\nTo accomplish this, a few temporary files are used:\n\ndlb2_hw_types_new.h\ndlb2_resources_new.h\ndlb2_resources_new.c\n\nAs dlb2_resources_new.c is populated with the new combined v2.0/v2.5\nlow level logic, the corresponding old code is removed from\ndlb2_resource.c, thus allowing both the original and new code to\ncontinue to compile and link cleanly. Once all of the code has been\nmigrated to the new model, the old versions of the files are removed,\nand the new versions are renamed, effectively replacing the old original\nfiles.\n\nAs you review the code, you can ignore the code deletions from\ndlb2_resource.c, as that file continues to shrink as the new\ncorresponding logic is added to dlb2_resource_new.c.\n\nChanges since V2:\n1) fix commit headers\n2) fix commit message repeated words\n3) remove FPGA reference\n4) split out new v2.5 register definitions into separate patch\n5) fixed documentation to use DLB and dlb_event exclusively,\n instead of the old names such as dlb1_event, dlb2_event,\n DLB2, ... Final doc updates are done in patch that performs\n device rename from DLB2 tosimply DLB\n6) use component event/dlb at commit which changes device name and\n all subsequent commits\n7) Move all DLB constants out of config/rte_config.h except QUELL_STATS,\n which is used in the fastpath. Exposed these as devarg command line\n parameters\n8) Removed \"TEMPORARY\" comment leftover in dlb2_osdep.h\n9) squashed 20-21 and 22-23 since they were logically the same as 19-20,\n which was requested to be squashed\n10) delete old dlb2.rst - dlb.rst has been updated for v2.0 and v2.1\n\nChanges since V1:\n1) Simplified subject text for all patches\n2) correct typos/spelling\n3) remove FPGA references\n4) remove stale sysconf() references\n5) fixed patches that had compilation issues\n6) updated release notes\n7) renamed dlb device from dlb2_event to dlb_event\n8) moved dlb2 directory to dlb,to match name change\n9) fixed other cases where \"dlb2\" was being used externally\n\nTimothy McDaniel (26):\n event/dlb2: add v2.5 probe\n event/dlb2: add v2.5 HW register definitions\n event/dlb2: add v2.5 HW init\n event/dlb2: add v2.5 get resources\n event/dlb2: add v2.5 create sched domain\n event/dlb2: add v2.5 domain reset\n event/dlb2: add V2.5 create ldb queue\n event/dlb2: add v2.5 create ldb port\n event/dlb2: add v2.5 create dir port\n event/dlb2: add v2.5 create dir queue\n event/dlb2: add v2.5 map qid\n event/dlb2: add v2.5 unmap queue\n event/dlb2: add v2.5 start domain\n event/dlb2: add v2.5 credit scheme\n event/dlb2: add v2.5 queue depth functions\n event/dlb2: add v2.5 finish map/unmap\n event/dlb2: add v2.5 sparse cq mode\n event/dlb2: add v2.5 sequence number management\n event/dlb2: use new implementation of resource header\n event/dlb2: use new implementation of resource file\n event/dlb2: use new implementation of HW types header\n event/dlb2: use new combined register map\n event/dlb2: update xstats for v2.5\n doc/dlb2: update documentation for v2.5\n event/dlb: remove version from device name\n event/dlb: move rte config defines to runtime devargs\n\n MAINTAINERS | 6 +-\n app/test/test_eventdev.c | 6 +-\n config/rte_config.h | 8 +-\n doc/api/doxy-api-index.md | 2 +-\n doc/api/doxy-api.conf.in | 2 +-\n doc/guides/eventdevs/{dlb2.rst => dlb.rst} | 155 +-\n doc/guides/eventdevs/index.rst | 2 +-\n doc/guides/rel_notes/release_21_05.rst | 5 +\n drivers/event/{dlb2 => dlb}/dlb2.c | 550 ++-\n drivers/event/{dlb2 => dlb}/dlb2_iface.c | 0\n drivers/event/{dlb2 => dlb}/dlb2_iface.h | 0\n drivers/event/{dlb2 => dlb}/dlb2_inline_fns.h | 0\n drivers/event/{dlb2 => dlb}/dlb2_log.h | 0\n drivers/event/{dlb2 => dlb}/dlb2_priv.h | 177 +-\n drivers/event/{dlb2 => dlb}/dlb2_selftest.c | 8 +-\n drivers/event/{dlb2 => dlb}/dlb2_user.h | 27 +-\n drivers/event/{dlb2 => dlb}/dlb2_xstats.c | 70 +-\n drivers/event/{dlb2 => dlb}/meson.build | 4 +-\n .../{dlb2 => dlb}/pf/base/dlb2_hw_types.h | 106 +-\n .../event/{dlb2 => dlb}/pf/base/dlb2_osdep.h | 2 +\n .../{dlb2 => dlb}/pf/base/dlb2_osdep_bitmap.h | 0\n .../{dlb2 => dlb}/pf/base/dlb2_osdep_list.h | 0\n .../{dlb2 => dlb}/pf/base/dlb2_osdep_types.h | 0\n drivers/event/dlb/pf/base/dlb2_regs.h | 4304 +++++++++++++++++\n .../{dlb2 => dlb}/pf/base/dlb2_resource.c | 3278 +++++++------\n .../{dlb2 => dlb}/pf/base/dlb2_resource.h | 28 +-\n drivers/event/{dlb2 => dlb}/pf/dlb2_main.c | 37 +-\n drivers/event/{dlb2 => dlb}/pf/dlb2_main.h | 0\n drivers/event/{dlb2 => dlb}/pf/dlb2_pf.c | 67 +-\n .../rte_pmd_dlb2.c => dlb/rte_pmd_dlb.c} | 6 +-\n .../rte_pmd_dlb2.h => dlb/rte_pmd_dlb.h} | 12 +-\n drivers/event/{dlb2 => dlb}/version.map | 2 +-\n drivers/event/dlb2/pf/base/dlb2_mbox.h | 596 ---\n drivers/event/dlb2/pf/base/dlb2_regs.h | 2527 ----------\n drivers/event/meson.build | 2 +-\n 35 files changed, 6921 insertions(+), 5068 deletions(-)\n rename doc/guides/eventdevs/{dlb2.rst => dlb.rst} (72%)\n rename drivers/event/{dlb2 => dlb}/dlb2.c (89%)\n rename drivers/event/{dlb2 => dlb}/dlb2_iface.c (100%)\n rename drivers/event/{dlb2 => dlb}/dlb2_iface.h (100%)\n rename drivers/event/{dlb2 => dlb}/dlb2_inline_fns.h (100%)\n rename drivers/event/{dlb2 => dlb}/dlb2_log.h (100%)\n rename drivers/event/{dlb2 => dlb}/dlb2_priv.h (77%)\n rename drivers/event/{dlb2 => dlb}/dlb2_selftest.c (99%)\n rename drivers/event/{dlb2 => dlb}/dlb2_user.h (97%)\n rename drivers/event/{dlb2 => dlb}/dlb2_xstats.c (94%)\n rename drivers/event/{dlb2 => dlb}/meson.build (89%)\n rename drivers/event/{dlb2 => dlb}/pf/base/dlb2_hw_types.h (80%)\n rename drivers/event/{dlb2 => dlb}/pf/base/dlb2_osdep.h (99%)\n rename drivers/event/{dlb2 => dlb}/pf/base/dlb2_osdep_bitmap.h (100%)\n rename drivers/event/{dlb2 => dlb}/pf/base/dlb2_osdep_list.h (100%)\n rename drivers/event/{dlb2 => dlb}/pf/base/dlb2_osdep_types.h (100%)\n create mode 100644 drivers/event/dlb/pf/base/dlb2_regs.h\n rename drivers/event/{dlb2 => dlb}/pf/base/dlb2_resource.c (68%)\n rename drivers/event/{dlb2 => dlb}/pf/base/dlb2_resource.h (99%)\n rename drivers/event/{dlb2 => dlb}/pf/dlb2_main.c (95%)\n rename drivers/event/{dlb2 => dlb}/pf/dlb2_main.h (100%)\n rename drivers/event/{dlb2 => dlb}/pf/dlb2_pf.c (91%)\n rename drivers/event/{dlb2/rte_pmd_dlb2.c => dlb/rte_pmd_dlb.c} (88%)\n rename drivers/event/{dlb2/rte_pmd_dlb2.h => dlb/rte_pmd_dlb.h} (88%)\n rename drivers/event/{dlb2 => dlb}/version.map (60%)\n delete mode 100644 drivers/event/dlb2/pf/base/dlb2_mbox.h\n delete mode 100644 drivers/event/dlb2/pf/base/dlb2_regs.h" }