Re: [ovs-dev] [PATCH net-next v6 3/3] openvswitch: enable NSH support

2017-09-04 Thread Hannes Frederic Sowa
Hello, Jan Scheurich writes: >> >> >> Does it makes sense to keep the context headers as part of the flow? >> >> >> What is the reasoning behind it? With mdtype 2 headers this might >> >> >> either not work very well or will increase sw_flow_key size causing >> >> >>

RE: [ovs-dev] [PATCH net-next v6 3/3] openvswitch: enable NSH support

2017-09-04 Thread Jan Scheurich
> >> >> Does it makes sense to keep the context headers as part of the flow? > >> >> What is the reasoning behind it? With mdtype 2 headers this might > >> >> either not work very well or will increase sw_flow_key size causing > >> >> slowdowns for all protocols. > >> > [Mooney, Sean K] > >> >

RE: [ovs-dev] [PATCH net-next v6 3/3] openvswitch: enable NSH support

2017-09-01 Thread Jan Scheurich
> > [Mooney, Sean K] > > Having the nsh context headers in the flow is quite useful It would > > allow loadblancing on values stored in the context headers Or other > > use. I belive odl previously used context header 4 to store a Flow id > > so this could potentialy be used with the multipath

Re: [ovs-dev] [PATCH net-next v6 3/3] openvswitch: enable NSH support

2017-08-31 Thread Hannes Frederic Sowa
Hello, "Mooney, Sean K" writes: [...] >> >> > +struct ovs_key_nsh { >> >> > + u8 flags; >> >> > + u8 ttl; >> >> > + u8 mdtype; >> >> > + u8 np; >> >> > + __be32 path_hdr; >> >> > + __be32 context[NSH_MD1_CONTEXT_SIZE]; }; >> >> > +

RE: [ovs-dev] [PATCH net-next v6 3/3] openvswitch: enable NSH support

2017-08-30 Thread Mooney, Sean K
ernel.org; jb...@redhat.com; e...@erig.me > Subject: Re: [ovs-dev] [PATCH net-next v6 3/3] openvswitch: enable NSH > support > > "Mooney, Sean K" <sean.k.moo...@intel.com> writes: > > >> -Original Message- > >> From: ovs-dev-boun...@openvswitch.

Re: [ovs-dev] [PATCH net-next v6 3/3] openvswitch: enable NSH support

2017-08-30 Thread Hannes Frederic Sowa
To: Yang, Yi Y <yi.y.y...@intel.com> >> Cc: d...@openvswitch.org; netdev@vger.kernel.org; jb...@redhat.com; >> e...@erig.me >> Subject: Re: [ovs-dev] [PATCH net-next v6 3/3] openvswitch: enable NSH >> support >> >> Hello, >> >> Yi Yang <yi

RE: [ovs-dev] [PATCH net-next v6 3/3] openvswitch: enable NSH support

2017-08-30 Thread Mooney, Sean K
tdev@vger.kernel.org; jb...@redhat.com; > e...@erig.me > Subject: Re: [ovs-dev] [PATCH net-next v6 3/3] openvswitch: enable NSH > support > > Hello, > > Yi Yang <yi.y.y...@intel.com> writes: > > [...] > > > +struct ovs_key_nsh { > > + u8 flags; &