lkml.org 
[lkml]   [2023]   [Dec]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [EXT] [PATCH iwl-next v6] ice: Add support for packet mirroring using hardware in switchdev mode
Date
> 
>
> -----Original Message-----
> From: Suman Ghosh <sumang@marvell.com>
> Sent: Tuesday, December 12, 2023 3:31 PM
> To: Staikov, Andrii <andrii.staikov@intel.com>
> Cc: netdev@vger.kernel.org; linux-kernel@vger.kernel.org; Szycik, Marcin <marcin.szycik@intel.com>; Drewek, Wojciech <wojciech.drewek@intel.com>
> Subject: RE: [EXT] [PATCH iwl-next v6] ice: Add support for packet mirroring using hardware in switchdev mode
>
> > break;
> >
> >+ case FLOW_ACTION_MIRRED:
> [Suman] Hi Andrei,
> I guess we have two flow_action_ids FLOW_ACTION_MIRRED (for egress) and FLOW_ACTION_MIRRED_INGRESS (for ingress). The sample command used in above is of type ingress
> and that is not handled here. My guess is, ingress command should show "not supported" error from tc right? Please let me know if I am missing something.

Yes, we'll return 'unsupported' since we currently don't support ingress action for mirror and redirect.

> >+ err = ice_tc_setup_mirror_action(filter_dev, fltr, act->dev);
> >+ if (err)
> >+ return err;
> >+ break;
> >+
> > default:
> > NL_SET_ERR_MSG_MOD(fltr->extack, "Unsupported action in
> >switchdev mode");
> > return -EINVAL;
> >diff --git a/drivers/net/ethernet/intel/ice/ice_type.h
> >b/drivers/net/ethernet/intel/ice/ice_type.h
> >index 5a80158e49ed..20c014e9b6c0 100644
> >--- a/drivers/net/ethernet/intel/ice/ice_type.h
> >+++ b/drivers/net/ethernet/intel/ice/ice_type.h
> >@@ -1055,6 +1055,7 @@ enum ice_sw_fwd_act_type {
> > ICE_FWD_TO_Q,
> > ICE_FWD_TO_QGRP,
> > ICE_DROP_PACKET,
> >+ ICE_MIRROR_PACKET,
> > ICE_NOP,
> > ICE_INVAL_ACT
> > };
> >--
> >2.25.1
> >
>
>

\
 
 \ /
  Last update: 2023-12-15 12:37    [W:0.078 / U:0.088 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site