netdev
[Top] [All Lists]

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

To: "David S. Miller" <davem@xxxxxxxxxx>
Subject: Re: 2.6.7-rc3: unregister_netdevice: waiting for tun0 to become free. Usage count = 1
From: Alexey Kuznetsov <kuznet@xxxxxxxxxxxxx>
Date: Thu, 17 Jun 2004 21:24:49 +0400
Cc: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>, schwab@xxxxxxx, netdev@xxxxxxxxxxx, yoshfuji@xxxxxxxxxxxxxx
In-reply-to: <20040617101016.44512cb1.davem@xxxxxxxxxx>
References: <20040614042216.GA28669@xxxxxxxxxxxxxxxxxxx> <20040614102858.GA12343@xxxxxxxxxxxxxxxxxxx> <20040614124402.GA28519@xxxxxxxxxxxxxxxxxxx> <20040616193731.GB29781@xxxxxxxxxxxxx> <20040616130950.6aadde3c.davem@xxxxxxxxxx> <20040616203748.GA30675@xxxxxxxxxxxxx> <20040616134711.499209c9.davem@xxxxxxxxxx> <20040617081722.GA8559@xxxxxxxxxxxxxxxxxxx> <20040617083303.GA8810@xxxxxxxxxxxxxxxxxxx> <20040617101016.44512cb1.davem@xxxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mutt/1.5.6i
Hello!

> It is to make sure that anyone (other cpus using the dst) who "saw"
> the old dst->dev is done doing whatever they were doing _BEFORE_
> we put the device reference.

Actually, he noticed right thing. I forgot that this function
works in context when dev, which is argument is of the function,
is held, so dev_put() here can be safely replaced with __dev_put().

So, synchonize_kernel() can be moved somewhere to unregister_netdevice(),
(maybe, it is already present there under surface)

Alexey

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