[dpdk-stable] patch 'net/cxgbe: add prefix to global functions' has been queued to LTS release 18.11.6
Kevin Traynor
ktraynor at redhat.com
Tue Dec 3 19:26:23 CET 2019
Hi,
FYI, your patch has been queued to LTS release 18.11.6
Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 12/10/19. So please
shout if anyone has objections.
Also note that after the patch there's a diff of the upstream commit vs the
patch applied to the branch. This will indicate if there was any rebasing
needed to apply to the stable branch. If there were code changes for rebasing
(ie: not only metadata diffs), please double check that the rebase was
correctly done.
Queued patches are on a temporary branch at:
https://github.com/kevintraynor/dpdk-stable-queue
This queued commit can be viewed at:
https://github.com/kevintraynor/dpdk-stable-queue/commit/193ed3b78aafed5b261da2d7aacc3062cda3a547
Thanks.
Kevin.
---
>From 193ed3b78aafed5b261da2d7aacc3062cda3a547 Mon Sep 17 00:00:00 2001
From: Rahul Lakkireddy <rahul.lakkireddy at chelsio.com>
Date: Sat, 28 Sep 2019 02:00:01 +0530
Subject: [PATCH] net/cxgbe: add prefix to global functions
[ upstream commit 71e9b334a23764a5baa55cbacbdd99aeb9733631 ]
To avoid name collisions, add cxgbe_ prefix to some global functions.
Also, make some local functions static in cxgbe_filter.c.
Fixes: ee61f5113b17 ("net/cxgbe: parse and validate flows")
Fixes: 9eb2c9a48072 ("net/cxgbe: implement flow create operation")
Fixes: 3a381a4116ed ("net/cxgbe: query firmware for HASH filter resources")
Fixes: af44a577988b ("net/cxgbe: support to offload flows to HASH region")
Fixes: 41dc98b0827a ("net/cxgbe: support to delete flows in HASH region")
Fixes: 23af667f1507 ("net/cxgbe: add API to program hardware layer 2 table")
Signed-off-by: Rahul Lakkireddy <rahul.lakkireddy at chelsio.com>
---
drivers/net/cxgbe/cxgbe_filter.c | 30 ++++++++++++++++--------------
drivers/net/cxgbe/cxgbe_filter.h | 19 +++++++++----------
drivers/net/cxgbe/cxgbe_flow.c | 6 +++---
drivers/net/cxgbe/cxgbe_main.c | 10 +++++-----
drivers/net/cxgbe/l2t.c | 3 ++-
drivers/net/cxgbe/l2t.h | 3 ++-
6 files changed, 37 insertions(+), 34 deletions(-)
diff --git a/drivers/net/cxgbe/cxgbe_filter.c b/drivers/net/cxgbe/cxgbe_filter.c
index 932127018..2eee7166c 100644
--- a/drivers/net/cxgbe/cxgbe_filter.c
+++ b/drivers/net/cxgbe/cxgbe_filter.c
@@ -14,5 +14,5 @@
* Initialize Hash Filters
*/
-int init_hash_filter(struct adapter *adap)
+int cxgbe_init_hash_filter(struct adapter *adap)
{
unsigned int n_user_filters;
@@ -53,5 +53,6 @@ int init_hash_filter(struct adapter *adap)
* if the requested features have been enabled
*/
-int validate_filter(struct adapter *adapter, struct ch_filter_specification *fs)
+int cxgbe_validate_filter(struct adapter *adapter,
+ struct ch_filter_specification *fs)
{
u32 fconf;
@@ -133,5 +134,5 @@ static unsigned int get_filter_steerq(struct rte_eth_dev *dev,
/* Return an error number if the indicated filter isn't writable ... */
-int writable_filter(struct filter_entry *f)
+static int writable_filter(struct filter_entry *f)
{
if (f->locked)
@@ -214,5 +215,5 @@ static inline void mk_set_tcb_field_ulp(struct filter_entry *f,
* Check if entry already filled.
*/
-bool is_filter_set(struct tid_info *t, int fidx, int family)
+bool cxgbe_is_filter_set(struct tid_info *t, int fidx, int family)
{
bool result = FALSE;
@@ -527,5 +528,5 @@ static int cxgbe_set_hash_filter(struct rte_eth_dev *dev,
int ret = 0;
- ret = validate_filter(adapter, fs);
+ ret = cxgbe_validate_filter(adapter, fs);
if (ret)
return ret;
@@ -618,5 +619,5 @@ out_err:
* clears the filter's "pending" status.
*/
-void clear_filter(struct filter_entry *f)
+static void clear_filter(struct filter_entry *f)
{
if (f->clipt)
@@ -690,5 +691,5 @@ static int del_filter_wr(struct rte_eth_dev *dev, unsigned int fidx)
}
-int set_filter_wr(struct rte_eth_dev *dev, unsigned int fidx)
+static int set_filter_wr(struct rte_eth_dev *dev, unsigned int fidx)
{
struct adapter *adapter = ethdev2adap(dev);
@@ -868,5 +869,5 @@ int cxgbe_del_filter(struct rte_eth_dev *dev, unsigned int filter_id,
chip_ver = CHELSIO_CHIP_VERSION(adapter->params.chip);
- ret = is_filter_set(&adapter->tids, filter_id, fs->type);
+ ret = cxgbe_is_filter_set(&adapter->tids, filter_id, fs->type);
if (!ret) {
dev_warn(adap, "%s: could not find filter entry: %u\n",
@@ -940,5 +941,5 @@ int cxgbe_set_filter(struct rte_eth_dev *dev, unsigned int filter_id,
chip_ver = CHELSIO_CHIP_VERSION(adapter->params.chip);
- ret = validate_filter(adapter, fs);
+ ret = cxgbe_validate_filter(adapter, fs);
if (ret)
return ret;
@@ -951,5 +952,5 @@ int cxgbe_set_filter(struct rte_eth_dev *dev, unsigned int filter_id,
filter_id &= ~(0x3);
- ret = is_filter_set(&adapter->tids, filter_id, fs->type);
+ ret = cxgbe_is_filter_set(&adapter->tids, filter_id, fs->type);
if (ret)
return -EBUSY;
@@ -1091,5 +1092,6 @@ free_tid:
* Handle a Hash filter write reply.
*/
-void hash_filter_rpl(struct adapter *adap, const struct cpl_act_open_rpl *rpl)
+void cxgbe_hash_filter_rpl(struct adapter *adap,
+ const struct cpl_act_open_rpl *rpl)
{
struct tid_info *t = &adap->tids;
@@ -1159,5 +1161,5 @@ void hash_filter_rpl(struct adapter *adap, const struct cpl_act_open_rpl *rpl)
* Handle a LE-TCAM filter write/deletion reply.
*/
-void filter_rpl(struct adapter *adap, const struct cpl_set_tcb_rpl *rpl)
+void cxgbe_filter_rpl(struct adapter *adap, const struct cpl_set_tcb_rpl *rpl)
{
struct filter_entry *f = NULL;
@@ -1308,6 +1310,6 @@ get_count:
* Handle a Hash filter delete reply.
*/
-void hash_del_filter_rpl(struct adapter *adap,
- const struct cpl_abort_rpl_rss *rpl)
+void cxgbe_hash_del_filter_rpl(struct adapter *adap,
+ const struct cpl_abort_rpl_rss *rpl)
{
struct tid_info *t = &adap->tids;
diff --git a/drivers/net/cxgbe/cxgbe_filter.h b/drivers/net/cxgbe/cxgbe_filter.h
index b7bcbf56a..6738ae4f1 100644
--- a/drivers/net/cxgbe/cxgbe_filter.h
+++ b/drivers/net/cxgbe/cxgbe_filter.h
@@ -249,9 +249,6 @@ cxgbe_bitmap_find_free_region(struct rte_bitmap *bmap, unsigned int size,
}
-bool is_filter_set(struct tid_info *, int fidx, int family);
-void filter_rpl(struct adapter *adap, const struct cpl_set_tcb_rpl *rpl);
-void clear_filter(struct filter_entry *f);
-int set_filter_wr(struct rte_eth_dev *dev, unsigned int fidx);
-int writable_filter(struct filter_entry *f);
+bool cxgbe_is_filter_set(struct tid_info *, int fidx, int family);
+void cxgbe_filter_rpl(struct adapter *adap, const struct cpl_set_tcb_rpl *rpl);
int cxgbe_set_filter(struct rte_eth_dev *dev, unsigned int filter_id,
struct ch_filter_specification *fs,
@@ -261,9 +258,11 @@ int cxgbe_del_filter(struct rte_eth_dev *dev, unsigned int filter_id,
struct filter_ctx *ctx);
int cxgbe_alloc_ftid(struct adapter *adap, unsigned int family);
-int init_hash_filter(struct adapter *adap);
-void hash_filter_rpl(struct adapter *adap, const struct cpl_act_open_rpl *rpl);
-void hash_del_filter_rpl(struct adapter *adap,
- const struct cpl_abort_rpl_rss *rpl);
-int validate_filter(struct adapter *adap, struct ch_filter_specification *fs);
+int cxgbe_init_hash_filter(struct adapter *adap);
+void cxgbe_hash_filter_rpl(struct adapter *adap,
+ const struct cpl_act_open_rpl *rpl);
+void cxgbe_hash_del_filter_rpl(struct adapter *adap,
+ const struct cpl_abort_rpl_rss *rpl);
+int cxgbe_validate_filter(struct adapter *adap,
+ struct ch_filter_specification *fs);
int cxgbe_get_filter_count(struct adapter *adapter, unsigned int fidx,
u64 *c, int hash, bool get_byte);
diff --git a/drivers/net/cxgbe/cxgbe_flow.c b/drivers/net/cxgbe/cxgbe_flow.c
index 7b87bdf58..17d5608fa 100644
--- a/drivers/net/cxgbe/cxgbe_flow.c
+++ b/drivers/net/cxgbe/cxgbe_flow.c
@@ -310,5 +310,5 @@ static int cxgbe_validate_fidxondel(struct filter_entry *f, unsigned int fidx)
return -EINVAL;
}
- if (!is_filter_set(&adap->tids, fidx, fs.type)) {
+ if (!cxgbe_is_filter_set(&adap->tids, fidx, fs.type)) {
dev_err(adap, "Already free fidx:%d f:%p\n", fidx, f);
return -EINVAL;
@@ -322,5 +322,5 @@ cxgbe_validate_fidxonadd(struct ch_filter_specification *fs,
struct adapter *adap, unsigned int fidx)
{
- if (is_filter_set(&adap->tids, fidx, fs->type)) {
+ if (cxgbe_is_filter_set(&adap->tids, fidx, fs->type)) {
dev_err(adap, "filter index: %d is busy.\n", fidx);
return -EBUSY;
@@ -1017,5 +1017,5 @@ cxgbe_flow_validate(struct rte_eth_dev *dev,
}
- if (validate_filter(adap, &flow->fs)) {
+ if (cxgbe_validate_filter(adap, &flow->fs)) {
t4_os_free(flow);
return rte_flow_error_set(e, EINVAL, RTE_FLOW_ERROR_TYPE_HANDLE,
diff --git a/drivers/net/cxgbe/cxgbe_main.c b/drivers/net/cxgbe/cxgbe_main.c
index 38938ecc0..2cc4b6d91 100644
--- a/drivers/net/cxgbe/cxgbe_main.c
+++ b/drivers/net/cxgbe/cxgbe_main.c
@@ -93,17 +93,17 @@ static int fwevtq_handler(struct sge_rspq *q, const __be64 *rsp,
const struct cpl_abort_rpl_rss *p = (const void *)rsp;
- hash_del_filter_rpl(q->adapter, p);
+ cxgbe_hash_del_filter_rpl(q->adapter, p);
} else if (opcode == CPL_SET_TCB_RPL) {
const struct cpl_set_tcb_rpl *p = (const void *)rsp;
- filter_rpl(q->adapter, p);
+ cxgbe_filter_rpl(q->adapter, p);
} else if (opcode == CPL_ACT_OPEN_RPL) {
const struct cpl_act_open_rpl *p = (const void *)rsp;
- hash_filter_rpl(q->adapter, p);
+ cxgbe_hash_filter_rpl(q->adapter, p);
} else if (opcode == CPL_L2T_WRITE_RPL) {
const struct cpl_l2t_write_rpl *p = (const void *)rsp;
- do_l2t_write_rpl(q->adapter, p);
+ cxgbe_do_l2t_write_rpl(q->adapter, p);
} else {
dev_err(adapter, "unexpected CPL %#x on FW event queue\n",
@@ -1180,5 +1180,5 @@ static int adap_init0(struct adapter *adap)
if ((caps_cmd.niccaps & cpu_to_be16(FW_CAPS_CONFIG_NIC_HASHFILTER)) &&
is_t6(adap->params.chip)) {
- if (init_hash_filter(adap) < 0)
+ if (cxgbe_init_hash_filter(adap) < 0)
goto bye;
}
diff --git a/drivers/net/cxgbe/l2t.c b/drivers/net/cxgbe/l2t.c
index 814188fea..deee261c6 100644
--- a/drivers/net/cxgbe/l2t.c
+++ b/drivers/net/cxgbe/l2t.c
@@ -22,5 +22,6 @@ void cxgbe_l2t_release(struct l2t_entry *e)
* the L2T index it refers to.
*/
-void do_l2t_write_rpl(struct adapter *adap, const struct cpl_l2t_write_rpl *rpl)
+void cxgbe_do_l2t_write_rpl(struct adapter *adap,
+ const struct cpl_l2t_write_rpl *rpl)
{
struct l2t_data *d = adap->l2t;
diff --git a/drivers/net/cxgbe/l2t.h b/drivers/net/cxgbe/l2t.h
index 22a34e388..d2cecf411 100644
--- a/drivers/net/cxgbe/l2t.h
+++ b/drivers/net/cxgbe/l2t.h
@@ -54,4 +54,5 @@ struct l2t_entry *cxgbe_l2t_alloc_switching(struct rte_eth_dev *dev, u16 vlan,
u8 port, u8 *dmac);
void cxgbe_l2t_release(struct l2t_entry *e);
-void do_l2t_write_rpl(struct adapter *p, const struct cpl_l2t_write_rpl *rpl);
+void cxgbe_do_l2t_write_rpl(struct adapter *p,
+ const struct cpl_l2t_write_rpl *rpl);
#endif /* _CXGBE_L2T_H_ */
--
2.21.0
---
Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- - 2019-12-03 17:29:52.631290349 +0000
+++ 0014-net-cxgbe-add-prefix-to-global-functions.patch 2019-12-03 17:29:51.713750723 +0000
@@ -1 +1 @@
-From 71e9b334a23764a5baa55cbacbdd99aeb9733631 Mon Sep 17 00:00:00 2001
+From 193ed3b78aafed5b261da2d7aacc3062cda3a547 Mon Sep 17 00:00:00 2001
@@ -5,0 +6,2 @@
+[ upstream commit 71e9b334a23764a5baa55cbacbdd99aeb9733631 ]
+
@@ -15 +16,0 @@
-Cc: stable at dpdk.org
@@ -28 +29 @@
-index 7fcee5c0a..cc8774c1d 100644
+index 932127018..2eee7166c 100644
@@ -31 +32 @@
-@@ -15,5 +15,5 @@
+@@ -14,5 +14,5 @@
@@ -38 +39 @@
-@@ -54,5 +54,6 @@ int init_hash_filter(struct adapter *adap)
+@@ -53,5 +53,6 @@ int init_hash_filter(struct adapter *adap)
@@ -46 +47 @@
-@@ -134,5 +135,5 @@ static unsigned int get_filter_steerq(struct rte_eth_dev *dev,
+@@ -133,5 +134,5 @@ static unsigned int get_filter_steerq(struct rte_eth_dev *dev,
@@ -53 +54 @@
-@@ -215,5 +216,5 @@ static inline void mk_set_tcb_field_ulp(struct filter_entry *f,
+@@ -214,5 +215,5 @@ static inline void mk_set_tcb_field_ulp(struct filter_entry *f,
@@ -60 +61 @@
-@@ -528,5 +529,5 @@ static int cxgbe_set_hash_filter(struct rte_eth_dev *dev,
+@@ -527,5 +528,5 @@ static int cxgbe_set_hash_filter(struct rte_eth_dev *dev,
@@ -67 +68 @@
-@@ -619,5 +620,5 @@ out_err:
+@@ -618,5 +619,5 @@ out_err:
@@ -74 +75 @@
-@@ -691,5 +692,5 @@ static int del_filter_wr(struct rte_eth_dev *dev, unsigned int fidx)
+@@ -690,5 +691,5 @@ static int del_filter_wr(struct rte_eth_dev *dev, unsigned int fidx)
@@ -81 +82 @@
-@@ -869,5 +870,5 @@ int cxgbe_del_filter(struct rte_eth_dev *dev, unsigned int filter_id,
+@@ -868,5 +869,5 @@ int cxgbe_del_filter(struct rte_eth_dev *dev, unsigned int filter_id,
@@ -88 +89 @@
-@@ -941,5 +942,5 @@ int cxgbe_set_filter(struct rte_eth_dev *dev, unsigned int filter_id,
+@@ -940,5 +941,5 @@ int cxgbe_set_filter(struct rte_eth_dev *dev, unsigned int filter_id,
@@ -95 +96 @@
-@@ -952,5 +953,5 @@ int cxgbe_set_filter(struct rte_eth_dev *dev, unsigned int filter_id,
+@@ -951,5 +952,5 @@ int cxgbe_set_filter(struct rte_eth_dev *dev, unsigned int filter_id,
@@ -102 +103 @@
-@@ -1092,5 +1093,6 @@ free_tid:
+@@ -1091,5 +1092,6 @@ free_tid:
@@ -110 +111 @@
-@@ -1160,5 +1162,5 @@ void hash_filter_rpl(struct adapter *adap, const struct cpl_act_open_rpl *rpl)
+@@ -1159,5 +1161,5 @@ void hash_filter_rpl(struct adapter *adap, const struct cpl_act_open_rpl *rpl)
@@ -117 +118 @@
-@@ -1358,6 +1360,6 @@ int cxgbe_clear_filter_count(struct adapter *adapter, unsigned int fidx,
+@@ -1308,6 +1310,6 @@ get_count:
@@ -127 +128 @@
-index 0c67d2d15..1964730ba 100644
+index b7bcbf56a..6738ae4f1 100644
@@ -160 +161 @@
-index d3de689c3..848c61f02 100644
+index 7b87bdf58..17d5608fa 100644
@@ -177 +178 @@
-@@ -1020,5 +1020,5 @@ cxgbe_flow_validate(struct rte_eth_dev *dev,
+@@ -1017,5 +1017,5 @@ cxgbe_flow_validate(struct rte_eth_dev *dev,
@@ -185 +186 @@
-index 620f60b4d..c3e6b9557 100644
+index 38938ecc0..2cc4b6d91 100644
@@ -218 +219 @@
-index 6faf624f7..f9d651fe0 100644
+index 814188fea..deee261c6 100644
@@ -221 +222 @@
-@@ -23,5 +23,6 @@ void cxgbe_l2t_release(struct l2t_entry *e)
+@@ -22,5 +22,6 @@ void cxgbe_l2t_release(struct l2t_entry *e)
@@ -230 +231 @@
-index 326abfde4..2c489e4aa 100644
+index 22a34e388..d2cecf411 100644
More information about the stable
mailing list