[dpdk-dev] i40e/ppc64/altivec: Fix compile errors on POWER8

Message ID 20170421031010.2389-1-aik@ozlabs.ru (mailing list archive)
State Not Applicable, archived
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Alexey Kardashevskiy April 21, 2017, 3:10 a.m. UTC
  Fixes: 67f038076657 ("net/i40e: enable per-device packet type mapping")
Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
---
 drivers/net/i40e/i40e_rxtx_vec_altivec.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
  

Comments

Alexey Kardashevskiy April 21, 2017, 3:12 a.m. UTC | #1
On 21/04/17 13:10, Alexey Kardashevskiy wrote:
> Fixes: 67f038076657 ("net/i40e: enable per-device packet type mapping")
> Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>


ah, never mind, fixed already.

> ---
>  drivers/net/i40e/i40e_rxtx_vec_altivec.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/net/i40e/i40e_rxtx_vec_altivec.c b/drivers/net/i40e/i40e_rxtx_vec_altivec.c
> index 07de31b01..f4036ea28 100644
> --- a/drivers/net/i40e/i40e_rxtx_vec_altivec.c
> +++ b/drivers/net/i40e/i40e_rxtx_vec_altivec.c
> @@ -208,13 +208,13 @@ desc_to_ptype_v(vector unsigned long descs[4], struct rte_mbuf **rx_pkts,
>  	ptype1 = vec_sr(ptype1, (vector unsigned long){30, 30});
>  
>  	rx_pkts[0]->packet_type =
> -		ptype_tbl[(*(vector unsigned char *)&ptype0)[0])];
> +		ptype_tbl[(*(vector unsigned char *)&ptype0)[0]];
>  	rx_pkts[1]->packet_type =
> -		ptype_tbl[(*(vector unsigned char *)&ptype0)[8])];
> +		ptype_tbl[(*(vector unsigned char *)&ptype0)[8]];
>  	rx_pkts[2]->packet_type =
> -		ptype_tbl[(*(vector unsigned char *)&ptype1)[0])];
> +		ptype_tbl[(*(vector unsigned char *)&ptype1)[0]];
>  	rx_pkts[3]->packet_type =
> -		ptype_tbl[(*(vector unsigned char *)&ptype1)[8])];
> +		ptype_tbl[(*(vector unsigned char *)&ptype1)[8]];
>  }
>  
>   /* Notice:
>
  
Qi Zhang April 21, 2017, 3:23 a.m. UTC | #2
> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Alexey
> Kardashevskiy
> Sent: Friday, April 21, 2017 11:10 AM
> To: dev@dpdk.org
> Cc: Alexey Kardashevskiy <aik@ozlabs.ru>
> Subject: [dpdk-dev] [PATCH dpdk] i40e/ppc64/altivec: Fix compile errors on
> POWER8
> 
> Fixes: 67f038076657 ("net/i40e: enable per-device packet type mapping")
> Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
> ---
>  drivers/net/i40e/i40e_rxtx_vec_altivec.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/net/i40e/i40e_rxtx_vec_altivec.c
> b/drivers/net/i40e/i40e_rxtx_vec_altivec.c
> index 07de31b01..f4036ea28 100644
> --- a/drivers/net/i40e/i40e_rxtx_vec_altivec.c
> +++ b/drivers/net/i40e/i40e_rxtx_vec_altivec.c
> @@ -208,13 +208,13 @@ desc_to_ptype_v(vector unsigned long descs[4],
> struct rte_mbuf **rx_pkts,
>  	ptype1 = vec_sr(ptype1, (vector unsigned long){30, 30});
> 
>  	rx_pkts[0]->packet_type =
> -		ptype_tbl[(*(vector unsigned char *)&ptype0)[0])];
> +		ptype_tbl[(*(vector unsigned char *)&ptype0)[0]];
>  	rx_pkts[1]->packet_type =
> -		ptype_tbl[(*(vector unsigned char *)&ptype0)[8])];
> +		ptype_tbl[(*(vector unsigned char *)&ptype0)[8]];
>  	rx_pkts[2]->packet_type =
> -		ptype_tbl[(*(vector unsigned char *)&ptype1)[0])];
> +		ptype_tbl[(*(vector unsigned char *)&ptype1)[0]];
>  	rx_pkts[3]->packet_type =
> -		ptype_tbl[(*(vector unsigned char *)&ptype1)[8])];
> +		ptype_tbl[(*(vector unsigned char *)&ptype1)[8]];
>  }
> 
>   /* Notice:
> --
> 2.11.0

Acked-by: Qi Zhang <qi.z.zhang@intel.com>
  

Patch

diff --git a/drivers/net/i40e/i40e_rxtx_vec_altivec.c b/drivers/net/i40e/i40e_rxtx_vec_altivec.c
index 07de31b01..f4036ea28 100644
--- a/drivers/net/i40e/i40e_rxtx_vec_altivec.c
+++ b/drivers/net/i40e/i40e_rxtx_vec_altivec.c
@@ -208,13 +208,13 @@  desc_to_ptype_v(vector unsigned long descs[4], struct rte_mbuf **rx_pkts,
 	ptype1 = vec_sr(ptype1, (vector unsigned long){30, 30});
 
 	rx_pkts[0]->packet_type =
-		ptype_tbl[(*(vector unsigned char *)&ptype0)[0])];
+		ptype_tbl[(*(vector unsigned char *)&ptype0)[0]];
 	rx_pkts[1]->packet_type =
-		ptype_tbl[(*(vector unsigned char *)&ptype0)[8])];
+		ptype_tbl[(*(vector unsigned char *)&ptype0)[8]];
 	rx_pkts[2]->packet_type =
-		ptype_tbl[(*(vector unsigned char *)&ptype1)[0])];
+		ptype_tbl[(*(vector unsigned char *)&ptype1)[0]];
 	rx_pkts[3]->packet_type =
-		ptype_tbl[(*(vector unsigned char *)&ptype1)[8])];
+		ptype_tbl[(*(vector unsigned char *)&ptype1)[8]];
 }
 
  /* Notice: