[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20160226154858.5338.90569.stgit@john-Precision-Tower-5810>
Date: Fri, 26 Feb 2016 07:53:26 -0800
From: John Fastabend <john.fastabend@...il.com>
To: jiri@...nulli.us, john.fastabend@...il.com, daniel@...earbox.net,
simon.horman@...ronome.com
Cc: netdev@...r.kernel.org, alexei.starovoitov@...il.com,
davem@...emloft.net, jhs@...atatu.com
Subject: [net-next PATCH v3 0/3] tc software only
This adds a software only flag to tc but incorporates a bunch of comments
from the original attempt at this.
First instead of having the offload decision logic be embedded in cls_u32
I lifted into cls_pkt.h so it can be used anywhere and named the flag
TCA_CLS_FLAGS_SKIP_HW (Thanks Jiri ;)
In order to do this I put the flag defines in pkt_cls.h as well. However
it was suggested that perhaps these flags could be lifted into the
upper layer of TCA_ as well but I'm afraid this can not be done with
existing tc design as far as I can tell. The problem is the filters are
packed and unpacked in the classifier specific code and pushing the flags
through the high level doesn't seem easily doable. And we already have
this design where classifiers handle generic options such as actions and
policers. So I think adding one more thing here is OK as 'tc', et. al.
already know how to handle this type of thing.
Thanks,
.John
---
John Fastabend (3):
net: sched: consolidate offload decision in cls_u32
net: cls_u32: move TC offload feature bit into cls_u32 offload logic
net: sched: cls_u32 add bit to specify software only rules
drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 3 --
include/net/pkt_cls.h | 17 +++++++++++
include/uapi/linux/pkt_cls.h | 1 +
net/sched/cls_u32.c | 37 ++++++++++++++++++-------
4 files changed, 45 insertions(+), 13 deletions(-)
--
Signature
Powered by blists - more mailing lists