Re: [PATCH v2 2/6] powerpc/kexec_file: Add KEXEC_SIG support.

From
Nayna <hidden>
Date
2021-12-09 20:22:06
Reply to
Michal Suchanek, 2021-11-25
On 11/25/21 13:02, Michal Suchanek wrote:
quoted hunk ↗ jump to hunk
Copy the code from s390x

Signed-off-by: Michal Suchanek<redacted>
---
  arch/powerpc/Kconfig        | 11 +++++++++++
  arch/powerpc/kexec/elf_64.c | 36 ++++++++++++++++++++++++++++++++++++
  2 files changed, 47 insertions(+)
diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig
index ac0c515552fd..ecc1227a77f1 100644
--- a/arch/powerpc/Kconfig
+++ b/arch/powerpc/Kconfig
@@ -561,6 +561,17 @@ config KEXEC_FILE
  config ARCH_HAS_KEXEC_PURGATORY
  	def_bool KEXEC_FILE

+config KEXEC_SIG
+	bool "Verify kernel signature during kexec_file_load() syscall"
+	depends on KEXEC_FILE && MODULE_SIG_FORMAT
After manually applying the patch, the build is failing with the following error:

build failed with error "arch/powerpc/kexec/elf_64.o: In function `elf64_verify_sig':
/root/kernel/linus/linux/arch/powerpc/kexec/elf_64.c:160: undefined reference to `verify_appended_signature'"

I see it happened because I didn't have MODULE_SIG enabled. Enabling MODULE_SIG fixes it.

I wonder why not to add "depends on MODULE_SIG" rather than on MODULE_SIG_FORMAT.

Thanks & Regards,

     - Nayna
Thread (24 messages) 24 messages, 5 authors, last reply 2021-12-13
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help