2024-01-09 08:22:57

by Juergen Gross

[permalink] [raw]
Subject: [PATCH] x86/paravirt: make BUG_func() usable by non-GPL modules

Several inlined functions subject to paravirt patching are referencing
BUG_func() since the recent switch to the alternative patching
mechanism.

As those functions can legally be used by non-GPL modules, BUG_func()
must be usable by those modules, too. So don't use EXPORT_SYMBOL_GPL()
for exporting BUG_func().

Fixes: 9824b00c2b58 ("x86/paravirt: Move some functions and defines to alternative.c")
Signed-off-by: Juergen Gross <[email protected]>
---
arch/x86/kernel/alternative.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/x86/kernel/alternative.c b/arch/x86/kernel/alternative.c
index 55e205b21271..692f597e414b 100644
--- a/arch/x86/kernel/alternative.c
+++ b/arch/x86/kernel/alternative.c
@@ -402,7 +402,7 @@ noinstr void BUG_func(void)
{
BUG();
}
-EXPORT_SYMBOL_GPL(BUG_func);
+EXPORT_SYMBOL(BUG_func);

#define CALL_RIP_REL_OPCODE 0xff
#define CALL_RIP_REL_MODRM 0x15
--
2.35.3



Subject: [tip: x86/urgent] x86/paravirt: Make BUG_func() usable by non-GPL modules

The following commit has been merged into the x86/urgent branch of tip:

Commit-ID: f7cfe7017b531e08c108ac6615b1ddedcc892428
Gitweb: https://git.kernel.org/tip/f7cfe7017b531e08c108ac6615b1ddedcc892428
Author: Juergen Gross <[email protected]>
AuthorDate: Tue, 09 Jan 2024 09:22:32 +01:00
Committer: Borislav Petkov (AMD) <[email protected]>
CommitterDate: Mon, 22 Jan 2024 15:51:07 +01:00

x86/paravirt: Make BUG_func() usable by non-GPL modules

Several inlined functions subject to paravirt patching are referencing
BUG_func() after the recent switch to the alternative patching
mechanism.

As those functions can legally be used by non-GPL modules, BUG_func()
must be usable by those modules, too. So use EXPORT_SYMBOL() when
exporting BUG_func().

Fixes: 9824b00c2b58 ("x86/paravirt: Move some functions and defines to alternative.c")
Signed-off-by: Juergen Gross <[email protected]>
Signed-off-by: Borislav Petkov (AMD) <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
---
arch/x86/kernel/alternative.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/x86/kernel/alternative.c b/arch/x86/kernel/alternative.c
index cc130b5..1d85cb7 100644
--- a/arch/x86/kernel/alternative.c
+++ b/arch/x86/kernel/alternative.c
@@ -403,7 +403,7 @@ noinstr void BUG_func(void)
{
BUG();
}
-EXPORT_SYMBOL_GPL(BUG_func);
+EXPORT_SYMBOL(BUG_func);

#define CALL_RIP_REL_OPCODE 0xff
#define CALL_RIP_REL_MODRM 0x15