From: Miroslav Benes <mbenes@suse.cz>
To: Seth Jennings <sjenning@redhat.com>
Cc: Josh Poimboeuf <jpoimboe@redhat.com>,
Jiri Kosina <jkosina@suse.cz>, Vojtech Pavlik <vojtech@suse.cz>,
Steven Rostedt <rostedt@goodmis.org>,
Petr Mladek <pmladek@suse.cz>,
Christoph Hellwig <hch@infradead.org>,
Greg KH <gregkh@linuxfoundation.org>,
Andy Lutomirski <luto@amacapital.net>,
Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>,
live-patching@vger.kernel.org, x86@kernel.org, kpatch@redhat.com,
linux-kernel@vger.kernel.org
Subject: Re: [PATCHv2 2/3] kernel: add support for live patching
Date: Fri, 21 Nov 2014 15:41:10 +0100 (CET) [thread overview]
Message-ID: <alpine.LNX.2.00.1411211538240.27891@pobox.suse.cz> (raw)
In-Reply-To: <20141120195637.GC13785@cerebellum.variantweb.net>
On Thu, 20 Nov 2014, Seth Jennings wrote:
> On Thu, Nov 20, 2014 at 11:35:52AM -0600, Josh Poimboeuf wrote:
> > On Thu, Nov 20, 2014 at 02:10:33PM +0100, Miroslav Benes wrote:
> > >
> > > On Sun, 16 Nov 2014, Seth Jennings wrote:
> > >
> > > > This commit introduces code for the live patching core. It implements
> > > > an ftrace-based mechanism and kernel interface for doing live patching
> > > > of kernel and kernel module functions.
> > > >
> > > > It represents the greatest common functionality set between kpatch and
> > > > kgraft and can accept patches built using either method.
> > > >
> > > > This first version does not implement any consistency mechanism that
> > > > ensures that old and new code do not run together. In practice, ~90% of
> > > > CVEs are safe to apply in this way, since they simply add a conditional
> > > > check. However, any function change that can not execute safely with
> > > > the old version of the function can _not_ be safely applied in this
> > > > version.
> > > >
> > > > Signed-off-by: Seth Jennings <sjenning@redhat.com>
> > >
> > > Hi,
> > >
> > > below is the patch which merges the internal and external data structures
> > > (so it is only one part of our original patch for version 1). Apart from
> > > that I tried to make minimal changes to the code. Only unnecessary
> > > kobjects were removed and I renamed lpc_create_* functions to lpc_init_*
> > > as it made more sense in this approach, I think.
> > >
> > > I hope this clearly shows our point of view stated previously. What do
> > > you say?
> >
> > Thanks for rebasing to v2 and splitting up the patches! Personally I'm
> > ok with this patch (though I do have a few comments below).
>
> Thanks Josh :)
>
> Miroslav, before you send out a revision on this patch, I'm merging it
> for v3 right now. I'll fixup any trivial fixes from this email.
>
> I'm putting the finishing touches on v3 now. Hopefully it will make
> everyone happy, or happier, with your changes merged. Should be getting
> close...
>
> Thanks,
> Seth
Ok, thank you for the fixes and merging. I'll take a closer look at v3 to
be sure that everything is ok.
Mira
next prev parent reply other threads:[~2014-11-21 14:41 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-17 1:29 [PATCHv2 0/3] Kernel Live Patching Seth Jennings
2014-11-17 1:29 ` [PATCHv2 1/3] kernel: add TAINT_LIVEPATCH Seth Jennings
2014-11-17 1:29 ` [PATCHv2 2/3] kernel: add support for live patching Seth Jennings
2014-11-17 18:45 ` Greg KH
2014-11-17 19:13 ` Seth Jennings
2014-11-18 14:11 ` Miroslav Benes
2014-11-18 14:26 ` Seth Jennings
2014-11-18 14:45 ` Miroslav Benes
2014-11-19 20:34 ` Seth Jennings
2014-11-20 13:22 ` Miroslav Benes
2014-11-19 15:27 ` Miroslav Benes
2014-11-19 16:05 ` Seth Jennings
2014-11-20 13:10 ` Miroslav Benes
2014-11-20 17:35 ` Josh Poimboeuf
2014-11-20 19:56 ` Seth Jennings
2014-11-21 14:41 ` Miroslav Benes [this message]
2014-11-21 14:38 ` Miroslav Benes
2014-11-20 15:19 ` Josh Poimboeuf
2014-11-20 16:48 ` Seth Jennings
2014-11-17 1:29 ` [PATCHv2 3/3] kernel: add sysfs documentation " Seth Jennings
2014-11-17 18:50 ` Greg KH
2014-11-17 5:33 ` [PATCHv2 0/3] Kernel Live Patching Masami Hiramatsu
2014-11-17 13:16 ` Steven Rostedt
2014-11-17 14:54 ` Seth Jennings
2014-11-18 14:23 ` Jiri Slaby
2014-11-18 14:42 ` Seth Jennings
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=alpine.LNX.2.00.1411211538240.27891@pobox.suse.cz \
--to=mbenes@suse.cz \
--cc=gregkh@linuxfoundation.org \
--cc=hch@infradead.org \
--cc=jkosina@suse.cz \
--cc=jpoimboe@redhat.com \
--cc=kpatch@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=live-patching@vger.kernel.org \
--cc=luto@amacapital.net \
--cc=masami.hiramatsu.pt@hitachi.com \
--cc=pmladek@suse.cz \
--cc=rostedt@goodmis.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 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).