Re: [PATCH v2 00/17] mt76 patches 2018-08-24 v2

[Stanislaw Gruszka <sgruszka-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>]

On Wed, Aug 29, 2018 at 06:10:01PM +0000, Sid Hayn wrote:
> I rebuilt wireless-testing (which updated today)  with
> CONFIG_DEBUG_KMEMLEAK=y.  I am still able to replicate the issue, and
> presently have 4 devices in the "No space left on device" state.
> 
> This is from /sys/kernel/debug/kmemleak:
> 
> unreferenced object 0xffff9183f9a4d000 (size 2048):
>   comm "iwconfig", pid 14872, jiffies 4295540797 (age 1233.338s)
>   hex dump (first 32 bytes):
>     46 00 00 00 00 00 00 00 05 00 00 00 00 00 00 00  F...............
>     12 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00  ................
>   backtrace:
>     [<000000003ec4c8c4>] sta_set_sinfo+0x5d2/0x890 [mac80211]
>     [<000000008bbf0699>] ieee80211_get_station+0x4b/0x70 [mac80211]
>     [<0000000030cbddbc>] cfg80211_wext_giwrate+0xdb/0x140 [cfg80211]
>     [<000000001e9277be>] ioctl_standard_call+0x49/0xd0
>     [<00000000a0eeae49>] wext_handle_ioctl+0xbe/0x120
>     [<00000000832bf9a4>] sock_ioctl+0x164/0x360
>     [<00000000d3578d89>] do_vfs_ioctl+0xa3/0x6c0
>     [<0000000036a4185e>] ksys_ioctl+0x6b/0x80
>     [<00000000e8443423>] __x64_sys_ioctl+0x11/0x20
>     [<00000000e2ddce89>] do_syscall_64+0x50/0xf0
>     [<000000005d6a8051>] entry_SYSCALL_64_after_hwframe+0x44/0xa9
>     [<00000000e724c32b>] 0xffffffffffffffff

I do not see what can leak here. Did you run clear and scan commands,
i.e:

echo clear > /sys/kernel/debug/kmemleak
Run test
echo scan > /sys/kernel/debug/kmemleak
cat /sys/kernel/debug/kmemleak

to get results ? I forgot that this is required (full docs can
be found here: 
https://www.kernel.org/doc/html/v4.16/dev-tools/kmemleak.html).

> None of my scripts directly use iwconfig, but it is possible that
> wpa_supplicant or dhcpcd invoke it (although a grep of their source
> code indicates they do not). In case it matters, this is what my
> wpa_supplicant invokation looks like:
> 
> wpa_supplicant -Dnl80211 -i ${interface} -c test_config/${conffile}
> 
> I am leaving the system in this state for now, I can resume from
> broken or reboot to working for whatever testing you suggest next.
> This is a test rig, so it takes a few but I'm happy to rebuild
> whatever you want however you want it to debug this.

You can move /usr/sbin/iwconfig somewhere and see what will fail,
or remove WEXT support from kernel, perhaps that itself will solve
the problem ;-)

Regards
Stanislaw