diff mbox series

net/dpaa2: fix RSS distribution size field

Message ID 20210408091744.21570-1-hemant.agrawal@nxp.com (mailing list archive)
State Accepted
Delegated to: Ferruh Yigit
Headers show
Series net/dpaa2: fix RSS distribution size field | expand

Checks

Context Check Description
ci/iol-testing success Testing PASS
ci/iol-abi-testing success Testing PASS
ci/iol-mellanox-Performance success Performance Testing PASS
ci/intel-Testing success Testing PASS
ci/Intel-compilation success Compilation OK
ci/iol-intel-Performance success Performance Testing PASS
ci/travis-robot success travis build: passed
ci/checkpatch success coding style OK

Commit Message

Hemant Agrawal April 8, 2021, 9:17 a.m. UTC
RSS distribution field is U64, while the DPAA2 code
was using integer value, which is causing a issue
for MPLS having bit position 33.
This patch fixes the code to use unsigned long long.

Fixes: 1832bc8ee8d4 ("net/dpaa2: support MPLS distribution")
Cc: apeksha.gupta@nxp.com

Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com>
---
 drivers/net/dpaa2/base/dpaa2_hw_dpni.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Ferruh Yigit April 9, 2021, 9:09 a.m. UTC | #1
On 4/8/2021 10:17 AM, Hemant Agrawal wrote:
> RSS distribution field is U64, while the DPAA2 code
> was using integer value, which is causing a issue
> for MPLS having bit position 33.
> This patch fixes the code to use unsigned long long.
> 
> Fixes: 1832bc8ee8d4 ("net/dpaa2: support MPLS distribution")
> Cc: apeksha.gupta@nxp.com
> 
> Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com>

     Cc: stable@dpdk.org

Applied to dpdk-next-net/main, thanks.
diff mbox series

Patch

diff --git a/drivers/net/dpaa2/base/dpaa2_hw_dpni.c b/drivers/net/dpaa2/base/dpaa2_hw_dpni.c
index 9d1565369d..b901b4342f 100644
--- a/drivers/net/dpaa2/base/dpaa2_hw_dpni.c
+++ b/drivers/net/dpaa2/base/dpaa2_hw_dpni.c
@@ -203,7 +203,7 @@  dpaa2_distset_to_dpkg_profile_cfg(
 	memset(kg_cfg, 0, sizeof(struct dpkg_profile_cfg));
 	while (req_dist_set) {
 		if (req_dist_set % 2 != 0) {
-			dist_field = 1U << loop;
+			dist_field = 1ULL << loop;
 			switch (dist_field) {
 			case ETH_RSS_L2_PAYLOAD: