Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/117595/?format=api
{ "id": 117595, "url": "http://patches.dpdk.org/api/patches/117595/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20221007174336.54354-16-andrew.boyer@amd.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": "<20221007174336.54354-16-andrew.boyer@amd.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20221007174336.54354-16-andrew.boyer@amd.com", "date": "2022-10-07T17:43:16", "name": "[15/35] net/ionic: free all buffers during Rx queue stop", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "d951941e128f42f26344ca451d61da7d7a743d6a", "submitter": { "id": 2861, "url": "http://patches.dpdk.org/api/people/2861/?format=api", "name": "Andrew Boyer", "email": "Andrew.Boyer@amd.com" }, "delegate": { "id": 319, "url": "http://patches.dpdk.org/api/users/319/?format=api", "username": "fyigit", "first_name": "Ferruh", "last_name": "Yigit", "email": "ferruh.yigit@amd.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/patch/20221007174336.54354-16-andrew.boyer@amd.com/mbox/", "series": [ { "id": 25037, "url": "http://patches.dpdk.org/api/series/25037/?format=api", "web_url": "http://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": "http://patches.dpdk.org/series/25037/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/117595/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/117595/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 03BE8A04FD;\n\tFri, 7 Oct 2022 19:46:04 +0200 (CEST)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 1008A42BB9;\n\tFri, 7 Oct 2022 19:44:47 +0200 (CEST)", "from NAM02-DM3-obe.outbound.protection.outlook.com\n (mail-dm3nam02on2078.outbound.protection.outlook.com [40.107.95.78])\n by mails.dpdk.org (Postfix) with ESMTP id 7C64A42BAD\n for <dev@dpdk.org>; Fri, 7 Oct 2022 19:44:43 +0200 (CEST)", "from MW4PR03CA0005.namprd03.prod.outlook.com (2603:10b6:303:8f::10)\n by SN7PR12MB6864.namprd12.prod.outlook.com (2603:10b6:806:263::10)\n with Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5676.24; Fri, 7 Oct\n 2022 17:44:41 +0000", "from CO1NAM11FT055.eop-nam11.prod.protection.outlook.com\n (2603:10b6:303:8f:cafe::8c) by MW4PR03CA0005.outlook.office365.com\n (2603:10b6:303:8f::10) 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:44:41 +0000", "from SATLEXMB04.amd.com (165.204.84.17) by\n CO1NAM11FT055.mail.protection.outlook.com (10.13.175.129) 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:44:40 +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:44:39 -0500" ], "ARC-Seal": "i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none;\n b=AWbKlKXb1XanufCELeOkBPw/gfJn5FmOSrByAqjjih6BllAO9Ubdw+W0ZDaV6KmGk0Ne9PAZSFR9dWYtHy3eXYXXd5AUXYAj/SUVh537afOLeezQhxNrm/aJMdrumW1gUW8Aknmz+wlQwQIyMQgRMmcoe9yUTI2rBR+/TBrMrM/Y11on9x6lQuPuHiWSAkqGJkf4Dm8LbTZ9wZnBMFHXl8eUl8UfUG6jIqKN/Ouxrzd/erJOqKbqnscSSek1jSNx0AY3bCRWcdfnUF7VuD7c9wwEAvdnDbghcEKXq+7TYB9sQY6/1PiCayJa3hxUkIPgS8imVPn7Pj34LX5MIDED9w==", "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=7RhvggWNLzhmXCqos9vjFZxKP84MIXmmsJ55mHfxVG0=;\n b=J7tbXNoqsIzUEsHV58U69WIWMJNBMDgPW2EwpCtQHN8cNojkEk2oIISpwdR5Gois1dEzvDgc9j4vgTzhuFyfhnwB4c0p1+XpnmqkbmHLWmTq5qY8fhzT3o9FXozIPzV/iMIyJGr+5hVCR2z3DM75vuP8oFk6jdZWBMu+N21qLYZNhzN2C6BmnEO/GtpPaLA+mGIIp+5SAB3U7ttG5JP+diSjOf7snzHQNABr7Gsp8gr596iW29JGaI1dTCmCEWpE2QD6uQKaM6lSbjPLZMkoLTHa0T4tuyD8NA6tbBAwHRTRLImuxQ4Xmv/ofj2csMGlvqC2c7ft14voMsV3KUmH8w==", "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=7RhvggWNLzhmXCqos9vjFZxKP84MIXmmsJ55mHfxVG0=;\n b=4t5tUfTLnD3mjhWA09fdTG7M5QhkaKuKv+BrgO87CjD8U7QU6we35+q0FZPWYebXnR3yuAtmP84bygksKoACMX7BBHC2U9wQ6AgFLMCsqrb12mv2QnCOGJN+V/XXJODCFRqhuhHxsRJhAoeQ/61v/VoAgX6n40D2U9rsdpDDIyU=", "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 15/35] net/ionic: free all buffers during Rx queue stop", "Date": "Fri, 7 Oct 2022 10:43:16 -0700", "Message-ID": "<20221007174336.54354-16-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": "CO1NAM11FT055:EE_|SN7PR12MB6864:EE_", "X-MS-Office365-Filtering-Correlation-Id": "8f48798a-dc68-4cbe-3335-08daa88b9ccc", "X-MS-Exchange-SenderADCheck": "1", "X-MS-Exchange-AntiSpam-Relay": "0", "X-Microsoft-Antispam": "BCL:0;", "X-Microsoft-Antispam-Message-Info": "\n XIk0nswQ5wphB5OlfZkN5SRRQRJ1RXRf7bFw3KoeU5kY4O502g/2TpNYpZaKZ3wCGAA+0LViHgH+vmaecK21Dl+v9o7neuNSWp1ylnoK4lIgjEfgkzKXrJLtqXMzzNedk1LYguIMtpcFELzxnGdrYsovQUzkSyBvwDJ/2UHbkto1i5bTw+hgTFEzwW4ht4tP6D4kA4aDp3m/pBFzFspKtOQg5pNq4vBCjV0Gd/VlJuhBmQkfPggelxolVdA8faRequJZFP7uSjn5x4ASMJ8i5wr/tXXkX0w3tsAmZNVMOXhrY/Uh8oTsV/HQBhvSMvExIQPfIuYSJsYbkX5vd9yiyin3iFmpWFyhwcq+DYLTKrsKjLUHgpeGmZpBR6O0GJ42Fns6mCuAI8uGKvpOUREcGpO/BJFYpFyg8SPap5oZ+wc89flwa+paN75G3hWr6uvPL/J2u84rwjVInXbji033DnS57oAriFQ4E1U5aL9riZ6Fu/0U9pSd98pcp6TGZvp5dvnwOC2uFMVBVm0xCOXSpkpkyu7cZHm0H39gPVBXHA+LtdDRF9nYaOtaO/UWYyHOAr7wQJlkwDX3UKeTgAJ5bl3259QosMNI2zZi9ypuG2T42zmZAxNx/COBJtdIEM6N+KzeTQNm+J9DcEEa2hb/FyAZG7Ke4xJf6acl+32st8a3End8vy5u5H3w9ps+xm2l7AFTfiDBTrsTYlhXTGsUfu8c7iJrfUn7UFFyW2KwG29BKVEjerEjVthrLDUyCSzgv8/YSt6raETLPvRE+756+u0EbrAfRVcnRTd6ye49Qn2TTZID4sqhPu6EseTfL5J+", "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)(346002)(396003)(136003)(376002)(451199015)(40470700004)(46966006)(36840700001)(70206006)(1076003)(356005)(336012)(81166007)(2616005)(82740400003)(186003)(47076005)(36860700001)(426003)(83380400001)(44832011)(26005)(5660300002)(82310400005)(16526019)(40480700001)(8936002)(478600001)(2906002)(40460700003)(41300700001)(6666004)(8676002)(70586007)(316002)(4326008)(30864003)(6916009)(36756003)(86362001)(36900700001);\n DIR:OUT; SFP:1101;", "X-OriginatorOrg": "amd.com", "X-MS-Exchange-CrossTenant-OriginalArrivalTime": "07 Oct 2022 17:44:40.9176 (UTC)", "X-MS-Exchange-CrossTenant-Network-Message-Id": "\n 8f48798a-dc68-4cbe-3335-08daa88b9ccc", "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 CO1NAM11FT055.eop-nam11.prod.protection.outlook.com", "X-MS-Exchange-CrossTenant-AuthAs": "Anonymous", "X-MS-Exchange-CrossTenant-FromEntityHeader": "HybridOnPrem", "X-MS-Exchange-Transport-CrossTenantHeadersStamped": "SN7PR12MB6864", "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": "Free all of the mbufs in the receive queue when the queue is\nstopped. This will allow them to be resized when the MTU is\nchanged.\n\nSigned-off-by: Andrew Boyer <andrew.boyer@amd.com>\n---\n drivers/net/ionic/ionic_dev.c | 16 +++++\n drivers/net/ionic/ionic_dev.h | 2 +\n drivers/net/ionic/ionic_lif.c | 32 ++++-----\n drivers/net/ionic/ionic_lif.h | 4 --\n drivers/net/ionic/ionic_rxtx.c | 116 +++++++++++++++------------------\n 5 files changed, 81 insertions(+), 89 deletions(-)", "diff": "diff --git a/drivers/net/ionic/ionic_dev.c b/drivers/net/ionic/ionic_dev.c\nindex d0d2ab1b17..e31ce4b2ef 100644\n--- a/drivers/net/ionic/ionic_dev.c\n+++ b/drivers/net/ionic/ionic_dev.c\n@@ -317,6 +317,15 @@ ionic_cq_init(struct ionic_cq *cq, uint16_t num_descs)\n \treturn 0;\n }\n \n+void\n+ionic_cq_reset(struct ionic_cq *cq)\n+{\n+\tcq->tail_idx = 0;\n+\tcq->done_color = 1;\n+\n+\tmemset(cq->base, 0, sizeof(struct ionic_nop_comp) * cq->num_descs);\n+}\n+\n void\n ionic_cq_map(struct ionic_cq *cq, void *base, rte_iova_t base_pa)\n {\n@@ -379,3 +388,10 @@ ionic_q_sg_map(struct ionic_queue *q, void *base, rte_iova_t base_pa)\n \tq->sg_base = base;\n \tq->sg_base_pa = base_pa;\n }\n+\n+void\n+ionic_q_reset(struct ionic_queue *q)\n+{\n+\tq->head_idx = 0;\n+\tq->tail_idx = 0;\n+}\ndiff --git a/drivers/net/ionic/ionic_dev.h b/drivers/net/ionic/ionic_dev.h\nindex 3b8e9fc3c2..f72c05342c 100644\n--- a/drivers/net/ionic/ionic_dev.h\n+++ b/drivers/net/ionic/ionic_dev.h\n@@ -225,6 +225,7 @@ struct ionic_doorbell __iomem *ionic_db_map(struct ionic_lif *lif,\n \tstruct ionic_queue *q);\n \n int ionic_cq_init(struct ionic_cq *cq, uint16_t num_descs);\n+void ionic_cq_reset(struct ionic_cq *cq);\n void ionic_cq_map(struct ionic_cq *cq, void *base, rte_iova_t base_pa);\n typedef bool (*ionic_cq_cb)(struct ionic_cq *cq, uint16_t cq_desc_index,\n \t\tvoid *cb_arg);\n@@ -232,6 +233,7 @@ uint32_t ionic_cq_service(struct ionic_cq *cq, uint32_t work_to_do,\n \tionic_cq_cb cb, void *cb_arg);\n \n int ionic_q_init(struct ionic_queue *q, uint32_t index, uint16_t num_descs);\n+void ionic_q_reset(struct ionic_queue *q);\n void ionic_q_map(struct ionic_queue *q, void *base, rte_iova_t base_pa);\n void ionic_q_sg_map(struct ionic_queue *q, void *base, rte_iova_t base_pa);\n \ndiff --git a/drivers/net/ionic/ionic_lif.c b/drivers/net/ionic/ionic_lif.c\nindex 220a4fdacb..578e36e60c 100644\n--- a/drivers/net/ionic/ionic_lif.c\n+++ b/drivers/net/ionic/ionic_lif.c\n@@ -30,25 +30,7 @@ static const uint8_t ionic_qtype_vers[IONIC_QTYPE_MAX] = {\n static int ionic_lif_addr_add(struct ionic_lif *lif, const uint8_t *addr);\n static int ionic_lif_addr_del(struct ionic_lif *lif, const uint8_t *addr);\n \n-int\n-ionic_qcq_enable(struct ionic_qcq *qcq)\n-{\n-\tstruct ionic_queue *q = &qcq->q;\n-\tstruct ionic_lif *lif = qcq->lif;\n-\tstruct ionic_admin_ctx ctx = {\n-\t\t.pending_work = true,\n-\t\t.cmd.q_control = {\n-\t\t\t.opcode = IONIC_CMD_Q_CONTROL,\n-\t\t\t.type = q->type,\n-\t\t\t.index = rte_cpu_to_le_32(q->index),\n-\t\t\t.oper = IONIC_Q_ENABLE,\n-\t\t},\n-\t};\n-\n-\treturn ionic_adminq_post_wait(lif, &ctx);\n-}\n-\n-int\n+static int\n ionic_qcq_disable(struct ionic_qcq *qcq)\n {\n \tstruct ionic_queue *q = &qcq->q;\n@@ -133,7 +115,6 @@ ionic_lif_get_abs_stats(const struct ionic_lif *lif, struct rte_eth_stats *stats\n \tfor (i = 0; i < lif->nrxqcqs; i++) {\n \t\tstruct ionic_rx_stats *rx_stats = &lif->rxqcqs[i]->stats;\n \t\tstats->ierrors +=\n-\t\t\trx_stats->no_cb_arg +\n \t\t\trx_stats->bad_cq_status +\n \t\t\trx_stats->no_room +\n \t\t\trx_stats->bad_len;\n@@ -154,7 +135,6 @@ ionic_lif_get_abs_stats(const struct ionic_lif *lif, struct rte_eth_stats *stats\n \t\tstats->q_ipackets[i] = rx_stats->packets;\n \t\tstats->q_ibytes[i] = rx_stats->bytes;\n \t\tstats->q_errors[i] =\n-\t\t\trx_stats->no_cb_arg +\n \t\t\trx_stats->bad_cq_status +\n \t\t\trx_stats->no_room +\n \t\t\trx_stats->bad_len;\n@@ -1146,12 +1126,16 @@ ionic_lif_rss_teardown(struct ionic_lif *lif)\n void\n ionic_lif_txq_deinit(struct ionic_tx_qcq *txq)\n {\n+\tionic_qcq_disable(&txq->qcq);\n+\n \ttxq->flags &= ~IONIC_QCQ_F_INITED;\n }\n \n void\n ionic_lif_rxq_deinit(struct ionic_rx_qcq *rxq)\n {\n+\tionic_qcq_disable(&rxq->qcq);\n+\n \trxq->flags &= ~IONIC_QCQ_F_INITED;\n }\n \n@@ -1488,6 +1472,9 @@ ionic_lif_txq_init(struct ionic_tx_qcq *txq)\n \t\tctx.cmd.q_init.ring_size);\n \tIONIC_PRINT(DEBUG, \"txq_init.ver %u\", ctx.cmd.q_init.ver);\n \n+\tionic_q_reset(q);\n+\tionic_cq_reset(cq);\n+\n \terr = ionic_adminq_post_wait(lif, &ctx);\n \tif (err)\n \t\treturn err;\n@@ -1536,6 +1523,9 @@ ionic_lif_rxq_init(struct ionic_rx_qcq *rxq)\n \t\tctx.cmd.q_init.ring_size);\n \tIONIC_PRINT(DEBUG, \"rxq_init.ver %u\", ctx.cmd.q_init.ver);\n \n+\tionic_q_reset(q);\n+\tionic_cq_reset(cq);\n+\n \terr = ionic_adminq_post_wait(lif, &ctx);\n \tif (err)\n \t\treturn err;\ndiff --git a/drivers/net/ionic/ionic_lif.h b/drivers/net/ionic/ionic_lif.h\nindex c3ae96d25b..b3f0ce720b 100644\n--- a/drivers/net/ionic/ionic_lif.h\n+++ b/drivers/net/ionic/ionic_lif.h\n@@ -39,7 +39,6 @@ struct ionic_tx_stats {\n struct ionic_rx_stats {\n \tuint64_t packets;\n \tuint64_t bytes;\n-\tuint64_t no_cb_arg;\n \tuint64_t bad_cq_status;\n \tuint64_t no_room;\n \tuint64_t bad_len;\n@@ -207,9 +206,6 @@ int ionic_tx_qcq_alloc(struct ionic_lif *lif, uint32_t socket_id,\n \tstruct ionic_tx_qcq **qcq_out);\n void ionic_qcq_free(struct ionic_qcq *qcq);\n \n-int ionic_qcq_enable(struct ionic_qcq *qcq);\n-int ionic_qcq_disable(struct ionic_qcq *qcq);\n-\n int ionic_lif_rxq_init(struct ionic_rx_qcq *rxq);\n void ionic_lif_rxq_deinit(struct ionic_rx_qcq *rxq);\n \ndiff --git a/drivers/net/ionic/ionic_rxtx.c b/drivers/net/ionic/ionic_rxtx.c\nindex 440e655e63..17ffaf6aac 100644\n--- a/drivers/net/ionic/ionic_rxtx.c\n+++ b/drivers/net/ionic/ionic_rxtx.c\n@@ -47,6 +47,34 @@\n #include \"ionic_lif.h\"\n #include \"ionic_rxtx.h\"\n \n+static void\n+ionic_empty_array(void **array, uint32_t cnt, uint16_t idx)\n+{\n+\tuint32_t i;\n+\n+\tfor (i = idx; i < cnt; i++)\n+\t\tif (array[i])\n+\t\t\trte_pktmbuf_free_seg(array[i]);\n+\n+\tmemset(array, 0, sizeof(void *) * cnt);\n+}\n+\n+static void __rte_cold\n+ionic_tx_empty(struct ionic_tx_qcq *txq)\n+{\n+\tstruct ionic_queue *q = &txq->qcq.q;\n+\n+\tionic_empty_array(q->info, q->num_descs, 0);\n+}\n+\n+static void __rte_cold\n+ionic_rx_empty(struct ionic_rx_qcq *rxq)\n+{\n+\tstruct ionic_queue *q = &rxq->qcq.q;\n+\n+\tionic_empty_array(q->info, q->num_descs, 0);\n+}\n+\n /*********************************************************************\n *\n * TX functions\n@@ -121,21 +149,16 @@ void __rte_cold\n ionic_dev_tx_queue_release(struct rte_eth_dev *dev, uint16_t qid)\n {\n \tstruct ionic_tx_qcq *txq = dev->data->tx_queues[qid];\n-\tstruct ionic_tx_stats *stats = &txq->stats;\n \n \tIONIC_PRINT_CALL();\n \n-\tIONIC_PRINT(DEBUG, \"TX queue %u pkts %ju tso %ju\",\n-\t\ttxq->qcq.q.index, stats->packets, stats->tso);\n-\n-\tionic_lif_txq_deinit(txq);\n-\n \tionic_qcq_free(&txq->qcq);\n }\n \n int __rte_cold\n ionic_dev_tx_queue_stop(struct rte_eth_dev *eth_dev, uint16_t tx_queue_id)\n {\n+\tstruct ionic_tx_stats *stats;\n \tstruct ionic_tx_qcq *txq;\n \n \tIONIC_PRINT(DEBUG, \"Stopping TX queue %u\", tx_queue_id);\n@@ -150,9 +173,14 @@ ionic_dev_tx_queue_stop(struct rte_eth_dev *eth_dev, uint16_t tx_queue_id)\n \t * before disabling Tx queue\n \t */\n \n-\tionic_qcq_disable(&txq->qcq);\n+\tionic_lif_txq_deinit(txq);\n \n-\tionic_tx_flush(txq);\n+\t/* Free all buffers from descriptor ring */\n+\tionic_tx_empty(txq);\n+\n+\tstats = &txq->stats;\n+\tIONIC_PRINT(DEBUG, \"TX queue %u pkts %ju tso %ju\",\n+\t\ttxq->qcq.q.index, stats->packets, stats->tso);\n \n \treturn 0;\n }\n@@ -236,13 +264,9 @@ ionic_dev_tx_queue_start(struct rte_eth_dev *eth_dev, uint16_t tx_queue_id)\n \tIONIC_PRINT(DEBUG, \"Starting TX queue %u, %u descs\",\n \t\ttx_queue_id, txq->qcq.q.num_descs);\n \n-\tif (!(txq->flags & IONIC_QCQ_F_INITED)) {\n-\t\terr = ionic_lif_txq_init(txq);\n-\t\tif (err)\n-\t\t\treturn err;\n-\t} else {\n-\t\tionic_qcq_enable(&txq->qcq);\n-\t}\n+\terr = ionic_lif_txq_init(txq);\n+\tif (err)\n+\t\treturn err;\n \n \ttx_queue_state[tx_queue_id] = RTE_ETH_QUEUE_STATE_STARTED;\n \n@@ -648,42 +672,16 @@ ionic_rxq_info_get(struct rte_eth_dev *dev, uint16_t queue_id,\n \tqinfo->conf.offloads = dev->data->dev_conf.rxmode.offloads;\n }\n \n-static void __rte_cold\n-ionic_rx_empty(struct ionic_rx_qcq *rxq)\n-{\n-\tstruct ionic_queue *q = &rxq->qcq.q;\n-\tstruct rte_mbuf *mbuf;\n-\tvoid **info;\n-\n-\twhile (q->tail_idx != q->head_idx) {\n-\t\tinfo = IONIC_INFO_PTR(q, q->tail_idx);\n-\t\tmbuf = info[0];\n-\t\trte_mempool_put(rxq->mb_pool, mbuf);\n-\n-\t\tq->tail_idx = Q_NEXT_TO_SRVC(q, 1);\n-\t}\n-}\n-\n void __rte_cold\n ionic_dev_rx_queue_release(struct rte_eth_dev *dev, uint16_t qid)\n {\n \tstruct ionic_rx_qcq *rxq = dev->data->rx_queues[qid];\n-\tstruct ionic_rx_stats *stats;\n \n \tif (!rxq)\n \t\treturn;\n \n \tIONIC_PRINT_CALL();\n \n-\tstats = &rxq->stats;\n-\n-\tIONIC_PRINT(DEBUG, \"RX queue %u pkts %ju mtod %ju\",\n-\t\trxq->qcq.q.index, stats->packets, stats->mtods);\n-\n-\tionic_rx_empty(rxq);\n-\n-\tionic_lif_rxq_deinit(rxq);\n-\n \tionic_qcq_free(&rxq->qcq);\n }\n \n@@ -787,17 +785,6 @@ ionic_rx_clean(struct ionic_rx_qcq *rxq,\n \n \trxm = info[0];\n \n-\tif (!rx_svc) {\n-\t\tstats->no_cb_arg++;\n-\t\t/* Flush */\n-\t\trte_pktmbuf_free(rxm);\n-\t\t/*\n-\t\t * Note: rte_mempool_put is faster with no segs\n-\t\t * rte_mempool_put(rxq->mb_pool, rxm);\n-\t\t */\n-\t\treturn;\n-\t}\n-\n \tif (cq_desc->status) {\n \t\tstats->bad_cq_status++;\n \t\tionic_rx_recycle(q, q_desc_index, rxm);\n@@ -1028,13 +1015,9 @@ ionic_dev_rx_queue_start(struct rte_eth_dev *eth_dev, uint16_t rx_queue_id)\n \tIONIC_PRINT(DEBUG, \"Starting RX queue %u, %u descs, size %u\",\n \t\trx_queue_id, rxq->qcq.q.num_descs, rxq->frame_size);\n \n-\tif (!(rxq->flags & IONIC_QCQ_F_INITED)) {\n-\t\terr = ionic_lif_rxq_init(rxq);\n-\t\tif (err)\n-\t\t\treturn err;\n-\t} else {\n-\t\tionic_qcq_enable(&rxq->qcq);\n-\t}\n+\terr = ionic_lif_rxq_init(rxq);\n+\tif (err)\n+\t\treturn err;\n \n \t/* Allocate buffers for descriptor rings */\n \tif (ionic_rx_fill(rxq) != 0) {\n@@ -1103,19 +1086,24 @@ ionic_rxq_service(struct ionic_rx_qcq *rxq, uint32_t work_to_do,\n int __rte_cold\n ionic_dev_rx_queue_stop(struct rte_eth_dev *eth_dev, uint16_t rx_queue_id)\n {\n+\tuint8_t *rx_queue_state = eth_dev->data->rx_queue_state;\n+\tstruct ionic_rx_stats *stats;\n \tstruct ionic_rx_qcq *rxq;\n \n \tIONIC_PRINT(DEBUG, \"Stopping RX queue %u\", rx_queue_id);\n \n \trxq = eth_dev->data->rx_queues[rx_queue_id];\n \n-\teth_dev->data->rx_queue_state[rx_queue_id] =\n-\t\tRTE_ETH_QUEUE_STATE_STOPPED;\n+\trx_queue_state[rx_queue_id] = RTE_ETH_QUEUE_STATE_STOPPED;\n+\n+\tionic_lif_rxq_deinit(rxq);\n \n-\tionic_qcq_disable(&rxq->qcq);\n+\t/* Free all buffers from descriptor ring */\n+\tionic_rx_empty(rxq);\n \n-\t/* Flush */\n-\tionic_rxq_service(rxq, -1, NULL);\n+\tstats = &rxq->stats;\n+\tIONIC_PRINT(DEBUG, \"RX queue %u pkts %ju mtod %ju\",\n+\t\trxq->qcq.q.index, stats->packets, stats->mtods);\n \n \treturn 0;\n }\n", "prefixes": [ "15/35" ] }