The attachments worked. I applied them to master and branch-2.8. I
added a NEWS entry:
- NSH implementation now conforms to latest draft
(draft-ietf-sfc-nsh-28).
Thank you!
On Wed, Nov 08, 2017 at 08:48:53AM +0000, Jan Scheurich wrote:
> Hi Ben,
>
> I think the first patch does not apply because somehow the email process has
> converted the leading tab to 7 spaces in line 24 of the patch.
>
> Unfortunately I can't use git send-email in our company network due to
> firewalls restrictions. Could you try to use the attached patch files instead?
>
> BR, Jan
>
> > -----Original Message-----
> > From: Ben Pfaff [mailto:b...@ovn.org]
> > Sent: Wednesday, 08 November, 2017 05:27
> > To: Jan Scheurich <jan.scheur...@ericsson.com>
> > Cc: 'd...@openvswitch.org' <d...@openvswitch.org>; Yang, Yi Y
> > (yi.y.y...@intel.com) <yi.y.y...@intel.com>; Jiri Benc
> > (jb...@redhat.com) <jb...@redhat.com>
> > Subject: Re: [PATCH v2 1/2] NSH: Minor bugfixes
> >
> > On Tue, Nov 07, 2017 at 03:34:06PM +0000, Jan Scheurich wrote:
> > > - Fix 2 incorrect length checks
> > > - Remove unnecessary limit of MD length to 16 bytes
> > > - Remove incorrect comments stating MD2 was not supported
> > > - Pad metadata in encap_nsh with zeroes if not multiple of 4 bytes
> > >
> > > Signed-off-by: Jan Scheurich <jan.scheur...@ericsson.com>
> >
> > There's something odd about this patch and it does not apply:
> >
> > Using index info to reconstruct a base tree...
> > error: patch failed:
> > datapath/linux/compat/include/linux/openvswitch.h:793
> > error: datapath/linux/compat/include/linux/openvswitch.h: patch does
> > not apply
> > error: Did you hand edit your patch?
> > It does not apply to blobs recorded in its index.
> >
> > I've never seen that particular error before.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
