net/iavf: fix build error

Message ID 20211027095116.2184749-1-radu.nicolau@intel.com (mailing list archive)
State Not Applicable, archived
Delegated to: Qi Zhang
Headers
Series net/iavf: fix build error |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/iol-testing warning apply patch failure
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS

Commit Message

Radu Nicolau Oct. 27, 2021, 9:51 a.m. UTC
  Fix Freebsd13 and Win10 build error.

Fixes: 9e84ce6ecd27 ("net/iavf: add iAVF IPsec inline crypto support")

Signed-off-by: Radu Nicolau <radu.nicolau@intel.com>
---
 drivers/net/iavf/iavf_ipsec_crypto.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Qi Zhang Oct. 28, 2021, 11:22 a.m. UTC | #1
> -----Original Message-----
> From: Nicolau, Radu <radu.nicolau@intel.com>
> Sent: Wednesday, October 27, 2021 5:51 PM
> To: Wu, Jingjing <jingjing.wu@intel.com>; Xing, Beilei <beilei.xing@intel.com>
> Cc: dev@dpdk.org; Zhang, Qi Z <qi.z.zhang@intel.com>; Nicolau, Radu
> <radu.nicolau@intel.com>
> Subject: [PATCH] net/iavf: fix build error
> 
> Fix Freebsd13 and Win10 build error.
> 
> Fixes: 9e84ce6ecd27 ("net/iavf: add iAVF IPsec inline crypto support")
> 
> Signed-off-by: Radu Nicolau <radu.nicolau@intel.com>
> ---
>  drivers/net/iavf/iavf_ipsec_crypto.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/iavf/iavf_ipsec_crypto.c
> b/drivers/net/iavf/iavf_ipsec_crypto.c
> index 4d4830f4c9..a8022f8ed7 100644
> --- a/drivers/net/iavf/iavf_ipsec_crypto.c
> +++ b/drivers/net/iavf/iavf_ipsec_crypto.c
> @@ -509,7 +509,7 @@ iavf_ipsec_crypto_security_association_add(struct
> iavf_adapter *adapter,
>  			htonl(conf->ipsec.tunnel.ipv4.dst_ip.s_addr);
>  	} else {
>  		uint32_t *v6_dst_addr =
> -			conf->ipsec.tunnel.ipv6.dst_addr.s6_addr32;
> +			(uint32_t *)conf->ipsec.tunnel.ipv6.dst_addr.s6_addr;
> 
>  		sa_cfg->virtchnl_ip_type = VIRTCHNL_IPV6;
> 
> --
> 2.25.1

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

Applied to dpdk-next-net-intel.

Thanks
Qi
  
Ferruh Yigit Oct. 28, 2021, 2:50 p.m. UTC | #2
On 10/27/2021 10:51 AM, Radu Nicolau wrote:
> Fix Freebsd13 and Win10 build error.
> 
> Fixes: 9e84ce6ecd27 ("net/iavf: add iAVF IPsec inline crypto support")
> 
> Signed-off-by: Radu Nicolau <radu.nicolau@intel.com>

Can you please squash this fix to the original set, in its new version?

> ---
>   drivers/net/iavf/iavf_ipsec_crypto.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/iavf/iavf_ipsec_crypto.c b/drivers/net/iavf/iavf_ipsec_crypto.c
> index 4d4830f4c9..a8022f8ed7 100644
> --- a/drivers/net/iavf/iavf_ipsec_crypto.c
> +++ b/drivers/net/iavf/iavf_ipsec_crypto.c
> @@ -509,7 +509,7 @@ iavf_ipsec_crypto_security_association_add(struct iavf_adapter *adapter,
>   			htonl(conf->ipsec.tunnel.ipv4.dst_ip.s_addr);
>   	} else {
>   		uint32_t *v6_dst_addr =
> -			conf->ipsec.tunnel.ipv6.dst_addr.s6_addr32;
> +			(uint32_t *)conf->ipsec.tunnel.ipv6.dst_addr.s6_addr;
>   
>   		sa_cfg->virtchnl_ip_type = VIRTCHNL_IPV6;
>   
>
  

Patch

diff --git a/drivers/net/iavf/iavf_ipsec_crypto.c b/drivers/net/iavf/iavf_ipsec_crypto.c
index 4d4830f4c9..a8022f8ed7 100644
--- a/drivers/net/iavf/iavf_ipsec_crypto.c
+++ b/drivers/net/iavf/iavf_ipsec_crypto.c
@@ -509,7 +509,7 @@  iavf_ipsec_crypto_security_association_add(struct iavf_adapter *adapter,
 			htonl(conf->ipsec.tunnel.ipv4.dst_ip.s_addr);
 	} else {
 		uint32_t *v6_dst_addr =
-			conf->ipsec.tunnel.ipv6.dst_addr.s6_addr32;
+			(uint32_t *)conf->ipsec.tunnel.ipv6.dst_addr.s6_addr;
 
 		sa_cfg->virtchnl_ip_type = VIRTCHNL_IPV6;