Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753756AbbDHMRc (ORCPT ); Wed, 8 Apr 2015 08:17:32 -0400 Received: from mga01.intel.com ([192.55.52.88]:16954 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751629AbbDHMR3 (ORCPT ); Wed, 8 Apr 2015 08:17:29 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.11,544,1422950400"; d="scan'208";a="710575907" Message-ID: <55251BD9.30700@intel.com> Date: Wed, 08 Apr 2015 15:15:21 +0300 From: Adrian Hunter Organization: Intel Finland Oy, Registered Address: PL 281, 00181 Helsinki, Business Identity Code: 0357606 - 4, Domiciled in Helsinki User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 MIME-Version: 1.0 To: Stephane Eranian CC: LKML , Arnaldo Carvalho de Melo , Peter Zijlstra , "mingo@elte.hu" , "ak@linux.intel.com" , Jiri Olsa , Namhyung Kim , Rose Belcher , Sukadev Bhattiprolu , Sonny Rao , John Mccutchan , David Ahern , Pawel Moll Subject: Re: [PATCH v6 3/4] perf inject: add jitdump mmap injection support References: <1427753974-13380-1-git-send-email-eranian@google.com> <1427753974-13380-4-git-send-email-eranian@google.com> <551B9731.7080505@intel.com> In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3350 Lines: 85 On 06/04/15 22:41, Stephane Eranian wrote: > > + if (inject.build_ids) { > > + /* > > + * to make sure the mmap records are ordered correctly > > + * and so that the correct especially due to jitted code > > + * mmaps. We cannot generate the buildid hit list and > > + * inject the jit mmaps at the same time for now. > > + */ > > + inject.tool.ordered_events = true; > > + inject.tool.ordering_requires_timestamps = true; > > + } > > + > > + if (inject.jit_mode) { > > + inject.tool.mmap2 = perf_event__repipe_mmap2; > > + inject.tool.mmap = perf_event__repipe_mmap; > > As suggested above, why not make your own tool fns e.g. > > inject.tool.mmap2 = perf_event__jit_mode_mmap2; > inject.tool.mmap = perf_event__jit_mode_mmap; > > > > + inject.tool.ordered_events = true; > > + inject.tool.ordering_requires_timestamps = true; > > You are taking advantage of a bug in perf-inject, that is the > "finished_round" events are not being processed. Really they should be > processed and you should inject in time order. > > I am not sure I understand. > Yes, I am trying to have inject reorder the samples instead of perf report. > You are likely to run inject once, and report many times. Also avoids a > warning in report about out-of-order events. Well forgetting about "finished_round", it seems to me you need to intercept all the delivered events (which will be in time order) and inject your own events at the right time. At the moment it seems to me you are injecting all your events in one go when you see the special jitdump mmap. So I would not expect the injected events to be ordered with respect to other events that come later. But maybe I misunderstand that? As I confusingly tried to suggest earlier, one way to see all the delivered events is to hook the ordered_events "deliver" callback. That will mean injecting one mmap event at a time. Here is just an idea. struct perf_inject { ... ordered_events__deliver_t deliver; }; int cmd_inject(int argc, const char **argv, const char *prefix __maybe_unused) { ... inject.deliver = inject.session->ordered_events.deliver; inject.session->ordered_events.deliver = inject_jit_mmap; ... } int inject_jit_mmap(struct ordered_events *oe, struct ordered_event *event) { struct perf_session *session = container_of(oe, struct perf_session, ordered_events); struct perf_inject *inject = container_of(session->tool, struct perf_inject, tool); /* Is it time to inject an event */ if (jit_next_timestamp(inject) < event->timestamp) { /* Yes, so inject it by delivery */ perf_session__deliver_synth_event(...); } inject->deliver(oe, event); } You would also need to inject any remaining events right at the end. Note the advantage of "delivering" events is that the same approach works whatever the tool. -- 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/