> -----Original Message-----
> From: Wu, Jingjing
> Sent: Thursday, March 30, 2017 2:17 PM
> To: Xing, Beilei <beilei.x...@intel.com>
> Cc: Zhang, Helin <helin.zh...@intel.com>; Iremonger, Bernard
> <bernard.iremon...@intel.com>; dev@dpdk.org
> Subject: RE: [PATCH v4 4/4] net/i40e: enable tunnel filter for MPLS
> 
> >  int
> >  i40e_dev_consistent_tunnel_filter_set(struct i40e_pf *pf,
> >                   struct i40e_tunnel_filter_conf *tunnel_filter, @@ -6954,6
> > +7063,8 @@ i40e_dev_consistent_tunnel_filter_set(struct i40e_pf *pf,
> >     struct i40e_tunnel_rule *tunnel_rule = &pf->tunnel;
> >     struct i40e_tunnel_filter *tunnel, *node;
> >     struct i40e_tunnel_filter check_filter; /* Check if filter exists */
> > +   uint32_t teid_le;
> > +   bool big_buffer = 0;
> >
> >     cld_filter = rte_zmalloc("tunnel_filter",
> >                      sizeof(struct
> i40e_aqc_add_rm_cloud_filt_elem_ext),
> > @@ -7001,6 +7112,32 @@ i40e_dev_consistent_tunnel_filter_set(struct
> > i40e_pf *pf,
> >     case I40E_TUNNEL_TYPE_IP_IN_GRE:
> >             tun_type = I40E_AQC_ADD_CLOUD_TNL_TYPE_IP;
> >             break;
> > +   case I40E_TUNNEL_TYPE_MPLSoUDP:
> > +           i40e_replace_mpls_l1_filter(pf);
> > +           i40e_replace_mpls_cloud_filter(pf);
> Do we need replace every time when add each rule?

No, I will add a flag to distinguish it.

> 
> Thanks
> Jingjing

Reply via email to