From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f71.google.com (mail-wm0-f71.google.com [74.125.82.71]) by kanga.kvack.org (Postfix) with ESMTP id 0E1F86B0007 for ; Sat, 3 Mar 2018 06:46:35 -0500 (EST) Received: by mail-wm0-f71.google.com with SMTP id i64so2267308wmd.8 for ; Sat, 03 Mar 2018 03:46:34 -0800 (PST) Received: from mail-sor-f41.google.com (mail-sor-f41.google.com. [209.85.220.41]) by mx.google.com with SMTPS id a6sor4324188wrh.51.2018.03.03.03.46.33 for (Google Transport Security); Sat, 03 Mar 2018 03:46:33 -0800 (PST) Subject: Re: "x86/boot/compressed/64: Prepare trampoline memory" breaks boot on Zotac CI-321 References: <12357ee3-0276-906a-0e7c-2c3055675af3@gmail.com> <8c6c0f9d-0f47-2fc9-5cb5-6335ef1152cd@gmail.com> <20180303100257.hzrqtshcnhzy5spl@gmail.com> From: Heiner Kallweit Message-ID: Date: Sat, 3 Mar 2018 12:46:28 +0100 MIME-Version: 1.0 In-Reply-To: <20180303100257.hzrqtshcnhzy5spl@gmail.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: Ingo Molnar Cc: Dexuan-Linux Cui , "Kirill A. Shutemov" , linux-mm@kvack.org, Linux Kernel Mailing List , Dexuan Cui , Thomas Gleixner Am 03.03.2018 um 11:02 schrieb Ingo Molnar: > > * Heiner Kallweit wrote: > >> Am 03.03.2018 um 00:50 schrieb Dexuan-Linux Cui: >>> On Fri, Mar 2, 2018 at 12:57 PM, Heiner Kallweit > wrote: >>> >>> Recently my Mini PC Zotac CI-321 started to reboot immediately before >>> anything was written to the console. >>> >>> Bisecting lead to b91993a87aff "x86/boot/compressed/64: Prepare >>> trampoline memory" being the change breaking boot. >>> >>> If you need any more information, please let me know. >>> >>> Rgds, Heiner >>> >>> >>> This may fix the issue: https://lkml.org/lkml/2018/2/13/668 >>> >>> Kirill posted a v2 patchset 3 days ago and I suppose the patchset should include the fix. >>> >> Thanks for the link. I bisected based on the latest next kernel including >> v2 of the patchset (IOW - the potential fix is included already). > > Are you sure? b91993a87aff is the old patch-set - which I just removed from -next > and which should thus be gone in the Monday iteration of -next. > > I have not merged v2 in -tip yet, did it get applied via some other tree? > > Thanks, > > Ingo > I wanted to apply the fix mentioned in the link but found that the statement was movq already. Therefore my (most likely false) understanding that it's v2. I'll re-test once v2 is out and let you know. Rgds, Heiner diff --git a/arch/x86/boot/compressed/head_64.S b/arch/x86/boot/compressed/head_64.S index 70b30f2bc9e0..99a0e7993252 100644 --- a/arch/x86/boot/compressed/head_64.S +++ b/arch/x86/boot/compressed/head_64.S @@ -332,7 +332,7 @@ ENTRY(startup_64) /* Make sure we have GDT with 32-bit code segment */ leaq gdt(%rip), %rax - movl %eax, gdt64+2(%rip) + movq %rax, gdt64+2(%rip) lgdt gdt64(%rip) /* -- Kirill A. Shutemov -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org