Hi Akhil,

Looks like you commented out ixgbe/txgbe from build in this patch,
but you didn't add them back in in the next patch?

Regards,
Fan

> -----Original Message-----
> From: Akhil Goyal <[email protected]>
> Sent: Thursday, September 30, 2021 3:50 PM
> To: [email protected]
> Cc: [email protected]; [email protected];
> [email protected]; [email protected]; De Lara Guarch, Pablo
> <[email protected]>; Trahe, Fiona <[email protected]>;
> Doherty, Declan <[email protected]>; [email protected];
> [email protected]; Zhang, Roy Fan <[email protected]>;
> [email protected]; [email protected]; [email protected];
> Ananyev, Konstantin <[email protected]>; Nicolau, Radu
> <[email protected]>; [email protected];
> [email protected]; [email protected]; Power, Ciara
> <[email protected]>; Akhil Goyal <[email protected]>; Jian Wang
> <[email protected]>; Jiawen Wu <[email protected]>; Wang,
> Haiyue <[email protected]>
> Subject: [PATCH 2/3] drivers/net: temporary disable ixgbe and txgbe
> 
> The PMDs ixgbe and txgbe are currently making
> rte_security_session private data as constant,
> but they are getting filled inside the API.
> This was feasible as the session_private_data
> and rte_security_session pointers were separate.
> But now these two will be from same memory chunk.
> Hence it cannot use them as const.
> Need help from PMD owners to fix this.
> Cc: Jian Wang <[email protected]>
> Cc: Jiawen Wu <[email protected]>
> Cc: Haiyue Wang <[email protected]>
> Cc: Fan Zhang <[email protected]>
> 
> Signed-off-by: Akhil Goyal <[email protected]>
> ---
>  drivers/net/meson.build | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/net/meson.build b/drivers/net/meson.build
> index bcf488f203..4c0cd50cd5 100644
> --- a/drivers/net/meson.build
> +++ b/drivers/net/meson.build
> @@ -30,7 +30,7 @@ drivers = [
>          'igc',
>          'ionic',
>          'ipn3ke',
> -        'ixgbe',
> +     #        'ixgbe',
>          'kni',
>          'liquidio',
>          'memif',
> @@ -55,7 +55,7 @@ drivers = [
>          'szedata2',
>          'tap',
>          'thunderx',
> -        'txgbe',
> +#        'txgbe',
>          'vdev_netvsc',
>          'vhost',
>          'virtio',
> --
> 2.25.1

Reply via email to