Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/45921/?format=api
{ "id": 45921, "url": "http://patches.dpdk.org/api/patches/45921/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/1538551336-19259-4-git-send-email-andrzej.ostruszka@gmail.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": "<1538551336-19259-4-git-send-email-andrzej.ostruszka@gmail.com>", "list_archive_url": "https://inbox.dpdk.org/dev/1538551336-19259-4-git-send-email-andrzej.ostruszka@gmail.com", "date": "2018-10-03T07:22:11", "name": "[v7,3/8] net/mvneta: support for setting of MTU", "commit_ref": null, "pull_url": null, "state": "accepted", "archived": true, "hash": "84cc5761eb77c68d36e399730af47fec3c085316", "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/1538551336-19259-4-git-send-email-andrzej.ostruszka@gmail.com/mbox/", "series": [ { "id": 1655, "url": "http://patches.dpdk.org/api/series/1655/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=1655", "date": "2018-10-03T07:22:08", "name": "Add Marvell NETA PMD", "version": 7, "mbox": "http://patches.dpdk.org/series/1655/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/45921/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/45921/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 EBA3F44BE;\n\tWed, 3 Oct 2018 09:22:28 +0200 (CEST)", "from mail-lj1-f195.google.com (mail-lj1-f195.google.com\n\t[209.85.208.195]) by dpdk.org (Postfix) with ESMTP id D58F02C37\n\tfor <dev@dpdk.org>; Wed, 3 Oct 2018 09:22:22 +0200 (CEST)", "by mail-lj1-f195.google.com with SMTP id p89-v6so4119317ljb.3\n\tfor <dev@dpdk.org>; Wed, 03 Oct 2018 00:22:22 -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\tf73-v6sm123659lfl.80.2018.10.03.00.22.21\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128);\n\tWed, 03 Oct 2018 00:22:21 -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=MVh48DRO/EzfdVkJVYpCQzNYoayZE/d3qIpcOPKTuys=;\n\tb=yW4H0AXDNMuylN3mLO93qhatRB4zpvo93EF2NjH6y75+HExfb8UFlcPHZXWZw/1lqg\n\tX/U/hHDreAxKKISJMXQxM5B6GZ/QoB3SnFKHTb77iqE1xG3K2HR1HOSXCVbuC7XSZaot\n\tRivn5rbgS/TuSNszHtcIIcw/taElRta4zqn62I8WknC/eG/oIxNuywmHJE1zOlJZp86B\n\t3yZQ6N7C9Qsckt8Yy0UKrGDe2USpV0E57NE8EgC49gExpK2kiBkrqM0pR3BXqPOgI8/D\n\tfiAbF9G6o3TlNZh9XexFq98QaXuvpuMZeDXkJQYbVHv8GFMLXNsCsAmSYRkweavjzcSV\n\td3ug==", "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=MVh48DRO/EzfdVkJVYpCQzNYoayZE/d3qIpcOPKTuys=;\n\tb=oozF6yNgm5NwLvA7daRTCP0kByCsxeq+Tw7CmApENqMlpdJmoiTn/ObtXmVc3W9GG3\n\t99qj9thsw1aQazzxPbbVls53G0bntjT6hNCmNAqz4JMDQJfgKLvVc72xLCfobzOVSV3D\n\tIbGzjl9FE8gCBEcQhEtdt1uc1PC2jo3ALroOjNGcSyKHTvuDrFdLJPHkhUjcjW6vyD1f\n\tTKgWzVii+yrUEeb5XbZzKrpoqMZdxiJEhbI6Qe7x5SG4qOoT0K3KAXXY1tEpT80JtCMH\n\tgithE3dKHAMJvIyU0f49HwURnVCpc9VY+WwfeU1W9A8tZF6HTBuFiP4JpP1PJuo70tDx\n\t/l2A==", "X-Gm-Message-State": "ABuFfoi3aSalAAVW8/xR++gnYZu6wWUBh516oC1DHL1BnY6OYsoAa4WF\n\t35OJe2ZppSKV5fw1ND2hMPWOj2ubN9uesQ==", "X-Google-Smtp-Source": "ACcGV63Q5rQe8ATfDGh0Rr/OY0uF1z7QO3q/gLotshGR3vOJFdByrhGjbv+QenN02pZ3hcoKntDgdw==", "X-Received": "by 2002:a2e:458b:: with SMTP id\n\ts133-v6mr182229lja.60.1538551342255; \n\tWed, 03 Oct 2018 00:22:22 -0700 (PDT)", "From": "Andrzej Ostruszka <amo@semihalf.com>", "X-Google-Original-From": "Andrzej Ostruszka <andrzej.ostruszka@gmail.com>", "To": "dev@dpdk.org", "Cc": "mw@semihalf.com, nadavh@marvell.com, zr@semihalf.com, tdu@semihalf.com", "Date": "Wed, 3 Oct 2018 09:22:11 +0200", "Message-Id": "<1538551336-19259-4-git-send-email-andrzej.ostruszka@gmail.com>", "X-Mailer": "git-send-email 2.7.4", "In-Reply-To": "<1538551336-19259-1-git-send-email-andrzej.ostruszka@gmail.com>", "References": "<1538385970-21260-1-git-send-email-andrzej.ostruszka@gmail.com>\n\t<1538551336-19259-1-git-send-email-andrzej.ostruszka@gmail.com>", "Subject": "[dpdk-dev] [PATCH v7 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 7ea003c..aadeacb 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 146b249..24f87e2 100644\n--- a/drivers/net/mvneta/mvneta_ethdev.c\n+++ b/drivers/net/mvneta/mvneta_ethdev.c\n@@ -218,6 +218,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@@ -307,6 +378,12 @@ mvneta_dev_start(struct rte_eth_dev *dev)\n \tif (ret)\n \t\tgoto out;\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@@ -408,6 +485,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": [ "v7", "3/8" ] }