Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1506210

Re: 4.9-rc1 boot regression, ambiguous bisect result

Show all headers | View raw

On Fri, Oct 21, 2016 at 12:00 AM, Ingo Molnar <mingo@kernel.org> wrote:
>
> * Dan Williams <dan.j.williams@intel.com> wrote:
>
>> On Thu, Oct 20, 2016 at 8:22 AM, Dan Williams <dan.j.williams@intel.com> wrote:
>> > On Thu, Oct 20, 2016 at 5:29 AM, Matt Fleming <matt@codeblueprint.co.uk> wrote:
>> >> On Wed, 19 Oct, at 09:04:29PM, Dan Williams wrote:
>> >>> Hi,
>> >>>
>> >>> I am currently unable to boot a Yoga 900 with latest mainline, but 4.8 boots.
>> >>>
>> >>> The symptom is a reboot before the video console is available.
>> >>>
>> >>> I bisected to commit 816e76129ed5 "efi: Allow drivers to reserve boot
>> >>> services forever".  However, that commit is known to be broken.  The
>> >>> proposed fix, commit 92dc33501bfb "x86/efi: Round EFI memmap
>> >>> reservations to EFI_PAGE_SIZE", also exhibits the reboot problem.
>> >>>
>> >>> During the bisect some of the stopping points landed on commits that
>> >>> caused the boot process to hang rather than cause a reboot.  The
>> >>> commits that resulted in a hang are marked "git bisect skip" in this
>> >>> log: https://gist.github.com/djbw/1b501daa98192a42ae848f03bb59c30e
>> >>>
>> >>> I'll try treating those hangs as bad bisect results and re-run the
>> >>> full bisect tomorrow.  In the meantime I wonder if the bisect log
>> >>> implicates a better regression candidate?
>> >>
>> >> Could you mail the dmesg output when booting a known working kernel
>> >> with efi=debug ?
>> >
>> > Here it is:
>> >
>> > https://gist.github.com/djbw/cae05e721b159d5ad7b146d7a93f5fa2
>>
>> I am able to build a kernel and boot the platform with the following
>> set of reverts:
>>
>>       Revert "x86/efi: Round EFI memmap reservations to EFI_PAGE_SIZE"
>>       Revert "x86/efi-bgrt: Use efi_mem_reserve() to avoid copying image data"
>>       Revert "efi/esrt: Use efi_mem_reserve() and avoid a kmalloc()"
>>       Revert "efi: Allow drivers to reserve boot services forever"
>
> Could you please describe the bootup behavior after each revert? I.e. wild guess:
>
>        vanilla kernel:
>    # spontaneous reboot
>        + Revert "x86/efi: Round EFI memmap reservations to EFI_PAGE_SIZE":
>    # spontaneous reboot
>        + Revert "x86/efi-bgrt: Use efi_mem_reserve() to avoid copying image data":
>    # hang
>        + Revert "efi/esrt: Use efi_mem_reserve() and avoid a kmalloc()":
>    # hang
>        + Revert "efi: Allow drivers to reserve boot services forever":
>    == works
>
> ?

In this case all but the last revert produce the same result, instant
reboot after loading the kernel.  I have not been able to pinpoint
what changes that behavior to the hang conditions I saw mid-bisect.

The first three reverts are just there to get the kernel to build
again after reverting "efi: Allow drivers to reserve boot services
forever"

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

4.9-rc1 boot regression, ambiguous bisect result Dan Williams <dan.j.williams@intel.com> - 2016-10-20 06:10 +0200
  Re: 4.9-rc1 boot regression, ambiguous bisect result Ingo Molnar <mingo@kernel.org> - 2016-10-20 08:30 +0200
  Re: 4.9-rc1 boot regression, ambiguous bisect result Matt Fleming <matt@codeblueprint.co.uk> - 2016-10-20 14:30 +0200
    Re: 4.9-rc1 boot regression, ambiguous bisect result Dan Williams <dan.j.williams@intel.com> - 2016-10-20 17:30 +0200
      Re: 4.9-rc1 boot regression, ambiguous bisect result Dan Williams <dan.j.williams@intel.com> - 2016-10-20 21:40 +0200
        Re: 4.9-rc1 boot regression, ambiguous bisect result Ingo Molnar <mingo@kernel.org> - 2016-10-21 09:10 +0200
          Re: 4.9-rc1 boot regression, ambiguous bisect result Dan Williams <dan.j.williams@intel.com> - 2016-10-21 20:30 +0200
        Re: 4.9-rc1 boot regression, ambiguous bisect result Matt Fleming <matt@codeblueprint.co.uk> - 2016-10-21 17:50 +0200
          Re: 4.9-rc1 boot regression, ambiguous bisect result Matt Fleming <matt@codeblueprint.co.uk> - 2016-10-21 22:30 +0200
            Re: 4.9-rc1 boot regression, ambiguous bisect result Dan Williams <dan.j.williams@intel.com> - 2016-10-22 01:30 +0200

csiph-web