So far, we're not clear how we can support MD type 2 better, as I explained
before, we need to reuse tun_metadata in struct flow_tnl which is the thing
Geneve is using. Geneve predefined 64 keys for this from tun_metadata0 to
tun_metadata63, we will reuse it for MD type 2. But you know NSH is not tunnel,
so it has to be changed to support both Geneve and NSH. Anyway, they won't be
part of ovs_key_nsh.
From: netdev-ow...@vger.kernel.org [mailto:netdev-ow...@vger.kernel.org] On
Behalf Of Jiri Benc
Sent: Friday, August 11, 2017 5:11 PM
To: Yang, Yi Y <yi.y.y...@intel.com>
Cc: net...@vger.kernel.org; d...@openvswitch.org
Subject: Re: [PATCH net-next v2] openvswitch: enable NSH support
On Fri, 11 Aug 2017 16:47:23 +0800, Yang, Yi wrote:
> is "__be32 context" ok?
Yes, that looks better.
> So define three new netlink attributes
> OVS_ACTION_ATTR_PUSH_NSH is nested netlink attribute, it will nest
> OVS_ACTION_ATTR_NSH_BASE_HEADER and OVS_ACTION_ATTR_NSH_MD1_DATA for
> MD type 1, it will nest OVS_ACTION_ATTR_NSH_BASE_HEADER and
> OVS_ACTION_ATTR_NSH_MD2_DATA for MD type 2. I'll compeletely remove
> struct ovs_action_push_nsh, is it ok?
Yes, that's the way to do it.
What should be done with struct ovs_key_nsh? Even with "c" renamed to
"context", it's still MD type 1 only structure. What is the plan for MD type 2
support wrt. this structure?
dev mailing list