Date: Fri, 19 Nov 2004 12:50:42 +0100
From: Gerd Knorr <kraxel@suse.de>
To: Takashi Iwai <tiwai@suse.de>
Cc: "Alexander E. Patrakov" <patrakov@ums.usu.ru>,
       linux-kernel@vger.kernel.org, Rusty Russell <rusty@rustcorp.com.au>
Subject: Re: modprobe + request_module() deadlock
Message-ID: <20041119115042.GA30334@bytesex>
References: <20041117222949.GA9006@linuxtv.org> <1100749702.5865.39.camel@localhost.localdomain> <20041118135522.GA16910@linuxtv.org> <s5h4qjnc6zs.wl@alsa2.suse.de> <cnjr8v$dcu$1@sea.gmane.org> <s5hpt2aayco.wl@alsa2.suse.de>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <s5hpt2aayco.wl@alsa2.suse.de>
User-Agent: Mutt/1.5.6i
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 991
Lines: 26

> > But IMHO Rusty should provide a way to specify those "recommended" install
> > lines in the modules themselves (so that they finally go
> > to /lib/modules/`uname -r`/modules.alias or a similar autogenerated file),
> > like that's already done wih aliases and char-majors.
> 
> Well, how about to add a new marker in the driver code such as
> 
> 	MODULE_DEPENDS_ON("somemodule");
> 
> so that depmod can pick it up?

Wouldn't work for me as this isn't static.  saa7134 has to look at the
hardware, then decide whenever it should load saa7134-empress,
saa7134-dvb or none of them.

On the other hand I don't depend on request_module() waiting for the
modprobe being finished.  So maybe we can solve that with a
request_module_async()?

  Gerd

-
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/