netdev
[Top] [All Lists]

Re: 2.6.7-rc3: unregister_netdevice: waiting for tun0 to become free. Us

To: Alexey Kuznetsov <kuznet@xxxxxxxxxxxxx>
Subject: Re: 2.6.7-rc3: unregister_netdevice: waiting for tun0 to become free. Usage count = 1
From: "David S. Miller" <davem@xxxxxxxxxx>
Date: Wed, 16 Jun 2004 13:47:11 -0700
Cc: herbert@xxxxxxxxxxxxxxxxxxx, schwab@xxxxxxx, netdev@xxxxxxxxxxx, yoshfuji@xxxxxxxxxxxxxx
In-reply-to: <20040616203748.GA30675@xxxxxxxxxxxxx>
References: <E1BZeW1-0008Ny-00@xxxxxxxxxxxxxxxxxxxxxxxx> <20040613234142.GA32329@xxxxxxxxxxxxxxxxxxx> <20040613183622.3a814506.davem@xxxxxxxxxx> <20040614015013.GA11048@xxxxxxxxxxxxxxxxxxx> <20040613210725.70dbd016.davem@xxxxxxxxxx> <20040614042216.GA28669@xxxxxxxxxxxxxxxxxxx> <20040614102858.GA12343@xxxxxxxxxxxxxxxxxxx> <20040614124402.GA28519@xxxxxxxxxxxxxxxxxxx> <20040616193731.GB29781@xxxxxxxxxxxxx> <20040616130950.6aadde3c.davem@xxxxxxxxxx> <20040616203748.GA30675@xxxxxxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
On Thu, 17 Jun 2004 00:37:48 +0400
Alexey Kuznetsov <kuznet@xxxxxxxxxxxxx> wrote:

> Sort of. I think it should happen after killing reference to dst->dev:
 ...
> but before dropping reference to dev:
 ...
> So, I think it should look like:
> 
>               dst->dev = &loopback_dev;
>               dev_hold(&loopback_dev);
> +             synchronize_kernel();
>               dev_put(dev);

Perfect, that's what I put into my tree then, plus some comment.

<Prev in Thread] Current Thread [Next in Thread>