Message-ID: <52277521.5020800@linux.vnet.ibm.com>
Date: Wed, 04 Sep 2013 23:30:01 +0530
From: Hemant <hkshaw@linux.vnet.ibm.com>
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130625 Thunderbird/17.0.7
MIME-Version: 1.0
To: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com>
CC: Namhyung Kim <namhyung@kernel.org>, linux-kernel@vger.kernel.org,
        srikar@linux.vnet.ibm.com, peterz@infradead.org, oleg@redhat.com,
        mingo@redhat.com, anton@redhat.com, systemtap@sourceware.org
Subject: Re: [PATCH 2/2] Support to perf to probe on SDT markers:
References: <20130903072944.4793.93584.stgit@hemant-fedora> <20130903073703.4793.32779.stgit@hemant-fedora> <8761uht6jk.fsf@sejong.aot.lge.com> <5226EAEA.1010606@hitachi.com>
In-Reply-To: <5226EAEA.1010606@hitachi.com>
Content-Type: text/plain; charset=ISO-2022-JP
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 912
Lines: 22

On 09/04/2013 01:40 PM, Masami Hiramatsu wrote:
> (2013/09/04 16:00), Namhyung Kim wrote:
>> On Tue, 03 Sep 2013 13:07:03 +0530, Hemant Kumar wrote:
>>> This patch enables perf to probe on the marker name specified on the command line.
>> It looks like you didn't consider prelinked libraries.  You need to check
>> the address of .stapsdt.base section too.  And obviously this patch
>> ignores any argument the SDT has which I think pretty important info.
>> But we can add it later once the uprobes arg fetch patches are in.
> Agreed, I'd like to see the argument support afterward.
> Thanks!
>

Will add the argument support in the next iteration.

Thanks
Hemant

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/