[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