svn commit: r272886 - in head/sys: netinet netinet6

[Bryan Venteicher]

On Tue, Jan 6, 2015 at 2:52 PM, John Nielsen <lists at jnielsen.net> wrote:

> Bryan-
>
> On Oct 10, 2014, at 12:09 AM, Bryan Venteicher <bryanv at freebsd.org> wrote:
>
> > Author: bryanv
> > Date: Fri Oct 10 06:08:59 2014
> > New Revision: 272886
> > URL: https://svnweb.freebsd.org/changeset/base/272886
> >
> > Log:
> >  Add context pointer and source address to the UDP tunnel callback
> >
> >  These are needed for the forthcoming vxlan implementation. The context
> >  pointer means we do not have to use a spare pointer field in the inpcb,
> >  and the source address is required to populate vxlan's forwarding table.
> >
> >  While I highly doubt there is an out of tree consumer of the UDP
> >  tunneling callback, this change may be a difficult to eventually MFC.
>
> I noticed this comment while doing an MFC of vxlan to my local tree. Do
> you think an MFC to 10-STABLE of this change (and vxlan generally) will be
> feasible? Is there precedent for ABI changes like this being sanctioned?
> Could symbol versioning help?
>
>

I'd like to get some consensus on whether this commit is OK to MFC. With
this commit, vxlan should be an easy to MFC.



> >  Phabricator: https://reviews.freebsd.org/D383
> >  Reviewed by: gnn
> >
> > Modified:
> >  head/sys/netinet/sctputil.c
> >  head/sys/netinet/udp_usrreq.c
> >  head/sys/netinet/udp_var.h
> >  head/sys/netinet6/udp6_usrreq.c
>
> Thanks,
>
> JN
>
>