lkml.org 
[lkml]   [2018]   [Apr]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: instant reboot caused by 194a9749c73d650c0
On Mon, Apr 16, 2018 at 08:07:09AM +0200, Ingo Molnar wrote:
>
> * Eric Dumazet <eric.dumazet@gmail.com> wrote:
>
> > Hi Kirill
> >
> > For some reason, my hosts instantly crash at boot time, with absolutely no log on console.
> >
> > Bisection pointed to :
> >
> > $ git bisect bad
> > 194a9749c73d650c0b1dfdee04fb0bdf0a888ba8 is the first bad commit
> > commit 194a9749c73d650c0b1dfdee04fb0bdf0a888ba8
> > Author: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
> > Date: Mon Mar 12 13:02:46 2018 +0300
> >
> > x86/boot/compressed/64: Handle 5-level paging boot if kernel is above 4G
>
> Could you please send your .config? These early boot problems are sometimes build
> and Kconfig environment sensitive.
>
> A high level description of your hardware and the distro you are using would also
> be useful.

And how do you start the kernel? EFI? Legacy boot? kexec?

>
> Kirill, I'm curious about this change:
>
> - /* Calculate address we are running at */
> - call 1f
> -1: popl %edi
> - subl $1b, %edi
> + /* Calculate address of paging_enabled() once we are executing in the
> trampoline */
> + leal paging_enabled - trampoline_32bit_src + TRAMPOLINE_32BIT_CODE_OFFSET(%ecx), %eax
>
> Here we change the calculation from a "discover where we are executing" method to
> a calculation method (which is fundamentally more fragile) - why?

I guess, I tried to save one register -- %rdi is used for return from
trampoline.

But you're right there's no reason to do this and it may be more fragile.

Eric, could you check if the patch makes any difference?

diff --git a/arch/x86/boot/compressed/head_64.S b/arch/x86/boot/compressed/head_64.S
index fca012baba19..395c122ef70b 100644
--- a/arch/x86/boot/compressed/head_64.S
+++ b/arch/x86/boot/compressed/head_64.S
@@ -562,7 +562,10 @@ ENTRY(trampoline_32bit_src)
movl %eax, %cr4

/* Calculate address of paging_enabled() once we are executing in the trampoline */
- leal paging_enabled - trampoline_32bit_src + TRAMPOLINE_32BIT_CODE_OFFSET(%ecx), %eax
+ call 1f
+1: popl %eax
+ subl $1b, %eax
+ leal paging_enabled(%eax), %eax

/* Prepare the stack for far return to Long Mode */
pushl $__KERNEL_CS
diff --git a/arch/x86/boot/compressed/pgtable.h b/arch/x86/boot/compressed/pgtable.h
index 91f75638f6e6..6ff7e81b5628 100644
--- a/arch/x86/boot/compressed/pgtable.h
+++ b/arch/x86/boot/compressed/pgtable.h
@@ -6,7 +6,7 @@
#define TRAMPOLINE_32BIT_PGTABLE_OFFSET 0

#define TRAMPOLINE_32BIT_CODE_OFFSET PAGE_SIZE
-#define TRAMPOLINE_32BIT_CODE_SIZE 0x60
+#define TRAMPOLINE_32BIT_CODE_SIZE 0x70

#define TRAMPOLINE_32BIT_STACK_END TRAMPOLINE_32BIT_SIZE

--
Kirill A. Shutemov
\
 
 \ /
  Last update: 2018-04-16 11:16    [W:0.042 / U:0.576 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site