Re: [net-next PATCH 0/5] net_sched: Add support for IFE action

[Daniel Borkmann]

Hi Jamal,

On 02/22/2016 02:21 PM, Jamal Hadi Salim wrote:

From: Jamal Hadi Salim <j...@mojatatu.com>


As agreed at netconf in Seville, here's the patch finally (1 year
was just too long to wait).
Described in netdev01 paper:
             "Distributing Linux Traffic Control Classifier-Action Subsystem"
              Authors: Jamal Hadi Salim and Damascene M. Joachimpillai

Allows for incremental updates for new metadatum support.
This patch set includes support for basic skb metadatum
Followup patches will have more examples of metadata

So, basically this is a L2 encap with TLVs, right?

And as TLVs you have skb->mark, skb->priority, skb->hash, skb->queue_mapping
that you transfer from one machine to another, where on the destination, you
are applying the above meta data to the skb itself. And, configuration is via
tc.

I couldn't parse from the commit log what the real world use case is, resp.
who is going to use this infrastructure?

Do you have some typical setup, where the above needs to be transferred in the
encap and restored?

Jamal Hadi Salim (5):
   introduce IFE action
   Support to encoding decoding skb mark on IFE action
   Support to encoding decoding skb prio on IFE action
   Support to encoding decoding skb hashid on IFE action
   Support to encoding decoding skb queue map on IFE action

  include/net/tc_act/tc_ife.h        |  60 +++
  include/uapi/linux/tc_act/tc_ife.h |  38 ++
  net/sched/Kconfig                  |  32 ++
  net/sched/Makefile                 |   5 +
  net/sched/act_ife.c                | 865 +++++++++++++++++++++++++++++++++++++
  net/sched/act_meta_mark.c          |  81 ++++
  net/sched/act_meta_qmap.c          | 100 +++++
  net/sched/act_meta_skbhash.c       |  87 ++++
  net/sched/act_meta_skbprio.c       |  80 ++++

Splitting these set/get functions into individual modules where you only
set/get a single skb member seems overkill to me. Could be done with a
simple switch statement inside ife?

  9 files changed, 1348 insertions(+)
  create mode 100644 include/net/tc_act/tc_ife.h
  create mode 100644 include/uapi/linux/tc_act/tc_ife.h
  create mode 100644 net/sched/act_ife.c
  create mode 100644 net/sched/act_meta_mark.c
  create mode 100644 net/sched/act_meta_qmap.c
  create mode 100644 net/sched/act_meta_skbhash.c
  create mode 100644 net/sched/act_meta_skbprio.c

Thanks,
Daniel