ACK: [SRU][F:linux-bluefield][PATCH v2 0/1] net: sched: allow flower to match vxlan options

[Bartlomiej Zolnierkiewicz]

Hi Tony,

While my ACK holds (as without this fix "ovs" won't even compile) I
wonder whether the backport of the commit d515d63cae2c ("net/mlx5e:
Don't support encap rules with gbp option") shouldn't also be
submitted as a follow-up?

--
Best regards,
Bartlomiej

On Wed, Apr 5, 2023 at 12:29 PM Bartlomiej Zolnierkiewicz
<bartlomiej.zolnierkiewicz at canonical.com> wrote:
>
> Acked-by: Bartlomiej Zolnierkiewicz <bartlomiej.zolnierkiewicz at canonical.com>
>
> On Sat, Apr 1, 2023 at 1:36 AM Tony Duan <yifeid at nvidia.com> wrote:
> >
> > Buglink: https://bugs.launchpad.net/bugs/2013422
> >
> > SRU Justification:
> >
> > [Impact]
> >
> > * This patch is to allow matching gbp option in vxlan. Introduce vxlan support to avoid "ovs" compile error.
> >
> > [Fix]
> >
> > * cherry-pick d8f9dfae49ce4ffb772dc10dd6578dc815b34c12 d8f9dfae49ce net: sched: allow flower to match vxlan options
> >
> > [Test Plan]
> >
> > 1) Compile "ovs" project, build succesfully.
> > 2) VXLAN option can be used as match condition in tc. eg:
> >       # ip link add name vxlan0 type vxlan dstport 0 external
> >       # tc qdisc add dev vxlan0 ingress
> >       # tc filter add dev vxlan0 protocol ip parent ffff: \
> >           flower \
> >             enc_src_ip 10.0.99.192 \
> >             enc_dst_ip 10.0.99.193 \
> >             enc_key_id 11 \
> >             vxlan_opts 01020304/ffffffff \
> >             ip_proto udp \
> >             action mirred egress redirect dev eth0
> >         Check the action is correcttly filtered
> >
> > [Where problems could occur]
> >
> > * Compile "ovs" without this fix, the compiling fails.
> >
> > [Other Info]
> >
> > * nothing
> >
> > -v2:
> > * Remove "UBUNTU: " prefix since it is already in upstream
> >
> > *** BLURB HERE ***
> >
> > Xin Long (1):
> >   net: sched: allow flower to match vxlan options
> >
> >  include/uapi/linux/pkt_cls.h |  13 +++++
> >  net/sched/cls_flower.c       | 109 +++++++++++++++++++++++++++++++++++
> >  2 files changed, 122 insertions(+)
> >