From d1c3a3f1c9a1dfc2b41696b7903972f4b3fbcd02 Mon Sep 17 00:00:00 2001 From: Cyrill Gorcunov Date: Sun, 16 Mar 2025 22:58:15 +0300 Subject: [PATCH] posix-timers: Drop redundant memset() invocation Initially in commit 6891c4509c79 memset() was required to clear a variable allocated on stack. Commit 2482097c6c0f removed the on stack variable and retained the memset() despite the fact that the memory is allocated via kmem_cache_zalloc() and therefore zereoed already. Drop the redundant memset(). Signed-off-by: Cyrill Gorcunov Signed-off-by: Thomas Gleixner Link: https://lore.kernel.org/all/Z9ctVxwaYOV4A2g4@grain --- kernel/time/posix-timers.c | 1 - 1 file changed, 1 deletion(-) diff --git a/kernel/time/posix-timers.c b/kernel/time/posix-timers.c index 2ca1c55d38a2..bc0bdf40eb50 100644 --- a/kernel/time/posix-timers.c +++ b/kernel/time/posix-timers.c @@ -526,7 +526,6 @@ static int do_timer_create(clockid_t which_clock, struct sigevent *event, } else { new_timer->it_sigev_notify = SIGEV_SIGNAL; new_timer->sigq.info.si_signo = SIGALRM; - memset(&new_timer->sigq.info.si_value, 0, sizeof(sigval_t)); new_timer->sigq.info.si_value.sival_int = new_timer->it_id; new_timer->it_pid = get_pid(task_tgid(current)); }