[ACK] [PATCH 1/1] tunnels: fix netns vs proto registration ordering -- fix stable backport

Stefan Bader stefan.bader at canonical.com
Tue Aug 30 13:04:28 UTC 2011


On 30.08.2011 14:44, Andy Whitcroft wrote:
> As per Stratos Psomadakis the backport of the upstream commit below is
> flawed and introduces crashes when using this ipv6 tunnel type:
> 
>   commit a14a3f90745c4831a1437528706b38daa1db4885
>   Author: Alexey Dobriyan <adobriyan at gmail.com>
>   Date:   Tue Feb 16 09:05:04 2010 +0000
> 
>     tunnels: fix netns vs proto registration ordering
> 
> Signed-off-by: Andy Whitcroft <apw at canonical.com>
> ---
>  net/ipv6/ip6_tunnel.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/net/ipv6/ip6_tunnel.c b/net/ipv6/ip6_tunnel.c
> index 7fb3e02..53e0d51 100644
> --- a/net/ipv6/ip6_tunnel.c
> +++ b/net/ipv6/ip6_tunnel.c
> @@ -1466,7 +1466,7 @@ static int __init ip6_tunnel_init(void)
>  {
>  	int  err;
>  
> -	err = register_pernet_device(&ip6_tnl_net_ops);
> +	err = register_pernet_gen_device(&ip6_tnl_net_id, &ip6_tnl_net_ops);
>  	if (err < 0)
>  		goto out_pernet;
>  

Looks like the proposed patch on the stable ml. Also looks obvious after being
pointed out (as  the ipv4 part makes no change there).

Acked-by: Stefan Bader <stefan.bader at canonical.com>




More information about the kernel-team mailing list