From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751451AbdBSEmZ (ORCPT ); Sat, 18 Feb 2017 23:42:25 -0500 Received: from mail.kernel.org ([198.145.29.136]:34488 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751266AbdBSEmX (ORCPT ); Sat, 18 Feb 2017 23:42:23 -0500 Date: Sun, 19 Feb 2017 13:42:11 +0900 From: Masami Hiramatsu To: Arnaldo Carvalho de Melo Cc: "Naveen N. Rao" , Ananth N Mavinakayanahalli , Michael Ellerman , Ingo Molnar , Namhyung Kim , linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org Subject: Re: [PATCH 0/2] powerpc: kretprobe updates Message-Id: <20170219134211.0a43dfb2534563af5b6d440d@kernel.org> In-Reply-To: <20170217204254.GH4109@kernel.org> References: <20170217194433.0b4707bbfa1de603d13fbd30@kernel.org> <20170217204254.GH4109@kernel.org> X-Mailer: Sylpheed 3.5.0 (GTK+ 2.24.30; x86_64-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 17 Feb 2017 17:42:54 -0300 Arnaldo Carvalho de Melo wrote: > Em Fri, Feb 17, 2017 at 07:44:33PM +0900, Masami Hiramatsu escreveu: > > On Thu, 16 Feb 2017 13:47:37 +0530 > > "Naveen N. Rao" wrote: > > > > > I am posting the powerpc bits in the same thread so as to keep these > > > changes together. I am not sure how this should be taken upstream as > > > there are atleast three different trees involved: one for the core > > > kprobes infrastructure, one for powerpc and one for perf. > > > Hmm, could you make these (and other related) patches and > > other series in one series? Or wait for the other series > > are merged correctly. > > Well, patches like these should be done in a way that the tooling parts > can deal with kernels with or without the kernel changes, so that older > tools work with new kernels and new tools work with older kernels. > > "work" as in the previous behaviour is kept when a new tool deals with > an older kernel and an older tool would warn the user that what it needs > is not present in that kernel. > > Is this the case? I just looked briefly at the patch commit logs. Thanks Arnaldo, Naveen, I think this one and your previous series are incompatible with older kernel. So those should be merged in one series and at least (1) update ftrace's README special file to show explicitly which can accept text+offset style for kretprobes, and (2) update perf probe side to ensure that (and fallback to previous logic if not). Thank you, > > If it is, then I can pick the tool ones, and the others can be submitted > to the relevant trees, at some point all will be in, kernels eventually > gets updated everywhere, ditto for the tooling, all gets well. > > Regards, > > - Arnaldo > > > > > Thank you, > > > > > > > > Thanks, > > > Naveen > > > > > > Naveen N. Rao (2): > > > powerpc: kretprobes: override default function entry offset > > > perf: powerpc: choose LEP with kretprobes > > > > > > arch/powerpc/kernel/kprobes.c | 9 +++++++++ > > > tools/perf/arch/powerpc/util/sym-handling.c | 5 +---- > > > 2 files changed, 10 insertions(+), 4 deletions(-) > > > > > > -- > > > 2.11.0 > > > > > > > > > -- > > Masami Hiramatsu -- Masami Hiramatsu