[PATCH] irqflags: explicitly ignore lockdep_hrtimer_exit() argument

Arnd Bergmann posted 1 patch 1 year, 10 months ago
There is a newer version of this series
include/linux/irqflags.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] irqflags: explicitly ignore lockdep_hrtimer_exit() argument
Posted by Arnd Bergmann 1 year, 10 months ago
From: Arnd Bergmann <arnd@arndb.de>

When building with 'make W=1' but CONFIG_TRACE_IRQFLAGS=n, the
unused argument to lockdep_hrtimer_exit() causes a warning:

kernel/time/hrtimer.c:1655:14: error: variable 'expires_in_hardirq' set but not used [-Werror=unused-but-set-variable]

This is intentional behavior, so add a cast to void to shut up the warning.

Fixes: 73d20564e0dc ("hrtimer: Don't dereference the hrtimer pointer after the callback")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 include/linux/irqflags.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/linux/irqflags.h b/include/linux/irqflags.h
index 147feebd508c..3f003d5fde53 100644
--- a/include/linux/irqflags.h
+++ b/include/linux/irqflags.h
@@ -114,7 +114,7 @@ do {						\
 # define lockdep_softirq_enter()		do { } while (0)
 # define lockdep_softirq_exit()			do { } while (0)
 # define lockdep_hrtimer_enter(__hrtimer)	false
-# define lockdep_hrtimer_exit(__context)	do { } while (0)
+# define lockdep_hrtimer_exit(__context)	do { (void)(__context); } while (0)
 # define lockdep_posixtimer_enter()		do { } while (0)
 # define lockdep_posixtimer_exit()		do { } while (0)
 # define lockdep_irq_work_enter(__work)		do { } while (0)
-- 
2.39.2
Re: [PATCH] irqflags: explicitly ignore lockdep_hrtimer_exit() argument
Posted by Sebastian Andrzej Siewior 1 year, 10 months ago
On 2024-03-22 14:20:41 [+0100], Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@arndb.de>
> 
> When building with 'make W=1' but CONFIG_TRACE_IRQFLAGS=n, the
> unused argument to lockdep_hrtimer_exit() causes a warning:
> 
> kernel/time/hrtimer.c:1655:14: error: variable 'expires_in_hardirq' set but not used [-Werror=unused-but-set-variable]
> 
> This is intentional behavior, so add a cast to void to shut up the warning.
> 
> Fixes: 73d20564e0dc ("hrtimer: Don't dereference the hrtimer pointer after the callback")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>

Hmm. Seems I forgot about it
   https://lore.kernel.org/all/20231120153953.kGBSHe5j@linutronix.de/

Maybe we could the commit message from the referenced patch. Either way,
Reviewed-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>

Sebastian