Received: by 2002:a25:868d:0:0:0:0:0 with SMTP id z13csp847339ybk; Wed, 20 May 2020 13:35:30 -0700 (PDT) X-Google-Smtp-Source: ABdhPJz/OPEV6kA0nTjcB/Jb9rldfgXJWxX9HwOTtCBjEfcoicGcxqfUFO0xuA4ukSoRTVZcVpSR X-Received: by 2002:a17:906:1b4a:: with SMTP id p10mr706559ejg.299.1590006930251; Wed, 20 May 2020 13:35:30 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1590006930; cv=none; d=google.com; s=arc-20160816; b=J4olfN+qjC+BoW/UlDxqzBYTQ1uM760rDgkr9TiWrWEWllymZTTZuPKaoVj06I8UCS ZfgSg8ByZhopGTd/fCEu+JcYSR0X0zkn1rAL+6DMq1wOdqLsv2i7AA0iN1AFQhMkKZYt YMXk8hDk8K12LS9FtXF0pNr+snBBzeVCuNqSZO3u76ltHLZCvnL9Qbs+bGegfSDN+XBp Cxc6KbjUVwRwLqAS8CI+HctH1T/dzcIDg3/ydW7BYSrxzb018BsTmAtirckWp+WlbtN9 eSb/QechuzipDbs4qaiwehrnyhT5+p8oCBpfWqq+c1Z4tu5g3WmP9NSmN3aaxjFjiBlB uv4Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:in-reply-to:content-transfer-encoding :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=5uXJBIfk9Q7QZF9nzfixlbtN7J1KaxdlddPN5ByIJAY=; b=yvvMkYdlIfR931J2hmfA+CNFfSHv1RxvXTsDQZbO/8UbkBNTQ5vPsTVH1neFDcgG4a s7iXid1qdkMMXFEunOeSQn/M8EJIlp/KdS8KVPdtA51pQGoZwadUFKE2TM5q5PmJ0Prk I/ubuFH5rnh9BTIeqgYk7ota2Pvy3N34v08QZo3y/zi+HEHyTKRPSyVZ0hhrnuuL9IgW LW+N7eou0mqx6pM7NdZGfNGPQPXKX6PymRWkaTIjYeaVQgQyqKPx0X+1WJDkT/MkkD5t UDMhCrT5xjVjb5GiXRynjUq96blOAUwtnksW9wIN53aX3VV1AAdoRo/jygT7+fYp9XUC E54w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=dYKlGwVJ; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id a17si1966217edj.251.2020.05.20.13.35.07; Wed, 20 May 2020 13:35:30 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=dYKlGwVJ; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727839AbgETUbc (ORCPT + 99 others); Wed, 20 May 2020 16:31:32 -0400 Received: from mail.kernel.org ([198.145.29.99]:37168 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726860AbgETUbc (ORCPT ); Wed, 20 May 2020 16:31:32 -0400 Received: from quaco.ghostprotocols.net (unknown [179.97.37.151]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 35763207E8; Wed, 20 May 2020 20:31:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1590006691; bh=2xWXSLA1iPuLF/4U+tLR1D4lYvQG7n2QJePYVkWCJy0=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=dYKlGwVJ9txWCu4XWhddSPDB9OnYexaWYo0MBtiGqs5nZehe8DK+ynPVGqHc/bwc9 65IEZpPK2mMKQaIHvyug76m1NYbGZVL2pynBTHZLQJeIl/jgBU4M/purLJxD3YuBGO hEicNJuiU5nn1FbbljTAhxeeG9qX/08LbqlIXVzw= Received: by quaco.ghostprotocols.net (Postfix, from userid 1000) id 8F7A040AFD; Wed, 20 May 2020 17:31:28 -0300 (-03) Date: Wed, 20 May 2020 17:31:28 -0300 From: Arnaldo Carvalho de Melo To: Changbin Du Cc: Jiri Olsa , Peter Zijlstra , Ingo Molnar , Namhyung Kim , Steven Rostedt , linux-kernel@vger.kernel.org Subject: Re: [PATCH 00/19] perf: ftrace enhancement Message-ID: <20200520203128.GN32678@kernel.org> References: <20200510150628.16610-1-changbin.du@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20200510150628.16610-1-changbin.du@gmail.com> X-Url: http://acmel.wordpress.com Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Em Sun, May 10, 2020 at 11:06:09PM +0800, Changbin Du escreveu: > The perf has basic kernel ftrace support but lack support of most tracing > options. This serias is target to enhance the perf ftrace functionality so > that we can make full use of kernel ftrace with only perf. Quite a lot! Going thru it now. - Arnaldo > In general, this serias be cataloged into two main changes: > 1) Improve usability of existing functions. For example, we don't need to type > extra option to select the tracer. > 2) Add new options to support all other ftrace functions. > > Here is a glance of all ftrace functions with this serias: > * - improved existing options. > + - new added options. > > $ sudo perf ftrace -h > > Usage: perf ftrace [] [] > or: perf ftrace [] -- [] > > * -a, --all-cpus system-wide collection from all CPUs > + -b, --buffer-size > size of per cpu buffer in kb > -C, --cpu list of cpus to monitor > + -d, --delay Wait ms before tracing > -D, --graph-depth > Max depth for function graph tracer > * -G, --graph-funcs > Set graph filter on given functions (imply to use function_graph tracer) > -g, --nograph-funcs > Set nograph filter on given functions (imply to use function_graph tracer) > + -L, --list-functions List available functions to filter > + -l, --long-info Show process names, PIDs, timestamps, irq-info if available > -N, --notrace-funcs > do not trace given functions > + -P, --no-pager Do not use pager > -p, --pid trace on existing process id > + -s, --func-stack-trace > Show kernel stack trace for function tracer > + -t, --tid trace on existing thread id (exclusive to --pid) > -T, --trace-funcs > trace given functions only > + -u, --userstacktrace Show stacktrace of the current user space thread > -v, --verbose be more verbose > + --funcgraph-tail Show function tails comment (function_graph only) > + --latency-format displays additional information about the latency (function_graph only) > + --nofuncgraph-irqs > Ignore functions that happen inside interrupt (function_graph only) > + --nosleep-time Measure on-CPU time only (function_graph only) > + --trace-children Trace children processes > + --tracing-thresh > Only show functions of which the duration is greater than ?s > > > Changbin Du (19): > perf ftrace: trace system wide if no target is given > perf ftrace: detect workload failure > perf ftrace: select function/function_graph tracer automatically > perf ftrace: add support for tracing option 'func_stack_trace' > perf ftrace: add option '-l/--list-functions' to list available > functions > perf ftrace: add support for trace option sleep-time > perf ftrace: add support for trace option funcgraph-irqs > perf ftrace: add option -l/--long-info to show more info > perf ftrace: add support for trace option tracing_thresh > perf ftrace: add support for trace option funcgraph-tail > perf ftrace: add option '-u/--userstacktrace' to show userspace > stacktrace > perf ftrace: add support for tracing children processes > perf ftrace: add option '-b/--buffer-size' to set per-cpu buffer size > perf ftrace: add option -P/--no-pager to disable pager > perf ftrace: show trace column header > perf ftrace: add option -t/--tid to filter by thread id > perf ftrace: add option -d/--delay to delay tracing > perf ftrace: add option --latency-format to display more info about > delay > perf ftrace: add change log > > tools/perf/Documentation/perf-config.txt | 5 - > tools/perf/builtin-ftrace.c | 388 ++++++++++++++++++++--- > 2 files changed, 345 insertions(+), 48 deletions(-) > > -- > 2.25.1 > -- - Arnaldo