Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/97841/?format=api
http://patches.dpdk.org/api/patches/97841/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20210902175955.9202-6-apeksha.gupta@nxp.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": "<20210902175955.9202-6-apeksha.gupta@nxp.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20210902175955.9202-6-apeksha.gupta@nxp.com", "date": "2021-09-02T17:59:55", "name": "[v2,5/5] net/enetfec: add features", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "6af313dbf3d744f6f7c5cb03827fca3b3483eb6f", "submitter": { "id": 1570, "url": "http://patches.dpdk.org/api/people/1570/?format=api", "name": "Apeksha Gupta", "email": "apeksha.gupta@nxp.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/20210902175955.9202-6-apeksha.gupta@nxp.com/mbox/", "series": [ { "id": 18636, "url": "http://patches.dpdk.org/api/series/18636/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=18636", "date": "2021-09-02T17:59:50", "name": "drivers/net: add NXP ENETFEC driver", "version": 2, "mbox": "http://patches.dpdk.org/series/18636/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/97841/comments/", "check": "fail", "checks": "http://patches.dpdk.org/api/patches/97841/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 E01CCA0C4C;\n\tThu, 2 Sep 2021 20:01:31 +0200 (CEST)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id B5E4D41104;\n\tThu, 2 Sep 2021 20:01:12 +0200 (CEST)", "from inva020.nxp.com (inva020.nxp.com [92.121.34.13])\n by mails.dpdk.org (Postfix) with ESMTP id DA22040E09\n for <dev@dpdk.org>; Thu, 2 Sep 2021 20:01:10 +0200 (CEST)", "from inva020.nxp.com (localhost [127.0.0.1])\n by inva020.eu-rdc02.nxp.com (Postfix) with ESMTP id B78811A1F22;\n Thu, 2 Sep 2021 20:01:10 +0200 (CEST)", "from aprdc01srsp001v.ap-rdc01.nxp.com\n (aprdc01srsp001v.ap-rdc01.nxp.com [165.114.16.16])\n by inva020.eu-rdc02.nxp.com (Postfix) with ESMTP id 5480B1A1F37;\n Thu, 2 Sep 2021 20:01:10 +0200 (CEST)", "from lsv03186.swis.in-blr01.nxp.com (lsv03186.swis.in-blr01.nxp.com\n [92.120.146.182])\n by aprdc01srsp001v.ap-rdc01.nxp.com (Postfix) with ESMTP id 5DF45183AD27;\n Fri, 3 Sep 2021 02:01:09 +0800 (+08)" ], "From": "Apeksha Gupta <apeksha.gupta@nxp.com>", "To": "andrew.rybchenko@oktetlabs.ru,\n\tferruh.yigit@intel.com", "Cc": "dev@dpdk.org, hemant.agrawal@nxp.com, sachin.saxena@nxp.com,\n Apeksha Gupta <apeksha.gupta@nxp.com>", "Date": "Thu, 2 Sep 2021 23:29:55 +0530", "Message-Id": "<20210902175955.9202-6-apeksha.gupta@nxp.com>", "X-Mailer": "git-send-email 2.17.1", "In-Reply-To": "<20210902175955.9202-1-apeksha.gupta@nxp.com>", "References": "<20210902175955.9202-1-apeksha.gupta@nxp.com>", "X-Virus-Scanned": "ClamAV using ClamSMTP", "Subject": "[dpdk-dev] [PATCH v2 5/5] net/enetfec: add features", "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 adds checksum and VLAN offloads in enetfec network\npoll mode driver.\n\nSigned-off-by: Sachin Saxena <sachin.saxena@nxp.com>\nSigned-off-by: Apeksha Gupta <apeksha.gupta@nxp.com>\n---\n doc/guides/nics/enetfec.rst | 2 ++\n doc/guides/nics/features/enetfec.ini | 3 ++\n drivers/net/enetfec/enet_ethdev.c | 17 +++++++++-\n drivers/net/enetfec/enet_regs.h | 10 ++++++\n drivers/net/enetfec/enet_rxtx.c | 51 +++++++++++++++++++++++++++-\n 5 files changed, 81 insertions(+), 2 deletions(-)", "diff": "diff --git a/doc/guides/nics/enetfec.rst b/doc/guides/nics/enetfec.rst\nindex 140279caa1..b48a195e67 100644\n--- a/doc/guides/nics/enetfec.rst\n+++ b/doc/guides/nics/enetfec.rst\n@@ -77,6 +77,8 @@ ENETFEC Features\n \n - Basic stats\n - Promiscuous\n+- VLAN offload\n+- L3/L4 checksum offload\n - ARMv8\n \n Supported ENETFEC SoCs\ndiff --git a/doc/guides/nics/features/enetfec.ini b/doc/guides/nics/features/enetfec.ini\nindex 0a151ba193..525df93ec7 100644\n--- a/doc/guides/nics/features/enetfec.ini\n+++ b/doc/guides/nics/features/enetfec.ini\n@@ -6,5 +6,8 @@\n [Features]\n Basic stats = Y\n Promiscuous mode = Y\n+VLAN offload\t\t= Y\n+L3 checksum offload\t= Y\n+L4 checksum offload\t= Y\n ARMv8 = Y\n Usage doc = Y\ndiff --git a/drivers/net/enetfec/enet_ethdev.c b/drivers/net/enetfec/enet_ethdev.c\nindex 6f512d2f96..c31c8a82ba 100644\n--- a/drivers/net/enetfec/enet_ethdev.c\n+++ b/drivers/net/enetfec/enet_ethdev.c\n@@ -107,7 +107,11 @@ enetfec_restart(struct rte_eth_dev *dev)\n \t\tval = rte_read32(fep->hw_baseaddr_v + ENETFEC_RACC);\n \t\t/* align IP header */\n \t\tval |= ENETFEC_RACC_SHIFT16;\n-\t\tval &= ~ENETFEC_RACC_OPTIONS;\n+\t\tif (fep->flag_csum & RX_FLAG_CSUM_EN)\n+\t\t\t/* set RX checksum */\n+\t\t\tval |= ENETFEC_RACC_OPTIONS;\n+\t\telse\n+\t\t\tval &= ~ENETFEC_RACC_OPTIONS;\n \t\trte_write32(rte_cpu_to_le_32(val),\n \t\t\t\tfep->hw_baseaddr_v + ENETFEC_RACC);\n \t\trte_write32(rte_cpu_to_le_32(PKT_MAX_BUF_SIZE),\n@@ -602,9 +606,20 @@ static int\n enetfec_eth_init(struct rte_eth_dev *dev)\n {\n \tstruct enetfec_private *fep = dev->data->dev_private;\n+\tstruct rte_eth_conf *eth_conf = &fep->dev->data->dev_conf;\n+\tuint64_t rx_offloads = eth_conf->rxmode.offloads;\n \n \tfep->full_duplex = FULL_DUPLEX;\n \tdev->dev_ops = &enetfec_ops;\n+\tif (fep->quirks & QUIRK_VLAN)\n+\t\t/* enable hw VLAN support */\n+\t\trx_offloads |= DEV_RX_OFFLOAD_VLAN;\n+\n+\tif (fep->quirks & QUIRK_CSUM) {\n+\t\t/* enable hw accelerator */\n+\t\trx_offloads |= DEV_RX_OFFLOAD_CHECKSUM;\n+\t\tfep->flag_csum |= RX_FLAG_CSUM_EN;\n+\t}\n \trte_eth_dev_probing_finish(dev);\n \n \treturn 0;\ndiff --git a/drivers/net/enetfec/enet_regs.h b/drivers/net/enetfec/enet_regs.h\nindex 5665e19dd3..63cba96733 100644\n--- a/drivers/net/enetfec/enet_regs.h\n+++ b/drivers/net/enetfec/enet_regs.h\n@@ -27,6 +27,12 @@\n #define RX_BD_EMPTY\t((ushort)0x8000) /* BD is empty */\n #define RX_BD_STATS\t((ushort)0x013f) /* All buffer descriptor status bits */\n \n+/* Ethernet receive use control and status of enhanced buffer descriptor */\n+#define BD_ENETFEC_RX_VLAN\t0x00000004\n+\n+#define RX_FLAG_CSUM_EN\t\t(RX_BD_ICE | RX_BD_PCR)\n+#define RX_FLAG_CSUM_ERR\t(RX_BD_ICE | RX_BD_PCR)\n+\n /* Ethernet transmit use control and status of buffer descriptor */\n #define TX_BD_TC\t((ushort)0x0400) /* Transmit CRC */\n #define TX_BD_LAST\t((ushort)0x0800) /* Last in frame */\n@@ -58,6 +64,10 @@\n #define QUIRK_GBIT\t\t(1 << 3)\n /* Controller has extended descriptor buffer */\n #define QUIRK_BUFDESC_EX\t(1 << 4)\n+/* Controller support hardware checksum */\n+#define QUIRK_CSUM\t\t(1 << 5)\n+/* Controller support hardware vlan*/\n+#define QUIRK_VLAN\t\t(1 << 6)\n /* RACC register supported by controller */\n #define QUIRK_RACC\t\t(1 << 12)\n /* i.MX8 ENETFEC IP version added the feature to generate the delayed TXC or\ndiff --git a/drivers/net/enetfec/enet_rxtx.c b/drivers/net/enetfec/enet_rxtx.c\nindex d168c22431..574d6c80a4 100644\n--- a/drivers/net/enetfec/enet_rxtx.c\n+++ b/drivers/net/enetfec/enet_rxtx.c\n@@ -245,9 +245,14 @@ enetfec_recv_pkts(void *rxq1, __rte_unused struct rte_mbuf **rx_pkts,\n \tunsigned short status;\n \tunsigned short pkt_len;\n \tint pkt_received = 0, index = 0;\n-\tvoid *data;\n+\tvoid *data, *mbuf_data;\n+\tuint16_t vlan_tag;\n+\tstruct bufdesc_ex *ebdp = NULL;\n+\tbool vlan_packet_rcvd = false;\n \tstruct enetfec_priv_rx_q *rxq = (struct enetfec_priv_rx_q *)rxq1;\n \tstruct rte_eth_stats *stats = &rxq->fep->stats;\n+\tstruct rte_eth_conf *eth_conf = &rxq->fep->dev->data->dev_conf;\n+\tuint64_t rx_offloads = eth_conf->rxmode.offloads;\n \tpool = rxq->pool;\n \tbdp = rxq->bd.cur;\n #if ENETFEC_LOOPBACK\n@@ -302,6 +307,7 @@ enetfec_recv_pkts(void *rxq1, __rte_unused struct rte_mbuf **rx_pkts,\n \t\tmbuf = rxq->rx_mbuf[index];\n \n \t\tdata = rte_pktmbuf_mtod(mbuf, uint8_t *);\n+\t\tmbuf_data = data;\n \t\trte_prefetch0(data);\n \t\trte_pktmbuf_append((struct rte_mbuf *)mbuf,\n \t\t\t\tpkt_len - 4);\n@@ -311,6 +317,45 @@ enetfec_recv_pkts(void *rxq1, __rte_unused struct rte_mbuf **rx_pkts,\n \n \t\trx_pkts[pkt_received] = mbuf;\n \t\tpkt_received++;\n+\n+\t\t/* Extract the enhanced buffer descriptor */\n+\t\tebdp = NULL;\n+\t\tif (rxq->fep->bufdesc_ex)\n+\t\t\tebdp = (struct bufdesc_ex *)bdp;\n+\n+\t\t/* If this is a VLAN packet remove the VLAN Tag */\n+\t\tvlan_packet_rcvd = false;\n+\t\tif ((rx_offloads & DEV_RX_OFFLOAD_VLAN) &&\n+\t\t\t\trxq->fep->bufdesc_ex &&\n+\t\t\t\t(rte_read32(&ebdp->bd_esc) &\n+\t\t\t\trte_cpu_to_le_32(BD_ENETFEC_RX_VLAN))) {\n+\t\t\t/* Push and remove the vlan tag */\n+\t\t\tstruct rte_vlan_hdr *vlan_header =\n+\t\t\t\t(struct rte_vlan_hdr *)(data + ETH_HLEN);\n+\t\t\tvlan_tag = rte_be_to_cpu_16(vlan_header->vlan_tci);\n+\n+\t\t\tvlan_packet_rcvd = true;\n+\t\t\tmemmove(mbuf_data + VLAN_HLEN, data, ETH_ALEN * 2);\n+\t\t\trte_pktmbuf_adj(mbuf, VLAN_HLEN);\n+\t\t}\n+\n+\t\tif (rxq->fep->bufdesc_ex &&\n+\t\t\t\t(rxq->fep->flag_csum & RX_FLAG_CSUM_EN)) {\n+\t\t\tif ((rte_read32(&ebdp->bd_esc) &\n+\t\t\t\t\trte_cpu_to_le_32(RX_FLAG_CSUM_ERR)) == 0) {\n+\t\t\t\t/* don't check it */\n+\t\t\t\tmbuf->ol_flags = PKT_RX_IP_CKSUM_BAD;\n+\t\t\t} else {\n+\t\t\t\tmbuf->ol_flags = PKT_RX_IP_CKSUM_GOOD;\n+\t\t\t}\n+\t\t}\n+\n+\t\t/* Handle received VLAN packets */\n+\t\tif (vlan_packet_rcvd) {\n+\t\t\tmbuf->vlan_tci = vlan_tag;\n+\t\t\tmbuf->ol_flags |= PKT_RX_VLAN_STRIPPED | PKT_RX_VLAN;\n+\t\t}\n+\n \t\trxq->rx_mbuf[index] = new_mbuf;\n \t\trte_write32(rte_cpu_to_le_32(rte_pktmbuf_iova(new_mbuf)),\n \t\t\t\t&bdp->bd_bufaddr);\n@@ -411,6 +456,10 @@ enetfec_xmit_pkts(void *tx_queue, struct rte_mbuf **tx_pkts, uint16_t nb_pkts)\n \n \t\tif (txq->fep->bufdesc_ex) {\n \t\t\tstruct bufdesc_ex *ebdp = (struct bufdesc_ex *)bdp;\n+\n+\t\t\tif (mbuf->ol_flags == PKT_RX_IP_CKSUM_GOOD)\n+\t\t\t\testatus |= TX_BD_PINS | TX_BD_IINS;\n+\n \t\t\trte_write32(0, &ebdp->bd_bdu);\n \t\t\trte_write32(rte_cpu_to_le_32(estatus),\n \t\t\t\t &ebdp->bd_esc);\n", "prefixes": [ "v2", "5/5" ] }{ "id": 97841, "url": "