On 02/07/16 16:34, Jamal Hadi Salim wrote: > From: Jamal Hadi Salim <j...@mojatatu.com> > > Often redirecting or mirroring requires that we set the dstMAC address > of the target device. While it is possible to pipe to a pedit action, > this patch obsoletes the need for that. This is a justified feature because > the dst MAC addresses rewrite is such a common use case. > > Sample usage: > sudo $TC filter add dev $ETH parent 1: protocol ip prio 10 \ > u32 match ip protocol 1 0xff flowid 1:2 \ > action mirred egress redirect dev $SPANPORT dst 02:15:15:15:15:15 > > This will match all icmp packets going out on dev $ETH and > redirect them to dev $SPANPORT while setting their dst MAC address > to 02:15:15:15:15:15 > > Signed-off-by: Jamal Hadi Salim <j...@mojatatu.com> > --- > include/net/tc_act/tc_mirred.h | 4 +++- > include/uapi/linux/tc_act/tc_mirred.h | 7 ++++--- > net/sched/act_mirred.c | 24 +++++++++++++++++++++++- > 3 files changed, 30 insertions(+), 5 deletions(-) >
LGTM, thanks! Reviewed-by: Nikolay Aleksandrov <niko...@cumulusnetworks.com>