linux-kernel - Re: [PATCH v2 22/31] ubifs: Switch to use hrtimer

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <184657480.17391174.1739216212140.JavaMail.zimbra@nod.at>
Date: Mon, 10 Feb 2025 20:36:52 +0100 (CET)
From: Richard Weinberger <richard@....at>
To: Nam Cao <namcao@...utronix.de>
Cc: Anna-Maria Behnsen <anna-maria@...utronix.de>, 
	frederic <frederic@...nel.org>, tglx <tglx@...utronix.de>, 
	linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 22/31] ubifs: Switch to use hrtimer_setup()

----- Ursprüngliche Mail -----
> Von: "Nam Cao" <namcao@...utronix.de>
> An: "Anna-Maria Behnsen" <anna-maria@...utronix.de>, "frederic" <frederic@...nel.org>, "tglx" <tglx@...utronix.de>,
> "linux-kernel" <linux-kernel@...r.kernel.org>
> CC: "Nam Cao" <namcao@...utronix.de>, "richard" <richard@....at>
> Gesendet: Mittwoch, 5. Februar 2025 11:39:06
> Betreff: [PATCH v2 22/31] ubifs: Switch to use hrtimer_setup()

> hrtimer_setup() takes the callback function pointer as argument and
> initializes the timer completely.
> 
> Replace hrtimer_init() and the open coded initialization of
> hrtimer::function with the new setup mechanism.
> 
> Patch was created by using Coccinelle.
> 
> Signed-off-by: Nam Cao <namcao@...utronix.de>
> Cc: Richard Weinberger <richard@....at>
> ---
> fs/ubifs/io.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/fs/ubifs/io.c b/fs/ubifs/io.c
> index 01d8eb170382..a79f229df475 100644
> --- a/fs/ubifs/io.c
> +++ b/fs/ubifs/io.c
> @@ -1179,8 +1179,7 @@ int ubifs_wbuf_init(struct ubifs_info *c, struct
> ubifs_wbuf *wbuf)
> 	wbuf->c = c;
> 	wbuf->next_ino = 0;
> 
> -	hrtimer_init(&wbuf->timer, CLOCK_MONOTONIC, HRTIMER_MODE_REL);
> -	wbuf->timer.function = wbuf_timer_callback_nolock;
> +	hrtimer_setup(&wbuf->timer, wbuf_timer_callback_nolock, CLOCK_MONOTONIC,
> HRTIMER_MODE_REL);
> 	return 0;

Acked-by: Richard Weinberger <richard@....at>

Thanks,
//richard