Re: bss_info_change_notify warning (hacked 3.3.0-rc5+)

[Ben Greear <greearb@candelatech.com>]

On 03/28/2012 11:38 PM, Johannes Berg wrote:
> On Wed, 2012-03-28 at 12:45 -0700, Ben Greear wrote:
>> On 03/28/2012 12:13 PM, Johannes Berg wrote:
>>> On Wed, 2012-03-28 at 12:10 -0700, Ben Greear wrote:
>>>> On 03/28/2012 11:59 AM, Johannes Berg wrote:
>>
>>>>> Hm. Maybe this is the problem?
>>>>>
>>>>> http://p.sipsolutions.net/d432de678ae3ff17.txt
>>>>
>>>> I don't see any code that matches that in 3.3.0 (nothing with nl80211_start_ap, for instance)
>>>
>>> ah, well, it's set_beacon there or so
>>>
>>> johannes
>>
>>
>> I tried this patch, but it still spews (see below)
>
>> WARNING: at /home/greearb/git/linux-3.3.dev.y/net/mac80211/driver-ops.h:12 check_sdata_in_driver+0x26/0x2b [mac80211]()
>
>>    [<c04263d1>] warn_slowpath_common+0x65/0x7a
>>    [<f96a28d1>] ? check_sdata_in_driver+0x26/0x2b [mac80211]
>>    [<c042644a>] warn_slowpath_fmt+0x26/0x2a
>>    [<f96a28d1>] check_sdata_in_driver+0x26/0x2b [mac80211]
>>    [<f96a3438>] ieee80211_set_txq_params+0x97/0xec [mac80211]
>
> but now it's coming from elsewhere, see this patch?
>
> http://p.sipsolutions.net/b8506c9d7ce8a119.txt

That fixes the problem for me, or at least the easily-reproducible part.

I did have to modify the patch to work on 3.3.0 since it has no start_ap
and the beacon method is named a bit differently.

I'm going to apply this to my patch set and will let you know if
I see anything else strange.

Thanks!
Ben

>
> johannes
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html


-- 
Ben Greear <greearb@candelatech.com>
Candela Technologies Inc  http://www.candelatech.com