MIME-Version: 1.0
In-Reply-To: <1339137689.5832.2.camel@jlt3.sipsolutions.net>
References: <CAHsu+b_wQOuhbBkx25CZ2cai5VVos7zK3s=EiPBy_+OnXWiWaA@mail.gmail.com>
	<1339014959.4523.0.camel@jlt3.sipsolutions.net>
	<CAHsu+b-ByqBaci9W-oS3OPEK3LdRcuGphtQPOTQ02hPWNpWgQw@mail.gmail.com>
	<1339053364.4517.0.camel@jlt3.sipsolutions.net>
	<CAHsu+b9UjkF6VM_YjJkHVDQzrZqsWqmG-XJKFNeSCdWGbdEngw@mail.gmail.com>
	<20120607180325.7752b3c0@milhouse>
	<1339094215.4630.7.camel@jlt3.sipsolutions.net>
	<1339102567.4630.8.camel@jlt3.sipsolutions.net>
	<4FD11B47.10002@lwfinger.net>
	<CAHsu+b9=4i1oSni1onKAKqH4FZ26nye=7GifnnJO-6c5gCaWpQ@mail.gmail.com>
	<1339136879.5832.1.camel@jlt3.sipsolutions.net>
	<1339137689.5832.2.camel@jlt3.sipsolutions.net>
Date: Fri, 8 Jun 2012 15:49:45 +0200
Message-ID: <CAHsu+b9uyohfFG46R7QHht6tD3MVUzk4_LJeXp+CC2bRv=NihA@mail.gmail.com> (sfid-20120608_154952_254390_11D5B88A)
Subject: Re: chip id 4318 regression: WARN_ON_ONCE(sdata->vif.hw_queue[i] >= n_queues))
From: Andre Heider <a.heider@gmail.com>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: Larry Finger <Larry.Finger@lwfinger.net>,
	=?ISO-8859-1?Q?Michael_B=FCsch?= <m@bues.ch>,
	linux-wireless@vger.kernel.org, b43-dev@lists.infradead.org,
	=?ISO-8859-2?Q?Rafa=B3_Mi=B3ecki?= <zajec5@gmail.com>
Content-Type: text/plain; charset=ISO-8859-1
Sender: linux-wireless-owner@vger.kernel.org

On Fri, Jun 8, 2012 at 8:41 AM, Johannes Berg <johannes@sipsolutions.net> wrote:
> On Fri, 2012-06-08 at 08:27 +0200, Johannes Berg wrote:
>> On Thu, 2012-06-07 at 23:47 +0200, Andre Heider wrote:
>> > On Thu, Jun 7, 2012 at 11:21 PM, Larry Finger <Larry.Finger@lwfinger.net> wrote:
>> > > On 06/07/2012 03:56 PM, Johannes Berg wrote:
>> > >>
>> > >> On Thu, 2012-06-07 at 20:36 +0200, Johannes Berg wrote:
>> > >>
>> > >>> However, it's also possible to actually *use* the new mac80211 feature.
>> > >>> That would also help with IEEE80211_TX_CTL_SEND_AFTER_DTIM, you could
>> > >>> ask mac80211 to use a separate HW queue for it (for tx_ring_mcast.) This
>> > >>> may be quicker.
>> > >>
>> > >>
>> > >> This might do it, I haven't tested it at all:
>> > >>
>> > >> http://p.sipsolutions.net/3c6b003bb06e5298.txt
>> > >
>> > >
>> > > With that patch, the driver never loads the firmware. I have not yet found
>> > > the reason.
>> >
>> > Same here.
>>
>> It doesn't start any interfaces, right?
>>
>> The patch was incomplete -- it has to register all 5 queues not but
>> might not use them, please try this:
>> http://p.sipsolutions.net/69a78bbb83cd6421.txt
>
> No, this was wrong too, I forgot to correct the condition in
> add_interface, this should be right:
>
> http://p.sipsolutions.net/854d460da251f9d6.txt

Sorry, still not loading any firmware.