[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CA+55aFxQg6=MAWEkEGz64Hz8OawvPyX=fpBOjY8cGBL5XgB9WQ@mail.gmail.com>
Date: Tue, 17 Dec 2013 14:06:06 -0800
From: Linus Torvalds <torvalds@...ux-foundation.org>
To: Sander Eikelenboom <linux@...elenboom.it>
Cc: Ben Hutchings <ben@...adent.org.uk>,
Julian Calaby <julian.calaby@...il.com>,
Arend van Spriel <arend@...adcom.com>,
"Luis R. Rodriguez" <mcgrof@...not-panic.com>,
"Berg, Johannes" <johannes.berg@...el.com>,
"Grumbach, Emmanuel" <emmanuel.grumbach@...el.com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"ilw@...ux.intel.com" <ilw@...ux.intel.com>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
"linux-wireless@...r.kernel.org" <linux-wireless@...r.kernel.org>,
"John W. Linville" <linville@...driver.com>,
Avinash Patil <avinashapatil@...il.com>
Subject: Re: [cfg80211 / iwlwifi] setting wireless regulatory domain doesn't work.
On Tue, Dec 17, 2013 at 1:49 PM, Sander Eikelenboom
<linux@...elenboom.it> wrote:
>
> So if i combine the two .. it's essentially just a very bad idea to compile the wireless stuff in.
> It needs a access to a userland program at module load time, or it will block forever.
No, it's a very stupid module if it does that.
It should require the crda hook not at module load time, but at first
ifconfig time.
We've had bugs like this before. Doing user-mode callbacks at module
loading time is a disaster exactly because it doesn't work well with
built-in modules.
The fact that those things apparently also don't time out or notice
when they fail seems to then just exacerbate the bad decision.
We have literally had this *exact* same issue with firmware loading.
Network drivers shouldn't try to load firmware at module load time.
Same deal.
What's the broken path? Is this driver-specific, or is it generic to
the 802.11 code?
Linus
--
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
Powered by blists - more mailing lists