Return-path: Received: from mx2.redhat.com ([66.187.237.31]:40260 "EHLO mx2.redhat.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1752091AbZHSQjg (ORCPT ); Wed, 19 Aug 2009 12:39:36 -0400 Subject: Re: question about ieee80211_tx.c (fwd) From: Dan Williams To: Greg KH Cc: Zhu Yi , =?ISO-8859-1?Q?G=E1bor?= Stefanik , Julia Lawall , "linux-wireless@vger.kernel.org" In-Reply-To: <20090819163328.GA3057@suse.de> References: <69e28c910908170450l65d493dbnb1d2e8f14d29dadc@mail.gmail.com> <1250558021.9651.35.camel@debian> <20090818013502.GD20566@suse.de> <1250699528.5351.18.camel@localhost.localdomain> <20090819163328.GA3057@suse.de> Content-Type: text/plain; charset="UTF-8" Date: Wed, 19 Aug 2009 11:39:36 -0500 Message-Id: <1250699976.5351.21.camel@localhost.localdomain> Mime-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Wed, 2009-08-19 at 09:33 -0700, Greg KH wrote: > On Wed, Aug 19, 2009 at 11:32:08AM -0500, Dan Williams wrote: > > On Mon, 2009-08-17 at 18:35 -0700, Greg KH wrote: > > > On Tue, Aug 18, 2009 at 09:13:41AM +0800, Zhu Yi wrote: > > > > On Mon, 2009-08-17 at 19:50 +0800, Gábor Stefanik wrote: > > > > > On Mon, Aug 17, 2009 at 10:03 AM, Julia Lawall wrote: > > > > > > The files ieee80211_tx.c in the directories > > > > > > drivers/staging/rtl8192su/ieee80211 and > > > > > drivers/staging/rtl8192e/ieee80211 > > > > > > both contain the following code in the function > > > > > > ieee80211_query_protectionmode: > > > > > > > > > > > > if (ieee->mode == IW_MODE_MASTER) > > > > > > goto NO_PROTECTION; > > > > > > > > > > > > Elsewhere in these files when there is a comparison against an > > > > > IW_MODE > > > > > > value, the field that is compared is iw_mode. Should that be the > > > > > case > > > > > > here as well? > > > > > > > > > > > > thanks, > > > > > > julia > > > > > > > > > > I don't know; rtl8192su & rtl8192e use a modified version of the > > > > > libipw stack, rather than the mac80211 stack found behind non-staging > > > > > drivers. > > > > > > > > This is clearly a bug. The question is: who is maintaining this driver > > > > and why it doesn't use/extend ieee80211/libipw? > > > > > > I maintain it, and I am working to convert it to use the existing > > > ieee80211 stack that is in the kernel. Remember, these are drivers in > > > the staging tree, they are usually "crap" :) > > > > When you say "ieee80211" you mean mac80211, right? > > Sorry, yes, whatever the wireless stack in the main kernel is called > these days :) We used to have the ieee80211 that rtl* forked its internal ieee80211 off of, but we just successfully made that code private to ipw2x00 (now called "libipw") earlier this year. Wanted to make sure any effort was correctly targeting mac80211 instead of libipw. Dan