Re: [dpdk-stable] patch 'examples/flow_filtering: add flow director config for i40e' has been queued to stable release 18.05.1

["Xu, Rosen" <rosen.xu@intel.com>]

Got, thanks Christian.

> -----Original Message-----
> From: Christian Ehrhardt [mailto:christian.ehrhardt@canonical.com]
> Sent: Tuesday, August 14, 2018 19:07
> To: Xu, Rosen <rosen.xu@intel.com>
> Cc: Ori Kam <orika@mellanox.com>; Yigit, Ferruh <ferruh.yigit@intel.com>;
> dpdk stable <stable@dpdk.org>
> Subject: patch 'examples/flow_filtering: add flow director config for i40e' has
> been queued to stable release 18.05.1
> 
> Hi,
> 
> FYI, your patch has been queued to stable release 18.05.1
> 
> Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
> It will be pushed if I get no objections before 08/16/18. So please shout if
> anyone has objections.
> 
> Thanks.
> 
> Christian Ehrhardt <christian.ehrhardt@canonical.com>
> 
> ---
> From 0fd2f90f51b1c8a4cc2370c71bc04e78d9df0cef Mon Sep 17 00:00:00
> 2001
> From: Rosen Xu <rosen.xu@intel.com>
> Date: Tue, 31 Jul 2018 20:52:40 +0800
> Subject: [PATCH] examples/flow_filtering: add flow director config for i40e
> 
> [ upstream commit 9a93446a0e8c7e95504ff0bfd26d139a8c1ef320 ]
> 
> Rte_fdir_conf of rte_eth_conf should be initialized before port initialization.
> It is a workaround solution when working with Intel I40e.
> 
> Fixes: 4a3ef59a10c8 ("examples/flow_filtering: add simple demo of flow API")
> 
> Signed-off-by: Rosen Xu <rosen.xu@intel.com>
> Acked-by: Ori Kam <orika@mellanox.com>
> Reviewed-by: Ferruh Yigit <ferruh.yigit@intel.com>
> ---
>  examples/flow_filtering/main.c | 16 ++++++++++++++++
>  1 file changed, 16 insertions(+)
> 
> diff --git a/examples/flow_filtering/main.c b/examples/flow_filtering/main.c
> index e0ee51679..9980a3ad5 100644
> --- a/examples/flow_filtering/main.c
> +++ b/examples/flow_filtering/main.c
> @@ -133,6 +133,22 @@ init_port(void)
>  				DEV_TX_OFFLOAD_SCTP_CKSUM  |
>  				DEV_TX_OFFLOAD_TCP_TSO,
>  		},
> +		/*
> +		 * Initialize fdir_conf of rte_eth_conf.
> +		 * Fdir is used in flow filtering for I40e,
> +		 * so rte_flow rules involve some fdir
> +		 * configurations. In long term it's better
> +		 * that drivers don't require any fdir
> +		 * configuration for rte_flow, but we need to
> +		 * get this workaround so that sample app can
> +		 * run on I40e.
> +		 */
> +		.fdir_conf = {
> +			.mode = RTE_FDIR_MODE_PERFECT,
> +			.pballoc = RTE_FDIR_PBALLOC_64K,
> +			.status = RTE_FDIR_REPORT_STATUS,
> +			.drop_queue = 127,
> +		},
>  	};
>  	struct rte_eth_txconf txq_conf;
>  	struct rte_eth_rxconf rxq_conf;
> --
> 2.17.1