[PATCH 1/1] fixup! Add VRF support

Thomas Haller thaller at redhat.com
Sun Nov 1 08:56:18 PST 2015


On Fri, 2015-10-23 at 09:23 -0600, David Ahern wrote:
> On 10/23/15 9:03 AM, Thomas Haller wrote:
> > ---
> > 
> >   If you'd agree, I'd add the following changes.
> > 
> >   - no TABs after first indention.
> > 
> >   - have same whitespace in our kernel header copy
> >     as upstream kernel (to have smaller diff).
> > 
> >   - sort entries in .sym file by name.
> > 
> >   - make vrf_alloc properly handle cases where the
> >     data is already allocated.
> > 
> > 
> >   include/linux-private/linux/if_link.h |  6 +++---
> >   lib/route/link/vrf.c                  | 27 +++++++++++++++-------
> > -----
> >   libnl-route-3.sym                     |  3 +--
> >   3 files changed, 19 insertions(+), 17 deletions(-)
> > 
> 
> Looks good to me. Thanks for the doing the cleanup.

Hi David,

v3 + fixup commit applied:
https://github.com/thom311/libnl/commit/493bbd13e6d96c61e844dd1c04aa7218d0b7da08


Thank you!!
Thomas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part
URL: <http://lists.infradead.org/pipermail/libnl/attachments/20151101/6b3026e8/attachment.sig>


More information about the libnl mailing list