Re: [PATCH] panic: remove the unneeded panic_print_get()

[Bradley Morgan]

On 2 September 2026 12:48:51 BST, Feng Tang <feng.tang@xxxxxxxxxxxxxxxxx>
wrote:

Feng!

>panic_print_get() was introduced in commit 2683df6539cb ("panic: add
>note that 'panic_print' parameter is deprecated") to print out warning
>message of the deprecation of 'panic_print' on read access.
>
>Since commit 90f3c123247e ("panic: only warn about deprecated panic_print
>on write access"), panic_print_get() wrapper is not needed anymore for
>read access, so remove it and use param_get_ulong() instead.
>

could you say that this is only a cleanup?

Reviewed-by: Bradley Morgan <brads@xxxxxxxxxxxxxx>


>Signed-off-by: Feng Tang <feng.tang@xxxxxxxxxxxxxxxxx>
>---
> kernel/panic.c | 7 +------
> 1 file changed, 1 insertion(+), 6 deletions(-)
>
>diff --git a/kernel/panic.c b/kernel/panic.c
>index 7dda841c16f9..50715f14cf04 100644
>--- a/kernel/panic.c
>+++ b/kernel/panic.c
>@@ -1216,14 +1216,9 @@ static int panic_print_set(const char *val, const struct kernel_param *kp)
> 	return  param_set_ulong(val, kp);
> }
> 
>-static int panic_print_get(char *val, const struct kernel_param *kp)
>-{
>-	return  param_get_ulong(val, kp);
>-}
>-
> static const struct kernel_param_ops panic_print_ops = {
> 	.set	= panic_print_set,
>-	.get	= panic_print_get,
>+	.get	= param_get_ulong,

Ack.


> };
> __core_param_cb(panic_print, &panic_print_ops, &panic_print, 0644);
> 
>

--- Thanks!
https://lore.kernel.org/all/EE579805-42F2-4C58-B752-F28779EEB717@xxxxxxxxx/