Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/41995/?format=api
{ "id": 41995, "url": "http://patches.dpdk.org/api/patches/41995/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/165fc0f3fa15ce2728f0a425531964c7063e1c28.1530295732.git.rahul.lakkireddy@chelsio.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": "<165fc0f3fa15ce2728f0a425531964c7063e1c28.1530295732.git.rahul.lakkireddy@chelsio.com>", "list_archive_url": "https://inbox.dpdk.org/dev/165fc0f3fa15ce2728f0a425531964c7063e1c28.1530295732.git.rahul.lakkireddy@chelsio.com", "date": "2018-06-29T18:12:23", "name": "[8/9] net/cxgbe: add support to match on ingress physical port", "commit_ref": null, "pull_url": null, "state": "accepted", "archived": true, "hash": "977928966797b3d17cf0dcaa023d43335dadce60", "submitter": { "id": 241, "url": "http://patches.dpdk.org/api/people/241/?format=api", "name": "Rahul Lakkireddy", "email": "rahul.lakkireddy@chelsio.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/165fc0f3fa15ce2728f0a425531964c7063e1c28.1530295732.git.rahul.lakkireddy@chelsio.com/mbox/", "series": [ { "id": 331, "url": "http://patches.dpdk.org/api/series/331/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=331", "date": "2018-06-29T18:12:15", "name": "net/cxgbe: add support for offloading flows to HASH region", "version": 1, "mbox": "http://patches.dpdk.org/series/331/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/41995/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/41995/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 A503B1BB09;\n\tFri, 29 Jun 2018 20:13:59 +0200 (CEST)", "from stargate.chelsio.com (stargate.chelsio.com [12.32.117.8])\n\tby dpdk.org (Postfix) with ESMTP id 65CC31B90F\n\tfor <dev@dpdk.org>; Fri, 29 Jun 2018 20:13:56 +0200 (CEST)", "from localhost (scalar.blr.asicdesigners.com [10.193.185.94])\n\tby stargate.chelsio.com (8.13.8/8.13.8) with ESMTP id w5TIDqu9028635; \n\tFri, 29 Jun 2018 11:13:53 -0700" ], "From": "Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>", "To": "dev@dpdk.org", "Cc": "shaguna@chelsio.com, indranil@chelsio.com, nirranjan@chelsio.com", "Date": "Fri, 29 Jun 2018 23:42:23 +0530", "Message-Id": "<165fc0f3fa15ce2728f0a425531964c7063e1c28.1530295732.git.rahul.lakkireddy@chelsio.com>", "X-Mailer": "git-send-email 2.5.3", "In-Reply-To": [ "<cover.1530295732.git.rahul.lakkireddy@chelsio.com>", "<cover.1530295732.git.rahul.lakkireddy@chelsio.com>" ], "References": [ "<cover.1530295732.git.rahul.lakkireddy@chelsio.com>", "<cover.1530295732.git.rahul.lakkireddy@chelsio.com>" ], "Subject": "[dpdk-dev] [PATCH 8/9] net/cxgbe: add support to match on ingress\n\tphysical port", "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: Shagun Agrawal <shaguna@chelsio.com>\n\nAdd support to match packets based on ingress physical port.\n\nSigned-off-by: Shagun Agrawal <shaguna@chelsio.com>\nSigned-off-by: Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>\n---\n drivers/net/cxgbe/cxgbe_filter.c | 16 +++++++++++++++-\n drivers/net/cxgbe/cxgbe_flow.c | 30 ++++++++++++++++++++++++++++++\n 2 files changed, 45 insertions(+), 1 deletion(-)", "diff": "diff --git a/drivers/net/cxgbe/cxgbe_filter.c b/drivers/net/cxgbe/cxgbe_filter.c\nindex ff43488b5..8c5890ea8 100644\n--- a/drivers/net/cxgbe/cxgbe_filter.c\n+++ b/drivers/net/cxgbe/cxgbe_filter.c\n@@ -65,11 +65,19 @@ int validate_filter(struct adapter *adapter, struct ch_filter_specification *fs)\n #define U(_mask, _field) \\\n \t(!(fconf & (_mask)) && S(_field))\n \n-\tif (U(F_ETHERTYPE, ethtype) || U(F_PROTOCOL, proto))\n+\tif (U(F_PORT, iport) || U(F_ETHERTYPE, ethtype) || U(F_PROTOCOL, proto))\n \t\treturn -EOPNOTSUPP;\n \n #undef S\n #undef U\n+\n+\t/*\n+\t * Don't allow various trivially obvious bogus out-of-range\n+\t * values ...\n+\t */\n+\tif (fs->val.iport >= adapter->params.nports)\n+\t\treturn -ERANGE;\n+\n \treturn 0;\n }\n \n@@ -228,6 +236,9 @@ static u64 hash_filter_ntuple(const struct filter_entry *f)\n \tu64 ntuple = 0;\n \tu16 tcp_proto = IPPROTO_TCP; /* TCP Protocol Number */\n \n+\tif (tp->port_shift >= 0)\n+\t\tntuple |= (u64)f->fs.mask.iport << tp->port_shift;\n+\n \tif (tp->protocol_shift >= 0) {\n \t\tif (!f->fs.val.proto)\n \t\t\tntuple |= (u64)tcp_proto << tp->protocol_shift;\n@@ -664,6 +675,9 @@ int set_filter_wr(struct rte_eth_dev *dev, unsigned int fidx)\n \t\tcpu_to_be16(V_FW_FILTER_WR_RX_CHAN(0) |\n \t\t\t V_FW_FILTER_WR_RX_RPL_IQ(adapter->sge.fw_evtq.abs_id\n \t\t\t\t\t\t ));\n+\tfwr->maci_to_matchtypem =\n+\t\tcpu_to_be32(V_FW_FILTER_WR_PORT(f->fs.val.iport) |\n+\t\t\t V_FW_FILTER_WR_PORTM(f->fs.mask.iport));\n \tfwr->ptcl = f->fs.val.proto;\n \tfwr->ptclm = f->fs.mask.proto;\n \trte_memcpy(fwr->lip, f->fs.val.lip, sizeof(fwr->lip));\ndiff --git a/drivers/net/cxgbe/cxgbe_flow.c b/drivers/net/cxgbe/cxgbe_flow.c\nindex 4f00ac4c6..823bc720c 100644\n--- a/drivers/net/cxgbe/cxgbe_flow.c\n+++ b/drivers/net/cxgbe/cxgbe_flow.c\n@@ -93,6 +93,8 @@ cxgbe_fill_filter_region(struct adapter *adap,\n \t\tntuple_mask |= (u64)fs->mask.proto << tp->protocol_shift;\n \tif (tp->ethertype_shift >= 0)\n \t\tntuple_mask |= (u64)fs->mask.ethtype << tp->ethertype_shift;\n+\tif (tp->port_shift >= 0)\n+\t\tntuple_mask |= (u64)fs->mask.iport << tp->port_shift;\n \n \tif (ntuple_mask != hash_filter_mask)\n \t\treturn;\n@@ -100,6 +102,27 @@ cxgbe_fill_filter_region(struct adapter *adap,\n \tfs->cap = 1;\t/* use hash region */\n }\n \n+static int\n+ch_rte_parsetype_port(const void *dmask, const struct rte_flow_item *item,\n+\t\t struct ch_filter_specification *fs,\n+\t\t struct rte_flow_error *e)\n+{\n+\tconst struct rte_flow_item_phy_port *val = item->spec;\n+\tconst struct rte_flow_item_phy_port *umask = item->mask;\n+\tconst struct rte_flow_item_phy_port *mask;\n+\n+\tmask = umask ? umask : (const struct rte_flow_item_phy_port *)dmask;\n+\n+\tif (val->index > 0x7)\n+\t\treturn rte_flow_error_set(e, EINVAL, RTE_FLOW_ERROR_TYPE_ITEM,\n+\t\t\t\t\t item,\n+\t\t\t\t\t \"port index upto 0x7 is supported\");\n+\n+\tCXGBE_FILL_FS(val->index, mask->index, iport);\n+\n+\treturn 0;\n+}\n+\n static int\n ch_rte_parsetype_udp(const void *dmask, const struct rte_flow_item *item,\n \t\t struct ch_filter_specification *fs,\n@@ -357,6 +380,13 @@ cxgbe_rtef_parse_actions(struct rte_flow *flow,\n }\n \n struct chrte_fparse parseitem[] = {\n+\t\t[RTE_FLOW_ITEM_TYPE_PHY_PORT] = {\n+\t\t.fptr = ch_rte_parsetype_port,\n+\t\t.dmask = &(const struct rte_flow_item_phy_port){\n+\t\t\t.index = 0x7,\n+\t\t}\n+\t},\n+\n \t[RTE_FLOW_ITEM_TYPE_IPV4] = {\n \t\t.fptr = ch_rte_parsetype_ipv4,\n \t\t.dmask = &rte_flow_item_ipv4_mask,\n", "prefixes": [ "8/9" ] }