kexec bzImage path setup data is shared with real-mode path, and
setup_header is copied together with setup_code.
Later 32bit just use whole area as boot_params for real_mode_data.
but those area for boot_params around setup_header is
not cleaned that will leave some field in boot_param as
non-zero value.

So clean whole buffer at first, and only copy setup_header for non
real-mode entry path.

Signed-off-by: Yinghai Lu <ying...@kernel.org>
---
 kexec/arch/i386/kexec-bzImage.c |   14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/kexec/arch/i386/kexec-bzImage.c b/kexec/arch/i386/kexec-bzImage.c
index 83a023d..35005c7 100644
--- a/kexec/arch/i386/kexec-bzImage.c
+++ b/kexec/arch/i386/kexec-bzImage.c
@@ -211,7 +211,19 @@ int do_bzImage_load(struct kexec_info *info,
        /* The argument/parameter segment */
        setup_size = kern16_size + command_line_len + PURGATORY_CMDLINE_SIZE;
        real_mode = xmalloc(setup_size);
-       memcpy(real_mode, kernel, kern16_size);
+       memset(real_mode, 0, setup_size);
+       if (!real_mode_entry) {
+               unsigned long setup_header_size = kernel[0x201] + 0x202 - 0x1f1;
+
+               /* only copy setup_header */
+               if (setup_header_size > 0x7f)
+                       setup_header_size = 0x7f;
+               memcpy((unsigned char *)real_mode + 0x1f1, kernel + 0x1f1,
+                       setup_header_size);
+       } else {
+               /* copy setup code and setup_header */
+               memcpy(real_mode, kernel, kern16_size);
+       }
 
        if (info->kexec_flags & (KEXEC_ON_CRASH | KEXEC_PRESERVE_CONTEXT)) {
                /* If using bzImage for capture kernel, then we will not be
-- 
1.7.10.4


_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec

Reply via email to