Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/44945/?format=api
{ "id": 44945, "url": "http://patches.dpdk.org/api/patches/44945/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/1537369294-17099-4-git-send-email-amo@semihalf.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": "<1537369294-17099-4-git-send-email-amo@semihalf.com>", "list_archive_url": "https://inbox.dpdk.org/dev/1537369294-17099-4-git-send-email-amo@semihalf.com", "date": "2018-09-19T15:01:29", "name": "[v4,3/8] net/mvneta: support for setting of MTU", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "8874f52cc74926cfd61ecb3473025e8c075e1d01", "submitter": { "id": 1112, "url": "http://patches.dpdk.org/api/people/1112/?format=api", "name": "Andrzej Ostruszka", "email": "amo@semihalf.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/1537369294-17099-4-git-send-email-amo@semihalf.com/mbox/", "series": [ { "id": 1394, "url": "http://patches.dpdk.org/api/series/1394/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=1394", "date": "2018-09-19T15:01:26", "name": "Add Marvell NETA PMD", "version": 4, "mbox": "http://patches.dpdk.org/series/1394/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/44945/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/44945/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@dpdk.org", "Delivered-To": "patchwork@dpdk.org", "Received": [ "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id F055B4F9B;\n\tWed, 19 Sep 2018 17:01:44 +0200 (CEST)", "from mail-lf1-f65.google.com (mail-lf1-f65.google.com\n\t[209.85.167.65]) by dpdk.org (Postfix) with ESMTP id BD0D24C96\n\tfor <dev@dpdk.org>; Wed, 19 Sep 2018 17:01:40 +0200 (CEST)", "by mail-lf1-f65.google.com with SMTP id c29-v6so5444113lfj.1\n\tfor <dev@dpdk.org>; Wed, 19 Sep 2018 08:01:40 -0700 (PDT)", "from amok.semihalf.local (31-172-191-173.noc.fibertech.net.pl.\n\t[31.172.191.173]) by smtp.googlemail.com with ESMTPSA id\n\tz3-v6sm1107711lfj.89.2018.09.19.08.01.39\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128);\n\tWed, 19 Sep 2018 08:01:39 -0700 (PDT)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=semihalf-com.20150623.gappssmtp.com; s=20150623;\n\th=from:to:cc:subject:date:message-id:in-reply-to:references;\n\tbh=TzTNspwkOECoxw0bkSuJkzZxLyl1ZzyjJ56zALZV6k4=;\n\tb=AjIDXtTvfC6LoKwEYBdUcwZrl9A55RsnqUzwgU7Wc13c8LNzHVyTJTzMw3s0KXCHsd\n\tbLVIBE649qKhgBTwmJf0E515tkGtT3/qGUSMTEIIDQioZbN/X8Et7rGsQK5qXYWVS2Gc\n\tf1swkswXbGIbKICjjKHf7TDUfMMAYig28+KRVxSVEid9iH/HjNWrpZlwXP5DlQrNwJft\n\t3Qa/rxu2c0Mv1AblRc6vazKdf1REUdkE9UR3wA4yy+9/uYhJwn/Swn5uXFtpR9PeoSEv\n\tCQiJpkoTeKoqjpzIz9etd2Fwlxn7ng2MejwGb7zpE5OVxNe9CJHu7Prd4WXHeP0HI1nt\n\tj5rQ==", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to\n\t:references;\n\tbh=TzTNspwkOECoxw0bkSuJkzZxLyl1ZzyjJ56zALZV6k4=;\n\tb=Ma81m1XyyoilbSzJevrpEbXNTJ0vnn1xYykG1BRMVoeKYPLfTtuEquaN6qkEcttBwH\n\t0bYEZRyMoXgL2GocIvlKwjRu+jIAWJasJEhwICy38Q86D3z1jULh766Z+/CqRCdh90az\n\tZ5mKOFF5pTFf0gHuUVrCJ357K/ICOFSuTn9uivTmMLtDRBEdq/xSWEPjBGPq2MFq0nG9\n\t1iv/13D4nPpWQwD9HGBC+KaKyc5fB9KSm3/X/OyRczBywOIMv6zmI5+Z+XpZtYdRaAfI\n\tDh1CqMZprA3IwiiFHRHT17hWsjRD7Fy8enhpToR89MIbnulKMDu8qr1ijLF8J16mqyCy\n\tkYOg==", "X-Gm-Message-State": "APzg51CngLvEq1/F6XP0jZOpl4Vz/Rn+AeehDutpFg3qZy+emOZfsRHC\n\t3cVfZRWUYP+Ye3B2PR9bL5R1BZOq4Bc=", "X-Google-Smtp-Source": "ANB0VdYmqNDuUa4nE1GKDAaXMKt0ZKoRSyF4RlS1HwsbxxsaeUNh8olXi+QH7MJbdeJsYSDp6zlrrQ==", "X-Received": "by 2002:a19:a141:: with SMTP id\n\tk62-v6mr22668850lfe.18.1537369300058; \n\tWed, 19 Sep 2018 08:01:40 -0700 (PDT)", "From": "Andrzej Ostruszka <amo@semihalf.com>", "To": "dev@dpdk.org", "Cc": "mw@semihalf.com, zr@semihalf.com, tdu@semihalf.com, nadavh@marvell.com", "Date": "Wed, 19 Sep 2018 17:01:29 +0200", "Message-Id": "<1537369294-17099-4-git-send-email-amo@semihalf.com>", "X-Mailer": "git-send-email 2.7.4", "In-Reply-To": "<1537369294-17099-1-git-send-email-amo@semihalf.com>", "References": "<1535720386-18775-1-git-send-email-amo@semihalf.com>\n\t<1537369294-17099-1-git-send-email-amo@semihalf.com>", "Subject": "[dpdk-dev] [PATCH v4 3/8] net/mvneta: support for setting of MTU", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.15", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dev>,\n\t<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\t<mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "From: Zyta Szpak <zr@semihalf.com>\n\nAdd callback for setting of MTU.\n\nSigned-off-by: Natalie Samsonov <nsamsono@marvell.com>\nSigned-off-by: Zyta Szpak <zr@semihalf.com>\n---\n doc/guides/nics/features/mvneta.ini | 1 +\n doc/guides/nics/mvneta.rst | 1 +\n drivers/net/mvneta/mvneta_ethdev.c | 78 +++++++++++++++++++++++++++++++++++++\n 3 files changed, 80 insertions(+)", "diff": "diff --git a/doc/guides/nics/features/mvneta.ini b/doc/guides/nics/features/mvneta.ini\nindex 0a89e2f..bc4e400 100644\n--- a/doc/guides/nics/features/mvneta.ini\n+++ b/doc/guides/nics/features/mvneta.ini\n@@ -5,6 +5,7 @@\n ;\n [Features]\n Speed capabilities = Y\n+MTU update = Y\n Jumbo frame = Y\n CRC offload = Y\n L3 checksum offload = Y\ndiff --git a/doc/guides/nics/mvneta.rst b/doc/guides/nics/mvneta.rst\nindex 9d25c40..55ffe57 100644\n--- a/doc/guides/nics/mvneta.rst\n+++ b/doc/guides/nics/mvneta.rst\n@@ -30,6 +30,7 @@ Features of the MVNETA PMD are:\n - tx/rx_burst\n - Speed capabilities\n - Jumbo frame\n+- MTU update\n - CRC offload\n - L3 checksum offload\n - L4 checksum offload\ndiff --git a/drivers/net/mvneta/mvneta_ethdev.c b/drivers/net/mvneta/mvneta_ethdev.c\nindex 3d6f97b..80ab9ba 100644\n--- a/drivers/net/mvneta/mvneta_ethdev.c\n+++ b/drivers/net/mvneta/mvneta_ethdev.c\n@@ -221,6 +221,77 @@ mvneta_dev_supported_ptypes_get(struct rte_eth_dev *dev __rte_unused)\n }\n \n /**\n+ * DPDK callback to change the MTU.\n+ *\n+ * Setting the MTU affects hardware MRU (packets larger than the MRU\n+ * will be dropped).\n+ *\n+ * @param dev\n+ * Pointer to Ethernet device structure.\n+ * @param mtu\n+ * New MTU.\n+ *\n+ * @return\n+ * 0 on success, negative error value otherwise.\n+ */\n+static int\n+mvneta_mtu_set(struct rte_eth_dev *dev, uint16_t mtu)\n+{\n+\tstruct mvneta_priv *priv = dev->data->dev_private;\n+\tuint16_t mbuf_data_size = 0; /* SW buffer size */\n+\tuint16_t mru;\n+\tint ret;\n+\n+\tmru = MRVL_NETA_MTU_TO_MRU(mtu);\n+\t/*\n+\t * min_rx_buf_size is equal to mbuf data size\n+\t * if pmd didn't set it differently\n+\t */\n+\tmbuf_data_size = dev->data->min_rx_buf_size - RTE_PKTMBUF_HEADROOM;\n+\t/* Prevent PMD from:\n+\t * - setting mru greater than the mbuf size resulting in\n+\t * hw and sw buffer size mismatch\n+\t * - setting mtu that requires the support of scattered packets\n+\t * when this feature has not been enabled/supported so far.\n+\t */\n+\tif (!dev->data->scattered_rx &&\n+\t (mru + MRVL_NETA_PKT_OFFS > mbuf_data_size)) {\n+\t\tmru = mbuf_data_size - MRVL_NETA_PKT_OFFS;\n+\t\tmtu = MRVL_NETA_MRU_TO_MTU(mru);\n+\t\tMVNETA_LOG(WARNING, \"MTU too big, max MTU possible limitted by\"\n+\t\t\t\" current mbuf size: %u. Set MTU to %u, MRU to %u\",\n+\t\t\tmbuf_data_size, mtu, mru);\n+\t}\n+\n+\tif (mtu < ETHER_MIN_MTU || mru > MVNETA_PKT_SIZE_MAX) {\n+\t\tMVNETA_LOG(ERR, \"Invalid MTU [%u] or MRU [%u]\", mtu, mru);\n+\t\treturn -EINVAL;\n+\t}\n+\n+\tdev->data->mtu = mtu;\n+\tdev->data->dev_conf.rxmode.max_rx_pkt_len = mru - MV_MH_SIZE;\n+\n+\tif (!priv->ppio)\n+\t\t/* It is OK. New MTU will be set later on mvneta_dev_start */\n+\t\treturn 0;\n+\n+\tret = neta_ppio_set_mru(priv->ppio, mru);\n+\tif (ret) {\n+\t\tMVNETA_LOG(ERR, \"Failed to change MRU\");\n+\t\treturn ret;\n+\t}\n+\n+\tret = neta_ppio_set_mtu(priv->ppio, mtu);\n+\tif (ret) {\n+\t\tMVNETA_LOG(ERR, \"Failed to change MTU\");\n+\t\treturn ret;\n+\t}\n+\tMVNETA_LOG(INFO, \"MTU changed to %u, MRU = %u\", mtu, mru);\n+\n+\treturn 0;\n+}\n+\n+/**\n * DPDK callback to bring the link up.\n *\n * @param dev\n@@ -318,6 +389,12 @@ mvneta_dev_start(struct rte_eth_dev *dev)\n \t\t}\n \t}\n \n+\tret = mvneta_mtu_set(dev, dev->data->mtu);\n+\tif (ret) {\n+\t\tMVNETA_LOG(ERR, \"Failed to set MTU %d\", dev->data->mtu);\n+\t\tgoto out;\n+\t}\n+\n \tret = mvneta_dev_set_link_up(dev);\n \tif (ret) {\n \t\tMVNETA_LOG(ERR, \"Failed to set link up\");\n@@ -433,6 +510,7 @@ static const struct eth_dev_ops mvneta_ops = {\n \t.dev_set_link_down = mvneta_dev_set_link_down,\n \t.dev_close = mvneta_dev_close,\n \t.mac_addr_set = mvneta_mac_addr_set,\n+\t.mtu_set = mvneta_mtu_set,\n \t.dev_infos_get = mvneta_dev_infos_get,\n \t.dev_supported_ptypes_get = mvneta_dev_supported_ptypes_get,\n \t.rxq_info_get = mvneta_rxq_info_get,\n", "prefixes": [ "v4", "3/8" ] }