[Intel-wired-lan] [PATCH net] ice: fix adding different tunnels
Michal Swiatkowski
michal.swiatkowski at linux.intel.com
Mon Nov 22 11:23:36 UTC 2021
On Wed, Nov 17, 2021 at 11:59:19PM +0000, Nguyen, Anthony L wrote:
> On Tue, 2021-11-16 at 11:24 +0100, Michal Swiatkowski wrote:
> > Adding filters with the same values inside for VXLAN and geneve
> > causes hw
> > error, because it looks exactly the same. To choose between different
> > type of tunnels new recipe is needed. Add storing tunnel types in
> > creatig recipes function and start checking it in finding function.
> >
> > Change getting open tunnels function to return port on correct tunnel
> > type. This is needed to copy correct port to dummy packet.
> >
> > Block user from adding enc_dst_port via tc flower, because VXLAN and
> > geneve filters can be created only with destination port which was
> > previously opened.
> >
> > Fixes: 8b032a55c1bd5 ("ice: low level support for tunnels")
> > Signed-off-by: Michal Swiatkowski
> > <michal.swiatkowski at linux.intel.com>
> > ---
> > .../net/ethernet/intel/ice/ice_ethtool_fdir.c | 4 ++--
> > drivers/net/ethernet/intel/ice/ice_fdir.c | 2 +-
> > drivers/net/ethernet/intel/ice/ice_flex_pipe.c | 6 ++++--
> > drivers/net/ethernet/intel/ice/ice_flex_pipe.h | 3 ++-
> > drivers/net/ethernet/intel/ice/ice_switch.c | 17 ++++++++++++---
> > --
> > drivers/net/ethernet/intel/ice/ice_tc_lib.c | 3 ++-
> > 6 files changed, 23 insertions(+), 12 deletions(-)
>
> The added function parameters need to be documented:
>
> drivers/net/ethernet/intel/ice/ice_switch.c:3801: warning: Function
> parameter or member 'tun_type' not described in 'ice_find_recp'
> drivers/net/ethernet/intel/ice/ice_flex_pipe.c:2047: warning: Function
> parameter or member 'type' not described in 'ice_get_open_tunnel_port'
>
Sorry for late answer, I missed it. Thanks, will send corrected v2 soon.
More information about the Intel-wired-lan
mailing list