Hi，yigit > -----Original Message----- > From: Yigit, Ferruh > Sent: Saturday, January 7, 2017 12:40 AM > To: Zhao1, Wei <wei.zh...@intel.com>; dev@dpdk.org > Cc: Lu, Wenzhuo <wenzhuo...@intel.com> > Subject: Re: [dpdk-dev] [PATCH v2 10/18] net/ixgbe: flush all the filters > > On 12/30/2016 7:53 AM, Wei Zhao wrote: > > Add support for flush all the filters in SW. > > > > Signed-off-by: Wenzhuo Lu <wenzhuo...@intel.com> > > Signed-off-by: Wei Zhao <wei.zh...@intel.com> > > > > --- > > > > v2: > > --change flush filter function call relationship > > --- > > drivers/net/ixgbe/ixgbe_ethdev.c | 118 > ++++++++++++++++++++++++++++++++++++++- > > drivers/net/ixgbe/ixgbe_ethdev.h | 9 +++ > > drivers/net/ixgbe/ixgbe_fdir.c | 24 ++++++++ > > drivers/net/ixgbe/ixgbe_pf.c | 1 + > > 4 files changed, 150 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/net/ixgbe/ixgbe_ethdev.c > > b/drivers/net/ixgbe/ixgbe_ethdev.c > > index d68de65..0de1318 100644 > > --- a/drivers/net/ixgbe/ixgbe_ethdev.c > > +++ b/drivers/net/ixgbe/ixgbe_ethdev.c > > @@ -61,6 +61,8 @@ > > #include <rte_random.h> > > #include <rte_dev.h> > > #include <rte_hash_crc.h> > > +#include <rte_flow.h> > > +#include <rte_flow_driver.h> > > > > #include "ixgbe_logs.h" > > #include "base/ixgbe_api.h" > > @@ -386,7 +388,8 @@ static int ixgbe_dev_udp_tunnel_port_del(struct > rte_eth_dev *dev, > > struct rte_eth_udp_tunnel > *udp_tunnel); static int > > ixgbe_filter_restore(struct rte_eth_dev *dev); static void > > ixgbe_l2_tunnel_conf(struct rte_eth_dev *dev); > > - > > +static int ixgbe_flow_flush(struct rte_eth_dev *dev, > > + struct rte_flow_error *error); > > I think it is good idea to move all rte_flow related code into a new file, as > done in i40e (i40e_flow.c)? What do you think, can it be done?
I will do as your suggestion in v3. > > <...>
