Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/117609/?format=api
{ "id": 117609, "url": "https://patches.dpdk.org/api/patches/117609/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/patch/20221007174336.54354-32-andrew.boyer@amd.com/", "project": { "id": 1, "url": "https://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": "<20221007174336.54354-32-andrew.boyer@amd.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20221007174336.54354-32-andrew.boyer@amd.com", "date": "2022-10-07T17:43:32", "name": "[31/35] net/ionic: allow client to specify Tx free threshold", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "3bd1a6bb2a1a0ef44035563d5f7d539851a04dc6", "submitter": { "id": 2861, "url": "https://patches.dpdk.org/api/people/2861/?format=api", "name": "Andrew Boyer", "email": "Andrew.Boyer@amd.com" }, "delegate": { "id": 319, "url": "https://patches.dpdk.org/api/users/319/?format=api", "username": "fyigit", "first_name": "Ferruh", "last_name": "Yigit", "email": "ferruh.yigit@amd.com" }, "mbox": "https://patches.dpdk.org/project/dpdk/patch/20221007174336.54354-32-andrew.boyer@amd.com/mbox/", "series": [ { "id": 25037, "url": "https://patches.dpdk.org/api/series/25037/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/list/?series=25037", "date": "2022-10-07T17:43:01", "name": "net/ionic: updates for 22.11 release", "version": 1, "mbox": "https://patches.dpdk.org/series/25037/mbox/" } ], "comments": "https://patches.dpdk.org/api/patches/117609/comments/", "check": "success", "checks": "https://patches.dpdk.org/api/patches/117609/checks/", "tags": {}, "related": [], "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 66D2FA04FD;\n\tFri, 7 Oct 2022 19:47:41 +0200 (CEST)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 3ABA142B76;\n\tFri, 7 Oct 2022 19:45:12 +0200 (CEST)", "from NAM12-BN8-obe.outbound.protection.outlook.com\n (mail-bn8nam12on2052.outbound.protection.outlook.com [40.107.237.52])\n by mails.dpdk.org (Postfix) with ESMTP id 238ED42C04\n for <dev@dpdk.org>; Fri, 7 Oct 2022 19:45:09 +0200 (CEST)", "from MW4PR03CA0340.namprd03.prod.outlook.com (2603:10b6:303:dc::15)\n by MW5PR12MB5600.namprd12.prod.outlook.com (2603:10b6:303:195::12)\n with Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5676.34; Fri, 7 Oct\n 2022 17:45:07 +0000", "from CO1NAM11FT068.eop-nam11.prod.protection.outlook.com\n (2603:10b6:303:dc:cafe::96) by MW4PR03CA0340.outlook.office365.com\n (2603:10b6:303:dc::15) with Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5676.24 via Frontend\n Transport; Fri, 7 Oct 2022 17:45:07 +0000", "from SATLEXMB04.amd.com (165.204.84.17) by\n CO1NAM11FT068.mail.protection.outlook.com (10.13.175.142) with Microsoft SMTP\n Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id\n 15.20.5709.10 via Frontend Transport; Fri, 7 Oct 2022 17:45:06 +0000", "from driver-dev1.pensando.io (10.180.168.240) by SATLEXMB04.amd.com\n (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.28; Fri, 7 Oct\n 2022 12:45:03 -0500" ], "ARC-Seal": "i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none;\n b=TVukYCoo/XbRB1sI2r/NDu+iE6WDgnA9nFIAtyAQapLezU5rmIoeMOSRq6ThJHTolQP0n6MJDOcTsfod1dVvT0003MMP5DrgZBfE3NRzmKsr1i/qSKO2z9VRjdPDOzGS+eCwhIE/0K8d2rHPktjxdbS2wxJRsj0H89qYWCaflsbvS8/BBOkCM/xhIlegcaCDiLdDrVlbpMQvSTL7ypXmyA4ICDLAbCe0LfAH4cg4QvW7BCy42TVO2AO0b9B7DoboV/jqBh4/eHGLKw+kT9AR6jPduk26mgH3K4SbFia9tM8itL+ht3foUecQOW4RKyoYAi2ZsH4Y3o+FQzGzeKZ5Rw==", "ARC-Message-Signature": "i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com;\n s=arcselector9901;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1;\n bh=+d24/ofUOlesiNWDNQDIv6rLFf3ZL534BBo/iy3t9iQ=;\n b=QAcV/a252wddd0gwzC7qEKGxMKiPfWtL56QUkMXZOBSRaJxOPhPeFzH4dKBKK7rGN0x6AIk225yKm0d+AKbxnPnC8S1I76kxjq6/ZZEBTN66U+fqtByEn6wuAa2cbsb0mQj8gsabWFWwez+d2UzsEzy1AQtVciGgc2SatA2LYwuox7bsyCG1bcRqR27dxWggv0jO49+R/DlaF4r0HX6cPaOVZmXnd7UFwUis4TukgvQYxml4cUrUprY1sgjSuym6y0orE87I4NnJxqdF5QRSSKqfLaQZRYGXGXIvI47/hI9n0eP2v4H/5sWZiepMx7zI0aNOnnyshTSFs0JsnsdgEw==", "ARC-Authentication-Results": "i=1; mx.microsoft.com 1; spf=pass (sender ip is\n 165.204.84.17) smtp.rcpttodomain=dpdk.org smtp.mailfrom=amd.com; dmarc=pass\n (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com;\n dkim=none (message not signed); arc=none", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1;\n h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;\n bh=+d24/ofUOlesiNWDNQDIv6rLFf3ZL534BBo/iy3t9iQ=;\n b=pgTBs1jR4kMgP+FqvDqAUw2dYLPmOlw0B+3Xqzi+7pM3v2Pz/bQFOM9mJ3X09F8NIO8cMgPGatOUjv+KCdcRrZYi81KgE3Kb2Qoie7X+7B5i0JzcdcSC1Vmde2j/OrzmHbuEfpoJpa4jQPb8N2Goi2yp4P7sooUZnr4LkyWSQ/Y=", "X-MS-Exchange-Authentication-Results": "spf=pass (sender IP is 165.204.84.17)\n smtp.mailfrom=amd.com; dkim=none (message not signed)\n header.d=none;dmarc=pass action=none header.from=amd.com;", "Received-SPF": "Pass (protection.outlook.com: domain of amd.com designates\n 165.204.84.17 as permitted sender) receiver=protection.outlook.com;\n client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C", "From": "Andrew Boyer <andrew.boyer@amd.com>", "To": "<dev@dpdk.org>", "CC": "Andrew Boyer <andrew.boyer@amd.com>", "Subject": "[PATCH 31/35] net/ionic: allow client to specify Tx free threshold", "Date": "Fri, 7 Oct 2022 10:43:32 -0700", "Message-ID": "<20221007174336.54354-32-andrew.boyer@amd.com>", "X-Mailer": "git-send-email 2.17.1", "In-Reply-To": "<20221007174336.54354-1-andrew.boyer@amd.com>", "References": "<20221007174336.54354-1-andrew.boyer@amd.com>", "MIME-Version": "1.0", "Content-Type": "text/plain", "X-Originating-IP": "[10.180.168.240]", "X-ClientProxiedBy": "SATLEXMB04.amd.com (10.181.40.145) To SATLEXMB04.amd.com\n (10.181.40.145)", "X-EOPAttributedMessage": "0", "X-MS-PublicTrafficType": "Email", "X-MS-TrafficTypeDiagnostic": "CO1NAM11FT068:EE_|MW5PR12MB5600:EE_", "X-MS-Office365-Filtering-Correlation-Id": "aaf39cbe-5574-4e3a-663b-08daa88bac3c", "X-MS-Exchange-SenderADCheck": "1", "X-MS-Exchange-AntiSpam-Relay": "0", "X-Microsoft-Antispam": "BCL:0;", "X-Microsoft-Antispam-Message-Info": "\n PuavUIFFvK3TPZSwLWVupDGoDMUlKr/cOMuWbs1MmWeACOKInZ/Qcny+F35XcIxywaXDDebscETiUfrUJYHRNYUz0EFuJ3Ij8rNb4v8NZqPGincgTpsvE9k4sMl5qL9OesSozRzvGAs3Hc4evVgb0+xMxXv0o4L/828L5mu/eb9dxllt/R0SeXGKu+t7hXD7kwZitnrZzqU7Dn6oC9aoPAH90TRQNJ0XnJ5dhqYrJvfsoAdJZZPPTD7WOy8l9vPA9ndtd5PALkQhAOV6D450cDmrxzEJ50k3s8FJNFCUu1WV4ThQBVpb2wCOSrj2ZVBYlFMp5j1nqd3FLQ9EuVKEHyBayq3j1MXV8zY/trK3pvYm5tDKE0XaL5ay10S+3TKpZDCjLd9Qd28tnms7/l9ZrOS946DSnibEX6geWeA1uHK2W3n+YsGMkCuTKDOEZxkFfX3DUDp/fQKRNAB3s+Z5zPxdCg5xBcgmOOpENMZbyA9JEDNxW6NCC+L+224s7eCYaaJ6RbwcFYc10Drgy8uDnb2/M33xDsV0geLCqXpZeIw+TejlDFugyFwy/l6zKFXHvLuVAnFKOKMPe6y7q+lJYgQZSqR7BgWlkg424uOxn4BRGbcpyBs3LEZIOwjRp1vXNw1lAPdBalsJXaDoal6ZmYsiYqDTbCDBt7cYk70wVYHs1yOGyH9C6GMi2BFhrDsSiN2/UafUSCj4FKYTjHbR0pBoWr59GGAstjHEXb3JChGW5e2qUPq8K8+bjpD3YorcHhE6Eq1a263t8b0lpu3m50aHjkVkiC4s2IbwayD06+AiD3AR8Xnb9yNiiKqAbU2e", "X-Forefront-Antispam-Report": "CIP:165.204.84.17; CTRY:US; LANG:en; SCL:1; SRV:;\n IPV:CAL; SFV:NSPM; H:SATLEXMB04.amd.com; PTR:InfoDomainNonexistent; CAT:NONE;\n SFS:(13230022)(4636009)(39860400002)(376002)(346002)(396003)(136003)(451199015)(36840700001)(40470700004)(46966006)(86362001)(2906002)(70586007)(5660300002)(4326008)(8676002)(81166007)(70206006)(41300700001)(36756003)(186003)(16526019)(82310400005)(40460700003)(426003)(83380400001)(44832011)(40480700001)(336012)(8936002)(356005)(36860700001)(6666004)(2616005)(82740400003)(478600001)(6916009)(1076003)(26005)(47076005)(316002)(36900700001);\n DIR:OUT; SFP:1101;", "X-OriginatorOrg": "amd.com", "X-MS-Exchange-CrossTenant-OriginalArrivalTime": "07 Oct 2022 17:45:06.7875 (UTC)", "X-MS-Exchange-CrossTenant-Network-Message-Id": "\n aaf39cbe-5574-4e3a-663b-08daa88bac3c", "X-MS-Exchange-CrossTenant-Id": "3dd8961f-e488-4e60-8e11-a82d994e183d", "X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp": "\n TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d; Ip=[165.204.84.17];\n Helo=[SATLEXMB04.amd.com]", "X-MS-Exchange-CrossTenant-AuthSource": "\n CO1NAM11FT068.eop-nam11.prod.protection.outlook.com", "X-MS-Exchange-CrossTenant-AuthAs": "Anonymous", "X-MS-Exchange-CrossTenant-FromEntityHeader": "HybridOnPrem", "X-MS-Exchange-Transport-CrossTenantHeadersStamped": "MW5PR12MB5600", "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" }, "content": "Some clients have opinions about how often to flush the\ntransmit ring.\n\nThe default value is the number of Tx descriptors minus the\ndefault Tx burst size.\n\nSigned-off-by: Andrew Boyer <andrew.boyer@amd.com>\n---\n drivers/net/ionic/ionic_dev.h | 1 +\n drivers/net/ionic/ionic_ethdev.c | 4 ++--\n drivers/net/ionic/ionic_lif.h | 1 +\n drivers/net/ionic/ionic_rxtx.c | 17 +++++++++++++++--\n 4 files changed, 19 insertions(+), 4 deletions(-)", "diff": "diff --git a/drivers/net/ionic/ionic_dev.h b/drivers/net/ionic/ionic_dev.h\nindex 6b0540f615..d696de45e0 100644\n--- a/drivers/net/ionic/ionic_dev.h\n+++ b/drivers/net/ionic/ionic_dev.h\n@@ -20,6 +20,7 @@\n #define IONIC_MAX_RING_DESC\t\t32768\n #define IONIC_MIN_RING_DESC\t\t16\n #define IONIC_DEF_TXRX_DESC\t\t4096\n+#define IONIC_DEF_TXRX_BURST\t\t32\n \n #define IONIC_DEVCMD_TIMEOUT\t\t5\t/* devcmd_timeout */\n #define IONIC_DEVCMD_CHECK_PERIOD_US\t10\t/* devcmd status chk period */\ndiff --git a/drivers/net/ionic/ionic_ethdev.c b/drivers/net/ionic/ionic_ethdev.c\nindex 36fb09b810..28297879cf 100644\n--- a/drivers/net/ionic/ionic_ethdev.c\n+++ b/drivers/net/ionic/ionic_ethdev.c\n@@ -429,8 +429,8 @@ ionic_dev_info_get(struct rte_eth_dev *eth_dev,\n \tdev_info->tx_desc_lim = tx_desc_lim_v1;\n \n \t/* Driver-preferred Rx/Tx parameters */\n-\tdev_info->default_rxportconf.burst_size = 32;\n-\tdev_info->default_txportconf.burst_size = 32;\n+\tdev_info->default_rxportconf.burst_size = IONIC_DEF_TXRX_BURST;\n+\tdev_info->default_txportconf.burst_size = IONIC_DEF_TXRX_BURST;\n \tdev_info->default_rxportconf.nb_queues = 1;\n \tdev_info->default_txportconf.nb_queues = 1;\n \tdev_info->default_rxportconf.ring_size = IONIC_DEF_TXRX_DESC;\ndiff --git a/drivers/net/ionic/ionic_lif.h b/drivers/net/ionic/ionic_lif.h\nindex ec9cb24a61..e4af138a51 100644\n--- a/drivers/net/ionic/ionic_lif.h\n+++ b/drivers/net/ionic/ionic_lif.h\n@@ -104,6 +104,7 @@ struct ionic_tx_qcq {\n \n \t/* cacheline2 */\n \tuint16_t num_segs_fw;\t/* # segs supported by current FW */\n+\tuint16_t free_thresh;\n \tuint16_t flags;\n \n \tstruct ionic_tx_stats stats;\ndiff --git a/drivers/net/ionic/ionic_rxtx.c b/drivers/net/ionic/ionic_rxtx.c\nindex fceb8333f6..9a346f4143 100644\n--- a/drivers/net/ionic/ionic_rxtx.c\n+++ b/drivers/net/ionic/ionic_rxtx.c\n@@ -223,6 +223,13 @@ ionic_dev_tx_queue_setup(struct rte_eth_dev *eth_dev, uint16_t tx_queue_id,\n \tif (!rte_is_power_of_2(nb_desc) || nb_desc < IONIC_MIN_RING_DESC)\n \t\treturn -EINVAL; /* or use IONIC_DEFAULT_RING_DESC */\n \n+\tif (tx_conf->tx_free_thresh > nb_desc) {\n+\t\tIONIC_PRINT(ERR,\n+\t\t\t\"tx_free_thresh must be less than nb_desc (%u)\",\n+\t\t\tnb_desc);\n+\t\treturn -EINVAL;\n+\t}\n+\n \t/* Free memory prior to re-allocation if needed... */\n \tif (eth_dev->data->tx_queues[tx_queue_id] != NULL) {\n \t\tionic_dev_tx_queue_release(eth_dev, tx_queue_id);\n@@ -252,6 +259,10 @@ ionic_dev_tx_queue_setup(struct rte_eth_dev *eth_dev, uint16_t tx_queue_id,\n \tif (offloads & RTE_ETH_TX_OFFLOAD_MBUF_FAST_FREE)\n \t\ttxq->flags |= IONIC_QCQ_F_FAST_FREE;\n \n+\ttxq->free_thresh =\n+\t\ttx_conf->tx_free_thresh ? tx_conf->tx_free_thresh :\n+\t\tnb_desc - IONIC_DEF_TXRX_BURST;\n+\n \teth_dev->data->tx_queues[tx_queue_id] = txq;\n \n \treturn 0;\n@@ -609,8 +620,10 @@ ionic_xmit_pkts(void *tx_queue, struct rte_mbuf **tx_pkts,\n \t\trte_mbuf_prefetch_part2(tx_pkts[0]);\n \t}\n \n-\t/* Cleaning old buffers */\n-\tionic_tx_flush(txq);\n+\tif (unlikely(ionic_q_space_avail(q) < txq->free_thresh)) {\n+\t\t/* Cleaning old buffers */\n+\t\tionic_tx_flush(txq);\n+\t}\n \n \tnb_avail = ionic_q_space_avail(q);\n \tif (unlikely(nb_avail < nb_pkts)) {\n", "prefixes": [ "31/35" ] }