DPDK patches and discussions
 help / color / mirror / Atom feed
* Re: [dpdk-dev] [PATCH] net/bnxt: fix variable size of port id
  2020-08-04 17:41 [dpdk-dev] [PATCH] net/bnxt: fix variable size of port id Chenbo Xia
@ 2020-08-04 16:22 ` Stephen Hemminger
  2020-08-05 20:18 ` Thomas Monjalon
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 15+ messages in thread
From: Stephen Hemminger @ 2020-08-04 16:22 UTC (permalink / raw)
  To: Chenbo Xia
  Cc: dev, ajit.khaparde, somnath.kotur, kishore.padmanabha,
	venkatkumar.duvvuru, stable

On Tue,  4 Aug 2020 17:41:47 +0000
Chenbo Xia <chenbo.xia@intel.com> wrote:

> Currenly the variable size of ethdev port id is 8 bits. This
> patch standarizes it to 16 bits.
> 
> Fixes: 769de16872ab ("net/bnxt: fix port default rule create/destroy")
> Cc: stable@dpdk.org
> 
> Reported-by: Yinan Wang <yinan.wang@intel.com>
> Signed-off-by: Chenbo Xia <chenbo.xia@intel.com>
> ---
	    BNXT_ETH_DEV_IS_REPRESENTOR(bp->eth_dev))

Acked-by: Stephen Hemminger <stephen@networkplumber.org>

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [dpdk-dev] [PATCH] net/bnxt: fix variable size of port id
@ 2020-08-04 17:41 Chenbo Xia
  2020-08-04 16:22 ` Stephen Hemminger
                   ` (3 more replies)
  0 siblings, 4 replies; 15+ messages in thread
From: Chenbo Xia @ 2020-08-04 17:41 UTC (permalink / raw)
  To: dev
  Cc: ajit.khaparde, somnath.kotur, kishore.padmanabha,
	venkatkumar.duvvuru, stable

Currenly the variable size of ethdev port id is 8 bits. This
patch standarizes it to 16 bits.

Fixes: 769de16872ab ("net/bnxt: fix port default rule create/destroy")
Cc: stable@dpdk.org

Reported-by: Yinan Wang <yinan.wang@intel.com>
Signed-off-by: Chenbo Xia <chenbo.xia@intel.com>
---
 drivers/net/bnxt/tf_ulp/ulp_def_rules.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/bnxt/tf_ulp/ulp_def_rules.c b/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
index 9fb1a028f..ff5aed3d7 100644
--- a/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
+++ b/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
@@ -397,7 +397,7 @@ void
 bnxt_ulp_destroy_df_rules(struct bnxt *bp, bool global)
 {
 	struct bnxt_ulp_df_rule_info *info;
-	uint8_t port_id;
+	uint16_t port_id;
 
 	if (!BNXT_TRUFLOW_EN(bp) ||
 	    BNXT_ETH_DEV_IS_REPRESENTOR(bp->eth_dev))
-- 
2.17.1


^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [dpdk-dev] [PATCH] net/bnxt: fix variable size of port id
  2020-08-04 17:41 [dpdk-dev] [PATCH] net/bnxt: fix variable size of port id Chenbo Xia
  2020-08-04 16:22 ` Stephen Hemminger
@ 2020-08-05 20:18 ` Thomas Monjalon
  2020-08-06  2:03   ` Xia, Chenbo
  2020-09-03  6:49 ` Xiao, QimaiX
  2020-09-25  5:37 ` [dpdk-dev] [PATCH v2 1/3] net: fix variable size of ethdev " Chenbo Xia
  3 siblings, 1 reply; 15+ messages in thread
From: Thomas Monjalon @ 2020-08-05 20:18 UTC (permalink / raw)
  To: Chenbo Xia, Yinan Wang
  Cc: dev, ajit.khaparde, somnath.kotur, kishore.padmanabha,
	venkatkumar.duvvuru, stable

04/08/2020 19:41, Chenbo Xia:
> Currenly the variable size of ethdev port id is 8 bits. This
> patch standarizes it to 16 bits.
> 
> Fixes: 769de16872ab ("net/bnxt: fix port default rule create/destroy")
> Cc: stable@dpdk.org
> 
> Reported-by: Yinan Wang <yinan.wang@intel.com>
> Signed-off-by: Chenbo Xia <chenbo.xia@intel.com>
> ---
>  drivers/net/bnxt/tf_ulp/ulp_def_rules.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/bnxt/tf_ulp/ulp_def_rules.c b/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
> index 9fb1a028f..ff5aed3d7 100644
> --- a/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
> +++ b/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
> @@ -397,7 +397,7 @@ void
>  bnxt_ulp_destroy_df_rules(struct bnxt *bp, bool global)
>  {
>  	struct bnxt_ulp_df_rule_info *info;
> -	uint8_t port_id;
> +	uint16_t port_id;

There are more occurences in bnxt, introduced by the same patch.

In general, in other drivers, it would be good to make this clean-up.
	git grep 'uint8_t port' | wc -l
	297



^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [dpdk-dev] [PATCH] net/bnxt: fix variable size of port id
  2020-08-05 20:18 ` Thomas Monjalon
@ 2020-08-06  2:03   ` Xia, Chenbo
  2020-08-06  7:57     ` Thomas Monjalon
  0 siblings, 1 reply; 15+ messages in thread
From: Xia, Chenbo @ 2020-08-06  2:03 UTC (permalink / raw)
  To: Thomas Monjalon, Wang, Yinan
  Cc: dev, ajit.khaparde, somnath.kotur, kishore.padmanabha,
	venkatkumar.duvvuru, stable

Hi Thomas,

> -----Original Message-----
> From: Thomas Monjalon <thomas@monjalon.net>
> Sent: Thursday, August 6, 2020 4:18 AM
> To: Xia, Chenbo <chenbo.xia@intel.com>; Wang, Yinan <yinan.wang@intel.com>
> Cc: dev@dpdk.org; ajit.khaparde@broadcom.com; somnath.kotur@broadcom.com;
> kishore.padmanabha@broadcom.com; venkatkumar.duvvuru@broadcom.com;
> stable@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH] net/bnxt: fix variable size of port id
> 
> 04/08/2020 19:41, Chenbo Xia:
> > Currenly the variable size of ethdev port id is 8 bits. This
> > patch standarizes it to 16 bits.
> >
> > Fixes: 769de16872ab ("net/bnxt: fix port default rule create/destroy")
> > Cc: stable@dpdk.org
> >
> > Reported-by: Yinan Wang <yinan.wang@intel.com>
> > Signed-off-by: Chenbo Xia <chenbo.xia@intel.com>
> > ---
> >  drivers/net/bnxt/tf_ulp/ulp_def_rules.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
> b/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
> > index 9fb1a028f..ff5aed3d7 100644
> > --- a/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
> > +++ b/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
> > @@ -397,7 +397,7 @@ void
> >  bnxt_ulp_destroy_df_rules(struct bnxt *bp, bool global)
> >  {
> >  	struct bnxt_ulp_df_rule_info *info;
> > -	uint8_t port_id;
> > +	uint16_t port_id;
> 
> There are more occurences in bnxt, introduced by the same patch.
> 
> In general, in other drivers, it would be good to make this clean-up.

Sure, I could do the clean-up. Btw, only net drivers, right?

> 	git grep 'uint8_t port' | wc -l
> 	297
> 

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [dpdk-dev] [PATCH] net/bnxt: fix variable size of port id
  2020-08-06  2:03   ` Xia, Chenbo
@ 2020-08-06  7:57     ` Thomas Monjalon
  0 siblings, 0 replies; 15+ messages in thread
From: Thomas Monjalon @ 2020-08-06  7:57 UTC (permalink / raw)
  To: Xia, Chenbo
  Cc: Wang, Yinan, dev, ajit.khaparde, somnath.kotur,
	kishore.padmanabha, venkatkumar.duvvuru, stable

06/08/2020 04:03, Xia, Chenbo:
> Hi Thomas,
> 
> From: Thomas Monjalon <thomas@monjalon.net>
> > 04/08/2020 19:41, Chenbo Xia:
> > > Currenly the variable size of ethdev port id is 8 bits. This
> > > patch standarizes it to 16 bits.
> > >
> > > Fixes: 769de16872ab ("net/bnxt: fix port default rule create/destroy")
> > > Cc: stable@dpdk.org
> > >
> > > Reported-by: Yinan Wang <yinan.wang@intel.com>
> > > Signed-off-by: Chenbo Xia <chenbo.xia@intel.com>
> > > ---
> > >  drivers/net/bnxt/tf_ulp/ulp_def_rules.c | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
> > b/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
> > > index 9fb1a028f..ff5aed3d7 100644
> > > --- a/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
> > > +++ b/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
> > > @@ -397,7 +397,7 @@ void
> > >  bnxt_ulp_destroy_df_rules(struct bnxt *bp, bool global)
> > >  {
> > >  	struct bnxt_ulp_df_rule_info *info;
> > > -	uint8_t port_id;
> > > +	uint16_t port_id;
> > 
> > There are more occurences in bnxt, introduced by the same patch.
> > 
> > In general, in other drivers, it would be good to make this clean-up.
> 
> Sure, I could do the clean-up. Btw, only net drivers, right?

ethdev drivers and applications referencing a port id should
use uint16_t, yes.



^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [dpdk-dev] [PATCH] net/bnxt: fix variable size of port id
  2020-08-04 17:41 [dpdk-dev] [PATCH] net/bnxt: fix variable size of port id Chenbo Xia
  2020-08-04 16:22 ` Stephen Hemminger
  2020-08-05 20:18 ` Thomas Monjalon
@ 2020-09-03  6:49 ` Xiao, QimaiX
  2020-09-25  5:37 ` [dpdk-dev] [PATCH v2 1/3] net: fix variable size of ethdev " Chenbo Xia
  3 siblings, 0 replies; 15+ messages in thread
From: Xiao, QimaiX @ 2020-09-03  6:49 UTC (permalink / raw)
  To: Xia, Chenbo, dev
  Cc: ajit.khaparde, somnath.kotur, kishore.padmanabha,
	venkatkumar.duvvuru, stable

Tested-by: Xiao, Qimai <qimaix.xiao@intel.com>

Regards,
Xiao Qimai

> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of Chenbo Xia
> Sent: Wednesday, August 5, 2020 1:42 AM
> To: dev@dpdk.org
> Cc: ajit.khaparde@broadcom.com; somnath.kotur@broadcom.com;
> kishore.padmanabha@broadcom.com;
> venkatkumar.duvvuru@broadcom.com; stable@dpdk.org
> Subject: [dpdk-dev] [PATCH] net/bnxt: fix variable size of port id
> 
> Currenly the variable size of ethdev port id is 8 bits. This patch standarizes it
> to 16 bits.
> 
> Fixes: 769de16872ab ("net/bnxt: fix port default rule create/destroy")
> Cc: stable@dpdk.org
> 
> Reported-by: Yinan Wang <yinan.wang@intel.com>
> Signed-off-by: Chenbo Xia <chenbo.xia@intel.com>
> ---
>  drivers/net/bnxt/tf_ulp/ulp_def_rules.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
> b/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
> index 9fb1a028f..ff5aed3d7 100644
> --- a/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
> +++ b/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
> @@ -397,7 +397,7 @@ void
>  bnxt_ulp_destroy_df_rules(struct bnxt *bp, bool global)  {
>  	struct bnxt_ulp_df_rule_info *info;
> -	uint8_t port_id;
> +	uint16_t port_id;
> 
>  	if (!BNXT_TRUFLOW_EN(bp) ||
>  	    BNXT_ETH_DEV_IS_REPRESENTOR(bp->eth_dev))
> --
> 2.17.1


^ permalink raw reply	[flat|nested] 15+ messages in thread

* [dpdk-dev] [PATCH v2 1/3] net: fix variable size of ethdev port id
  2020-08-04 17:41 [dpdk-dev] [PATCH] net/bnxt: fix variable size of port id Chenbo Xia
                   ` (2 preceding siblings ...)
  2020-09-03  6:49 ` Xiao, QimaiX
@ 2020-09-25  5:37 ` Chenbo Xia
  2020-09-25  5:37   ` [dpdk-dev] [PATCH v2 2/3] app: " Chenbo Xia
                     ` (3 more replies)
  3 siblings, 4 replies; 15+ messages in thread
From: Chenbo Xia @ 2020-09-25  5:37 UTC (permalink / raw)
  To: dev
  Cc: shepard.siegel, ed.czeck, john.miller, ajit.khaparde,
	somnath.kotur, qiming.yang, qi.z.zhang, rmody, shshaikh, sbhosle,
	kishore.padmanabha, michael.baucom, venkatkumar.duvvuru,
	xiaoyun.li, ferruh.yigit, jingjing.wu, irusskikh, wenzhuo.lu,
	beilei.xing, bernard.iremonger, nikhil.rao, konstantin.ananyev,
	jerin.jacob, marko.kovacevic, orika, bruce.richardson,
	radu.nicolau, akhil.goyal, tomasz.kantecki, skori, pbhagavatula,
	john.mcnamara, stable

The ethdev port id should be 16 bits now. This patch changes the
variable size of port id in some net drivers from 8 bits to 16
bits.

Fixes: cf18d4def295 ("net/ark: provide API for hardware modules UDM and DDM")
Fixes: 09b23f8b9df6 ("net/bnxt: fix port stop process and cleanup resources")
Fixes: 769de16872ab ("net/bnxt: fix port default rule create/destroy")
Fixes: 50370662b727 ("net/ice: support device and queue ops")
Fixes: a50d7cbbdad7 ("net/qede: support registers dump")
Cc: stable@dpdk.org

Signed-off-by: Chenbo Xia <chenbo.xia@intel.com>
---
 drivers/net/ark/ark_udm.h               | 2 +-
 drivers/net/bnxt/tf_ulp/bnxt_ulp.c      | 2 +-
 drivers/net/bnxt/tf_ulp/ulp_def_rules.c | 6 +++---
 drivers/net/ice/ice_rxtx.h              | 4 ++--
 drivers/net/qede/base/bcm_osal.h        | 2 +-
 drivers/net/qede/qede_regs.c            | 2 +-
 6 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/drivers/net/ark/ark_udm.h b/drivers/net/ark/ark_udm.h
index 5846c825b..32eb82faa 100644
--- a/drivers/net/ark/ark_udm.h
+++ b/drivers/net/ark/ark_udm.h
@@ -19,7 +19,7 @@
 struct ark_rx_meta {
 	uint64_t timestamp;
 	uint64_t user_data;
-	uint8_t port;
+	uint16_t port;
 	uint8_t dst_queue;
 	uint16_t pkt_len;
 };
diff --git a/drivers/net/bnxt/tf_ulp/bnxt_ulp.c b/drivers/net/bnxt/tf_ulp/bnxt_ulp.c
index e8927f629..3d148029e 100644
--- a/drivers/net/bnxt/tf_ulp/bnxt_ulp.c
+++ b/drivers/net/bnxt/tf_ulp/bnxt_ulp.c
@@ -646,7 +646,7 @@ static void
 bnxt_ulp_destroy_vfr_default_rules(struct bnxt *bp, bool global)
 {
 	struct bnxt_ulp_vfr_rule_info *info;
-	uint8_t port_id;
+	uint16_t port_id;
 	struct rte_eth_dev *vfr_eth_dev;
 	struct bnxt_vf_representor *vfr_bp;
 
diff --git a/drivers/net/bnxt/tf_ulp/ulp_def_rules.c b/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
index 2d0c3bccc..fdd2692f6 100644
--- a/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
+++ b/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
@@ -402,7 +402,7 @@ void
 bnxt_ulp_destroy_df_rules(struct bnxt *bp, bool global)
 {
 	struct bnxt_ulp_df_rule_info *info;
-	uint8_t port_id;
+	uint16_t port_id;
 
 	if (!BNXT_TRUFLOW_EN(bp) ||
 	    BNXT_ETH_DEV_IS_REPRESENTOR(bp->eth_dev))
@@ -466,7 +466,7 @@ int32_t
 bnxt_ulp_create_df_rules(struct bnxt *bp)
 {
 	struct bnxt_ulp_df_rule_info *info;
-	uint8_t port_id;
+	uint16_t port_id;
 	int rc;
 
 	if (!BNXT_TRUFLOW_EN(bp) ||
@@ -540,7 +540,7 @@ bnxt_ulp_create_vfr_default_rules(struct rte_eth_dev *vfr_ethdev)
 	struct rte_eth_dev *parent_dev = vfr->parent_dev;
 	struct bnxt *bp = parent_dev->data->dev_private;
 	uint16_t vfr_port_id = vfr_ethdev->data->port_id;
-	uint8_t port_id;
+	uint16_t port_id;
 	int rc;
 
 	if (!bp || !BNXT_TRUFLOW_EN(bp))
diff --git a/drivers/net/ice/ice_rxtx.h b/drivers/net/ice/ice_rxtx.h
index 9fa57b3b2..e4309b8b9 100644
--- a/drivers/net/ice/ice_rxtx.h
+++ b/drivers/net/ice/ice_rxtx.h
@@ -68,7 +68,7 @@ struct ice_rx_queue {
 	uint16_t rxrearm_start;	/**< the idx we start the re-arming from */
 	uint64_t mbuf_initializer; /**< value to init mbufs */
 
-	uint8_t port_id; /* device port ID */
+	uint16_t port_id; /* device port ID */
 	uint8_t crc_len; /* 0 if CRC stripped, 4 otherwise */
 	uint8_t fdir_enabled; /* 0 if FDIR disabled, 1 when enabled */
 	uint16_t queue_id; /* RX queue index */
@@ -112,7 +112,7 @@ struct ice_tx_queue {
 	uint8_t pthresh; /**< Prefetch threshold register. */
 	uint8_t hthresh; /**< Host threshold register. */
 	uint8_t wthresh; /**< Write-back threshold reg. */
-	uint8_t port_id; /* Device port identifier. */
+	uint16_t port_id; /* Device port identifier. */
 	uint16_t queue_id; /* TX queue index. */
 	uint32_t q_teid; /* TX schedule node id. */
 	uint16_t reg_idx;
diff --git a/drivers/net/qede/base/bcm_osal.h b/drivers/net/qede/base/bcm_osal.h
index 5d4df5907..814000039 100644
--- a/drivers/net/qede/base/bcm_osal.h
+++ b/drivers/net/qede/base/bcm_osal.h
@@ -372,7 +372,7 @@ void qede_hw_err_notify(struct ecore_hwfn *p_hwfn,
 /* TODO: */
 #define OSAL_SCHEDULE_RECOVERY_HANDLER(hwfn) nothing
 
-int qede_save_fw_dump(uint8_t port_id);
+int qede_save_fw_dump(uint16_t port_id);
 
 #define OSAL_SAVE_FW_DUMP(port_id) qede_save_fw_dump(port_id)
 
diff --git a/drivers/net/qede/qede_regs.c b/drivers/net/qede/qede_regs.c
index 1f2dbc6e7..d2ea1c910 100644
--- a/drivers/net/qede/qede_regs.c
+++ b/drivers/net/qede/qede_regs.c
@@ -242,7 +242,7 @@ qede_write_fwdump(const char *dump_file, void *dump, size_t len)
 }
 
 int
-qede_save_fw_dump(uint8_t port_id)
+qede_save_fw_dump(uint16_t port_id)
 {
 	struct rte_eth_dev *eth_dev = &rte_eth_devices[port_id];
 	struct rte_dev_reg_info regs;
-- 
2.17.1


^ permalink raw reply	[flat|nested] 15+ messages in thread

* [dpdk-dev] [PATCH v2 2/3] app: fix variable size of ethdev port id
  2020-09-25  5:37 ` [dpdk-dev] [PATCH v2 1/3] net: fix variable size of ethdev " Chenbo Xia
@ 2020-09-25  5:37   ` Chenbo Xia
  2020-09-25  5:37   ` [dpdk-dev] [PATCH v2 3/3] doc: " Chenbo Xia
                     ` (2 subsequent siblings)
  3 siblings, 0 replies; 15+ messages in thread
From: Chenbo Xia @ 2020-09-25  5:37 UTC (permalink / raw)
  To: dev
  Cc: shepard.siegel, ed.czeck, john.miller, ajit.khaparde,
	somnath.kotur, qiming.yang, qi.z.zhang, rmody, shshaikh, sbhosle,
	kishore.padmanabha, michael.baucom, venkatkumar.duvvuru,
	xiaoyun.li, ferruh.yigit, jingjing.wu, irusskikh, wenzhuo.lu,
	beilei.xing, bernard.iremonger, nikhil.rao, konstantin.ananyev,
	jerin.jacob, marko.kovacevic, orika, bruce.richardson,
	radu.nicolau, akhil.goyal, tomasz.kantecki, skori, pbhagavatula,
	john.mcnamara, stable

The ethdev port id should be 16 bits now. This patch changes the
variable size of port id in applications from 8 bits to 16 bits.

Fixes: e977e4199a8d ("app/testpmd: add commands to load/unload BPF filters")
Fixes: 46cf97e4bbfa ("eventdev: add test for eth Tx adapter")
Cc: stable@dpdk.org

Signed-off-by: Chenbo Xia <chenbo.xia@intel.com>
---
 app/test-pmd/bpf_cmd.c               | 4 ++--
 app/test/test_event_eth_tx_adapter.c | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/app/test-pmd/bpf_cmd.c b/app/test-pmd/bpf_cmd.c
index 0f984ccf4..16e3c3b3b 100644
--- a/app/test-pmd/bpf_cmd.c
+++ b/app/test-pmd/bpf_cmd.c
@@ -55,7 +55,7 @@ static const struct rte_bpf_xsym bpf_xsym[] = {
 struct cmd_bpf_ld_result {
 	cmdline_fixed_string_t bpf;
 	cmdline_fixed_string_t dir;
-	uint8_t port;
+	uint16_t port;
 	uint16_t queue;
 	cmdline_fixed_string_t op;
 	cmdline_fixed_string_t flags;
@@ -153,7 +153,7 @@ cmdline_parse_inst_t cmd_operate_bpf_ld_parse = {
 struct cmd_bpf_unld_result {
 	cmdline_fixed_string_t bpf;
 	cmdline_fixed_string_t dir;
-	uint8_t port;
+	uint16_t port;
 	uint16_t queue;
 };
 
diff --git a/app/test/test_event_eth_tx_adapter.c b/app/test/test_event_eth_tx_adapter.c
index 3af749280..707303090 100644
--- a/app/test/test_event_eth_tx_adapter.c
+++ b/app/test/test_event_eth_tx_adapter.c
@@ -45,7 +45,7 @@ static uint64_t eid = ~0ULL;
 static uint32_t tid;
 
 static inline int
-port_init_common(uint8_t port, const struct rte_eth_conf *port_conf,
+port_init_common(uint16_t port, const struct rte_eth_conf *port_conf,
 		struct rte_mempool *mp)
 {
 	const uint16_t rx_ring_size = RING_SIZE, tx_ring_size = RING_SIZE;
@@ -104,7 +104,7 @@ port_init_common(uint8_t port, const struct rte_eth_conf *port_conf,
 }
 
 static inline int
-port_init(uint8_t port, struct rte_mempool *mp)
+port_init(uint16_t port, struct rte_mempool *mp)
 {
 	struct rte_eth_conf conf = { 0 };
 	return port_init_common(port, &conf, mp);
-- 
2.17.1


^ permalink raw reply	[flat|nested] 15+ messages in thread

* [dpdk-dev] [PATCH v2 3/3] doc: fix variable size of ethdev port id
  2020-09-25  5:37 ` [dpdk-dev] [PATCH v2 1/3] net: fix variable size of ethdev " Chenbo Xia
  2020-09-25  5:37   ` [dpdk-dev] [PATCH v2 2/3] app: " Chenbo Xia
@ 2020-09-25  5:37   ` Chenbo Xia
  2020-09-29 18:51   ` [dpdk-dev] [PATCH v2 1/3] net: " Ed Czeck
  2020-09-30  4:23   ` [dpdk-dev] [PATCH v3 " Chenbo Xia
  3 siblings, 0 replies; 15+ messages in thread
From: Chenbo Xia @ 2020-09-25  5:37 UTC (permalink / raw)
  To: dev
  Cc: shepard.siegel, ed.czeck, john.miller, ajit.khaparde,
	somnath.kotur, qiming.yang, qi.z.zhang, rmody, shshaikh, sbhosle,
	kishore.padmanabha, michael.baucom, venkatkumar.duvvuru,
	xiaoyun.li, ferruh.yigit, jingjing.wu, irusskikh, wenzhuo.lu,
	beilei.xing, bernard.iremonger, nikhil.rao, konstantin.ananyev,
	jerin.jacob, marko.kovacevic, orika, bruce.richardson,
	radu.nicolau, akhil.goyal, tomasz.kantecki, skori, pbhagavatula,
	john.mcnamara, stable

The ethdev port id should be 16 bits now. This patch changes the
variable size of port id in docs from 8 bits to 16 bits.

Fixes: fdec9301f52d ("doc: add flow classify guides")
Fixes: 4a3ef59a10c8 ("examples/flow_filtering: add simple demo of flow API")
Cc: stable@dpdk.org

Signed-off-by: Chenbo Xia <chenbo.xia@intel.com>
---
 doc/guides/sample_app_ug/flow_classify.rst  | 2 +-
 doc/guides/sample_app_ug/flow_filtering.rst | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/doc/guides/sample_app_ug/flow_classify.rst b/doc/guides/sample_app_ug/flow_classify.rst
index dc40b4d6f..31175cff0 100644
--- a/doc/guides/sample_app_ug/flow_classify.rst
+++ b/doc/guides/sample_app_ug/flow_classify.rst
@@ -271,7 +271,7 @@ Forwarding application is shown below:
 .. code-block:: c
 
     static inline int
-    port_init(uint8_t port, struct rte_mempool *mbuf_pool)
+    port_init(uint16_t port, struct rte_mempool *mbuf_pool)
     {
         struct rte_eth_conf port_conf = port_conf_default;
         const uint16_t rx_rings = 1, tx_rings = 1;
diff --git a/doc/guides/sample_app_ug/flow_filtering.rst b/doc/guides/sample_app_ug/flow_filtering.rst
index 5e5a6cd8a..d3653e57b 100644
--- a/doc/guides/sample_app_ug/flow_filtering.rst
+++ b/doc/guides/sample_app_ug/flow_filtering.rst
@@ -384,7 +384,7 @@ This function is located in the ``flow_blocks.c`` file.
 .. code-block:: c
 
    static struct rte_flow *
-   generate_ipv4_flow(uint8_t port_id, uint16_t rx_q,
+   generate_ipv4_flow(uint16_t port_id, uint16_t rx_q,
                    uint32_t src_ip, uint32_t src_mask,
                    uint32_t dest_ip, uint32_t dest_mask,
                    struct rte_flow_error *error)
-- 
2.17.1


^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [dpdk-dev] [PATCH v2 1/3] net: fix variable size of ethdev port id
  2020-09-25  5:37 ` [dpdk-dev] [PATCH v2 1/3] net: fix variable size of ethdev " Chenbo Xia
  2020-09-25  5:37   ` [dpdk-dev] [PATCH v2 2/3] app: " Chenbo Xia
  2020-09-25  5:37   ` [dpdk-dev] [PATCH v2 3/3] doc: " Chenbo Xia
@ 2020-09-29 18:51   ` Ed Czeck
  2020-09-30  4:23   ` [dpdk-dev] [PATCH v3 " Chenbo Xia
  3 siblings, 0 replies; 15+ messages in thread
From: Ed Czeck @ 2020-09-29 18:51 UTC (permalink / raw)
  To: Chenbo Xia
  Cc: dev, Shepard Siegel, John Miller, ajit.khaparde, somnath.kotur,
	qiming.yang, qi.z.zhang, rmody, shshaikh, sbhosle,
	kishore.padmanabha, michael.baucom, venkatkumar.duvvuru,
	xiaoyun.li, Ferruh Yigit, jingjing.wu, irusskikh, wenzhuo.lu,
	beilei.xing, bernard.iremonger, nikhil.rao, konstantin.ananyev,
	jerin.jacob, marko.kovacevic, orika, Richardson, Bruce,
	radu.nicolau, akhil.goyal, tomasz.kantecki, skori, pbhagavatula,
	John McNamara, stable

On Fri, Sep 25, 2020 at 1:41 AM Chenbo Xia <chenbo.xia@intel.com> wrote:
>
> The ethdev port id should be 16 bits now. This patch changes the
> variable size of port id in some net drivers from 8 bits to 16
> bits.
>
> Fixes: cf18d4def295 ("net/ark: provide API for hardware modules UDM and DDM")
> Fixes: 09b23f8b9df6 ("net/bnxt: fix port stop process and cleanup resources")
> Fixes: 769de16872ab ("net/bnxt: fix port default rule create/destroy")
> Fixes: 50370662b727 ("net/ice: support device and queue ops")
> Fixes: a50d7cbbdad7 ("net/qede: support registers dump")
> Cc: stable@dpdk.org
>
> Signed-off-by: Chenbo Xia <chenbo.xia@intel.com>
> ---
>  drivers/net/ark/ark_udm.h               | 2 +-
>  drivers/net/bnxt/tf_ulp/bnxt_ulp.c      | 2 +-
>  drivers/net/bnxt/tf_ulp/ulp_def_rules.c | 6 +++---
>  drivers/net/ice/ice_rxtx.h              | 4 ++--
>  drivers/net/qede/base/bcm_osal.h        | 2 +-
>  drivers/net/qede/qede_regs.c            | 2 +-
>  6 files changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/net/ark/ark_udm.h b/drivers/net/ark/ark_udm.h
> index 5846c825b..32eb82faa 100644
> --- a/drivers/net/ark/ark_udm.h
> +++ b/drivers/net/ark/ark_udm.h
> @@ -19,7 +19,7 @@
>  struct ark_rx_meta {
>         uint64_t timestamp;
>         uint64_t user_data;
> -       uint8_t port;
> +       uint16_t port;
>         uint8_t dst_queue;
>         uint16_t pkt_len;
>  };

NACK - this struct correlates with an identical structure in the
hardware module.  The C-level cannot be changed without a hardware
update.  For the net/ark driver we will be content with 8-bit port
ids.
Ed.

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [dpdk-dev] [PATCH v3 1/3] net: fix variable size of ethdev port id
  2020-09-25  5:37 ` [dpdk-dev] [PATCH v2 1/3] net: fix variable size of ethdev " Chenbo Xia
                     ` (2 preceding siblings ...)
  2020-09-29 18:51   ` [dpdk-dev] [PATCH v2 1/3] net: " Ed Czeck
@ 2020-09-30  4:23   ` Chenbo Xia
  2020-09-30  4:23     ` [dpdk-dev] [PATCH v3 2/3] app: " Chenbo Xia
                       ` (3 more replies)
  3 siblings, 4 replies; 15+ messages in thread
From: Chenbo Xia @ 2020-09-30  4:23 UTC (permalink / raw)
  To: dev
  Cc: ajit.khaparde, somnath.kotur, qiming.yang, qi.z.zhang, rmody,
	shshaikh, sbhosle, kishore.padmanabha, michael.baucom,
	venkatkumar.duvvuru, xiaoyun.li, ferruh.yigit, jingjing.wu,
	irusskikh, wenzhuo.lu, beilei.xing, bernard.iremonger,
	nikhil.rao, konstantin.ananyev, jerin.jacob, marko.kovacevic,
	orika, bruce.richardson, radu.nicolau, akhil.goyal,
	tomasz.kantecki, skori, pbhagavatula, john.mcnamara, stable

The ethdev port id should be 16 bits now. This patch changes the
variable size of port id in some net drivers from 8 bits to 16
bits.

Fixes: 09b23f8b9df6 ("net/bnxt: fix port stop process and cleanup resources")
Fixes: 769de16872ab ("net/bnxt: fix port default rule create/destroy")
Fixes: 50370662b727 ("net/ice: support device and queue ops")
Fixes: a50d7cbbdad7 ("net/qede: support registers dump")
Cc: stable@dpdk.org

Signed-off-by: Chenbo Xia <chenbo.xia@intel.com>
---
 drivers/net/bnxt/tf_ulp/bnxt_ulp.c      | 2 +-
 drivers/net/bnxt/tf_ulp/ulp_def_rules.c | 6 +++---
 drivers/net/ice/ice_rxtx.h              | 4 ++--
 drivers/net/qede/base/bcm_osal.h        | 2 +-
 drivers/net/qede/qede_regs.c            | 2 +-
 5 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/drivers/net/bnxt/tf_ulp/bnxt_ulp.c b/drivers/net/bnxt/tf_ulp/bnxt_ulp.c
index e8927f629..3d148029e 100644
--- a/drivers/net/bnxt/tf_ulp/bnxt_ulp.c
+++ b/drivers/net/bnxt/tf_ulp/bnxt_ulp.c
@@ -646,7 +646,7 @@ static void
 bnxt_ulp_destroy_vfr_default_rules(struct bnxt *bp, bool global)
 {
 	struct bnxt_ulp_vfr_rule_info *info;
-	uint8_t port_id;
+	uint16_t port_id;
 	struct rte_eth_dev *vfr_eth_dev;
 	struct bnxt_vf_representor *vfr_bp;
 
diff --git a/drivers/net/bnxt/tf_ulp/ulp_def_rules.c b/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
index 2d0c3bccc..fdd2692f6 100644
--- a/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
+++ b/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
@@ -402,7 +402,7 @@ void
 bnxt_ulp_destroy_df_rules(struct bnxt *bp, bool global)
 {
 	struct bnxt_ulp_df_rule_info *info;
-	uint8_t port_id;
+	uint16_t port_id;
 
 	if (!BNXT_TRUFLOW_EN(bp) ||
 	    BNXT_ETH_DEV_IS_REPRESENTOR(bp->eth_dev))
@@ -466,7 +466,7 @@ int32_t
 bnxt_ulp_create_df_rules(struct bnxt *bp)
 {
 	struct bnxt_ulp_df_rule_info *info;
-	uint8_t port_id;
+	uint16_t port_id;
 	int rc;
 
 	if (!BNXT_TRUFLOW_EN(bp) ||
@@ -540,7 +540,7 @@ bnxt_ulp_create_vfr_default_rules(struct rte_eth_dev *vfr_ethdev)
 	struct rte_eth_dev *parent_dev = vfr->parent_dev;
 	struct bnxt *bp = parent_dev->data->dev_private;
 	uint16_t vfr_port_id = vfr_ethdev->data->port_id;
-	uint8_t port_id;
+	uint16_t port_id;
 	int rc;
 
 	if (!bp || !BNXT_TRUFLOW_EN(bp))
diff --git a/drivers/net/ice/ice_rxtx.h b/drivers/net/ice/ice_rxtx.h
index 9fa57b3b2..e4309b8b9 100644
--- a/drivers/net/ice/ice_rxtx.h
+++ b/drivers/net/ice/ice_rxtx.h
@@ -68,7 +68,7 @@ struct ice_rx_queue {
 	uint16_t rxrearm_start;	/**< the idx we start the re-arming from */
 	uint64_t mbuf_initializer; /**< value to init mbufs */
 
-	uint8_t port_id; /* device port ID */
+	uint16_t port_id; /* device port ID */
 	uint8_t crc_len; /* 0 if CRC stripped, 4 otherwise */
 	uint8_t fdir_enabled; /* 0 if FDIR disabled, 1 when enabled */
 	uint16_t queue_id; /* RX queue index */
@@ -112,7 +112,7 @@ struct ice_tx_queue {
 	uint8_t pthresh; /**< Prefetch threshold register. */
 	uint8_t hthresh; /**< Host threshold register. */
 	uint8_t wthresh; /**< Write-back threshold reg. */
-	uint8_t port_id; /* Device port identifier. */
+	uint16_t port_id; /* Device port identifier. */
 	uint16_t queue_id; /* TX queue index. */
 	uint32_t q_teid; /* TX schedule node id. */
 	uint16_t reg_idx;
diff --git a/drivers/net/qede/base/bcm_osal.h b/drivers/net/qede/base/bcm_osal.h
index 5d4df5907..814000039 100644
--- a/drivers/net/qede/base/bcm_osal.h
+++ b/drivers/net/qede/base/bcm_osal.h
@@ -372,7 +372,7 @@ void qede_hw_err_notify(struct ecore_hwfn *p_hwfn,
 /* TODO: */
 #define OSAL_SCHEDULE_RECOVERY_HANDLER(hwfn) nothing
 
-int qede_save_fw_dump(uint8_t port_id);
+int qede_save_fw_dump(uint16_t port_id);
 
 #define OSAL_SAVE_FW_DUMP(port_id) qede_save_fw_dump(port_id)
 
diff --git a/drivers/net/qede/qede_regs.c b/drivers/net/qede/qede_regs.c
index 1f2dbc6e7..d2ea1c910 100644
--- a/drivers/net/qede/qede_regs.c
+++ b/drivers/net/qede/qede_regs.c
@@ -242,7 +242,7 @@ qede_write_fwdump(const char *dump_file, void *dump, size_t len)
 }
 
 int
-qede_save_fw_dump(uint8_t port_id)
+qede_save_fw_dump(uint16_t port_id)
 {
 	struct rte_eth_dev *eth_dev = &rte_eth_devices[port_id];
 	struct rte_dev_reg_info regs;
-- 
2.17.1


^ permalink raw reply	[flat|nested] 15+ messages in thread

* [dpdk-dev] [PATCH v3 2/3] app: fix variable size of ethdev port id
  2020-09-30  4:23   ` [dpdk-dev] [PATCH v3 " Chenbo Xia
@ 2020-09-30  4:23     ` Chenbo Xia
  2020-09-30  4:23     ` [dpdk-dev] [PATCH v3 3/3] doc: " Chenbo Xia
                       ` (2 subsequent siblings)
  3 siblings, 0 replies; 15+ messages in thread
From: Chenbo Xia @ 2020-09-30  4:23 UTC (permalink / raw)
  To: dev
  Cc: ajit.khaparde, somnath.kotur, qiming.yang, qi.z.zhang, rmody,
	shshaikh, sbhosle, kishore.padmanabha, michael.baucom,
	venkatkumar.duvvuru, xiaoyun.li, ferruh.yigit, jingjing.wu,
	irusskikh, wenzhuo.lu, beilei.xing, bernard.iremonger,
	nikhil.rao, konstantin.ananyev, jerin.jacob, marko.kovacevic,
	orika, bruce.richardson, radu.nicolau, akhil.goyal,
	tomasz.kantecki, skori, pbhagavatula, john.mcnamara, stable

The ethdev port id should be 16 bits now. This patch changes the
variable size of port id in applications from 8 bits to 16 bits.

Fixes: e977e4199a8d ("app/testpmd: add commands to load/unload BPF filters")
Fixes: 46cf97e4bbfa ("eventdev: add test for eth Tx adapter")
Cc: stable@dpdk.org

Signed-off-by: Chenbo Xia <chenbo.xia@intel.com>
---
 app/test-pmd/bpf_cmd.c               | 4 ++--
 app/test/test_event_eth_tx_adapter.c | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/app/test-pmd/bpf_cmd.c b/app/test-pmd/bpf_cmd.c
index 0f984ccf4..16e3c3b3b 100644
--- a/app/test-pmd/bpf_cmd.c
+++ b/app/test-pmd/bpf_cmd.c
@@ -55,7 +55,7 @@ static const struct rte_bpf_xsym bpf_xsym[] = {
 struct cmd_bpf_ld_result {
 	cmdline_fixed_string_t bpf;
 	cmdline_fixed_string_t dir;
-	uint8_t port;
+	uint16_t port;
 	uint16_t queue;
 	cmdline_fixed_string_t op;
 	cmdline_fixed_string_t flags;
@@ -153,7 +153,7 @@ cmdline_parse_inst_t cmd_operate_bpf_ld_parse = {
 struct cmd_bpf_unld_result {
 	cmdline_fixed_string_t bpf;
 	cmdline_fixed_string_t dir;
-	uint8_t port;
+	uint16_t port;
 	uint16_t queue;
 };
 
diff --git a/app/test/test_event_eth_tx_adapter.c b/app/test/test_event_eth_tx_adapter.c
index 3af749280..707303090 100644
--- a/app/test/test_event_eth_tx_adapter.c
+++ b/app/test/test_event_eth_tx_adapter.c
@@ -45,7 +45,7 @@ static uint64_t eid = ~0ULL;
 static uint32_t tid;
 
 static inline int
-port_init_common(uint8_t port, const struct rte_eth_conf *port_conf,
+port_init_common(uint16_t port, const struct rte_eth_conf *port_conf,
 		struct rte_mempool *mp)
 {
 	const uint16_t rx_ring_size = RING_SIZE, tx_ring_size = RING_SIZE;
@@ -104,7 +104,7 @@ port_init_common(uint8_t port, const struct rte_eth_conf *port_conf,
 }
 
 static inline int
-port_init(uint8_t port, struct rte_mempool *mp)
+port_init(uint16_t port, struct rte_mempool *mp)
 {
 	struct rte_eth_conf conf = { 0 };
 	return port_init_common(port, &conf, mp);
-- 
2.17.1


^ permalink raw reply	[flat|nested] 15+ messages in thread

* [dpdk-dev] [PATCH v3 3/3] doc: fix variable size of ethdev port id
  2020-09-30  4:23   ` [dpdk-dev] [PATCH v3 " Chenbo Xia
  2020-09-30  4:23     ` [dpdk-dev] [PATCH v3 2/3] app: " Chenbo Xia
@ 2020-09-30  4:23     ` Chenbo Xia
  2020-09-30 13:40     ` [dpdk-dev] [PATCH v3 1/3] net: " Ajit Khaparde
  2020-09-30 17:13     ` Ferruh Yigit
  3 siblings, 0 replies; 15+ messages in thread
From: Chenbo Xia @ 2020-09-30  4:23 UTC (permalink / raw)
  To: dev
  Cc: ajit.khaparde, somnath.kotur, qiming.yang, qi.z.zhang, rmody,
	shshaikh, sbhosle, kishore.padmanabha, michael.baucom,
	venkatkumar.duvvuru, xiaoyun.li, ferruh.yigit, jingjing.wu,
	irusskikh, wenzhuo.lu, beilei.xing, bernard.iremonger,
	nikhil.rao, konstantin.ananyev, jerin.jacob, marko.kovacevic,
	orika, bruce.richardson, radu.nicolau, akhil.goyal,
	tomasz.kantecki, skori, pbhagavatula, john.mcnamara, stable

The ethdev port id should be 16 bits now. This patch changes the
variable size of port id in docs from 8 bits to 16 bits.

Fixes: fdec9301f52d ("doc: add flow classify guides")
Fixes: 4a3ef59a10c8 ("examples/flow_filtering: add simple demo of flow API")
Cc: stable@dpdk.org

Signed-off-by: Chenbo Xia <chenbo.xia@intel.com>
---
 doc/guides/sample_app_ug/flow_classify.rst  | 2 +-
 doc/guides/sample_app_ug/flow_filtering.rst | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/doc/guides/sample_app_ug/flow_classify.rst b/doc/guides/sample_app_ug/flow_classify.rst
index dc40b4d6f..31175cff0 100644
--- a/doc/guides/sample_app_ug/flow_classify.rst
+++ b/doc/guides/sample_app_ug/flow_classify.rst
@@ -271,7 +271,7 @@ Forwarding application is shown below:
 .. code-block:: c
 
     static inline int
-    port_init(uint8_t port, struct rte_mempool *mbuf_pool)
+    port_init(uint16_t port, struct rte_mempool *mbuf_pool)
     {
         struct rte_eth_conf port_conf = port_conf_default;
         const uint16_t rx_rings = 1, tx_rings = 1;
diff --git a/doc/guides/sample_app_ug/flow_filtering.rst b/doc/guides/sample_app_ug/flow_filtering.rst
index 5e5a6cd8a..d3653e57b 100644
--- a/doc/guides/sample_app_ug/flow_filtering.rst
+++ b/doc/guides/sample_app_ug/flow_filtering.rst
@@ -384,7 +384,7 @@ This function is located in the ``flow_blocks.c`` file.
 .. code-block:: c
 
    static struct rte_flow *
-   generate_ipv4_flow(uint8_t port_id, uint16_t rx_q,
+   generate_ipv4_flow(uint16_t port_id, uint16_t rx_q,
                    uint32_t src_ip, uint32_t src_mask,
                    uint32_t dest_ip, uint32_t dest_mask,
                    struct rte_flow_error *error)
-- 
2.17.1


^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [dpdk-dev] [PATCH v3 1/3] net: fix variable size of ethdev port id
  2020-09-30  4:23   ` [dpdk-dev] [PATCH v3 " Chenbo Xia
  2020-09-30  4:23     ` [dpdk-dev] [PATCH v3 2/3] app: " Chenbo Xia
  2020-09-30  4:23     ` [dpdk-dev] [PATCH v3 3/3] doc: " Chenbo Xia
@ 2020-09-30 13:40     ` Ajit Khaparde
  2020-09-30 17:13     ` Ferruh Yigit
  3 siblings, 0 replies; 15+ messages in thread
From: Ajit Khaparde @ 2020-09-30 13:40 UTC (permalink / raw)
  To: Chenbo Xia
  Cc: dpdk-dev, Somnath Kotur, Qiming Yang, Qi Zhang, Rasesh Mody,
	Shahed Shaikh, Shahaji Bhosle, Kishore Padmanabha,
	Michael Baucom, Venkat Duvvuru, Xiaoyun Li, Ferruh Yigit,
	Jingjing Wu, irusskikh, Wenzhuo Lu, Beilei Xing, Iremonger,
	Bernard, nikhil.rao, Ananyev, Konstantin, Jerin Jacob,
	Marko Kovacevic, Ori Kam, Bruce Richardson, radu.nicolau,
	Akhil Goyal, tomasz.kantecki, skori, pbhagavatula, Mcnamara,
	John, dpdk stable

On Tue, Sep 29, 2020 at 9:28 PM Chenbo Xia <chenbo.xia@intel.com> wrote:
>
> The ethdev port id should be 16 bits now. This patch changes the
> variable size of port id in some net drivers from 8 bits to 16
> bits.
>
> Fixes: 09b23f8b9df6 ("net/bnxt: fix port stop process and cleanup resources")
> Fixes: 769de16872ab ("net/bnxt: fix port default rule create/destroy")
> Fixes: 50370662b727 ("net/ice: support device and queue ops")
> Fixes: a50d7cbbdad7 ("net/qede: support registers dump")
> Cc: stable@dpdk.org
>
> Signed-off-by: Chenbo Xia <chenbo.xia@intel.com>

For bnxt,
Acked-by: Ajit Khaparde <ajit.khaparde@broadcom.com>

>
> ---
>  drivers/net/bnxt/tf_ulp/bnxt_ulp.c      | 2 +-
>  drivers/net/bnxt/tf_ulp/ulp_def_rules.c | 6 +++---
>  drivers/net/ice/ice_rxtx.h              | 4 ++--
>  drivers/net/qede/base/bcm_osal.h        | 2 +-
>  drivers/net/qede/qede_regs.c            | 2 +-
>  5 files changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/net/bnxt/tf_ulp/bnxt_ulp.c b/drivers/net/bnxt/tf_ulp/bnxt_ulp.c
> index e8927f629..3d148029e 100644
> --- a/drivers/net/bnxt/tf_ulp/bnxt_ulp.c
> +++ b/drivers/net/bnxt/tf_ulp/bnxt_ulp.c
> @@ -646,7 +646,7 @@ static void
>  bnxt_ulp_destroy_vfr_default_rules(struct bnxt *bp, bool global)
>  {
>         struct bnxt_ulp_vfr_rule_info *info;
> -       uint8_t port_id;
> +       uint16_t port_id;
>         struct rte_eth_dev *vfr_eth_dev;
>         struct bnxt_vf_representor *vfr_bp;
>
> diff --git a/drivers/net/bnxt/tf_ulp/ulp_def_rules.c b/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
> index 2d0c3bccc..fdd2692f6 100644
> --- a/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
> +++ b/drivers/net/bnxt/tf_ulp/ulp_def_rules.c
> @@ -402,7 +402,7 @@ void
>  bnxt_ulp_destroy_df_rules(struct bnxt *bp, bool global)
>  {
>         struct bnxt_ulp_df_rule_info *info;
> -       uint8_t port_id;
> +       uint16_t port_id;
>
>         if (!BNXT_TRUFLOW_EN(bp) ||
>             BNXT_ETH_DEV_IS_REPRESENTOR(bp->eth_dev))
> @@ -466,7 +466,7 @@ int32_t
>  bnxt_ulp_create_df_rules(struct bnxt *bp)
>  {
>         struct bnxt_ulp_df_rule_info *info;
> -       uint8_t port_id;
> +       uint16_t port_id;
>         int rc;
>
>         if (!BNXT_TRUFLOW_EN(bp) ||
> @@ -540,7 +540,7 @@ bnxt_ulp_create_vfr_default_rules(struct rte_eth_dev *vfr_ethdev)
>         struct rte_eth_dev *parent_dev = vfr->parent_dev;
>         struct bnxt *bp = parent_dev->data->dev_private;
>         uint16_t vfr_port_id = vfr_ethdev->data->port_id;
> -       uint8_t port_id;
> +       uint16_t port_id;
>         int rc;
>
>         if (!bp || !BNXT_TRUFLOW_EN(bp))
> diff --git a/drivers/net/ice/ice_rxtx.h b/drivers/net/ice/ice_rxtx.h
> index 9fa57b3b2..e4309b8b9 100644
> --- a/drivers/net/ice/ice_rxtx.h
> +++ b/drivers/net/ice/ice_rxtx.h
> @@ -68,7 +68,7 @@ struct ice_rx_queue {
>         uint16_t rxrearm_start; /**< the idx we start the re-arming from */
>         uint64_t mbuf_initializer; /**< value to init mbufs */
>
> -       uint8_t port_id; /* device port ID */
> +       uint16_t port_id; /* device port ID */
>         uint8_t crc_len; /* 0 if CRC stripped, 4 otherwise */
>         uint8_t fdir_enabled; /* 0 if FDIR disabled, 1 when enabled */
>         uint16_t queue_id; /* RX queue index */
> @@ -112,7 +112,7 @@ struct ice_tx_queue {
>         uint8_t pthresh; /**< Prefetch threshold register. */
>         uint8_t hthresh; /**< Host threshold register. */
>         uint8_t wthresh; /**< Write-back threshold reg. */
> -       uint8_t port_id; /* Device port identifier. */
> +       uint16_t port_id; /* Device port identifier. */
>         uint16_t queue_id; /* TX queue index. */
>         uint32_t q_teid; /* TX schedule node id. */
>         uint16_t reg_idx;
> diff --git a/drivers/net/qede/base/bcm_osal.h b/drivers/net/qede/base/bcm_osal.h
> index 5d4df5907..814000039 100644
> --- a/drivers/net/qede/base/bcm_osal.h
> +++ b/drivers/net/qede/base/bcm_osal.h
> @@ -372,7 +372,7 @@ void qede_hw_err_notify(struct ecore_hwfn *p_hwfn,
>  /* TODO: */
>  #define OSAL_SCHEDULE_RECOVERY_HANDLER(hwfn) nothing
>
> -int qede_save_fw_dump(uint8_t port_id);
> +int qede_save_fw_dump(uint16_t port_id);
>
>  #define OSAL_SAVE_FW_DUMP(port_id) qede_save_fw_dump(port_id)
>
> diff --git a/drivers/net/qede/qede_regs.c b/drivers/net/qede/qede_regs.c
> index 1f2dbc6e7..d2ea1c910 100644
> --- a/drivers/net/qede/qede_regs.c
> +++ b/drivers/net/qede/qede_regs.c
> @@ -242,7 +242,7 @@ qede_write_fwdump(const char *dump_file, void *dump, size_t len)
>  }
>
>  int
> -qede_save_fw_dump(uint8_t port_id)
> +qede_save_fw_dump(uint16_t port_id)
>  {
>         struct rte_eth_dev *eth_dev = &rte_eth_devices[port_id];
>         struct rte_dev_reg_info regs;
> --
> 2.17.1
>

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [dpdk-dev] [PATCH v3 1/3] net: fix variable size of ethdev port id
  2020-09-30  4:23   ` [dpdk-dev] [PATCH v3 " Chenbo Xia
                       ` (2 preceding siblings ...)
  2020-09-30 13:40     ` [dpdk-dev] [PATCH v3 1/3] net: " Ajit Khaparde
@ 2020-09-30 17:13     ` Ferruh Yigit
  3 siblings, 0 replies; 15+ messages in thread
From: Ferruh Yigit @ 2020-09-30 17:13 UTC (permalink / raw)
  To: Chenbo Xia, dev
  Cc: ajit.khaparde, somnath.kotur, qiming.yang, qi.z.zhang, rmody,
	shshaikh, sbhosle, kishore.padmanabha, michael.baucom,
	venkatkumar.duvvuru, xiaoyun.li, jingjing.wu, irusskikh,
	wenzhuo.lu, beilei.xing, bernard.iremonger, nikhil.rao,
	konstantin.ananyev, jerin.jacob, marko.kovacevic, orika,
	bruce.richardson, radu.nicolau, akhil.goyal, tomasz.kantecki,
	skori, pbhagavatula, john.mcnamara, stable

On 9/30/2020 5:23 AM, Chenbo Xia wrote:
> The ethdev port id should be 16 bits now. This patch changes the
> variable size of port id in some net drivers from 8 bits to 16
> bits.
> 
> Fixes: 09b23f8b9df6 ("net/bnxt: fix port stop process and cleanup resources")
> Fixes: 769de16872ab ("net/bnxt: fix port default rule create/destroy")
> Fixes: 50370662b727 ("net/ice: support device and queue ops")
> Fixes: a50d7cbbdad7 ("net/qede: support registers dump")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Chenbo Xia <chenbo.xia@intel.com>
 >

For series,
Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>

Series applied to dpdk-next-net/main, thanks.


^ permalink raw reply	[flat|nested] 15+ messages in thread

end of thread, other threads:[~2020-09-30 17:13 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-04 17:41 [dpdk-dev] [PATCH] net/bnxt: fix variable size of port id Chenbo Xia
2020-08-04 16:22 ` Stephen Hemminger
2020-08-05 20:18 ` Thomas Monjalon
2020-08-06  2:03   ` Xia, Chenbo
2020-08-06  7:57     ` Thomas Monjalon
2020-09-03  6:49 ` Xiao, QimaiX
2020-09-25  5:37 ` [dpdk-dev] [PATCH v2 1/3] net: fix variable size of ethdev " Chenbo Xia
2020-09-25  5:37   ` [dpdk-dev] [PATCH v2 2/3] app: " Chenbo Xia
2020-09-25  5:37   ` [dpdk-dev] [PATCH v2 3/3] doc: " Chenbo Xia
2020-09-29 18:51   ` [dpdk-dev] [PATCH v2 1/3] net: " Ed Czeck
2020-09-30  4:23   ` [dpdk-dev] [PATCH v3 " Chenbo Xia
2020-09-30  4:23     ` [dpdk-dev] [PATCH v3 2/3] app: " Chenbo Xia
2020-09-30  4:23     ` [dpdk-dev] [PATCH v3 3/3] doc: " Chenbo Xia
2020-09-30 13:40     ` [dpdk-dev] [PATCH v3 1/3] net: " Ajit Khaparde
2020-09-30 17:13     ` Ferruh Yigit

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).