Re: Iwlwifi 3160 AC (firmware?) clashes with CONFIG_IWLWIFI_BCAST_FILTERING

On Monday, August 25, 2014 11:20:29 PM Emmanuel Grumbach wrote:

quoted

Hi Christian,

I'm not the best person to talk about the iwlwifi firmwares (that would
be Emmanuel, but he is not available now), but I'll try.

Here I am - thanks Luca for stepping in.

Hello you both!
 

quoted

quoted

[...]
[ 1021.094192] Call Trace:
[ 1021.094205]  [<ffffffff813ce9bd>] ? dump_stack+0x41/0x51
[ 1021.094217]  [<ffffffffa0b9cd8b>] ? iwl_trans_pcie_send_hcmd+0x367/0x438 [iwlwifi]
[ 1021.094226]  [<ffffffffa0b9b52c>] ? iwl_pcie_enqueue_hcmd+0x5e3/0x5fe [iwlwifi]
[ 1021.094235]  [<ffffffff8105fd4c>] ? finish_wait+0x5d/0x5d
[ 1021.094250]  [<ffffffffa0d50106>] ? iwl_mvm_send_cmd+0xd/0x22 [iwlmvm]
[ 1021.094262]  [<ffffffffa0d50160>] ? iwl_mvm_send_cmd_pdu+0x45/0x52 [iwlmvm]
[ 1021.094273]  [<ffffffffa0d4c5fb>] ? iwl_mvm_configure_bcast_filter.isra.14.part.15+0x36/0x3f [iwlmvm]

(Full log attached below)

What cough my eye was the "BAD_COMMAND" in the error log. I disabled
"CONFIG_IWLWIFI_BCAST_FILTERING" option and the 3160 is now working.

As far as I can tell, there is already a firmware feature flag:
"IWL_UCODE_TLV_FLAGS_BCAST_FILTERING" which can be used to
distinguish whenever the feature is supported or not. But it
seems to be "stuck on" for the 3160.

Weird, but it's possible that they set the flag mistakenly for the 3160
firmware.  I'll check this with the firmware team.

[...]
Actually, we do have an internal ticket with this issue - but I don't
know if it was on 3160 or on 7260. Your report explicitly says that
you saw this on 3160, so I am only now connecting between the dots.
I can't check the internal bug since I am on vacation right now. Note
that 3160 is much less common than 7260 - so this kind of stupid bugs
can stay unseen for a fairly long time...

I have some 7260s too. As far as I can tell, they both 
(one is a 7260-N, the other a 7260-AC) support the 
BCAST_FILTERING feature.

[I'll open a bug report @ iwlwifi#Bugzilla - and
check out 25.222.9.0 too [0]]

Regards
Christian

[0] <https://git.kernel.org/cgit/linux/kernel/git/egrumbach/linux-firmware.git/commit/?id=13dfdae12b8a8018abed2224db6ad1856817ab05>