From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753065AbYJaPcw (ORCPT ); Fri, 31 Oct 2008 11:32:52 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751481AbYJaPco (ORCPT ); Fri, 31 Oct 2008 11:32:44 -0400 Received: from rv-out-0506.google.com ([209.85.198.232]:50101 "EHLO rv-out-0506.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751267AbYJaPcn (ORCPT ); Fri, 31 Oct 2008 11:32:43 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:to:subject:cc:in-reply-to:mime-version :content-type:content-transfer-encoding:content-disposition :references; b=vKmRC4d/TRSMtTXMKGwz1isWyEGQ4bSU34qr6ZpfFm4+Xj2TYUsf6/JJNSZr3aFD5W +61xoxfRtlu6XZGltzC97nQPaUyNs7o6o9ILA6w/OUoaNvxX2lGVan4/vZllTaHLIDdq 76o5zp0URbLPnIG4jZC/yibULE+YC/Wik786E= Message-ID: <863e9df20810310832uadf4daay7b5a98b677d86561@mail.gmail.com> Date: Fri, 31 Oct 2008 11:32:42 -0400 From: "Abhishek Sagar" To: "Steven Rostedt" Subject: Re: [PATCH] ftrace: distinguish kretprobe'd functions in trace logs Cc: "Ingo Molnar" , "Thomas Gleixner" , LKML , "Al Viro" , ananth@in.ibm.com, jkenisto@us.ibm.com, mhiramat@redhat.com In-Reply-To: MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <483C53EE.2060602@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Oct 31, 2008 at 10:36 AM, Steven Rostedt wrote: >> Hi Abhishek, >> >> Now that this code is in mainline, it has become apparent that >> kretprobe_trampoline is not defined on all archs as a global like it is in >> x86. This needs to be fixed (it breaks ia64 build). >> > > Also note that kretprobe_trampoline will not be the return address. > > Hmm, I think I can solve this by not even doing the test here, but simply > on the name itself. Ok. I wonder if it would be a good idea to export kretprobe_trampoline (or kretprobe_trampoline->ip) via asm/kprobes.h for all archs, for the kind of checks done here for ftrace (to have them elsewhere too like in OOPs generation etc). Adding Kprobe folks in CC -- Abhishek