[PATCH 1/1] fixup! Add VRF support
David Ahern
dsa at cumulusnetworks.com
Fri Oct 23 08:23:32 PDT 2015
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.
David
More information about the libnl
mailing list