Received: by 2002:a25:23cc:0:0:0:0:0 with SMTP id j195csp3161ybj; Wed, 6 May 2020 10:18:44 -0700 (PDT) X-Google-Smtp-Source: APiQypLiuPkb8vLeyurq+y+Spft5apX0sbxp+wD8wOyFvBGHpLFwJuCS02MYISe5AEFxOWk3I2+z X-Received: by 2002:a17:906:b2c2:: with SMTP id cf2mr8103087ejb.262.1588785524507; Wed, 06 May 2020 10:18:44 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1588785524; cv=none; d=google.com; s=arc-20160816; b=CYB1WQnwLIeJt/DInh+c34TwowiAmg5FZkH5d6mcXXj7yARAdZW76WfshhDt1BGQDe OxrGmWVtxsGMko+DJmVK6TWqXIayH57JnwtaH0KWdZCaqr3TXAzJ2HhaegR8uLc6O8zP EMgY0qlWgGAiBg1IRQ2Xk/wvFg+z1ACcS2Cv/VUNBC4JDAaaebIBXa9ZSYfNNBAT9LIX XD2QuP/OAWcgJvInh/PZaMa4sTlb3ejIdUS4th96NNFEdS1HaS7DOo+fo/3Ol6Xo78dV /QvSHPF1RvZ4vQ6nro4vlSErQNK/4ZzlL7l3uXLBncVCsjX+g4O7tRPl2PbG1AzIh2tt yxRw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:mime-version:user-agent:date:message-id :organization:cc:to:subject:from:ironport-sdr:ironport-sdr; bh=NA78kEqx4Ev5+8pnFjb8XtuzcwdffLHTNSf0ahFNIHk=; b=udpdzqrdbZC8G97xBAri8HJ55N9VcdUOpO+fawWIp+1n0SqGDERxct9DlplndzbLst Ep/xJNKCkLuLuTk+QSRSsUjjmqqIAUAONBjX8Ixt8l9aJmPlVqsue5J91rwnbqdUqGkV qpemLI5FvgS5goEozmFII/ipV2aeTL/dP0cK7LWcNfWF3Vqdi1qFEnuGysvBAQANwX5V PHkgYVjh68Wrt/q7TVJe6XtW2iPRQH3vR42SiIE0w3ccOw4LYlCz47bfLs/VRtJPvgmZ NSruLAm5acjIfP5ZUbZrvulHCV31ABbfrGLzH76jVnpPNx8bgVtDkoRtOl+GqDSadNev kRCQ== ARC-Authentication-Results: i=1; mx.google.com; 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=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id v29si1648382eda.533.2020.05.06.10.18.20; Wed, 06 May 2020 10:18:44 -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; 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=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728948AbgEFRQR (ORCPT + 99 others); Wed, 6 May 2020 13:16:17 -0400 Received: from mga01.intel.com ([192.55.52.88]:55139 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725799AbgEFRQR (ORCPT ); Wed, 6 May 2020 13:16:17 -0400 IronPort-SDR: JZiZOhXHPO8roHD5zsIcpU4Ecc/Cm5Dr92z82CQ0MmWVCkqBvJ2PSe3LDRTfkCBSz2ebLeGlVt ztA6v3Psm2hw== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga008.jf.intel.com ([10.7.209.65]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 06 May 2020 10:16:13 -0700 IronPort-SDR: 8GMkJ0IwKqfes7ax+V+jV+bh8iJp/P/4zHzD1dSLKooKzVjHVczptuoRa+T5iMGIzgSVqLiZk8 gtBkVFQUMGdA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.73,360,1583222400"; d="scan'208";a="296252742" Received: from linux.intel.com ([10.54.29.200]) by orsmga008.jf.intel.com with ESMTP; 06 May 2020 10:16:13 -0700 Received: from [10.249.228.227] (abudanko-mobl.ccr.corp.intel.com [10.249.228.227]) by linux.intel.com (Postfix) with ESMTP id 4020458043A; Wed, 6 May 2020 10:16:11 -0700 (PDT) From: Alexey Budankov Subject: [PATCH v2 00/11] perf: support enable and disable commands in stat and record modes To: Arnaldo Carvalho de Melo , Jiri Olsa Cc: Namhyung Kim , Alexander Shishkin , Peter Zijlstra , Ingo Molnar , Andi Kleen , linux-kernel Organization: Intel Corp. Message-ID: Date: Wed, 6 May 2020 20:16:10 +0300 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0 MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Changes in v2: - renamed resume and pause commands to enable and disable ones, renamed CTL_CMD_RESUME and CTL_CMD_PAUSE to CTL_CMD_ENABLE and CTL_CMD_DISABLE to fit to the appropriate ioctls and avoid mixing up with PAUSE_OUTPUT ioctl; - factored out event handling loop into a handle_events() for stat mode; - separated -D,--delay=-1 into separate patches for stat and record modes; v1: https://lore.kernel.org/lkml/825a5132-b58d-c0b6-b050-5a6040386ec7@linux.intel.com/ repo: tip of git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git perf/core The patch set implements handling of 'start disabled', 'enable' and 'disable' external control commands which can be provided for stat and record modes of the tool from an external controlling process. 'start disabled' command can be used to postpone enabling of events in the beginning of a monitoring session. 'enable' and 'disable' commands can be used to enable and disable events correspondingly any time after the start of the session. The 'start disabled', 'enable' and 'disable' external control commands can be used to focus measurement on specially selected time intervals of workload execution. Focused measurement reduces tool intrusion and influence on workload behavior, reduces distortion and amount of collected and stored data, mitigates data accuracy loss because measurement and data capturing happen only during intervals of interest. A controlling process can be a bash shell script [1], native executable or any other language program that can directly work with file descriptors, e.g. pipes [2], and spawn a process, specially the tool one. -D,--delay option is extended with -1 value to skip events enabling in the beginning of a monitoring session ('start disabled' command). --ctl-fd and --ctl-fd-ack command line options are introduced to provide the tool with a pair of file descriptors to listen to control commands and reply to the controlling process on the completion of received commands. The tool reads two byte control command message from ctl-fd descriptor, handles the command and optionally replies two bytes acknowledgement message to fd-ack descriptor, if it is specified on the command line. 'enable' command is recognized as 'e' character message and 'disable' command is recognized as 'd' character message both received from ctl-fd descriptor. Completion message is 'a''\n' and sent to fd-ack descriptor. example bash script demonstrating simple use case: #!/bin/bash ctl_dir=/tmp/ ctl_fifo=${ctl_dir}perf_ctl.fifo test -p ${ctl_fifo} && unlink ${ctl_fifo} mkfifo ${ctl_fifo} && exec {ctl_fd}<>${ctl_fifo} ctl_ack_fifo=${ctl_dir}perf_ctl_ack.fifo test -p ${ctl_ack_fifo} && unlink ${ctl_ack_fifo} mkfifo ${ctl_ack_fifo} && exec {ctl_fd_ack}<>${ctl_ack_fifo} perf stat -D -1 -e cpu-cycles -a -I 1000 \ --ctl-fd ${ctl_fd} --ctl-fd-ack ${ctl_fd_ack} \ -- sleep 40 & perf_pid=$! sleep 5 && echo 'e' >&${ctl_fd} && read -u ${ctl_fd_ack} e1 && echo "enabled(${e1})" sleep 10 && echo 'd' >&${ctl_fd} && read -u ${ctl_fd_ack} d1 && echo "disabled(${d1})" sleep 5 && echo 'e' >&${ctl_fd} && read -u ${ctl_fd_ack} e2 && echo "enabled(${e2})" sleep 10 && echo 'd' >&${ctl_fd} && read -u ${ctl_fd_ack} d2 && echo "disabled(${d2})" exec {ctl_fd_ack}>&- && unlink ${ctl_ack_fifo} exec {ctl_fd}>&- && unlink ${ctl_fifo} wait -n ${perf_pid} exit $? Script output: [root@host acme] example Events disabled # time counts unit events 1.001101062 cpu-cycles 2.002994944 cpu-cycles 3.004864340 cpu-cycles 4.006727177 cpu-cycles Events enabled enabled(a) 4.993808464 3,124,246 cpu-cycles 5.008597004 3,325,624 cpu-cycles 6.010387483 83,472,992 cpu-cycles 7.012266598 55,877,621 cpu-cycles 8.014175695 97,892,729 cpu-cycles 9.016056093 68,461,242 cpu-cycles 10.017937507 55,449,643 cpu-cycles 11.019830154 68,938,167 cpu-cycles 12.021719952 55,164,101 cpu-cycles 13.023627550 70,535,720 cpu-cycles 14.025580995 53,240,125 cpu-cycles disabled(a) 14.997518260 53,558,068 cpu-cycles Events disabled 15.027216416 cpu-cycles 16.029052729 cpu-cycles 17.030904762 cpu-cycles 18.032073424 cpu-cycles 19.033805074 cpu-cycles Events enabled enabled(a) 20.001279097 3,021,022 cpu-cycles 20.035044381 6,434,367 cpu-cycles 21.036923813 89,358,251 cpu-cycles 22.038825169 72,516,351 cpu-cycles # time counts unit events 23.040715596 55,046,157 cpu-cycles 24.042643757 78,128,649 cpu-cycles 25.044558535 61,052,428 cpu-cycles 26.046452785 62,142,806 cpu-cycles 27.048353021 74,477,971 cpu-cycles 28.050241286 61,001,623 cpu-cycles 29.052149961 61,653,502 cpu-cycles disabled(a) 30.004980264 82,729,640 cpu-cycles Events disabled 30.053516176 cpu-cycles 31.055348366 cpu-cycles 32.057202097 cpu-cycles 33.059040702 cpu-cycles 34.060843288 cpu-cycles 35.000888624 cpu-cycles [root@host acme]# [1] http://man7.org/linux/man-pages/man1/bash.1.html [2] http://man7.org/linux/man-pages/man2/pipe.2.html --- Alexey Budankov (11): perf evlist: introduce control file descriptors perf evlist: implement control command handling functions perf stat: factor out event handling loop into a function perf stat: extend -D,--delay option with -1 value perf stat: implement control commands handling perf stat: introduce --ctl-fd[-ack] options perf docs: extend stat mode docs with info on --ctl-fd[-ack] options perf record: extend -D,--delay option with -1 value perf record: implement control commands handling perf record: introduce --ctl-fd[-ack] options perf docs: extend record mode docs with info on --ctl-fd[-ack] options tools/perf/Documentation/perf-record.txt | 39 +++++++ tools/perf/Documentation/perf-stat.txt | 40 +++++++ tools/perf/builtin-record.c | 39 ++++++- tools/perf/builtin-stat.c | 137 ++++++++++++++++------- tools/perf/builtin-trace.c | 2 +- tools/perf/util/evlist.c | 103 +++++++++++++++++ tools/perf/util/evlist.h | 18 +++ tools/perf/util/record.h | 4 +- tools/perf/util/stat.h | 4 +- 9 files changed, 339 insertions(+), 47 deletions(-) -- 2.24.1