From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.4 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8956DC4363A for ; Mon, 26 Oct 2020 12:05:23 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 46AEA22281 for ; Mon, 26 Oct 2020 12:05:23 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1774713AbgJZMFW (ORCPT ); Mon, 26 Oct 2020 08:05:22 -0400 Received: from jptosegrel01.sonyericsson.com ([124.215.201.71]:2659 "EHLO JPTOSEGREL01.sonyericsson.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1770973AbgJZMFV (ORCPT ); Mon, 26 Oct 2020 08:05:21 -0400 Subject: Re: [RFC PATCH 0/6] Sleepable tracepoints To: Michael Jeanson , CC: , Steven Rostedt , Peter Zijlstra , Alexei Starovoitov , Yonghong Song , "Paul E . McKenney" , Ingo Molnar , Arnaldo Carvalho de Melo , Mark Rutland , Alexander Shishkin , Jiri Olsa , Namhyung Kim , Joel Fernandes , References: <20201023195352.26269-1-mjeanson@efficios.com> From: peter enderborg Message-ID: <083f3ffa-3395-d66b-bb8b-d6a3fd7f6177@sony.com> Date: Mon, 26 Oct 2020 13:05:09 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: <20201023195352.26269-1-mjeanson@efficios.com> Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Content-Language: en-GB X-SEG-SpamProfiler-Analysis: v=2.3 cv=EbK2v8uC c=1 sm=1 tr=0 a=CGteIMthFL3x4Fb36c5kWA==:117 a=IkcTkHD0fZMA:10 a=afefHYAZSVUA:10 a=meVymXHHAAAA:8 a=JfrnYn6hAAAA:8 a=VwQbUJbxAAAA:8 a=FOH2dFAWAAAA:8 a=20KFwNOVAAAA:8 a=7CQSdrXTAAAA:8 a=QyXUC8HyAAAA:8 a=qqdB56dbAAAA:8 a=3C5PsIaMDjZkp0q5FAMA:9 a=QEXdDO2ut3YA:10 a=2JgSa4NbpEOStq-L5dxp:22 a=1CNFftbPRP8L7MoqJWF3:22 a=AjGcO6oz07-iQ99wixmX:22 a=i3VuKzQdj-NEYjvDI-p3:22 a=a-qgeE7W1pNrGK8U0ZQC:22 a=ccaIO3UgQCpleZvgly2v:22 X-SEG-SpamProfiler-Score: 0 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 10/23/20 9:53 PM, Michael Jeanson wrote: > When invoked from system call enter/exit instrumentation, accessing > user-space data is a common use-case for tracers. However, tracepoints > currently disable preemption around iteration on the registered > tracepoint probes and invocation of the probe callbacks, which prevents > tracers from handling page faults. > > Extend the tracepoint and trace event APIs to allow specific tracer > probes to take page faults. Adapt ftrace, perf, and ebpf to allow being > called from sleepable context, and convert the system call enter/exit > instrumentation to sleepable tracepoints. Will this not be a problem for analyse of the trace? It get two relevant times, one it when it is called and one when it returns. It makes things harder to correlate in what order things happen. And handling of tracing of contexts that already are not preamptable? Eg the same tracepoint are used in different places and contexts. > This series only implements the tracepoint infrastructure required to > allow tracers to handle page faults. Modifying each tracer to handle > those page faults would be a next step after we all agree on this piece > of instrumentation infrastructure. > > This patchset is base on v5.9.1. > > Cc: Steven Rostedt (VMware) > Cc: Peter Zijlstra > Cc: Alexei Starovoitov > Cc: Yonghong Song > Cc: Paul E. McKenney > Cc: Ingo Molnar > Cc: Arnaldo Carvalho de Melo > Cc: Mark Rutland > Cc: Alexander Shishkin > Cc: Jiri Olsa > Cc: Namhyung Kim > Cc: Joel Fernandes (Google) > Cc: bpf@vger.kernel.org > > Mathieu Desnoyers (1): > tracing: use sched-RCU instead of SRCU for rcuidle tracepoints > > Michael Jeanson (5): > tracing: introduce sleepable tracepoints > tracing: ftrace: add support for sleepable tracepoints > tracing: bpf-trace: add support for sleepable tracepoints > tracing: perf: add support for sleepable tracepoints > tracing: convert sys_enter/exit to sleepable tracepoints > > include/linux/tracepoint-defs.h | 11 ++++ > include/linux/tracepoint.h | 104 +++++++++++++++++++++----------- > include/trace/bpf_probe.h | 23 ++++++- > include/trace/define_trace.h | 7 +++ > include/trace/events/syscalls.h | 4 +- > include/trace/perf.h | 26 ++++++-- > include/trace/trace_events.h | 79 ++++++++++++++++++++++-- > init/Kconfig | 1 + > kernel/trace/bpf_trace.c | 5 +- > kernel/trace/trace_events.c | 15 ++++- > kernel/trace/trace_syscalls.c | 68 +++++++++++++-------- > kernel/tracepoint.c | 104 +++++++++++++++++++++++++------- > 12 files changed, 351 insertions(+), 96 deletions(-) >