Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/30269/?format=api
{ "id": 30269, "url": "http://patches.dpdk.org/api/patches/30269/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/dfc33efe83f3a6fbd0770b85988e519733350d83.1507809961.git.adrien.mazarguil@6wind.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": "<dfc33efe83f3a6fbd0770b85988e519733350d83.1507809961.git.adrien.mazarguil@6wind.com>", "list_archive_url": "https://inbox.dpdk.org/dev/dfc33efe83f3a6fbd0770b85988e519733350d83.1507809961.git.adrien.mazarguil@6wind.com", "date": "2017-10-12T12:19:32", "name": "[dpdk-dev,v2,18/29] net/mlx4: add VLAN filter configuration support", "commit_ref": null, "pull_url": null, "state": "accepted", "archived": true, "hash": "09897fd4e923695b8dec05d6e5c5159035584f6d", "submitter": { "id": 165, "url": "http://patches.dpdk.org/api/people/165/?format=api", "name": "Adrien Mazarguil", "email": "adrien.mazarguil@6wind.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/dfc33efe83f3a6fbd0770b85988e519733350d83.1507809961.git.adrien.mazarguil@6wind.com/mbox/", "series": [], "comments": "http://patches.dpdk.org/api/patches/30269/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/30269/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 53AC61B313;\n\tThu, 12 Oct 2017 14:20:42 +0200 (CEST)", "from mail-wm0-f54.google.com (mail-wm0-f54.google.com\n\t[74.125.82.54]) by dpdk.org (Postfix) with ESMTP id 4D1391B2ED\n\tfor <dev@dpdk.org>; Thu, 12 Oct 2017 14:20:31 +0200 (CEST)", "by mail-wm0-f54.google.com with SMTP id q132so12824347wmd.2\n\tfor <dev@dpdk.org>; Thu, 12 Oct 2017 05:20:31 -0700 (PDT)", "from 6wind.com (host.78.145.23.62.rev.coltfrance.com.\n\t[62.23.145.78]) by smtp.gmail.com with ESMTPSA id\n\t200sm102420wmu.44.2017.10.12.05.20.30\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tThu, 12 Oct 2017 05:20:30 -0700 (PDT)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=6wind-com.20150623.gappssmtp.com; s=20150623;\n\th=from:to:cc:subject:date:message-id:in-reply-to:references;\n\tbh=yBBGlPNUMNLV7vJSoTpboXniPYCZAeZAXjcE5CxPUG8=;\n\tb=ESFvcc8mI7Ba0M1jqc4PtuW30AppD03dcqUp1nOilpxxQ8o7RFWMCUSw3OkPDk6O14\n\t2aNb18TBMheVpYIYuZVPqA18rAZPv8y60EfXjO4Acq8eH5Txb2+LwvgDNmK1yJCAdhMY\n\tamRNy3MKgjDcLEC5/xkgTLWdb6gatxYRDn+hNI5ywNk7hWkC5YQJQoOBso11MFC4td0x\n\tGK4KfiW+eSIQV4f3S8tCpj5Iyau+BAiKBNbDOAxyO6bZKUEpUfp3ejRv/mDB6q9x1Xa8\n\tSTYGpKcfDuWExeSTXBDDUEOf8QE9V7ixhra3LCtss9TmCxVq4pUXG00UMMldlvBvHATi\n\tzWDA==", "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=yBBGlPNUMNLV7vJSoTpboXniPYCZAeZAXjcE5CxPUG8=;\n\tb=FRVaDrLXPLOk0FejfvMlLW5YCaRWHt7tPA85o2nBsBrEJAYIGiizKEoJvCPEQLpiPH\n\t5sfWdEjkSiVcJPFX8sPgHVS9HVVysNBTP7ijaVqfwTrsx7uiL8+bkpXj23cxk1qMR/eO\n\tLue0mofh6FIdkIjDGz9O0hlARj++cJe/hUmdpvdab+/uts5t1AIfo8U0vHtImZzi/xJR\n\tT8+eGTQeJvUYTlKEEDyXtsmoHSmjEtCFSOrWw0AbXS2G7Tapi42S159e3TVCufgjiGbZ\n\tNkCFB18tz4jKMzshPiK0NOj68xPGX0/4ng0EEXTdriurNwfCguvX8FaU37BKZWm2bmgW\n\t6gCQ==", "X-Gm-Message-State": "AMCzsaVPHNJ7ieAwOG8PutRJsrtm05vq557m8jT864MXYE5B6T4Q53nZ\n\torlkUlUz/eyohnaZDSMbmSMTaA==", "X-Google-Smtp-Source": "AOwi7QCIiyqge8IAPYos3XxY39/mTXCEUKChQu1Lr1Is6e6lc+x8orohpUtqP+RaRjP9zelo3rU7Tw==", "X-Received": "by 10.223.182.80 with SMTP id i16mr2341836wre.110.1507810830895; \n\tThu, 12 Oct 2017 05:20:30 -0700 (PDT)", "From": "Adrien Mazarguil <adrien.mazarguil@6wind.com>", "To": "Ferruh Yigit <ferruh.yigit@intel.com>", "Cc": "Nelio Laranjeiro <nelio.laranjeiro@6wind.com>,\n\tdev@dpdk.org", "Date": "Thu, 12 Oct 2017 14:19:32 +0200", "Message-Id": "<dfc33efe83f3a6fbd0770b85988e519733350d83.1507809961.git.adrien.mazarguil@6wind.com>", "X-Mailer": "git-send-email 2.1.4", "In-Reply-To": "<cover.1507809961.git.adrien.mazarguil@6wind.com>", "References": "<cover.1507730496.git.adrien.mazarguil@6wind.com>\n\t<cover.1507809961.git.adrien.mazarguil@6wind.com>", "Subject": "[dpdk-dev] [PATCH v2 18/29] net/mlx4: add VLAN filter configuration\n\tsupport", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.15", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<http://dpdk.org/ml/options/dev>,\n\t<mailto:dev-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://dpdk.org/ml/archives/dev/>", "List-Post": "<mailto:dev@dpdk.org>", "List-Help": "<mailto:dev-request@dpdk.org?subject=help>", "List-Subscribe": "<http://dpdk.org/ml/listinfo/dev>,\n\t<mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "This commit brings back VLAN filter configuration support without any\nartificial limitation on the number of simultaneous VLANs that can be\nconfigured (previously 127).\n\nAlso thanks to the fact it does not rely on fixed per-queue arrays for\npotential Verbs flow handle storage anymore, this version wastes a lot less\nmemory (previously 128 * 127 * pointer size, i.e. 130 kiB per Rx queue,\nonly one of which actually had any use for this room: the RSS parent\nqueue).\n\nThe number of internal flow rules generated still depends on the number of\nconfigured MAC addresses times that of configured VLAN filters though.\n\nSigned-off-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>\nAcked-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>\n---\n doc/guides/nics/features/mlx4.ini | 1 +\n drivers/net/mlx4/mlx4.c | 1 +\n drivers/net/mlx4/mlx4.h | 1 +\n drivers/net/mlx4/mlx4_ethdev.c | 42 +++++++++++++++++++++\n drivers/net/mlx4/mlx4_flow.c | 67 ++++++++++++++++++++++++++++++++++\n 5 files changed, 112 insertions(+)", "diff": "diff --git a/doc/guides/nics/features/mlx4.ini b/doc/guides/nics/features/mlx4.ini\nindex d17774f..bfe0eb1 100644\n--- a/doc/guides/nics/features/mlx4.ini\n+++ b/doc/guides/nics/features/mlx4.ini\n@@ -14,6 +14,7 @@ MTU update = Y\n Jumbo frame = Y\n Unicast MAC filter = Y\n SR-IOV = Y\n+VLAN filter = Y\n Basic stats = Y\n Stats per queue = Y\n Other kdrv = Y\ndiff --git a/drivers/net/mlx4/mlx4.c b/drivers/net/mlx4/mlx4.c\nindex 99c87ff..e25e958 100644\n--- a/drivers/net/mlx4/mlx4.c\n+++ b/drivers/net/mlx4/mlx4.c\n@@ -227,6 +227,7 @@ static const struct eth_dev_ops mlx4_dev_ops = {\n \t.stats_get = mlx4_stats_get,\n \t.stats_reset = mlx4_stats_reset,\n \t.dev_infos_get = mlx4_dev_infos_get,\n+\t.vlan_filter_set = mlx4_vlan_filter_set,\n \t.rx_queue_setup = mlx4_rx_queue_setup,\n \t.tx_queue_setup = mlx4_tx_queue_setup,\n \t.rx_queue_release = mlx4_rx_queue_release,\ndiff --git a/drivers/net/mlx4/mlx4.h b/drivers/net/mlx4/mlx4.h\nindex 15ecd95..cc403ea 100644\n--- a/drivers/net/mlx4/mlx4.h\n+++ b/drivers/net/mlx4/mlx4.h\n@@ -128,6 +128,7 @@ void mlx4_mac_addr_remove(struct rte_eth_dev *dev, uint32_t index);\n int mlx4_mac_addr_add(struct rte_eth_dev *dev, struct ether_addr *mac_addr,\n \t\t uint32_t index, uint32_t vmdq);\n void mlx4_mac_addr_set(struct rte_eth_dev *dev, struct ether_addr *mac_addr);\n+int mlx4_vlan_filter_set(struct rte_eth_dev *dev, uint16_t vlan_id, int on);\n int mlx4_stats_get(struct rte_eth_dev *dev, struct rte_eth_stats *stats);\n void mlx4_stats_reset(struct rte_eth_dev *dev);\n void mlx4_dev_infos_get(struct rte_eth_dev *dev,\ndiff --git a/drivers/net/mlx4/mlx4_ethdev.c b/drivers/net/mlx4/mlx4_ethdev.c\nindex 52924df..7721f13 100644\n--- a/drivers/net/mlx4/mlx4_ethdev.c\n+++ b/drivers/net/mlx4/mlx4_ethdev.c\n@@ -588,6 +588,48 @@ mlx4_mac_addr_add(struct rte_eth_dev *dev, struct ether_addr *mac_addr,\n }\n \n /**\n+ * DPDK callback to configure a VLAN filter.\n+ *\n+ * @param dev\n+ * Pointer to Ethernet device structure.\n+ * @param vlan_id\n+ * VLAN ID to filter.\n+ * @param on\n+ * Toggle filter.\n+ *\n+ * @return\n+ * 0 on success, negative errno value otherwise and rte_errno is set.\n+ */\n+int\n+mlx4_vlan_filter_set(struct rte_eth_dev *dev, uint16_t vlan_id, int on)\n+{\n+\tstruct priv *priv = dev->data->dev_private;\n+\tstruct rte_flow_error error;\n+\tunsigned int vidx = vlan_id / 64;\n+\tunsigned int vbit = vlan_id % 64;\n+\tuint64_t *v;\n+\tint ret;\n+\n+\tif (vidx >= RTE_DIM(dev->data->vlan_filter_conf.ids)) {\n+\t\trte_errno = EINVAL;\n+\t\treturn -rte_errno;\n+\t}\n+\tv = &dev->data->vlan_filter_conf.ids[vidx];\n+\t*v &= ~(UINT64_C(1) << vbit);\n+\t*v |= (uint64_t)!!on << vbit;\n+\tret = mlx4_flow_sync(priv, &error);\n+\tif (!ret)\n+\t\treturn 0;\n+\tERROR(\"failed to synchronize flow rules after %s VLAN filter on ID %u\"\n+\t \" (code %d, \\\"%s\\\"), \"\n+\t \" flow error type %d, cause %p, message: %s\",\n+\t on ? \"enabling\" : \"disabling\", vlan_id,\n+\t rte_errno, strerror(rte_errno), error.type, error.cause,\n+\t error.message ? error.message : \"(unspecified)\");\n+\treturn ret;\n+}\n+\n+/**\n * DPDK callback to set the primary MAC address.\n *\n * @param dev\ndiff --git a/drivers/net/mlx4/mlx4_flow.c b/drivers/net/mlx4/mlx4_flow.c\nindex 14d2ed3..377b48b 100644\n--- a/drivers/net/mlx4/mlx4_flow.c\n+++ b/drivers/net/mlx4/mlx4_flow.c\n@@ -1009,11 +1009,36 @@ mlx4_flow_flush(struct rte_eth_dev *dev,\n }\n \n /**\n+ * Helper function to determine the next configured VLAN filter.\n+ *\n+ * @param priv\n+ * Pointer to private structure.\n+ * @param vlan\n+ * VLAN ID to use as a starting point.\n+ *\n+ * @return\n+ * Next configured VLAN ID or a high value (>= 4096) if there is none.\n+ */\n+static uint16_t\n+mlx4_flow_internal_next_vlan(struct priv *priv, uint16_t vlan)\n+{\n+\twhile (vlan < 4096) {\n+\t\tif (priv->dev->data->vlan_filter_conf.ids[vlan / 64] &\n+\t\t (UINT64_C(1) << (vlan % 64)))\n+\t\t\treturn vlan;\n+\t\t++vlan;\n+\t}\n+\treturn vlan;\n+}\n+\n+/**\n * Generate internal flow rules.\n *\n * - MAC flow rules are generated from @p dev->data->mac_addrs\n * (@p priv->mac array).\n * - An additional flow rule for Ethernet broadcasts is also generated.\n+ * - All these are per-VLAN if @p dev->data->dev_conf.rxmode.hw_vlan_filter\n+ * is enabled and VLAN filters are configured.\n *\n * @param priv\n * Pointer to private structure.\n@@ -1034,6 +1059,10 @@ mlx4_flow_internal(struct priv *priv, struct rte_flow_error *error)\n \tconst struct rte_flow_item_eth eth_mask = {\n \t\t.dst.addr_bytes = \"\\xff\\xff\\xff\\xff\\xff\\xff\",\n \t};\n+\tstruct rte_flow_item_vlan vlan_spec;\n+\tconst struct rte_flow_item_vlan vlan_mask = {\n+\t\t.tci = RTE_BE16(0x0fff),\n+\t};\n \tstruct rte_flow_item pattern[] = {\n \t\t{\n \t\t\t.type = MLX4_FLOW_ITEM_TYPE_INTERNAL,\n@@ -1044,6 +1073,10 @@ mlx4_flow_internal(struct priv *priv, struct rte_flow_error *error)\n \t\t\t.mask = ð_mask,\n \t\t},\n \t\t{\n+\t\t\t/* Replaced with VLAN if filtering is enabled. */\n+\t\t\t.type = RTE_FLOW_ITEM_TYPE_END,\n+\t\t},\n+\t\t{\n \t\t\t.type = RTE_FLOW_ITEM_TYPE_END,\n \t\t},\n \t};\n@@ -1059,10 +1092,33 @@ mlx4_flow_internal(struct priv *priv, struct rte_flow_error *error)\n \t\t},\n \t};\n \tstruct ether_addr *rule_mac = ð_spec.dst;\n+\trte_be16_t *rule_vlan =\n+\t\tpriv->dev->data->dev_conf.rxmode.hw_vlan_filter ?\n+\t\t&vlan_spec.tci :\n+\t\tNULL;\n+\tuint16_t vlan = 0;\n \tstruct rte_flow *flow;\n \tunsigned int i;\n \tint err = 0;\n \n+\t/*\n+\t * Set up VLAN item if filtering is enabled and at least one VLAN\n+\t * filter is configured.\n+\t */\n+\tif (rule_vlan) {\n+\t\tvlan = mlx4_flow_internal_next_vlan(priv, 0);\n+\t\tif (vlan < 4096) {\n+\t\t\tpattern[2] = (struct rte_flow_item){\n+\t\t\t\t.type = RTE_FLOW_ITEM_TYPE_VLAN,\n+\t\t\t\t.spec = &vlan_spec,\n+\t\t\t\t.mask = &vlan_mask,\n+\t\t\t};\n+next_vlan:\n+\t\t\t*rule_vlan = rte_cpu_to_be_16(vlan);\n+\t\t} else {\n+\t\t\trule_vlan = NULL;\n+\t\t}\n+\t}\n \tfor (i = 0; i != RTE_DIM(priv->mac) + 1; ++i) {\n \t\tconst struct ether_addr *mac;\n \n@@ -1087,6 +1143,12 @@ mlx4_flow_internal(struct priv *priv, struct rte_flow_error *error)\n \t\t\tassert(flow->ibv_attr->type == IBV_FLOW_ATTR_NORMAL);\n \t\t\tassert(flow->ibv_attr->num_of_specs == 1);\n \t\t\tassert(eth->type == IBV_FLOW_SPEC_ETH);\n+\t\t\tif (rule_vlan &&\n+\t\t\t (eth->val.vlan_tag != *rule_vlan ||\n+\t\t\t eth->mask.vlan_tag != RTE_BE16(0x0fff)))\n+\t\t\t\tcontinue;\n+\t\t\tif (!rule_vlan && eth->mask.vlan_tag)\n+\t\t\t\tcontinue;\n \t\t\tfor (j = 0; j != sizeof(mac->addr_bytes); ++j)\n \t\t\t\tif (eth->val.dst_mac[j] != mac->addr_bytes[j] ||\n \t\t\t\t eth->mask.dst_mac[j] != UINT8_C(0xff) ||\n@@ -1109,6 +1171,11 @@ mlx4_flow_internal(struct priv *priv, struct rte_flow_error *error)\n \t\tflow->select = 1;\n \t\tflow->mac = 1;\n \t}\n+\tif (!err && rule_vlan) {\n+\t\tvlan = mlx4_flow_internal_next_vlan(priv, vlan + 1);\n+\t\tif (vlan < 4096)\n+\t\t\tgoto next_vlan;\n+\t}\n \t/* Clear selection and clean up stale MAC flow rules. */\n \tflow = LIST_FIRST(&priv->flows);\n \twhile (flow && flow->internal) {\n", "prefixes": [ "dpdk-dev", "v2", "18/29" ] }