On Tue, Jun 07, 2016 at 08:37:40AM -0700, John Fastabend wrote: > On 16-06-05 07:11 AM, Amir Vadai wrote: > > From: Amir Vadai <ami...@mellanox.com> > > > > In order to make a filter processed only by hardware, skip_sw flag > > should be supplied. This is an addition to the already existing skip_hw > > flag (filter will be processed by software only). If no flag is > > specified, filter will be processed by both software and hardware. > > > > If only hardware offloaded filters exist, fl_classify() will return > > without doing anything. > > > > A following userspace patch will be sent once kernel patch is accepted. > > > > Example: > > > > tc filter add dev enp0s9 protocol ip prio 20 parent ffff: \ > > flower \ > > ip_proto 6 \ > > indev enp0s9 \ > > skip_sw \ > > action skbedit mark 0x1234 > > > > Signed-off-by: Amir Vadai <ami...@mellanox.com> > > --- > > > > Looks good to me. Although we need to do the same error propagation in > flower that Jakub just added to cls_u32. Thanks John, I will send a patch to return error when add to hw is failing and skip_sw is set.
> > Acked-by: John Fastabend <john.r.fastab...@intel.com> >
