Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id ; Thu, 11 Jul 2002 13:59:06 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id ; Thu, 11 Jul 2002 13:59:05 -0400 Received: from pD952A525.dip.t-dialin.net ([217.82.165.37]:63617 "EHLO hawkeye.luckynet.adm") by vger.kernel.org with ESMTP id ; Thu, 11 Jul 2002 13:59:04 -0400 Date: Thu, 11 Jul 2002 12:01:25 -0600 (MDT) From: Thunder from the hill X-X-Sender: thunder@hawkeye.luckynet.adm To: Roman Zippel cc: Daniel Phillips , Rusty Russell , Alexander Viro , "David S. Miller" , , , Subject: Re: Rusty's module talk at the Kernel Summit In-Reply-To: Message-ID: X-Location: Potsdam; Germany MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1235 Lines: 41 Hi, On Thu, 11 Jul 2002, Roman Zippel wrote: > On Thu, 11 Jul 2002, Daniel Phillips wrote: > > Closing the rmmod race with this interface is easy. We can for example just > > keep a state variable in the module struct (protected by a lock) to say the > > module is in the process of being deregistered. > > Please check try_inc_mod_count(). It's already done. Btw, couldn't the module/non-module issue be solved like this: int module_do_blah(struct blah *blah, didel_t dei) #ifdef __MODULE__ { locking_code(); pure_module_do_blah(blah, dei) unlocking_code(); } int pure_module_do_blah(struct blah *blah, didel_t dei) #endif /* __MODULE__ */ Just an idea... Regards, Thunder -- (Use http://www.ebb.org/ungeek if you can't decode) ------BEGIN GEEK CODE BLOCK------ Version: 3.12 GCS/E/G/S/AT d- s++:-- a? C++$ ULAVHI++++$ P++$ L++++(+++++)$ E W-$ N--- o? K? w-- O- M V$ PS+ PE- Y- PGP+ t+ 5+ X+ R- !tv b++ DI? !D G e++++ h* r--- y- ------END GEEK CODE BLOCK------ - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/