linux-efi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
To: Matt Fleming <matt-mF/unelCI9GS6iBeEJttW/XRex20P6io@public.gmane.org>
Cc: Srikar Dronamraju
	<srikar-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org>,
	Ingo Molnar <mingo-X9Un+BFzKDI@public.gmane.org>,
	LKML <linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Linus Torvalds
	<torvalds-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org>
Subject: Re: v4.4-rc8 based tip tree kernel hangs on boot.
Date: Tue, 12 Jan 2016 11:30:44 +0100	[thread overview]
Message-ID: <20160112103043.GA5707@gmail.com> (raw)
In-Reply-To: <20160111140140.GA2644-mF/unelCI9GS6iBeEJttW/XRex20P6io@public.gmane.org>


* Matt Fleming <matt-mF/unelCI9GS6iBeEJttW/XRex20P6io@public.gmane.org> wrote:

> > Tried reverting Commit 21cdb6b56 "x86/mm: Page align the '_end'
> > symbol to avoid pfn conversion bugs" but that doesnt help.
> 
> Odd. Could you try booting the tip/x86/efi branch instead? There's
> only a handful of patches on that branch so it should be pretty easy
> to narrow down the problematic change.

So he already appears to have done that, when performing the bisection:

 # first bad commit: [21cdb6b568435738cc0b303b2b3b82742396310c] x86/mm: Page align the '_end' symbol to avoid pfn conversion bugs

commit 21cdb6b568 is part of tip:x86/efi:

 triton:~/tip> gll linus..x86/efi
 e2c90dd7e11e x86/efi-bgrt: Replace early_memremap() with memremap()
 98f91276900f Merge tag 'efi-next' of git://git.kernel.org/pub/scm/linux/kernel/git/mfleming/efi into x86/efi
 50a0cb565246 x86/efi-bgrt: Fix kernel panic when mapping BGRT data
 26d7f65fbd22 x86/efi: Preface all print statements with efi* tag
 3bb9eee61c64 doc: efi-stub.txt: Fix arm64 paths
 7f83773ced2f efi/esrt: Don't preformat name
 ff3d0a12fb2d Documentation/x86: Update EFI memory region description
 67a9108ed431 x86/efi: Build our own page table structures
 c9f2a9a65e48 x86/efi: Hoist page table switching code into efi_call_virt()
 b61a76f8850d x86/efi: Map RAM into the identity page table for mixed mode
 edc3b9129cec x86/mm/pat: Ensure cpa->pfn only contains page frame numbers
 21cdb6b56843 x86/mm: Page align the '_end' symbol to avoid pfn conversion bugs

it's the first commit in the branch in fact.

To double check, it would be nice to check that applying that commit to v4.4 
breaks the boot too:

  git checkout v4.4
  git cherry-pick 21cdb6b568

... but assuming the bisection is reliable, I'd say the evidence is pretty 
conclusive already.

Srikar, could you please send the good and bad System.map files, so that we see 
what the effects of the aligning were? The two interesting sha1's to test are:

  21cdb6b56843: first bad commit
  6ffeba960734: last good commit (parent commit of 21cdb6b56843)

Thanks,

	Ingo

  parent reply	other threads:[~2016-01-12 10:30 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-10 19:29 v4.4-rc8 based tip tree kernel hangs on boot Srikar Dronamraju
2016-01-11 14:01 ` Matt Fleming
     [not found]   ` <20160111140140.GA2644-mF/unelCI9GS6iBeEJttW/XRex20P6io@public.gmane.org>
2016-01-11 15:10     ` Ingo Molnar
     [not found]       ` <20160111151030.GA23096-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-01-11 16:47         ` Srikar Dronamraju
2016-01-12 10:30     ` Ingo Molnar [this message]
2016-01-14 12:25       ` Matt Fleming
     [not found]       ` <20160112103043.GA5707-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-01-15 16:20         ` Srikar Dronamraju
     [not found]           ` <20160115162011.GA2935-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org>
2016-01-15 22:29             ` Matt Fleming
2016-01-17 17:32               ` Srikar Dronamraju
     [not found]                 ` <20160117173248.GA28841-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org>
2016-01-18 15:40                   ` Matt Fleming
2016-01-19 10:47                     ` Srikar Dronamraju
     [not found]                       ` <20160119104715.GA15850-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org>
2016-01-19 15:42                         ` Matt Fleming
     [not found]                           ` <20160119154241.GA3615-mF/unelCI9GS6iBeEJttW/XRex20P6io@public.gmane.org>
2016-01-19 18:17                             ` Srikar Dronamraju

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160112103043.GA5707@gmail.com \
    --to=mingo-dgejt+ai2ygdnm+yrofe0a@public.gmane.org \
    --cc=linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=matt-mF/unelCI9GS6iBeEJttW/XRex20P6io@public.gmane.org \
    --cc=mingo-X9Un+BFzKDI@public.gmane.org \
    --cc=srikar-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org \
    --cc=torvalds-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).