From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.0 required=3.0 tests=MAILING_LIST_MULTI,SPF_PASS autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 42D71C43387 for ; Thu, 20 Dec 2018 08:33:10 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 0513721720 for ; Thu, 20 Dec 2018 08:33:10 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728008AbeLTIdJ (ORCPT ); Thu, 20 Dec 2018 03:33:09 -0500 Received: from mx2.suse.de ([195.135.220.15]:44902 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1725300AbeLTIdI (ORCPT ); Thu, 20 Dec 2018 03:33:08 -0500 X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay1.suse.de (unknown [195.135.220.254]) by mx1.suse.de (Postfix) with ESMTP id 28B92AF0E; Thu, 20 Dec 2018 08:33:06 +0000 (UTC) Date: Thu, 20 Dec 2018 09:33:05 +0100 (CET) From: Miroslav Benes To: Jiri Kosina , Josh Poimboeuf cc: yamada.masahiro@socionext.com, michal.lkml@markovi.net, jeyu@kernel.org, pmladek@suse.com, linux-kbuild@vger.kernel.org, live-patching@vger.kernel.org, linux-kernel@vger.kernel.org, Joao Moreira Subject: Re: [PATCH] kbuild: use -flive-patching when CONFIG_LIVEPATCH is enabled In-Reply-To: Message-ID: References: <20181219141744.32392-1-mbenes@suse.cz> <20181219165428.5udrppedpdvf2u7k@treble> <20181219172127.o753af7pkrsttcgl@treble> User-Agent: Alpine 2.21 (LSU 202 2017-01-01) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 19 Dec 2018, Jiri Kosina wrote: > On Wed, 19 Dec 2018, Josh Poimboeuf wrote: > > > Also the commit message needs an analysis of the performance impacts. > > Agreed. Especially as it's expected (*) to be completely in the noise > particularly for the kernel, it'd be good to have that documented in the > changelog. > > (*) actually measured already for some subset of the IPA optimizations Ok, we can do that. I don't expect the results to be different from the last measurement as Jiri mentions. The sets of disabled optimizations are similar. I'll add it to v2. On Wed, 19 Dec 2018, Jiri Kosina wrote: > On Wed, 19 Dec 2018, Josh Poimboeuf wrote: > > > > > This option only makes sense for source-based patch generation, so isn't > > > > it a bit premature to make this change without proper source-based patch > > > > tooling? > > > > > > The reality is though that before the full-fledged patch tooling exists, > > > people are actually already writing livepatches by hand, so this option is > > > useful for them. > > > > Fair enough. Yes, that was the reason I sent it. It would not make sense to wait for the tooling in this case, because -flive-patching is useful even now, since there is a way to prepare livepatches without any tooling. > > Though, upstream, almost everybody seems to use kpatch-build, for which > > this patch doesn't help. And people will continue to do so until we > > have decent source-based tooling. Will the klp-convert patches be > > revived soon? > > Let me add Joao, who's working on that. > > Joao, I think you had something basically ready for upstream exposure, > right? I think that when Joao posted it a long time ago, the conclusion was that it would be better to wait for the source-based tooling and have the complete solution. I may misremember though. If Josh thinks that it would be acceptable to have klp-convert merged even without the tooling, I'm all for it. We're about to start using it at SUSE and staying close to upstream would definitely be better. Miroslav