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


Groups > linux.kernel > #1567059

Re: About commit 65fe935d ("x86/KASLR, x86/power: Remove x86 hibernation restrictions")

From Kees Cook <keescook@chromium.org>
Newsgroups linux.kernel
Subject Re: About commit 65fe935d ("x86/KASLR, x86/power: Remove x86 hibernation restrictions")
Date 2017-01-26 02:10 +0100
Message-ID <t3GJz-SE-5@gated-at.bofh.it> (permalink)
References <t2o2l-8c0-1@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Sun, Jan 22, 2017 at 2:53 AM, Baoquan He <bhe@redhat.com> wrote:
> Hi Kees,
>
> In your commit 65fe935dd, you mentioned with the fix of 70595b479ce1,
> the x86 hibernation restrictions can be removed. However I didn't find
> it in Linus's tree. I found commit 65c0554 ("x86/power/64: Fix kernel text
> mapping corruption during image restoration"), it should be the one you
> mentioned, but not very sure since it was merged later than your commit.
> Could you help confirm this?

That sounds correct, yes. There was some last minute additional fixing
which I think made the referenced commit vanish before Linus merged or
something like that. 65c0554 looks like the fix it was meaning to
reference. :)

-Kees

>
> commit 65fe935dd2387a4faf15314c73f5e6d31ef0217e
> Author: Kees Cook <keescook@chromium.org>
> Date:   Mon Jun 13 15:10:02 2016 -0700
>
>     x86/KASLR, x86/power: Remove x86 hibernation restrictions
>
>     With the following fix:
>
>       70595b479ce1 ("x86/power/64: Fix crash whan the hibernation code passes control to the image kernel")
>
>     ... there is no longer a problem with hibernation resuming a
>     KASLR-booted kernel image, so remove the restriction.
>
> commit 65c0554b73c920023cc8998802e508b798113b46
> Author: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> Date:   Thu Jun 30 18:11:41 2016 +0200
>
>     x86/power/64: Fix kernel text mapping corruption during image restoration
>
>     Logan Gunthorpe reports that hibernation stopped working reliably for
>     him after commit ab76f7b4ab23 (x86/mm: Set NX on gap between __ex_table
>     and rodata).
>
> Thanks
> Baoquan



-- 
Kees Cook
Nexus Security

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


Thread

About commit 65fe935d ("x86/KASLR, x86/power: Remove x86 hibernation  restrictions") Baoquan He <bhe@redhat.com> - 2017-01-22 12:00 +0100
  Re: About commit 65fe935d ("x86/KASLR, x86/power: Remove x86  hibernation restrictions") Kees Cook <keescook@chromium.org> - 2017-01-26 02:10 +0100
    Re: About commit 65fe935d ("x86/KASLR, x86/power: Remove x86  hibernation restrictions") Baoquan He <bhe@redhat.com> - 2017-01-26 02:20 +0100

csiph-web