From mboxrd@z Thu Jan 1 00:00:00 1970 From: Johannes Berg Subject: Re: [RFC][PATCH] mac80211_hwsim: No parent is better than an illegimate one. Date: Mon, 12 Jul 2010 08:46:40 +0200 Message-ID: <1278917200.3667.1.camel@jlt3.sipsolutions.net> References: <20100621150826.762ac9f2.akpm@linux-foundation.org> <20100621222938.GA20583@suse.de> <20100621231058.GA1066@suse.de> <20100622035631.GA3755@suse.de> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: Andrew Morton , "Rafael J. Wysocki" , "Maciej W. Rozycki" , Kay Sievers , Greg KH , "Greg KH netdev" To: "Eric W. Biederman" Return-path: Received: from he.sipsolutions.net ([78.46.109.217]:55370 "EHLO sipsolutions.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754216Ab0GLGs0 (ORCPT ); Mon, 12 Jul 2010 02:48:26 -0400 In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: On Thu, 2010-07-08 at 09:37 -0700, Eric W. Biederman wrote: > Don't call SET_IEEE80211_DEV. This weakens the connections between > the phy files in sysfs slightly but otherwise it makes the driver work > in the face of tagged sysfs support. > > Signed-off-by: Eric W. Biederman > --- > > Johannes does this change look usable? It'll probably work, but anything in userspace relying on it will fail, and you'll have some unused virtual devices in sysfs? Not too keen on it, but if we really can't fix it .... johannes > drivers/net/wireless/mac80211_hwsim.c | 1 - > 1 files changed, 0 insertions(+), 1 deletions(-) > > diff --git a/drivers/net/wireless/mac80211_hwsim.c b/drivers/net/wireless/mac80211_hwsim.c > index 6f8cb3e..b387222 100644 > --- a/drivers/net/wireless/mac80211_hwsim.c > +++ b/drivers/net/wireless/mac80211_hwsim.c > @@ -1282,7 +1282,6 @@ static int __init init_mac80211_hwsim(void) > } > data->dev->driver = &mac80211_hwsim_driver; > > - SET_IEEE80211_DEV(hw, data->dev); > addr[3] = i >> 8; > addr[4] = i; > memcpy(data->addresses[0].addr, addr, ETH_ALEN);