From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752758AbaINPnU (ORCPT ); Sun, 14 Sep 2014 11:43:20 -0400 Received: from mail-pa0-f43.google.com ([209.85.220.43]:61924 "EHLO mail-pa0-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752635AbaINPnM (ORCPT ); Sun, 14 Sep 2014 11:43:12 -0400 Message-ID: <5415B790.5010607@gmail.com> Date: Sun, 14 Sep 2014 09:43:12 -0600 From: David Ahern User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:24.0) Gecko/20100101 Thunderbird/24.6.0 MIME-Version: 1.0 To: Arnaldo Carvalho de Melo CC: Pawel Moll , Richard Cochran , Steven Rostedt , Ingo Molnar , Peter Zijlstra , Paul Mackerras , John Stultz , linux-kernel@vger.kernel.org, linux-api@vger.kernel.org Subject: Re: [RFC 2/2] perf: Marker software event and ioctl References: <1410522513-1045-1-git-send-email-pawel.moll@arm.com> <1410522513-1045-3-git-send-email-pawel.moll@arm.com> <54132F63.1010401@gmail.com> <20140912204444.GA11199@kernel.org> In-Reply-To: <20140912204444.GA11199@kernel.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 9/12/14, 2:44 PM, Arnaldo Carvalho de Melo wrote: > Em Fri, Sep 12, 2014 at 10:37:39AM -0700, David Ahern escreveu: >> On 9/12/14, 4:48 AM, Pawel Moll wrote: >>> This patch adds a PERF_COUNT_SW_MARKER event type, which >>> can be requested by user and a PERF_EVENT_IOC_MARKER >>> ioctl command which will inject an event of said type into >>> the perf buffer. The ioctl can take a zero-terminated >>> string argument, similar to tracing_marker in ftrace, >>> which will be kept in the "raw" field of the sample. >>> >>> The main use case for this is synchronisation of >>> performance data generated in user space with the perf >>> stream coming from the kernel. For example, the marker >>> can be inserted by a JIT engine after it generated >>> portion of the code, but before the code is executed >>> for the first time, allowing the post-processor to >>> pick the correct debugging information. Other example >>> is a system profiling tool taking data from other >>> sources than just perf, which generates a marker >>> at the beginning at at the end of the session >>> (also possibly periodically during the session) to >>> synchronise kernel timestamps with clock values >>> obtained in userspace (gtod or raw_monotonic). >> >> Seems really similar to what I proposed in the past: >> >> https://lkml.org/lkml/2011/2/27/159 >> >> Which was rejected. > > I took a look at that thread, but just barely, emphasis on that. > > Injecting something from userspace, a la ftrace, seems to be something, > as tglx mentioned, "buried" in that patchset. Thomas object to an ioctl buried deep in a patch -- newbie mistake. Peter objected to the ioctl https://lkml.org/lkml/2011/3/1/229 It was not userspace injecting random data into the stream but rather forcing the sample to be generated and added to the stream. David David