Skip to contentSkip to conversation

iwlwifi/iwldvm has a 30s delay in EAP authentication

v3.9.x stable regression, bisected Kshitij Kulshreshtha 2 participants 4 messages May 21, 2013 – May 22, 2013

lore
Share & tools
lore

Kshitij Kulshreshtha

author Reviewed-by Johannes Berg
#0 lore raw

Hello,

I am currently running the openSUSE Tumbleweed distribution and have a problem starting with the kernel v3.9.1 and also with kernel v3.9.2.

The iwldvm driver has an artificial delay of 30s when authenticating with a radius server using EAP-TLS. This was not the case using kernel v3.9.

I bisected the problem using local builds of the tumbleweed desktop configuration. This is the result:

c30f37f8f1f8d06c142d67ba74955fd48b5cf5a4 is the first bad commit commit c30f37f8f1f8d06c142d67ba74955fd48b5cf5a4

Author: Emmanuel Grumbach <[email protected]>
Date:   Wed Apr 17 09:47:00 2013 +0300

iwlwifi: dvm: don't send zeroed LQ cmd

commit 63b77bf489881747c5118476918cc8c29378ee63 upstream.

When the stations are being restored because of unassoc RXON, the LQ cmd may not have been initialized because it is initialized only after association.
Sending zeroed LQ_CMD makes the fw unhappy: it raises SYSASSERT_2078.

:040000 040000 fb6795b349a1765f357df8fcdf309e6e2613230b
5e542c8aef84287968515803efd193462c4a0d6b M drivers

Reverting the above commit on top of kernel v3.9.3 fixes this 30s delay but I've never seen a firmware restart of the iwldvm/iwlwifi driver in my logs that would suggest that the SYSASSERT mentioned in the commit message is happening. My hardware is Intel Centrino Advanced-N 6200 (rev
35) (pci_id 8086:422c).

I'll be happy to send any further information if required (Please CC me in replies).

Thanks and regards.
Kshitij

The full log of the bisection is as follows:

[kulshres@apollo 3.9.y]$ git bisect log
# bad: [57049bb1dd0461d8423c3feceea36148d4335317] Linux 3.9.2 # good: [c1be5a5b1b355d40e6cf79cc979eb66dafa24ad1] Linux 3.9 git bisect start 'HEAD' 'v3.9'
# bad: [50f3a76bf2179283bba925a84839f23fcb42cd92] wireless: regulatory:
fix channel disabling race condition
git bisect bad 50f3a76bf2179283bba925a84839f23fcb42cd92
# bad: [5504e82764f41969a5f4e29af9f950f79ec3538a] crypto: crc32-pclmul - Use gas macro for pclmulqdq
git bisect bad 5504e82764f41969a5f4e29af9f950f79ec3538a
# bad: [ad3b8a5451b342d58c16806dc6c0f750973c169b] USB: ftdi_sio: correct ST Micro Connect Lite PIDs
git bisect bad ad3b8a5451b342d58c16806dc6c0f750973c169b
# good: [efaa9d79ccb5491181ea100dc6a9c575f626ef5a] powerpc: Add isync to copy_and_flush
git bisect good efaa9d79ccb5491181ea100dc6a9c575f626ef5a
# bad: [af1f921702d4074dd61da0d4a2974d51ecd9c05d] mwifiex: Use pci_release_region() instead of a pci_release_regions()
git bisect bad af1f921702d4074dd61da0d4a2974d51ecd9c05d
# good: [1a36af179e195ab85e1444cc760fd7a620289fa6] powerpc/spufs:
Initialise inode->i_ino in spufs_new_inode()
git bisect good 1a36af179e195ab85e1444cc760fd7a620289fa6
# bad: [c30f37f8f1f8d06c142d67ba74955fd48b5cf5a4] iwlwifi: dvm: don't send zeroed LQ cmd
git bisect bad c30f37f8f1f8d06c142d67ba74955fd48b5cf5a4
# good: [34e3e78befc3c6f65f3e7ce70f0367745cd550b0] iwlwifi: fix freeing uninitialized pointer
git bisect good 34e3e78befc3c6f65f3e7ce70f0367745cd550b0

Signature · 12 lines
Dr. Kshitij Kulshreshtha

Institut für Mathematik,
Universität Paderborn,
Warburger Straße 100,
33098 Paderborn.

Büro: A3.235

Privatanschrift:
Arnikaweg 62
33100 Paderborn.
#1 lore raw
Quoted 27 lines · Kshitij Kulshreshtha

Hello,

I am currently running the openSUSE Tumbleweed distribution and have a problem starting with the kernel v3.9.1 and also with kernel v3.9.2.

The iwldvm driver has an artificial delay of 30s when authenticating with a radius server using EAP-TLS. This was not the case using kernel v3.9.

I bisected the problem using local builds of the tumbleweed desktop configuration. This is the result:

c30f37f8f1f8d06c142d67ba74955fd48b5cf5a4 is the first bad commit commit c30f37f8f1f8d06c142d67ba74955fd48b5cf5a4

Author: Emmanuel Grumbach <[email protected]>
Date:   Wed Apr 17 09:47:00 2013 +0300

iwlwifi: dvm: don't send zeroed LQ cmd

commit 63b77bf489881747c5118476918cc8c29378ee63 upstream.

When the stations are being restored because of unassoc RXON, the LQ cmd may not have been initialized because it is initialized only after association.
Sending zeroed LQ_CMD makes the fw unhappy: it raises SYSASSERT_2078.

Quoted 6 lines

Thanks for the bisection.
Can you please try the patch in
https://bugzilla.kernel.org/show_bug.cgi?id=58341 and report? Thanks

#2 lore raw

As on 2013-05-22 10:17h, Emmanuel Grumbach did write:

Quoted 5 lines

Thanks for the bisection. Can you please try the patch in
https://bugzilla.kernel.org/show_bug.cgi?id=58341 and report? Thanks

I can confirm that the patch in this bugzilla entry does indeed get rid of the delay in EAP authentication.

Thanks.
Kshitij

Signature · 12 lines
Dr. Kshitij Kulshreshtha

Institut für Mathematik,
Universität Paderborn,
Warburger Straße 100,
33098 Paderborn.

Büro: A3.235

Privatanschrift:
Arnikaweg 62
33100 Paderborn.
#3 lore raw
Quoted 9 lines
Quoted 5 lines

Thanks for the bisection. Can you please try the patch in
https://bugzilla.kernel.org/show_bug.cgi?id=58341 and report? Thanks

I can confirm that the patch in this bugzilla entry does indeed get rid of the delay in EAP authentication.

Thanks for the testing - patch on the way to upstream / stable