netdev - Re: [PATCH 2/8] netback: add module unload function

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20130304135831.4d52eddc@nehalam.linuxnetplumber.net>
Date:	Mon, 4 Mar 2013 13:58:31 -0800
From:	Stephen Hemminger <stephen@...workplumber.org>
To:	Wei Liu <wei.liu2@...rix.com>
Cc:	<xen-devel@...ts.xen.org>, <netdev@...r.kernel.org>,
	<ian.campbell@...rix.com>, <konrad.wilk@...cle.com>,
	<annie.li@...cle.com>
Subject: Re: [PATCH 2/8] netback: add module unload function

On Fri, 15 Feb 2013 16:00:03 +0000
Wei Liu <wei.liu2@...rix.com> wrote:

> Enable users to unload netback module. Users should make sure there is not vif
> runnig.


Isn't it likely that some admin might be trying to cleanup or do shutdown and there
is an ordering problem. What happens if vif's are still running?

Why depend on users? You should allow it anytime and do refcounting and auto-destroy the vif's for them.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html