Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751429AbaLaHoW (ORCPT ); Wed, 31 Dec 2014 02:44:22 -0500 Received: from mga02.intel.com ([134.134.136.20]:64012 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750710AbaLaHoU convert rfc822-to-8bit (ORCPT ); Wed, 31 Dec 2014 02:44:20 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.04,691,1406617200"; d="scan'208";a="506034713" From: "Grumbach, Emmanuel" To: Jiri Kosina , Linus Torvalds CC: Borislav Petkov , "linux-wireless@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "egrumbach@gmail.com" , "peter@hurleysoftware.com" , "ilw@linux.intel.com" , "Berg, Johannes" , "Larry Finger" , "johannes@sipsolutions.net" Subject: RE: [PATCH] Revert "cfg80211: make WEXT compatibility unselectable" Thread-Topic: [PATCH] Revert "cfg80211: make WEXT compatibility unselectable" Thread-Index: AQHQJINHASoXmCzt7UaJrD69LgOnMJypUBRQ Date: Wed, 31 Dec 2014 07:44:14 +0000 Message-ID: <0BA3FCBA62E2DC44AF3030971E174FB31B5DD408@hasmsx107.ger.corp.intel.com> References: <54A2B807.1020202@hurleysoftware.com> <0BA3FCBA62E2DC44AF3030971E174FB31B5DC915@hasmsx107.ger.corp.intel.com> <1419971304.30412.0.camel@egrumbacBox> <20141230212326.GA29263@pd.tnic> <54A328C2.5080606@lwfinger.net> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.184.70.11] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > Subject: [PATCH] Revert "cfg80211: make WEXT compatibility unselectable" > > This reverts commit 24a0aa212ee2dbe44360288684478d76a8e20a0a. > > It's causing severe userspace breakage. Namely, all the utilities from > wireless-utils which are relying on CONFIG_WEXT (which means tools like > 'iwconfig', 'iwlist', etc) are not working anymore. There is a 'iw' utility in > newer wireless-tools, which is supposed to be a replacement for all the > "deprecated" binaries, but it's far away from being massively adopted. > > Please see [1] for example of the userspace breakage this is causing. > > In addition to that, Larry Finger reports [2] that this patch is also causing > ipw2200 driver being impossible to build. > > To me this clearly shows that CONFIG_WEXT is far, far away from being > "deprecated enough" to be removed. > > [1] http://thread.gmane.org/gmane.linux.kernel/1857010 > [2] http://thread.gmane.org/gmane.linux.network/343688 > > Signed-off-by: Jiri Kosina Wait. I don't want to discuss the userspace API deprecation here but I do think that this patch should be applied to the proper tree if at all. The proper tree is mac80211.git. Now you are unhappy with what happens there and want to escalate - fine. But then, please do so following the natural hierarchy of the trees: net.git. Then mac80211.git's maintainer (Johannes) will be able to sync with your change more easily. Johannes doesn't need me as an advocate, but I feel that reverting a mac80211 patch in linux.git while we are only in -rc2 while everybody is on holiday leave is a bit unfair. > --- > net/wireless/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/net/wireless/Kconfig b/net/wireless/Kconfig index > 22ba971..29c8675 100644 > --- a/net/wireless/Kconfig > +++ b/net/wireless/Kconfig > @@ -175,7 +175,7 @@ config CFG80211_INTERNAL_REGDB > Most distributions have a CRDA package. So if unsure, say N. > > config CFG80211_WEXT > - bool > + bool "cfg80211 wireless extensions compatibility" > depends on CFG80211 > select WEXT_CORE > help > -- > Jiri Kosina > SUSE Labs -- 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/