* [dpdk-stable] [PATCH v3 02/14] net/mlx5: fix field reference for PPC [not found] ` <20211103075838.1486056-1-xuemingl@nvidia.com> @ 2021-11-03 7:58 ` Xueming Li 2021-11-03 7:58 ` [dpdk-stable] [PATCH v3 05/14] net/mlx5: fix Rx queue memory allocation return value Xueming Li 1 sibling, 0 replies; 6+ messages in thread From: Xueming Li @ 2021-11-03 7:58 UTC (permalink / raw) To: dev Cc: xuemingl, Lior Margalit, viacheslavo, stable, David Christensen, Matan Azrad, Yongseok Koh This patch fixes stale field reference. Fixes: a18ac6113331 ("net/mlx5: add metadata support to Rx datapath") Cc: viacheslavo@nvidia.com Cc: stable@dpdk.org Signed-off-by: Xueming Li <xuemingl@nvidia.com> --- drivers/net/mlx5/mlx5_rxtx_vec_altivec.h | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/net/mlx5/mlx5_rxtx_vec_altivec.h b/drivers/net/mlx5/mlx5_rxtx_vec_altivec.h index bcf487c34e9..1d00c1c43d1 100644 --- a/drivers/net/mlx5/mlx5_rxtx_vec_altivec.h +++ b/drivers/net/mlx5/mlx5_rxtx_vec_altivec.h @@ -974,10 +974,10 @@ rxq_cq_process_v(struct mlx5_rxq_data *rxq, volatile struct mlx5_cqe *cq, (vector unsigned short)cqe_tmp1, cqe_sel_mask1); cqe_tmp2 = (vector unsigned char)(vector unsigned long){ *(__rte_aligned(8) unsigned long *) - &cq[pos + p3].rsvd3[9], 0LL}; + &cq[pos + p3].rsvd4[2], 0LL}; cqe_tmp1 = (vector unsigned char)(vector unsigned long){ *(__rte_aligned(8) unsigned long *) - &cq[pos + p2].rsvd3[9], 0LL}; + &cq[pos + p2].rsvd4[2], 0LL}; cqes[3] = (vector unsigned char) vec_sel((vector unsigned short)cqes[3], (vector unsigned short)cqe_tmp2, @@ -1037,10 +1037,10 @@ rxq_cq_process_v(struct mlx5_rxq_data *rxq, volatile struct mlx5_cqe *cq, (vector unsigned short)cqe_tmp1, cqe_sel_mask1); cqe_tmp2 = (vector unsigned char)(vector unsigned long){ *(__rte_aligned(8) unsigned long *) - &cq[pos + p1].rsvd3[9], 0LL}; + &cq[pos + p1].rsvd4[2], 0LL}; cqe_tmp1 = (vector unsigned char)(vector unsigned long){ *(__rte_aligned(8) unsigned long *) - &cq[pos].rsvd3[9], 0LL}; + &cq[pos].rsvd4[2], 0LL}; cqes[1] = (vector unsigned char) vec_sel((vector unsigned short)cqes[1], (vector unsigned short)cqe_tmp2, cqe_sel_mask2); -- 2.33.0 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [dpdk-stable] [PATCH v3 05/14] net/mlx5: fix Rx queue memory allocation return value [not found] ` <20211103075838.1486056-1-xuemingl@nvidia.com> 2021-11-03 7:58 ` [dpdk-stable] [PATCH v3 02/14] net/mlx5: fix field reference for PPC Xueming Li @ 2021-11-03 7:58 ` Xueming Li 1 sibling, 0 replies; 6+ messages in thread From: Xueming Li @ 2021-11-03 7:58 UTC (permalink / raw) To: dev Cc: xuemingl, Lior Margalit, akozyrev, stable, Matan Azrad, Viacheslav Ovsiienko If error happened during Rx queue mbuf allocation, boolean value returned. From description, return value should be error number. This patch returns negative error number. Fixes: 0f20acbf5eda ("net/mlx5: implement vectorized MPRQ burst") Cc: akozyrev@nvidia.com Cc: stable@dpdk.org Signed-off-by: Xueming Li <xuemingl@nvidia.com> --- drivers/net/mlx5/mlx5_rxq.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers/net/mlx5/mlx5_rxq.c b/drivers/net/mlx5/mlx5_rxq.c index 9220bb2c15c..4567b43c1b6 100644 --- a/drivers/net/mlx5/mlx5_rxq.c +++ b/drivers/net/mlx5/mlx5_rxq.c @@ -129,7 +129,7 @@ rxq_alloc_elts_mprq(struct mlx5_rxq_ctrl *rxq_ctrl) * Pointer to RX queue structure. * * @return - * 0 on success, errno value on failure. + * 0 on success, negative errno value on failure. */ static int rxq_alloc_elts_sprq(struct mlx5_rxq_ctrl *rxq_ctrl) @@ -220,7 +220,7 @@ rxq_alloc_elts_sprq(struct mlx5_rxq_ctrl *rxq_ctrl) * Pointer to RX queue structure. * * @return - * 0 on success, errno value on failure. + * 0 on success, negative errno value on failure. */ int rxq_alloc_elts(struct mlx5_rxq_ctrl *rxq_ctrl) @@ -233,7 +233,9 @@ rxq_alloc_elts(struct mlx5_rxq_ctrl *rxq_ctrl) */ if (mlx5_rxq_mprq_enabled(&rxq_ctrl->rxq)) ret = rxq_alloc_elts_mprq(rxq_ctrl); - return (ret || rxq_alloc_elts_sprq(rxq_ctrl)); + if (ret == 0) + ret = rxq_alloc_elts_sprq(rxq_ctrl); + return ret; } /** -- 2.33.0 ^ permalink raw reply [flat|nested] 6+ messages in thread
[parent not found: <20211104123320.1638915-1-xuemingl@nvidia.com>]
* [dpdk-stable] [PATCH v4 02/14] net/mlx5: fix field reference for PPC [not found] ` <20211104123320.1638915-1-xuemingl@nvidia.com> @ 2021-11-04 12:33 ` Xueming Li 2021-11-04 17:07 ` [dpdk-stable] [dpdk-dev] " Raslan Darawsheh 2021-11-04 17:49 ` [dpdk-stable] " David Christensen 2021-11-04 12:33 ` [dpdk-stable] [PATCH v4 05/14] net/mlx5: fix Rx queue memory allocation return value Xueming Li 1 sibling, 2 replies; 6+ messages in thread From: Xueming Li @ 2021-11-04 12:33 UTC (permalink / raw) To: dev Cc: xuemingl, Lior Margalit, viacheslavo, stable, David Christensen, Matan Azrad, Yongseok Koh This patch fixes stale field reference. Fixes: a18ac6113331 ("net/mlx5: add metadata support to Rx datapath") Cc: viacheslavo@nvidia.com Cc: stable@dpdk.org Signed-off-by: Xueming Li <xuemingl@nvidia.com> Acked-by: Slava Ovsiienko <viacheslavo@nvidia.com> --- drivers/net/mlx5/mlx5_rxtx_vec_altivec.h | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/net/mlx5/mlx5_rxtx_vec_altivec.h b/drivers/net/mlx5/mlx5_rxtx_vec_altivec.h index bcf487c34e9..1d00c1c43d1 100644 --- a/drivers/net/mlx5/mlx5_rxtx_vec_altivec.h +++ b/drivers/net/mlx5/mlx5_rxtx_vec_altivec.h @@ -974,10 +974,10 @@ rxq_cq_process_v(struct mlx5_rxq_data *rxq, volatile struct mlx5_cqe *cq, (vector unsigned short)cqe_tmp1, cqe_sel_mask1); cqe_tmp2 = (vector unsigned char)(vector unsigned long){ *(__rte_aligned(8) unsigned long *) - &cq[pos + p3].rsvd3[9], 0LL}; + &cq[pos + p3].rsvd4[2], 0LL}; cqe_tmp1 = (vector unsigned char)(vector unsigned long){ *(__rte_aligned(8) unsigned long *) - &cq[pos + p2].rsvd3[9], 0LL}; + &cq[pos + p2].rsvd4[2], 0LL}; cqes[3] = (vector unsigned char) vec_sel((vector unsigned short)cqes[3], (vector unsigned short)cqe_tmp2, @@ -1037,10 +1037,10 @@ rxq_cq_process_v(struct mlx5_rxq_data *rxq, volatile struct mlx5_cqe *cq, (vector unsigned short)cqe_tmp1, cqe_sel_mask1); cqe_tmp2 = (vector unsigned char)(vector unsigned long){ *(__rte_aligned(8) unsigned long *) - &cq[pos + p1].rsvd3[9], 0LL}; + &cq[pos + p1].rsvd4[2], 0LL}; cqe_tmp1 = (vector unsigned char)(vector unsigned long){ *(__rte_aligned(8) unsigned long *) - &cq[pos].rsvd3[9], 0LL}; + &cq[pos].rsvd4[2], 0LL}; cqes[1] = (vector unsigned char) vec_sel((vector unsigned short)cqes[1], (vector unsigned short)cqe_tmp2, cqe_sel_mask2); -- 2.33.0 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-stable] [dpdk-dev] [PATCH v4 02/14] net/mlx5: fix field reference for PPC 2021-11-04 12:33 ` [dpdk-stable] [PATCH v4 02/14] net/mlx5: fix field reference for PPC Xueming Li @ 2021-11-04 17:07 ` Raslan Darawsheh 2021-11-04 17:49 ` [dpdk-stable] " David Christensen 1 sibling, 0 replies; 6+ messages in thread From: Raslan Darawsheh @ 2021-11-04 17:07 UTC (permalink / raw) To: Xueming(Steven) Li, dev Cc: Xueming(Steven) Li, Lior Margalit, Slava Ovsiienko, stable, David Christensen, Matan Azrad, Yongseok Koh Hi, > -----Original Message----- > From: dev <dev-bounces@dpdk.org> On Behalf Of Xueming Li > Sent: Thursday, November 4, 2021 2:33 PM > To: dev@dpdk.org > Cc: Xueming(Steven) Li <xuemingl@nvidia.com>; Lior Margalit > <lmargalit@nvidia.com>; Slava Ovsiienko <viacheslavo@nvidia.com>; > stable@dpdk.org; David Christensen <drc@linux.vnet.ibm.com>; Matan > Azrad <matan@nvidia.com>; Yongseok Koh <yskoh@mellanox.com> > Subject: [dpdk-dev] [PATCH v4 02/14] net/mlx5: fix field reference for PPC > > This patch fixes stale field reference. > > Fixes: a18ac6113331 ("net/mlx5: add metadata support to Rx datapath") > Cc: viacheslavo@nvidia.com > Cc: stable@dpdk.org > This should be the first patch in the series since the first patch is removing the rsvd3 from the structure. I'll change the order during integration, Kindest regards, Raslan Darawsheh ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [dpdk-stable] [PATCH v4 02/14] net/mlx5: fix field reference for PPC 2021-11-04 12:33 ` [dpdk-stable] [PATCH v4 02/14] net/mlx5: fix field reference for PPC Xueming Li 2021-11-04 17:07 ` [dpdk-stable] [dpdk-dev] " Raslan Darawsheh @ 2021-11-04 17:49 ` David Christensen 1 sibling, 0 replies; 6+ messages in thread From: David Christensen @ 2021-11-04 17:49 UTC (permalink / raw) To: Xueming Li, dev Cc: Lior Margalit, viacheslavo, stable, Matan Azrad, Yongseok Koh On 11/4/21 5:33 AM, Xueming Li wrote: > This patch fixes stale field reference. > > Fixes: a18ac6113331 ("net/mlx5: add metadata support to Rx datapath") > Cc: viacheslavo@nvidia.com > Cc: stable@dpdk.org > > Signed-off-by: Xueming Li <xuemingl@nvidia.com> > Acked-by: Slava Ovsiienko <viacheslavo@nvidia.com> > --- > drivers/net/mlx5/mlx5_rxtx_vec_altivec.h | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/net/mlx5/mlx5_rxtx_vec_altivec.h b/drivers/net/mlx5/mlx5_rxtx_vec_altivec.h > index bcf487c34e9..1d00c1c43d1 100644 > --- a/drivers/net/mlx5/mlx5_rxtx_vec_altivec.h > +++ b/drivers/net/mlx5/mlx5_rxtx_vec_altivec.h > @@ -974,10 +974,10 @@ rxq_cq_process_v(struct mlx5_rxq_data *rxq, volatile struct mlx5_cqe *cq, > (vector unsigned short)cqe_tmp1, cqe_sel_mask1); > cqe_tmp2 = (vector unsigned char)(vector unsigned long){ > *(__rte_aligned(8) unsigned long *) > - &cq[pos + p3].rsvd3[9], 0LL}; > + &cq[pos + p3].rsvd4[2], 0LL}; > cqe_tmp1 = (vector unsigned char)(vector unsigned long){ > *(__rte_aligned(8) unsigned long *) > - &cq[pos + p2].rsvd3[9], 0LL}; > + &cq[pos + p2].rsvd4[2], 0LL}; > cqes[3] = (vector unsigned char) > vec_sel((vector unsigned short)cqes[3], > (vector unsigned short)cqe_tmp2, > @@ -1037,10 +1037,10 @@ rxq_cq_process_v(struct mlx5_rxq_data *rxq, volatile struct mlx5_cqe *cq, > (vector unsigned short)cqe_tmp1, cqe_sel_mask1); > cqe_tmp2 = (vector unsigned char)(vector unsigned long){ > *(__rte_aligned(8) unsigned long *) > - &cq[pos + p1].rsvd3[9], 0LL}; > + &cq[pos + p1].rsvd4[2], 0LL}; > cqe_tmp1 = (vector unsigned char)(vector unsigned long){ > *(__rte_aligned(8) unsigned long *) > - &cq[pos].rsvd3[9], 0LL}; > + &cq[pos].rsvd4[2], 0LL}; > cqes[1] = (vector unsigned char) > vec_sel((vector unsigned short)cqes[1], > (vector unsigned short)cqe_tmp2, cqe_sel_mask2); > Reviewed-by: David Christensen <drc@linux.vnet.ibm.com> ^ permalink raw reply [flat|nested] 6+ messages in thread
* [dpdk-stable] [PATCH v4 05/14] net/mlx5: fix Rx queue memory allocation return value [not found] ` <20211104123320.1638915-1-xuemingl@nvidia.com> 2021-11-04 12:33 ` [dpdk-stable] [PATCH v4 02/14] net/mlx5: fix field reference for PPC Xueming Li @ 2021-11-04 12:33 ` Xueming Li 1 sibling, 0 replies; 6+ messages in thread From: Xueming Li @ 2021-11-04 12:33 UTC (permalink / raw) To: dev Cc: xuemingl, Lior Margalit, akozyrev, stable, Slava Ovsiienko, Matan Azrad If error happened during Rx queue mbuf allocation, boolean value returned. From description, return value should be error number. This patch returns negative error number. Fixes: 0f20acbf5eda ("net/mlx5: implement vectorized MPRQ burst") Cc: akozyrev@nvidia.com Cc: stable@dpdk.org Signed-off-by: Xueming Li <xuemingl@nvidia.com> Acked-by: Slava Ovsiienko <viacheslavo@nvidia.com> --- drivers/net/mlx5/mlx5_rxq.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers/net/mlx5/mlx5_rxq.c b/drivers/net/mlx5/mlx5_rxq.c index 9220bb2c15c..4567b43c1b6 100644 --- a/drivers/net/mlx5/mlx5_rxq.c +++ b/drivers/net/mlx5/mlx5_rxq.c @@ -129,7 +129,7 @@ rxq_alloc_elts_mprq(struct mlx5_rxq_ctrl *rxq_ctrl) * Pointer to RX queue structure. * * @return - * 0 on success, errno value on failure. + * 0 on success, negative errno value on failure. */ static int rxq_alloc_elts_sprq(struct mlx5_rxq_ctrl *rxq_ctrl) @@ -220,7 +220,7 @@ rxq_alloc_elts_sprq(struct mlx5_rxq_ctrl *rxq_ctrl) * Pointer to RX queue structure. * * @return - * 0 on success, errno value on failure. + * 0 on success, negative errno value on failure. */ int rxq_alloc_elts(struct mlx5_rxq_ctrl *rxq_ctrl) @@ -233,7 +233,9 @@ rxq_alloc_elts(struct mlx5_rxq_ctrl *rxq_ctrl) */ if (mlx5_rxq_mprq_enabled(&rxq_ctrl->rxq)) ret = rxq_alloc_elts_mprq(rxq_ctrl); - return (ret || rxq_alloc_elts_sprq(rxq_ctrl)); + if (ret == 0) + ret = rxq_alloc_elts_sprq(rxq_ctrl); + return ret; } /** -- 2.33.0 ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-11-04 17:49 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <20210727034204.20649-1-xuemingl@nvidia.com> [not found] ` <20211103075838.1486056-1-xuemingl@nvidia.com> 2021-11-03 7:58 ` [dpdk-stable] [PATCH v3 02/14] net/mlx5: fix field reference for PPC Xueming Li 2021-11-03 7:58 ` [dpdk-stable] [PATCH v3 05/14] net/mlx5: fix Rx queue memory allocation return value Xueming Li [not found] ` <20211104123320.1638915-1-xuemingl@nvidia.com> 2021-11-04 12:33 ` [dpdk-stable] [PATCH v4 02/14] net/mlx5: fix field reference for PPC Xueming Li 2021-11-04 17:07 ` [dpdk-stable] [dpdk-dev] " Raslan Darawsheh 2021-11-04 17:49 ` [dpdk-stable] " David Christensen 2021-11-04 12:33 ` [dpdk-stable] [PATCH v4 05/14] net/mlx5: fix Rx queue memory allocation return value Xueming Li
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).