Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756411AbdCGUnY (ORCPT ); Tue, 7 Mar 2017 15:43:24 -0500 Received: from mx0b-001b2d01.pphosted.com ([148.163.158.5]:33510 "EHLO mx0a-001b2d01.pphosted.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1756268AbdCGUmM (ORCPT ); Tue, 7 Mar 2017 15:42:12 -0500 Subject: [PATCH v8 0/6] perf: Add support for analyzing events for containers From: Hari Bathini To: ast@fb.com, peterz@infradead.org, lkml , acme@kernel.org, alexander.shishkin@linux.intel.com, mingo@redhat.com Cc: daniel@iogearbox.net, rostedt@goodmis.org, Ananth N Mavinakayanahalli , ebiederm@xmission.com, sargun@sargun.me, Aravinda Prasad , brendan.d.gregg@gmail.com, jolsa@redhat.com Date: Wed, 08 Mar 2017 02:11:06 +0530 User-Agent: StGit/0.17.1-dirty MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit X-TM-AS-MML: disable x-cbid: 17030720-0016-0000-0000-00000222E1CD X-IBM-AV-DETECTION: SAVI=unused REMOTE=unused XFE=unused x-cbparentid: 17030720-0017-0000-0000-00000690000E Message-Id: <148891921533.25309.8328657213311313206.stgit@hbathini.in.ibm.com> X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2017-03-07_16:,, signatures=0 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 spamscore=0 suspectscore=0 malwarescore=0 phishscore=0 adultscore=0 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1702020001 definitions=main-1703070162 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4221 Lines: 87 Currently, there is no trivial mechanism to analyze events based on containers. perf -G can be used, but it will not filter events for the containers created after perf is invoked, making it difficult to assess/ analyze performance issues of multiple containers at once. This patch-set is aimed at addressing this limitation by introducing a new PERF_RECORD_NAMESPACES event that records namespaces related info. As containers are created with namespaces, the new data can be used to in assessment/analysis of multiple containers. The first patch introduces PERF_RECORD_NAMESPACES in kernel while the second patch makes the corresponding changes in perf tool to read this PERF_RECORD_NAMESPACES events. The third patch demonstrates analysis of containers with this data by adding a cgroup identifier column in perf report, which contains the cgroup namespace's device and inode numbers. This is based on the assumption that each container is created with it's own cgroup namespace. The third patch has scope for improvement based on the conventions a container is attributed with, going forward. Changes from v7: * Folded doc related patches into corresponding code changes * Got rid of NAMESPACES_MAX macro and using a zero array instead * Updated changelog of few patches Changes from v6: * Updated changelog of patch 1 * Split patch 2 into smaller patches * Updated record and script documenatation * Dropped name field from ns_link_info struct Changes from v5: * Updated changelogs of patches 1 & 3 * Rebased the patches on perf/core in tip --- Hari Bathini (6): perf: Add PERF_RECORD_NAMESPACES to include namespaces related info perf tools: Add PERF_RECORD_NAMESPACES to include namespaces related info perf record: Synthesize namespace events for current processes perf tools: Add print support for namespace events perf script: Add script print support for namespace events perf tools: Add cgroup identifier sort order keyword include/linux/perf_event.h | 2 include/uapi/linux/perf_event.h | 32 ++++++ kernel/events/core.c | 139 ++++++++++++++++++++++++++++ kernel/fork.c | 2 kernel/nsproxy.c | 3 + tools/include/uapi/linux/perf_event.h | 32 ++++++ tools/perf/Documentation/perf-record.txt | 3 + tools/perf/Documentation/perf-report.txt | 4 + tools/perf/Documentation/perf-script.txt | 3 + tools/perf/builtin-annotate.c | 1 tools/perf/builtin-diff.c | 1 tools/perf/builtin-inject.c | 13 +++ tools/perf/builtin-kmem.c | 1 tools/perf/builtin-kvm.c | 2 tools/perf/builtin-lock.c | 1 tools/perf/builtin-mem.c | 1 tools/perf/builtin-record.c | 35 ++++++- tools/perf/builtin-report.c | 1 tools/perf/builtin-sched.c | 1 tools/perf/builtin-script.c | 41 ++++++++ tools/perf/builtin-trace.c | 3 - tools/perf/perf.h | 1 tools/perf/util/Build | 1 tools/perf/util/data-convert-bt.c | 1 tools/perf/util/event.c | 150 +++++++++++++++++++++++++++++- tools/perf/util/event.h | 19 ++++ tools/perf/util/evsel.c | 3 + tools/perf/util/hist.c | 7 + tools/perf/util/hist.h | 1 tools/perf/util/machine.c | 34 +++++++ tools/perf/util/machine.h | 3 + tools/perf/util/namespaces.c | 36 +++++++ tools/perf/util/namespaces.h | 26 +++++ tools/perf/util/session.c | 7 + tools/perf/util/sort.c | 41 ++++++++ tools/perf/util/sort.h | 7 + tools/perf/util/thread.c | 44 ++++++++- tools/perf/util/thread.h | 6 + tools/perf/util/tool.h | 2 39 files changed, 694 insertions(+), 16 deletions(-) create mode 100644 tools/perf/util/namespaces.c create mode 100644 tools/perf/util/namespaces.h