Kevin Hao
e5c6b312ce
cpufreq: schedutil: Use kobject release() method to free sugov_tunables
The struct sugov_tunables is protected by the kobject, so we can't free
it directly. Otherwise we would get a call trace like this:
ODEBUG: free active (active state 0) object type: timer_list hint: delayed_work_timer_fn+0x0/0x30
WARNING: CPU: 3 PID: 720 at lib/debugobjects.c:505 debug_print_object+0xb8/0x100
Modules linked in:
CPU: 3 PID: 720 Comm: a.sh Tainted: G W 5.14.0-rc1-next-20210715-yocto-standard+ #507
Hardware name: Marvell OcteonTX CN96XX board (DT)
pstate: 40400009 (nZcv daif +PAN -UAO -TCO BTYPE=--)
pc : debug_print_object+0xb8/0x100
lr : debug_print_object+0xb8/0x100
sp : ffff80001ecaf910
x29: ffff80001ecaf910 x28: ffff00011b10b8d0 x27: ffff800011043d80
x26: ffff00011a8f0000 x25: ffff800013cb3ff0 x24: 0000000000000000
x23: ffff80001142aa68 x22: ffff800011043d80 x21: ffff00010de46f20
x20: ffff800013c0c520 x19: ffff800011d8f5b0 x18: 0000000000000010
x17: 6e6968207473696c x16: 5f72656d6974203a x15: 6570797420746365
x14: 6a626f2029302065 x13: 303378302f307830 x12: 2b6e665f72656d69
x11: ffff8000124b1560 x10: ffff800012331520 x9 : ffff8000100ca6b0
x8 : 000000000017ffe8 x7 : c0000000fffeffff x6 : 0000000000000001
x5 : ffff800011d8c000 x4 : ffff800011d8c740 x3 : 0000000000000000
x2 : ffff0001108301c0 x1 : ab3c90eedf9c0f00 x0 : 0000000000000000
Call trace:
debug_print_object+0xb8/0x100
__debug_check_no_obj_freed+0x1c0/0x230
debug_check_no_obj_freed+0x20/0x88
slab_free_freelist_hook+0x154/0x1c8
kfree+0x114/0x5d0
sugov_exit+0xbc/0xc0
cpufreq_exit_governor+0x44/0x90
cpufreq_set_policy+0x268/0x4a8
store_scaling_governor+0xe0/0x128
store+0xc0/0xf0
sysfs_kf_write+0x54/0x80
kernfs_fop_write_iter+0x128/0x1c0
new_sync_write+0xf0/0x190
vfs_write+0x2d4/0x478
ksys_write+0x74/0x100
__arm64_sys_write+0x24/0x30
invoke_syscall.constprop.0+0x54/0xe0
do_el0_svc+0x64/0x158
el0_svc+0x2c/0xb0
el0t_64_sync_handler+0xb0/0xb8
el0t_64_sync+0x198/0x19c
irq event stamp: 5518
hardirqs last enabled at (5517): [<ffff8000100cbd7c>] console_unlock+0x554/0x6c8
hardirqs last disabled at (5518): [<ffff800010fc0638>] el1_dbg+0x28/0xa0
softirqs last enabled at (5504): [<ffff8000100106e0>] __do_softirq+0x4d0/0x6c0
softirqs last disabled at (5483): [<ffff800010049548>] irq_exit+0x1b0/0x1b8
So split the original sugov_tunables_free() into two functions,
sugov_clear_global_tunables() is just used to clear the global_tunables
and the new sugov_tunables_free() is used as kobj_type::release to
release the sugov_tunables safely.
Fixes: 9bdcb44e39 ("cpufreq: schedutil: New governor based on scheduler utilization data")
Cc: 4.7+ <stable@vger.kernel.org> # 4.7+
Signed-off-by: Kevin Hao <haokexin@gmail.com>
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
2021-08-06 15:34:55 +02:00
..
2021-07-29 00:27:52 +02:00
2021-07-21 06:39:20 -10:00
2021-05-07 00:26:34 -07:00
2021-07-12 11:03:35 -05:00
2021-07-16 11:30:26 +02:00
2021-05-31 10:14:49 +02:00
2021-06-29 17:29:11 -07:00
2021-06-22 11:07:18 -07:00
2021-07-09 15:35:13 +02:00
2021-07-04 12:29:16 -07:00
2021-04-27 18:14:38 -07:00
2021-07-11 11:06:09 -07:00
2021-07-08 11:48:21 -07:00
2021-06-29 12:07:18 -07:00
2021-07-06 15:53:12 -07:00
2021-08-06 15:34:55 +02:00
2021-07-15 01:23:54 +02:00
2021-07-23 08:46:02 -04:00
2021-05-02 00:43:35 +09:00
2020-12-15 22:46:15 -08:00
2021-05-07 00:26:33 -07:00
2021-02-23 10:25:27 -05:00
2021-06-08 22:18:35 -04:00
2020-12-03 14:58:35 +01:00
2021-03-22 15:23:32 -04:00
2021-06-10 20:59:05 -04:00
2021-03-22 15:23:32 -04:00
2021-06-10 20:59:05 -04:00
2021-01-24 14:27:16 +01:00
2021-04-08 16:04:20 -07:00
2021-06-29 12:23:02 -07:00
2021-07-08 11:48:22 -07:00
2021-06-28 20:39:26 -07:00
2021-05-12 11:43:25 +02:00
2021-06-28 20:39:26 -07:00
2020-12-15 22:46:19 -08:00
2021-06-29 17:29:11 -07:00
2021-06-18 11:43:08 +02:00
2021-06-28 11:45:29 -07:00
2021-05-27 04:01:50 +09:00
2021-02-26 09:41:03 -08:00
2021-07-02 12:08:10 -07:00
2021-06-10 10:00:08 +02:00
2021-07-05 10:46:20 +02:00
2021-07-08 11:48:22 -07:00
2020-12-15 19:36:48 -08:00
2021-06-28 22:43:05 +02:00
2020-11-02 18:00:20 -08:00
2021-07-01 11:06:04 -07:00
2021-05-07 00:26:32 -07:00
2021-02-22 12:33:26 +00:00
2021-05-07 00:26:33 -07:00
2021-07-11 11:06:09 -07:00
2021-06-29 17:29:11 -07:00
2021-05-02 00:43:35 +09:00
2021-01-19 10:24:45 +01:00
2021-01-19 10:24:45 +01:00
2021-07-08 11:48:22 -07:00
2020-12-14 16:40:27 -08:00
2021-07-01 11:06:04 -07:00
2020-12-17 13:01:31 -08:00
2020-12-14 16:40:27 -08:00
2020-12-15 19:36:48 -08:00
2021-04-10 13:35:54 +02:00
2021-06-18 11:43:09 +02:00
2021-06-21 13:08:36 +01:00
2020-12-15 22:46:18 -08:00
2020-11-25 18:52:35 +01:00
2021-05-14 19:41:32 -07:00
2021-04-14 18:04:09 +02:00
2021-07-06 12:37:55 -07:00
2020-12-01 10:30:28 +00:00
2021-06-28 12:49:52 -07:00
2021-07-08 11:43:24 -07:00
2021-05-06 15:33:49 +02:00
2021-07-25 11:06:37 -07:00
2021-06-18 11:43:07 +02:00
2021-07-05 10:46:33 +02:00
2021-03-23 16:01:58 +01:00
2021-07-08 11:48:21 -07:00
2021-06-28 20:39:26 -07:00
2021-06-23 16:41:24 -06:00
2021-07-02 12:08:10 -07:00
2021-04-30 11:20:42 -07:00
2020-11-16 08:08:54 -08:00
2021-03-08 14:22:28 -08:00
2021-07-23 08:46:22 -04:00
2021-06-28 20:39:26 -07:00
2021-05-07 00:26:34 -07:00
2021-05-09 13:07:03 -07:00
2021-06-28 20:39:26 -07:00
2021-04-30 14:14:02 -05:00
2021-07-03 11:41:14 -07:00
2021-01-26 11:16:34 +00:00
2021-06-29 10:53:46 -07:00
2021-07-21 06:42:31 -10:00