* [PATCH] Add a check for remove callback in spmi driver removal API
@ 2022-12-21 5:37 Jishnu Prakash
2022-12-21 5:37 ` [PATCH] spmi: Add a check for remove callback when removing a SPMI driver Jishnu Prakash
0 siblings, 1 reply; 3+ messages in thread
From: Jishnu Prakash @ 2022-12-21 5:37 UTC (permalink / raw)
To: gregkh, sboyd, agross, devicetree, robh+dt, linus.walleij,
quic_kamalw, quic_subbaram, quic_collinsd
Cc: linux-arm-msm, linux-kernel, linux-arm-msm-owner, Jishnu Prakash
When removing a SPMI driver, there can be a crash due to NULL pointer
dereference if it does not have a remove callback defined, as the remove
callback gets called directly in spmi_drv_remove(). This is one such call
trace observed when removing the QCOM SPMI PMIC driver:
dump_backtrace.cfi_jt+0x0/0x8
dump_stack_lvl+0xd8/0x16c
panic+0x188/0x498
__cfi_slowpath+0x0/0x214
__cfi_slowpath+0x1dc/0x214
spmi_drv_remove+0x16c/0x1e0
device_release_driver_internal+0x468/0x79c
driver_detach+0x11c/0x1a0
bus_remove_driver+0xc4/0x124
driver_unregister+0x58/0x84
cleanup_module+0x1c/0xc24 [qcom_spmi_pmic]
__do_sys_delete_module+0x3ec/0x53c
__arm64_sys_delete_module+0x18/0x28
el0_svc_common+0xdc/0x294
el0_svc+0x38/0x9c
el0_sync_handler+0x8c/0xf0
el0_sync+0x1b4/0x1c0
If a driver has all its resources allocated through devm_() APIs and
does not need any other explicit cleanup, it would not require a
remove callback to be defined. The SPMI framework also does not enforce
the presence of a remove callback when a client driver registers with it.
Hence, add a check for remove callback presence before calling it
when removing a SPMI driver.
Jishnu Prakash (1):
spmi: Add a check for remove callback when removing a SPMI driver
drivers/spmi/spmi.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
--
2.7.4
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] spmi: Add a check for remove callback when removing a SPMI driver
2022-12-21 5:37 [PATCH] Add a check for remove callback in spmi driver removal API Jishnu Prakash
@ 2022-12-21 5:37 ` Jishnu Prakash
2023-04-13 4:15 ` Stephen Boyd
0 siblings, 1 reply; 3+ messages in thread
From: Jishnu Prakash @ 2022-12-21 5:37 UTC (permalink / raw)
To: gregkh, sboyd, agross, devicetree, robh+dt, linus.walleij,
quic_kamalw, quic_subbaram, quic_collinsd, linux-kernel
Cc: linux-arm-msm, linux-arm-msm-owner, Jishnu Prakash
When removing a SPMI driver, there can be a crash due to NULL pointer
dereference if it does not have a remove callback defined. This is
one such call trace observed when removing the QCOM SPMI PMIC driver:
dump_backtrace.cfi_jt+0x0/0x8
dump_stack_lvl+0xd8/0x16c
panic+0x188/0x498
__cfi_slowpath+0x0/0x214
__cfi_slowpath+0x1dc/0x214
spmi_drv_remove+0x16c/0x1e0
device_release_driver_internal+0x468/0x79c
driver_detach+0x11c/0x1a0
bus_remove_driver+0xc4/0x124
driver_unregister+0x58/0x84
cleanup_module+0x1c/0xc24 [qcom_spmi_pmic]
__do_sys_delete_module+0x3ec/0x53c
__arm64_sys_delete_module+0x18/0x28
el0_svc_common+0xdc/0x294
el0_svc+0x38/0x9c
el0_sync_handler+0x8c/0xf0
el0_sync+0x1b4/0x1c0
If a driver has all its resources allocated through devm_() APIs and
does not need any other explicit cleanup, it would not require a
remove callback to be defined. Hence, add a check for remove callback
presence before calling it when removing a SPMI driver.
Signed-off-by: Jishnu Prakash <quic_jprakash@quicinc.com>
---
drivers/spmi/spmi.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/spmi/spmi.c b/drivers/spmi/spmi.c
index a456ce5..6b34356 100644
--- a/drivers/spmi/spmi.c
+++ b/drivers/spmi/spmi.c
@@ -350,7 +350,8 @@ static void spmi_drv_remove(struct device *dev)
const struct spmi_driver *sdrv = to_spmi_driver(dev->driver);
pm_runtime_get_sync(dev);
- sdrv->remove(to_spmi_device(dev));
+ if (sdrv->remove)
+ sdrv->remove(to_spmi_device(dev));
pm_runtime_put_noidle(dev);
pm_runtime_disable(dev);
--
2.7.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] spmi: Add a check for remove callback when removing a SPMI driver
2022-12-21 5:37 ` [PATCH] spmi: Add a check for remove callback when removing a SPMI driver Jishnu Prakash
@ 2023-04-13 4:15 ` Stephen Boyd
0 siblings, 0 replies; 3+ messages in thread
From: Stephen Boyd @ 2023-04-13 4:15 UTC (permalink / raw)
To: Jishnu Prakash, agross, devicetree, gregkh, linus.walleij,
linux-kernel, quic_collinsd, quic_kamalw, quic_subbaram, robh+dt
Cc: linux-arm-msm, linux-arm-msm-owner, Jishnu Prakash
Quoting Jishnu Prakash (2022-12-20 21:37:12)
> When removing a SPMI driver, there can be a crash due to NULL pointer
> dereference if it does not have a remove callback defined. This is
> one such call trace observed when removing the QCOM SPMI PMIC driver:
>
> dump_backtrace.cfi_jt+0x0/0x8
> dump_stack_lvl+0xd8/0x16c
> panic+0x188/0x498
> __cfi_slowpath+0x0/0x214
> __cfi_slowpath+0x1dc/0x214
> spmi_drv_remove+0x16c/0x1e0
> device_release_driver_internal+0x468/0x79c
> driver_detach+0x11c/0x1a0
> bus_remove_driver+0xc4/0x124
> driver_unregister+0x58/0x84
> cleanup_module+0x1c/0xc24 [qcom_spmi_pmic]
> __do_sys_delete_module+0x3ec/0x53c
> __arm64_sys_delete_module+0x18/0x28
> el0_svc_common+0xdc/0x294
> el0_svc+0x38/0x9c
> el0_sync_handler+0x8c/0xf0
> el0_sync+0x1b4/0x1c0
>
> If a driver has all its resources allocated through devm_() APIs and
> does not need any other explicit cleanup, it would not require a
> remove callback to be defined. Hence, add a check for remove callback
> presence before calling it when removing a SPMI driver.
>
> Signed-off-by: Jishnu Prakash <quic_jprakash@quicinc.com>
> ---
Applied to spmi-next
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-04-13 4:17 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-12-21 5:37 [PATCH] Add a check for remove callback in spmi driver removal API Jishnu Prakash
2022-12-21 5:37 ` [PATCH] spmi: Add a check for remove callback when removing a SPMI driver Jishnu Prakash
2023-04-13 4:15 ` Stephen Boyd
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).