On Tue, 12 Aug 2003 09:48:50 -0700
Max Krasnyansky <maxk@xxxxxxxxxxxx> wrote:
> At 01:18 AM 8/9/2003, David S. Miller wrote:
> >On Fri, 8 Aug 2003 11:34:04 -0700
> >Stephen Hemminger <shemminger@xxxxxxxx> wrote:
> >
> >> Less grotty version, applies over earlier patch.
> >> - keep a private list.
> >> - fix debug format strings.
> >> - drop the name entry in the private data structure since it already
> >> has a pointer to netdev that has name.
> >
> >Applied, thanks for following up on this Stephen.
>
> Folks,
>
> Sorry for jumping in late.
> I didn't implement cleanup logic in module_exit() because TUN module is not
> supposed
> to be unloaded if it has network devices, _even if those devices are down_.
> TUN registers net device only when user application asks for it.
> fd = open("/dev/net/tun") -> ioctl(fd, CREATE_TUN_DEV) ->
> read(fd)/write(fd);
> Net device must not be destroyed while fd is open.
>
> So instead of cleaning up in tun_module_exit() we should fix misc driver to
> do refcounting
> for misc devices so that we could bump ref count for tun driver when
> application creates
> net device.
Not necessary to change anything. If user process has /dev/net/tun open, then
the owner
field in the fops causes the module reference count to correctly increment.
Verified this
and it works. The issue is that it is possible to create TUN devices with
TUN_PERSIST
set and they have to be cleaned up upon. module_exit.
The patch works as expected.
|