LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [PATCH] wireless: fix regression caused by regulatory config option
@ 2008-10-26 17:30 Arjan van de Ven
  2008-10-26 20:09 ` John W. Linville
  2008-10-28 15:13 ` Bob Copeland
  0 siblings, 2 replies; 10+ messages in thread
From: Arjan van de Ven @ 2008-10-26 17:30 UTC (permalink / raw)
  To: torvalds; +Cc: netdev, linux-kernel, rjw, rodriguez, johannes

>From 9095a0f6005a8f50c79e51f4aaceda95e361a891 Mon Sep 17 00:00:00 2001
From: Arjan van de Ven <arjan@linux.intel.com>
Date: Sun, 26 Oct 2008 10:25:30 -0700
Subject: [PATCH] wireless: fix regression caused by regulatory config option

The default for the regulatory compatibility option is wrong;
if you picked the default you ended up with a non-functional wifi
system (at least I did on Fedora 9 with iwl4965).
I don't think even the October 2008 releases of the various distros
has the new userland so clearly the default is wrong, and also
we can't just go about deleting this in 2.6.29...

Change the default to "y" and also adjust the config text a little to
reflect this.

This patch fixes regression #11859

With thanks to Johannes Berg for the diagnostics

Signed-off-by: Arjan van de Ven <arjan@linux.intel.com>
---
 net/wireless/Kconfig |   11 +++++------
 1 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/net/wireless/Kconfig b/net/wireless/Kconfig
index 7d82be0..646c712 100644
--- a/net/wireless/Kconfig
+++ b/net/wireless/Kconfig
@@ -16,7 +16,7 @@ config NL80211
 
 config WIRELESS_OLD_REGULATORY
 	bool "Old wireless static regulatory definitions"
-	default n
+	default y
 	---help---
 	  This option enables the old static regulatory information
 	  and uses it within the new framework. This is available
@@ -40,11 +40,10 @@ config WIRELESS_OLD_REGULATORY
 	  ieee80211_regdom module parameter. This is being phased out and you
 	  should stop using them ASAP.
 
-	  Say N unless you cannot install a new userspace application
-	  or have one currently depending on the ieee80211_regdom module
-	  parameter and cannot port it to use the new userspace interfaces.
-
-	  This is scheduled for removal for 2.6.29.
+	  Say Y unless you have installed a new userspace application.
+	  Also say Y if have one currently depending on the ieee80211_regdom
+	  module parameter and cannot port it to use the new userspace
+	  interfaces.
 
 config WIRELESS_EXT
 	bool "Wireless extensions"
-- 
1.5.6.5



-- 
Arjan van de Ven 	Intel Open Source Technology Centre
For development, discussion and tips for power savings, 
visit http://www.lesswatts.org

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] wireless: fix regression caused by regulatory config option
  2008-10-26 17:30 [PATCH] wireless: fix regression caused by regulatory config option Arjan van de Ven
@ 2008-10-26 20:09 ` John W. Linville
  2008-10-26 20:16   ` Linus Torvalds
  2008-10-26 20:18   ` Arjan van de Ven
  2008-10-28 15:13 ` Bob Copeland
  1 sibling, 2 replies; 10+ messages in thread
From: John W. Linville @ 2008-10-26 20:09 UTC (permalink / raw)
  To: Arjan van de Ven; +Cc: torvalds, netdev, linux-kernel, rjw, rodriguez, johannes

On Sun, Oct 26, 2008 at 10:30:24AM -0700, Arjan van de Ven wrote:
> From 9095a0f6005a8f50c79e51f4aaceda95e361a891 Mon Sep 17 00:00:00 2001
> From: Arjan van de Ven <arjan@linux.intel.com>
> Date: Sun, 26 Oct 2008 10:25:30 -0700
> Subject: [PATCH] wireless: fix regression caused by regulatory config option
> 
> The default for the regulatory compatibility option is wrong;
> if you picked the default you ended up with a non-functional wifi
> system (at least I did on Fedora 9 with iwl4965).
> I don't think even the October 2008 releases of the various distros
> has the new userland so clearly the default is wrong, and also
> we can't just go about deleting this in 2.6.29...
> 
> Change the default to "y" and also adjust the config text a little to
> reflect this.

Is it common practice to have compatibility options default to
"y"?  I'm not saying it shouldn't be, just wondering if it has been
traditionally?

John
-- 
John W. Linville		Linux should be at the core
linville@tuxdriver.com			of your literate lifestyle.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] wireless: fix regression caused by regulatory config option
  2008-10-26 20:09 ` John W. Linville
@ 2008-10-26 20:16   ` Linus Torvalds
  2008-10-26 20:18   ` Arjan van de Ven
  1 sibling, 0 replies; 10+ messages in thread
From: Linus Torvalds @ 2008-10-26 20:16 UTC (permalink / raw)
  To: John W. Linville
  Cc: Arjan van de Ven, netdev, linux-kernel, rjw, rodriguez, johannes



On Sun, 26 Oct 2008, John W. Linville wrote:
> 
> Is it common practice to have compatibility options default to
> "y"?  I'm not saying it shouldn't be, just wondering if it has been
> traditionally?

Yes. Generally the rule should be that

	make oldconfig

with 'default values for all new options' should give you a configuration 
that is essentially identical to the old one.

So if it's a config option to enable compatibility with old kernels, it 
should default to 'y'. If it's a new driver or a new feature that didn't 
exist in previous kernels, it should default to 'n'.

			Linus

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] wireless: fix regression caused by regulatory config option
  2008-10-26 20:09 ` John W. Linville
  2008-10-26 20:16   ` Linus Torvalds
@ 2008-10-26 20:18   ` Arjan van de Ven
  2008-10-26 20:22     ` Johannes Berg
                       ` (2 more replies)
  1 sibling, 3 replies; 10+ messages in thread
From: Arjan van de Ven @ 2008-10-26 20:18 UTC (permalink / raw)
  To: John W. Linville; +Cc: torvalds, netdev, linux-kernel, rjw, rodriguez, johannes

On Sun, 26 Oct 2008 16:09:39 -0400
"John W. Linville" <linville@tuxdriver.com> wrote:
f
> > Change the default to "y" and also adjust the config text a little
> > to reflect this.
> 
> Is it common practice to have compatibility options default to
> "y"?  I'm not saying it shouldn't be, just wondering if it has been
> traditionally?

yes

default should be "keep working as before"

it's not always nice, especially if you're trying to get rid of some
nasty stuff, but think of it this way: you should be able to use a
new kernel on an existing distro, at least for a reasonable type of
distro (eg something shipped in, say, the last 2 years). In this case:
even Fedora 10 is not likely to work!



-- 
Arjan van de Ven 	Intel Open Source Technology Centre
For development, discussion and tips for power savings, 
visit http://www.lesswatts.org

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] wireless: fix regression caused by regulatory config option
  2008-10-26 20:18   ` Arjan van de Ven
@ 2008-10-26 20:22     ` Johannes Berg
  2008-10-27 20:42       ` Pavel Machek
  2008-10-26 20:44     ` John W. Linville
  2008-10-26 22:12     ` Marcel Holtmann
  2 siblings, 1 reply; 10+ messages in thread
From: Johannes Berg @ 2008-10-26 20:22 UTC (permalink / raw)
  To: Arjan van de Ven
  Cc: John W. Linville, torvalds, netdev, linux-kernel, rjw, rodriguez

[-- Attachment #1: Type: text/plain, Size: 795 bytes --]

On Sun, 2008-10-26 at 13:18 -0700, Arjan van de Ven wrote:

> > Is it common practice to have compatibility options default to
> > "y"?  I'm not saying it shouldn't be, just wondering if it has been
> > traditionally?
> 
> yes
> 
> default should be "keep working as before"
> 
> it's not always nice, especially if you're trying to get rid of some
> nasty stuff, but think of it this way: you should be able to use a
> new kernel on an existing distro, at least for a reasonable type of
> distro (eg something shipped in, say, the last 2 years). In this case:
> even Fedora 10 is not likely to work!

Well, actually, it _ought_ to work fine, with a smaller set of channels,
but you seem to be hit an iwlwifi bug that triggers only with this,
which is rather odd.

johannes

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] wireless: fix regression caused by regulatory config option
  2008-10-26 20:18   ` Arjan van de Ven
  2008-10-26 20:22     ` Johannes Berg
@ 2008-10-26 20:44     ` John W. Linville
  2008-10-26 22:12     ` Marcel Holtmann
  2 siblings, 0 replies; 10+ messages in thread
From: John W. Linville @ 2008-10-26 20:44 UTC (permalink / raw)
  To: Arjan van de Ven; +Cc: torvalds, netdev, linux-kernel, rjw, rodriguez, johannes

On Sun, Oct 26, 2008 at 01:18:18PM -0700, Arjan van de Ven wrote:
> On Sun, 26 Oct 2008 16:09:39 -0400
> "John W. Linville" <linville@tuxdriver.com> wrote:
> f
> > > Change the default to "y" and also adjust the config text a little
> > > to reflect this.
> > 
> > Is it common practice to have compatibility options default to
> > "y"?  I'm not saying it shouldn't be, just wondering if it has been
> > traditionally?
> 
> yes
> 
> default should be "keep working as before"
> 
> it's not always nice, especially if you're trying to get rid of some
> nasty stuff, but think of it this way: you should be able to use a
> new kernel on an existing distro, at least for a reasonable type of
> distro (eg something shipped in, say, the last 2 years). In this case:
> even Fedora 10 is not likely to work!

Well, I intended to make sure the Fedora guys picked the right option. :-)

Anyway, I'm fine with default 'y'...however, Luis is probably somewhere
buying a gun and looking-up your home address...just kidding...mostly... :-)

John
-- 
John W. Linville		Linux should be at the core
linville@tuxdriver.com			of your literate lifestyle.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] wireless: fix regression caused by regulatory config option
  2008-10-26 20:18   ` Arjan van de Ven
  2008-10-26 20:22     ` Johannes Berg
  2008-10-26 20:44     ` John W. Linville
@ 2008-10-26 22:12     ` Marcel Holtmann
  2 siblings, 0 replies; 10+ messages in thread
From: Marcel Holtmann @ 2008-10-26 22:12 UTC (permalink / raw)
  To: Arjan van de Ven
  Cc: John W. Linville, torvalds, netdev, linux-kernel, rjw, rodriguez,
	johannes

Hi Arjan,

> > > Change the default to "y" and also adjust the config text a little
> > > to reflect this.
> > 
> > Is it common practice to have compatibility options default to
> > "y"?  I'm not saying it shouldn't be, just wondering if it has been
> > traditionally?
> 
> yes
> 
> default should be "keep working as before"
> 
> it's not always nice, especially if you're trying to get rid of some
> nasty stuff, but think of it this way: you should be able to use a
> new kernel on an existing distro, at least for a reasonable type of
> distro (eg something shipped in, say, the last 2 years). In this case:
> even Fedora 10 is not likely to work!

I think with the new patches from Johannes to make the regulatory
interaction easier for hardware that has hardware/EEPROM based
regulatory enforcement like our wireless cards, it should be all good.

That seems to be 2.6.20 stuff and then it seems to be safe to remove the
WIRELESS_OLD_REGULATORY option and it would still work with old
userspace (or missing crda/iw) in cases the hardware does regulatory
enforcement.

Regards

Marcel



^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] wireless: fix regression caused by regulatory config option
  2008-10-26 20:22     ` Johannes Berg
@ 2008-10-27 20:42       ` Pavel Machek
  2008-10-28  7:29         ` Johannes Berg
  0 siblings, 1 reply; 10+ messages in thread
From: Pavel Machek @ 2008-10-27 20:42 UTC (permalink / raw)
  To: Johannes Berg
  Cc: Arjan van de Ven, John W. Linville, torvalds, netdev,
	linux-kernel, rjw, rodriguez

On Sun 2008-10-26 21:22:16, Johannes Berg wrote:
> On Sun, 2008-10-26 at 13:18 -0700, Arjan van de Ven wrote:
> 
> > > Is it common practice to have compatibility options default to
> > > "y"?  I'm not saying it shouldn't be, just wondering if it has been
> > > traditionally?
> > 
> > yes
> > 
> > default should be "keep working as before"
> > 
> > it's not always nice, especially if you're trying to get rid of some
> > nasty stuff, but think of it this way: you should be able to use a
> > new kernel on an existing distro, at least for a reasonable type of
> > distro (eg something shipped in, say, the last 2 years). In this case:
> > even Fedora 10 is not likely to work!
> 
> Well, actually, it _ought_ to work fine, with a smaller set of channels,

Really? It will still break if your AP uses one of those channels, right?


-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] wireless: fix regression caused by regulatory config option
  2008-10-27 20:42       ` Pavel Machek
@ 2008-10-28  7:29         ` Johannes Berg
  0 siblings, 0 replies; 10+ messages in thread
From: Johannes Berg @ 2008-10-28  7:29 UTC (permalink / raw)
  To: Pavel Machek
  Cc: Arjan van de Ven, John W. Linville, torvalds, netdev,
	linux-kernel, rjw, rodriguez

[-- Attachment #1: Type: text/plain, Size: 783 bytes --]

On Mon, 2008-10-27 at 21:42 +0100, Pavel Machek wrote:

> > > it's not always nice, especially if you're trying to get rid of some
> > > nasty stuff, but think of it this way: you should be able to use a
> > > new kernel on an existing distro, at least for a reasonable type of
> > > distro (eg something shipped in, say, the last 2 years). In this case:
> > > even Fedora 10 is not likely to work!
> > 
> > Well, actually, it _ought_ to work fine, with a smaller set of channels,
> 
> Really? It will still break if your AP uses one of those channels, right?

Well yes, but the driver itself shouldn't simply fail as it did in
Arjan's case, it should just not find the AP rather than saying "sorry,
found error and can't continue" or something like that.

johannes

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH] wireless: fix regression caused by regulatory config option
  2008-10-26 17:30 [PATCH] wireless: fix regression caused by regulatory config option Arjan van de Ven
  2008-10-26 20:09 ` John W. Linville
@ 2008-10-28 15:13 ` Bob Copeland
  1 sibling, 0 replies; 10+ messages in thread
From: Bob Copeland @ 2008-10-28 15:13 UTC (permalink / raw)
  To: Arjan van de Ven; +Cc: torvalds, netdev, linux-kernel, rjw, rodriguez, johannes

> -         Say N unless you cannot install a new userspace application
> -         or have one currently depending on the ieee80211_regdom module
> -         parameter and cannot port it to use the new userspace interfaces.
> -
> -         This is scheduled for removal for 2.6.29.
> +         Say Y unless you have installed a new userspace application.
> +         Also say Y if have one currently depending on the ieee80211_regdom
> +         module parameter and cannot port it to use the new userspace
> +         interfaces.

Ignoring the main debate :) -- I might point out the actual help text here in
is a bit user-unfriendly.  "installed a new userspace application" (which? I
just installed Firefox, does that count?)  "Also say Y if [you] have [an
application]..."

-- 
Bob Copeland %% www.bobcopeland.com

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2008-10-28 15:13 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-10-26 17:30 [PATCH] wireless: fix regression caused by regulatory config option Arjan van de Ven
2008-10-26 20:09 ` John W. Linville
2008-10-26 20:16   ` Linus Torvalds
2008-10-26 20:18   ` Arjan van de Ven
2008-10-26 20:22     ` Johannes Berg
2008-10-27 20:42       ` Pavel Machek
2008-10-28  7:29         ` Johannes Berg
2008-10-26 20:44     ` John W. Linville
2008-10-26 22:12     ` Marcel Holtmann
2008-10-28 15:13 ` Bob Copeland

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).