linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] genirq: Export irq_check_status_bit
@ 2020-12-30 15:45 Arnd Bergmann
  2020-12-31  6:36 ` Thorsten Leemhuis
  0 siblings, 1 reply; 2+ messages in thread
From: Arnd Bergmann @ 2020-12-30 15:45 UTC (permalink / raw)
  To: Thomas Gleixner; +Cc: Arnd Bergmann, Marc Zyngier, linux-kernel

From: Arnd Bergmann <arnd@arndb.de>

Changing some inline functions to use the new irq_check_status_bit
function out of line breaks calling them from loadable modules:

ERROR: modpost: "irq_check_status_bit" [drivers/perf/arm_spe_pmu.ko] undefined!

Export the function to make it work again. One can debate over
whether this should be EXPORT_SYMBOL() or EXPORT_SYMBOL_GPL(),
as it could be called from any module before and making it GPL-only
changes behavior. However all other symbols in this file are
EXPORT_SYMBOL_GPL(), so I went with that for consistency.

Fixes: fdd029630434 ("genirq: Move status flag checks to core")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
---
 kernel/irq/manage.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/kernel/irq/manage.c b/kernel/irq/manage.c
index ab8567f32501..dec3f73e8db9 100644
--- a/kernel/irq/manage.c
+++ b/kernel/irq/manage.c
@@ -2859,3 +2859,4 @@ bool irq_check_status_bit(unsigned int irq, unsigned int bitmask)
 	rcu_read_unlock();
 	return res;
 }
+EXPORT_SYMBOL_GPL(irq_check_status_bit);
-- 
2.29.2


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] genirq: Export irq_check_status_bit
  2020-12-30 15:45 [PATCH] genirq: Export irq_check_status_bit Arnd Bergmann
@ 2020-12-31  6:36 ` Thorsten Leemhuis
  0 siblings, 0 replies; 2+ messages in thread
From: Thorsten Leemhuis @ 2020-12-31  6:36 UTC (permalink / raw)
  To: Arnd Bergmann, Thomas Gleixner; +Cc: Arnd Bergmann, Marc Zyngier, linux-kernel

Am 30.12.20 um 16:45 schrieb Arnd Bergmann:
> From: Arnd Bergmann <arnd@arndb.de>
> 
> Changing some inline functions to use the new irq_check_status_bit
> function out of line breaks calling them from loadable modules:
> 
> ERROR: modpost: "irq_check_status_bit" [drivers/perf/arm_spe_pmu.ko] undefined!

Just FYI: Levi Yun sent a similar patch a on Dec, 26 already:
https://lore.kernel.org/lkml/20201226123818.GA693525@ubuntu/

But nothing happened afaics, the festival season is taking its toll. ;-)

I'd like to see this fixed, too, as it broke my
Kernel Vanilla builds for Fedora
(https://fedoraproject.org/wiki/Kernel_Vanilla_Repositories) just a day
after I finally started building packages for aarch64, too. Bad timing. :-D

Ciao, Thorsten



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-12-31  7:03 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-30 15:45 [PATCH] genirq: Export irq_check_status_bit Arnd Bergmann
2020-12-31  6:36 ` Thorsten Leemhuis

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).