From: Borislav Petkov <bp@alien8.de>
To: Jiri Kosina <jkosina@suse.cz>
Cc: x86@kernel.org, Josh Poimboeuf <jpoimboe@redhat.com>,
Kees Cook <keescook@chromium.org>,
Seth Jennings <sjenning@redhat.com>,
Vojtech Pavlik <vojtech@suse.cz>,
linux-kernel@vger.kernel.org, live-patching@vger.kernel.org
Subject: Re: [PATCH v2 1/2] x86: introduce kaslr_offset()
Date: Tue, 28 Apr 2015 17:59:58 +0200 [thread overview]
Message-ID: <20150428155958.GG19025@pd.tnic> (raw)
In-Reply-To: <alpine.LNX.2.00.1504281755480.16113@pobox.suse.cz>
On Tue, Apr 28, 2015 at 05:57:14PM +0200, Jiri Kosina wrote:
> On Tue, 28 Apr 2015, Jiri Kosina wrote:
>
> > Offset that has been chosen for kaslr during kernel decompression can be
> > easily computed as a difference between _text and __START_KERNEL. We are
> > already making use of this in dump_kernel_offset() notifier and in
> > arch_crash_save_vmcoreinfo().
> >
> > Introduce kaslr_offset() that makes this computation instead of
> > hard-coding it, so that other kernel code (such as live patching) can make
> > use of it. Also convert existing users to make use of it.
> >
> > Signed-off-by: Jiri Kosina <jkosina@suse.cz>
> > ---
> >
> > It'd be great to potentially have Ack from x86 guys for this patch so that
> > I could take it through livepatching.git with the depending 2/2 patch.
> > Thanks.
> >
> > v1 -> v2: convert arch_crash_save_vmcoreinfo(), as spotted by Josh
> > Poimboeuf.
>
> FWIW this patch is equivalent transofrmation without any effects on the
> resulting code:
>
> $ diff -u vmlinux.old.asm vmlinux.new.asm
> --- vmlinux.old.asm 2015-04-28 17:55:19.520983368 +0200
> +++ vmlinux.new.asm 2015-04-28 17:55:24.141206072 +0200
> @@ -1,5 +1,5 @@
>
> -vmlinux.old: file format elf64-x86-64
> +vmlinux.new: file format elf64-x86-64
>
>
> Disassembly of section .text:
> $
Then those are easy. Please add that piece of infomation to the commit
message.
With that:
Acked-by: Borislav Petkov <bp@suse.de>
--
Regards/Gruss,
Boris.
ECO tip #101: Trim your mails when you reply.
--
next prev parent reply other threads:[~2015-04-28 16:00 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-27 14:27 [PATCH 0/2] introduce kaslr_offset() and its users Jiri Kosina
2015-04-27 14:28 ` [PATCH 1/2] x86: introduce kaslr_offset() Jiri Kosina
2015-04-28 12:08 ` Josh Poimboeuf
2015-04-28 15:15 ` [PATCH v2 " Jiri Kosina
2015-04-28 15:57 ` Jiri Kosina
2015-04-28 15:59 ` Borislav Petkov [this message]
2015-04-29 14:56 ` Jiri Kosina
2015-04-29 16:16 ` Jiri Kosina
2015-04-27 14:28 ` [PATCH 2/2] livepatch: x86: make kASLR logic more accurate Jiri Kosina
2015-04-27 14:41 ` Minfei Huang
2015-04-27 23:29 ` Jiri Kosina
2015-04-28 0:08 ` Minfei Huang
2015-04-28 12:09 ` Josh Poimboeuf
2015-04-29 14:56 ` Jiri Kosina
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=20150428155958.GG19025@pd.tnic \
--to=bp@alien8.de \
--cc=jkosina@suse.cz \
--cc=jpoimboe@redhat.com \
--cc=keescook@chromium.org \
--cc=linux-kernel@vger.kernel.org \
--cc=live-patching@vger.kernel.org \
--cc=sjenning@redhat.com \
--cc=vojtech@suse.cz \
--cc=x86@kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.