]> www.infradead.org Git - users/hch/misc.git/commit
clk: test: Fix some memory leaks
authorJinjie Ruan <ruanjinjie@huawei.com>
Wed, 16 Oct 2024 02:26:58 +0000 (10:26 +0800)
committerStephen Boyd <sboyd@kernel.org>
Wed, 16 Oct 2024 21:39:18 +0000 (14:39 -0700)
commit6b5cca7868fdd2499384e21279fdab86bfa04997
tree90434ea7a92d345fe22b0b4fd72416bd4ecb23b5
parent23dbbe8889250bed10cdd8a328d97efd15f808a7
clk: test: Fix some memory leaks

CONFIG_CLK_KUNIT_TEST=y, CONFIG_DEBUG_KMEMLEAK=y
and CONFIG_DEBUG_KMEMLEAK_AUTO_SCAN=y, the following memory leak occurs.

If the KUNIT_ASSERT_*() fails, the latter (exit() or testcases)
clk_put() or clk_hw_unregister() will fail to release the clk resource
and cause memory leaks, use new clk_hw_register_kunit()
and clk_hw_get_clk_kunit() to automatically release them.

unreferenced object 0xffffff80c6af5000 (size 512):
  comm "kunit_try_catch", pid 371, jiffies 4294896001
  hex dump (first 32 bytes):
    20 4c c0 86 e1 ff ff ff e0 1a c0 86 e1 ff ff ff   L..............
    c0 75 e3 c6 80 ff ff ff 00 00 00 00 00 00 00 00  .u..............
  backtrace (crc 8ca788fa):
    [<00000000e21852d0>] kmemleak_alloc+0x34/0x40
    [<000000009c583f7b>] __kmalloc_cache_noprof+0x26c/0x2f4
    [<00000000d1bc850c>] __clk_register+0x80/0x1ecc
    [<00000000b08c78c5>] clk_hw_register+0xc4/0x110
    [<00000000b16d6df8>] clk_multiple_parents_mux_test_init+0x238/0x288
    [<0000000014a7e804>] kunit_try_run_case+0x10c/0x3ac
    [<0000000026b41f03>] kunit_generic_run_threadfn_adapter+0x80/0xec
    [<0000000066619fb8>] kthread+0x2e8/0x374
    [<00000000a1157f53>] ret_from_fork+0x10/0x20
unreferenced object 0xffffff80c6e37880 (size 96):
  comm "kunit_try_catch", pid 371, jiffies 4294896002
  hex dump (first 32 bytes):
    00 50 af c6 80 ff ff ff 00 00 00 00 00 00 00 00  .P..............
    00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00  ................
  backtrace (crc b4b766dd):
    [<00000000e21852d0>] kmemleak_alloc+0x34/0x40
    [<000000009c583f7b>] __kmalloc_cache_noprof+0x26c/0x2f4
    [<0000000086e7dd64>] clk_hw_create_clk.part.0.isra.0+0x58/0x2f4
    [<00000000dcf1ac31>] clk_hw_get_clk+0x8c/0x114
    [<000000006fab5bfa>] clk_test_multiple_parents_mux_set_range_set_parent_get_rate+0x3c/0xa0
    [<00000000c97db55a>] kunit_try_run_case+0x13c/0x3ac
    [<0000000026b41f03>] kunit_generic_run_threadfn_adapter+0x80/0xec
    [<0000000066619fb8>] kthread+0x2e8/0x374
    [<00000000a1157f53>] ret_from_fork+0x10/0x20
unreferenced object 0xffffff80c2b56900 (size 96):
  comm "kunit_try_catch", pid 395, jiffies 4294896107
  hex dump (first 32 bytes):
    00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00  ................
    00 00 00 00 00 00 00 00 e0 49 c0 86 e1 ff ff ff  .........I......
  backtrace (crc 2e59b327):
    [<00000000e21852d0>] kmemleak_alloc+0x34/0x40
    [<00000000c6c715a8>] __kmalloc_noprof+0x2bc/0x3c0
    [<00000000f04a7951>] __clk_register+0x70c/0x1ecc
    [<00000000b08c78c5>] clk_hw_register+0xc4/0x110
    [<00000000cafa9563>] clk_orphan_transparent_multiple_parent_mux_test_init+0x1a8/0x1dc
    [<0000000014a7e804>] kunit_try_run_case+0x10c/0x3ac
    [<0000000026b41f03>] kunit_generic_run_threadfn_adapter+0x80/0xec
    [<0000000066619fb8>] kthread+0x2e8/0x374
    [<00000000a1157f53>] ret_from_fork+0x10/0x20
unreferenced object 0xffffff80c87c9400 (size 512):
  comm "kunit_try_catch", pid 483, jiffies 4294896907
  hex dump (first 32 bytes):
    a0 44 c0 86 e1 ff ff ff e0 1a c0 86 e1 ff ff ff  .D..............
    20 05 a8 c8 80 ff ff ff 00 00 00 00 00 00 00 00   ...............
  backtrace (crc c25b43fb):
    [<00000000e21852d0>] kmemleak_alloc+0x34/0x40
    [<000000009c583f7b>] __kmalloc_cache_noprof+0x26c/0x2f4
    [<00000000d1bc850c>] __clk_register+0x80/0x1ecc
    [<00000000b08c78c5>] clk_hw_register+0xc4/0x110
    [<000000002688be48>] clk_single_parent_mux_test_init+0x1a0/0x1d4
    [<0000000014a7e804>] kunit_try_run_case+0x10c/0x3ac
    [<0000000026b41f03>] kunit_generic_run_threadfn_adapter+0x80/0xec
    [<0000000066619fb8>] kthread+0x2e8/0x374
    [<00000000a1157f53>] ret_from_fork+0x10/0x20
unreferenced object 0xffffff80c6dd2380 (size 96):
  comm "kunit_try_catch", pid 483, jiffies 4294896908
  hex dump (first 32 bytes):
    00 94 7c c8 80 ff ff ff 00 00 00 00 00 00 00 00  ..|.............
    00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00  ................
  backtrace (crc 4401212):
    [<00000000e21852d0>] kmemleak_alloc+0x34/0x40
    [<000000009c583f7b>] __kmalloc_cache_noprof+0x26c/0x2f4
    [<0000000086e7dd64>] clk_hw_create_clk.part.0.isra.0+0x58/0x2f4
    [<00000000dcf1ac31>] clk_hw_get_clk+0x8c/0x114
    [<0000000063eb2c90>] clk_test_single_parent_mux_set_range_disjoint_child_last+0x3c/0xa0
    [<00000000c97db55a>] kunit_try_run_case+0x13c/0x3ac
    [<0000000026b41f03>] kunit_generic_run_threadfn_adapter+0x80/0xec
    [<0000000066619fb8>] kthread+0x2e8/0x374
    [<00000000a1157f53>] ret_from_fork+0x10/0x20
......

Fixes: 02cdeace1e1e ("clk: tests: Add tests for single parent mux")
Fixes: 2e9cad1abc71 ("clk: tests: Add some tests for orphan with multiple parents")
Fixes: 433fb8a611ca ("clk: tests: Add missing test case for ranges")
Signed-off-by: Jinjie Ruan <ruanjinjie@huawei.com>
Link: https://lore.kernel.org/r/20241016022658.2131826-1-ruanjinjie@huawei.com
Reviewed-by: Maxime Ripard <mripard@kernel.org>
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/clk_test.c