Subject: Re: [PATCH net-next 0/4] vrf: cleanups part 2

From: David Ahern <[email-masked]>
Date: Wed, 19 Aug 2015 10:47:10 -0700

On 8/18/15 8:27 PM, Nikolay Aleksandrov wrote:
> From: Nikolay Aleksandrov <[email-masked]>
>
> Hi,
> This is the next part of vrf cleanups, patch 1 drops the SLAB_PANIC when
> creating kmem cache since it's handled, patch 02 removes a slave duplicate
> check which is already done by the lower/upper code, patch 3 moves the
> ndo_add_slave code around a bit so we can drop an error label and patch 4
> drops the master device checks which are unnecessary because the ops are
> taken from the master device itself so it can't be different.
>
> Cheers,
>   Nik
>
> Nikolay Aleksandrov (4):
>    vrf: don't panic on cache create failure
>    vrf: remove unnecessary duplicate check
>    vrf: move vrf_insert_slave so we can drop a goto label
>    vrf: ndo_add|del_slave drop unnecessary checks
>
>   drivers/net/vrf.c | 24 ++++--------------------
>   1 file changed, 4 insertions(+), 20 deletions(-)
>

Looks good to me. Thanks, Nikolay.

Acked-by: David Ahern <[email-masked]>