Re: kmemleaks on ac3b43283923 ("module: replace module_layout with module_memory")

From: jim . cromie
Date: Fri Mar 31 2023 - 13:08:55 EST


On Fri, Mar 31, 2023 at 1:00 AM Luis Chamberlain <mcgrof@xxxxxxxxxx> wrote:
>
> On Thu, Mar 30, 2023 at 04:45:43PM -0600, jim.cromie@xxxxxxxxx wrote:
> > hi Luis, etal
> >
> > kmemleak is reporting 19 leaks during boot
> >
> > because the hexdumps appeared to have module-names,
> > and Ive been hacking nearby, and see the same names
> > every time I boot my test-vm, I needed a clearer picture
> > Jason corroborated and bisected.
> >
> > the 19 leaks split into 2 groups,
> > 9 with names of builtin modules in the hexdump,
> > all with the same backtrace
> > 9 without module-names (with a shared backtrace)
> > +1 wo name-ish and a separate backtrace
>
> Song, please take a look.
>
> Thanks for the report Jim, what kernel are you on exactly?
>
> Luis

:#> uptime
09:45:32 up 1 day, 23:07, 0 users, load average: 0.07, 0.04, 0.01
:#> uname -a
Linux (none) 6.3.0-rc1-f2-00001-gac3b43283923 #359 SMP PREEMPT_DYNAMIC
Wed Mar 29 09:33:11 MDT 2023 x86_64 x86_64 x86_64 GNU/Linux

the leaks I sent previously might be from/on a different commit,
heres the relevant one

fwiw, the config is unremarkable. it started with
CONFIG_BUILD_SALT="5.16.8-200.fc35.x86_64"
then `make localmodconfig` to drop anything I dont have hw for
then `virtme-configkernel --update` to pick up the 9p,etc config options
And some extra DEBUG_* options
If you'd like to see runs with others, or see the config itself, please ask.

:#> uname -a
Linux (none) 6.3.0-rc1-f2-00001-gac3b43283923 #359 SMP PREEMPT_DYNAMIC
Wed Mar 29 09:33:11 MDT 2023 x86_64 x86_64 x86_64 GNU/Linux
:#> ./grok_kmemleak -n
not: bless( {
'backtraces' => {
'[<0000000058fb276d>] __kmalloc_node_track_caller+0x4a/0x140
[<00000000a2f80203>] memdup_user+0x26/0x90
[<00000000f7cd3624>] strndup_user+0x3f/0x60
[<0000000098fd26c5>] load_module+0x188b/0x20e0
[<0000000074361279>] __do_sys_finit_module+0x93/0xf0
[<000000004caeb948>] do_syscall_64+0x34/0x80
[<000000009f5d036c>] entry_SYSCALL_64_after_hwframe+0x46/0xb0' => 16,
'[<0000000094c136c3>] kmalloc_trace+0x26/0x90
[<00000000700fd414>] resolve_symbol+0x2a5/0x3a0
[<000000001dd9228b>] load_module+0x1465/0x20e0
[<0000000074361279>] __do_sys_finit_module+0x93/0xf0
[<000000004caeb948>] do_syscall_64+0x34/0x80
[<000000009f5d036c>] entry_SYSCALL_64_after_hwframe+0x46/0xb0' => 3
},
'hexdumps' => {
'00 b3 af 5a de 87 df 17 ...Z....' => 1,
'00 b6 af 5a de 87 d8 cf ...Z....' => 1,
'00 b7 af 5a de 87 d5 77 ...Z...w' => 1,
'00 c8 b9 5a de 91 a2 2f ...Z.../' => 1,
'00 ca b9 5a de 91 a6 3f ...Z...?' => 1,
'00 cf b9 5a de 91 a2 07 ...Z....' => 1,
'00 e0 4c 56 d2 64 8b 77 ..LV.d.w' => 1,
'00 e0 f2 59 dd da 85 7f ...Y....' => 1,
'00 e3 4c 56 d2 64 89 1f ..LV.d..' => 1,
'00 e4 f2 59 dd da 8f 4f ...Y...O' => 1,
'00 e5 4c 56 d2 64 87 bf ..LV.d..' => 1,
'00 e6 f2 59 dd da 89 57 ...Y...W' => 1,
'00 e8 f2 59 dd da 83 17 ...Y....' => 1,
'00 e9 4c 56 d2 64 8e df ..LV.d..' => 1,
'00 eb 4c 56 d2 64 80 67 ..LV.d.g' => 1,
'00 ec 4c 56 d2 64 8f 7f ..LV.d..' => 1,
'40 d4 1c 08 80 88 ff ff 88 99 37 c0 ff ff ff ff @.........7.....' => 1,
'88 99 37 c0 ff ff ff ff 40 09 e8 13 80 88 ff ff ..7.....@.......' => 1,
'c8 8a 23 c0 ff ff ff ff c8 8a 23 c0 ff ff ff ff ..#.......#.....' => 1
},
'users' => {
'comm "(udev-worker)", pid 219,' => 1,
'comm "(udev-worker)", pid 221,' => 4,
'comm "(udev-worker)", pid 224,' => 3,
'comm "(udev-worker)", pid 229,' => 1,
'comm "(udev-worker)", pid 230,' => 1,
'comm "modprobe", pid 728,' => 1,
'comm "modprobe", pid 814,' => 1,
'comm "modprobe", pid 825,' => 4,
'comm "modprobe", pid 832,' => 1,
'comm "modprobe", pid 835,' => 2
}
}, 'LeakSet' )
mods: bless( {
'backtraces' => {
'[<0000000058fb276d>] __kmalloc_node_track_caller+0x4a/0x140
[<00000000ab7b01fd>] kstrdup+0x32/0x60
[<000000005ed25b98>] kobject_set_name_vargs+0x1c/0x90
[<0000000090fe19ca>] kobject_init_and_add+0x4d/0x90
[<0000000045666935>] mod_sysfs_setup+0xa9/0x6e0
[<00000000d6f7187b>] load_module+0x1de3/0x20e0
[<0000000074361279>] __do_sys_finit_module+0x93/0xf0
[<000000004caeb948>] do_syscall_64+0x34/0x80
[<000000009f5d036c>] entry_SYSCALL_64_after_hwframe+0x46/0xb0' => 16
},
'hexdumps' => {
'63 65 63 00 d2 64 80 7f cec..d..' => 1,
'63 72 63 33 32 5f 70 63 6c 6d 75 6c 00 24 14 48 crc32_pclmul.$.H' => 1,
'63 72 63 33 32 63 5f 69 6e 74 65 6c 00 a7 e0 f8 crc32c_intel....' => 1,
'63 72 63 74 31 30 64 69 66 5f 70 63 6c 6d 75 6c crct10dif_pclmul' => 1,
'67 68 61 73 68 5f 63 6c 6d 75 6c 6e 69 5f 69 6e ghash_clmulni_in' => 1,
'69 32 63 5f 61 6c 67 6f 5f 62 69 74 00 c4 b6 08 i2c_algo_bit....' => 1,
'69 32 63 5f 70 69 69 78 34 00 cb 8a 66 a7 e2 48 i2c_piix4...f..H' => 1,
'69 6e 74 65 6c 5f 72 61 70 6c 5f 63 6f 6d 6d 6f intel_rapl_commo' => 1,
'69 6e 74 65 6c 5f 72 61 70 6c 5f 6d 73 72 00 98 intel_rapl_msr..' => 1,
'69 6f 6d 6d 75 5f 76 32 00 70 a8 80 6c c4 bd 08 iommu_v2.p..l...' => 1,
'6d 78 6d 5f 77 6d 69 00 mxm_wmi.' => 1,
'70 63 73 70 6b 72 00 8f pcspkr..' => 1,
'73 65 72 69 6f 5f 72 61 77 00 cb 8a 66 a7 ed b8 serio_raw...f...' => 1,
'74 65 73 74 5f 64 79 6e 61 6d 69 63 5f 64 65 62 test_dynamic_deb' => 1,
'76 69 64 65 6f 00 d9 bf video...' => 1,
'77 6d 69 00 dd da 80 df wmi.....' => 1
},
'users' => {
'comm "(udev-worker)", pid 219,' => 1,
'comm "(udev-worker)", pid 221,' => 4,
'comm "(udev-worker)", pid 224,' => 2,
'comm "(udev-worker)", pid 229,' => 1,
'comm "(udev-worker)", pid 230,' => 1,
'comm "modprobe", pid 728,' => 1,
'comm "modprobe", pid 814,' => 1,
'comm "modprobe", pid 825,' => 3,
'comm "modprobe", pid 832,' => 1,
'comm "modprobe", pid 835,' => 1
}
}, 'LeakSet' )
:#> lsmod
Module Size Used by
mxm_wmi 12288 0
iommu_v2 20480 0
video 65536 0
i2c_algo_bit 12288 0
wmi 32768 2 video,mxm_wmi
cec 57344 0
test_dynamic_debug 20480 0
intel_rapl_msr 16384 0
crc32_pclmul 12288 0
intel_rapl_common 28672 1 intel_rapl_msr
ghash_clmulni_intel 12288 0
crct10dif_pclmul 12288 1
crc32c_intel 20480 0
serio_raw 16384 0
pcspkr 12288 0
i2c_piix4 28672 0
:#>