Received: by 2002:a05:6a10:206:0:0:0:0 with SMTP id 6csp4511245pxj; Tue, 22 Jun 2021 01:46:58 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwD2rbpg3nh1spch7DgSDBbfhVnyNIrG3uTotgB8Tss6ElecqMM9V6N+mUq6L2VHU9vYbM9 X-Received: by 2002:a05:6402:151:: with SMTP id s17mr3368055edu.233.1624351618126; Tue, 22 Jun 2021 01:46:58 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1624351618; cv=none; d=google.com; s=arc-20160816; b=uZHp92944fEaVfly5HbZoQkBcWxeHqmsXByle6C7exvISsJC2innAfHaenRTsmnyWQ 6Gj7pcE3apQrXV8UVLdgUclnqJioxFU0ROPGDlLuMmRn/bJHJRG2rlFbKJSlYaMsJnn0 lHJaasMx9drQH29ic24uxbNJTVHtzQRJ37Ro5NS+CVBs2WcrlzgK0T1RajHiaY+2AGi3 Z4dd+xEh+oFU/Xqeq+31AhckBvIdIGsNW+ycrhmrkf9EFv2EU0oH5WORvscjb0n3/9UT FcDqOV4zXDiEZJi77Xc8zQ5b27hvxbru41CfP65bDr0DenG47gi5UPDGpVoZkTpJCNxh MIKA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :ironport-sdr:ironport-sdr; bh=UJgf5zDApD1QeTx3NaQTfviYbUXKPjXxjkylTivFUK4=; b=v5Z8f5Smp8DOzEvaInYjYbRCKhOxzHvaeL9kscp+WFLOX1QfFtzVl5lVqjC/453flq iLeTKbdU/oM5BdjmBhRQ+qlqwJ6jWXpUWoYQ2RIiN63gic5Oxor7W1OGwnfLXeJaLvOZ iPqD5HkWdlnDzq5SkU64xadmOYCd0P+LxQs5FXesPzkVGBR883U/PrNxUQYpS0d0Fijj 5tYOo8MUUqavS5WK5Ptih5YVKWYmaUFx79JYxGqkjkrLp4NoRzoln21ZGEfEkMwKxYtb a7BvxDfvApVibHTq+6Mdjy5kNp8aibjpe1InFlU36NMEDrcyG+vXmKZEiSceU8cjxk4V Fe1A== 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 n17si1555254edr.161.2021.06.22.01.46.36; Tue, 22 Jun 2021 01:46:58 -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 S231216AbhFVIpS (ORCPT + 99 others); Tue, 22 Jun 2021 04:45:18 -0400 Received: from mga05.intel.com ([192.55.52.43]:57595 "EHLO mga05.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231174AbhFVIpO (ORCPT ); Tue, 22 Jun 2021 04:45:14 -0400 IronPort-SDR: yDl1/wIjJQIEh7H3zB8HSW7jTpX/F9rwmMVCfwuFrJ3D3obX2KQvUG5YKGG/9T7m52Z0YC1ZyW zsy0XLjwVhkw== X-IronPort-AV: E=McAfee;i="6200,9189,10022"; a="292641565" X-IronPort-AV: E=Sophos;i="5.83,291,1616482800"; d="scan'208";a="292641565" Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by fmsmga105.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 22 Jun 2021 01:42:58 -0700 IronPort-SDR: D18rjCsHIdTQn4PHLi3GSow2jkYTyqzp1nqqiQg7TI/7XWyq7T7navPStt6rvei/fyxre0v9lk QSVE+mPOzMfg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.83,291,1616482800"; d="scan'208";a="417332548" Received: from nntpat99-84.inn.intel.com ([10.125.99.84]) by fmsmga007.fm.intel.com with ESMTP; 22 Jun 2021 01:42:55 -0700 From: Alexey Bayduraev To: Arnaldo Carvalho de Melo Cc: Jiri Olsa , Namhyung Kim , Alexander Shishkin , Peter Zijlstra , Ingo Molnar , linux-kernel , Andi Kleen , Adrian Hunter , Alexander Antonov , Alexei Budankov , Riccardo Mancini Subject: [PATCH v7 08/20] perf record: Init data file at mmap buffer object Date: Tue, 22 Jun 2021 11:42:17 +0300 Message-Id: X-Mailer: git-send-email 2.19.0 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Initialize data files located at mmap buffer objects so trace data can be written into several data file located at data directory. Acked-by: Andi Kleen Signed-off-by: Alexey Bayduraev --- tools/perf/builtin-record.c | 41 ++++++++++++++++++++++++++++++------- tools/perf/util/record.h | 1 + 2 files changed, 35 insertions(+), 7 deletions(-) diff --git a/tools/perf/builtin-record.c b/tools/perf/builtin-record.c index c9fd31211600..1b6716778650 100644 --- a/tools/perf/builtin-record.c +++ b/tools/perf/builtin-record.c @@ -160,6 +160,11 @@ static const char *affinity_tags[PERF_AFFINITY_MAX] = { "SYS", "NODE", "CPU" }; +static int record__threads_enabled(struct record *rec) +{ + return rec->opts.threads_spec; +} + static bool switch_output_signal(struct record *rec) { return rec->switch_output.signal && @@ -1070,7 +1075,7 @@ static void record__free_thread_data(struct record *rec) static int record__mmap_evlist(struct record *rec, struct evlist *evlist) { - int ret; + int i, ret; struct record_opts *opts = &rec->opts; bool auxtrace_overwrite = opts->auxtrace_snapshot_mode || opts->auxtrace_sample_mode; @@ -1109,6 +1114,18 @@ static int record__mmap_evlist(struct record *rec, if (ret) return ret; + if (record__threads_enabled(rec)) { + ret = perf_data__create_dir(&rec->data, evlist->core.nr_mmaps); + if (ret) + return ret; + for (i = 0; i < evlist->core.nr_mmaps; i++) { + if (evlist->mmap) + evlist->mmap[i].file = &rec->data.dir.files[i]; + if (evlist->overwrite_mmap) + evlist->overwrite_mmap[i].file = &rec->data.dir.files[i]; + } + } + return 0; } @@ -1416,8 +1433,12 @@ static int record__mmap_read_evlist(struct record *rec, struct evlist *evlist, /* * Mark the round finished in case we wrote * at least one event. + * + * No need for round events in directory mode, + * because per-cpu maps and files have data + * sorted by kernel. */ - if (bytes_written != rec->bytes_written) + if (!record__threads_enabled(rec) && bytes_written != rec->bytes_written) rc = record__write(rec, NULL, &finished_round_event, sizeof(finished_round_event)); if (overwrite) @@ -1532,7 +1553,9 @@ static void record__init_features(struct record *rec) if (!rec->opts.use_clockid) perf_header__clear_feat(&session->header, HEADER_CLOCK_DATA); - perf_header__clear_feat(&session->header, HEADER_DIR_FORMAT); + if (!record__threads_enabled(rec)) + perf_header__clear_feat(&session->header, HEADER_DIR_FORMAT); + if (!record__comp_enabled(rec)) perf_header__clear_feat(&session->header, HEADER_COMPRESSED); @@ -1543,15 +1566,21 @@ static void record__finish_output(struct record *rec) { struct perf_data *data = &rec->data; - int fd = perf_data__fd(data); + int i, fd = perf_data__fd(data); if (data->is_pipe) return; rec->session->header.data_size += rec->bytes_written; data->file.size = lseek(perf_data__fd(data), 0, SEEK_CUR); + if (record__threads_enabled(rec)) { + for (i = 0; i < data->dir.nr; i++) + data->dir.files[i].size = lseek(data->dir.files[i].fd, 0, SEEK_CUR); + } if (!rec->no_buildid) { + /* this will be recalculated during process_buildids() */ + rec->samples = 0; process_buildids(rec); if (rec->buildid_all) @@ -2489,8 +2518,6 @@ static int __cmd_record(struct record *rec, int argc, const char **argv) status = err; record__synthesize(rec, true); - /* this will be recalculated during process_buildids() */ - rec->samples = 0; if (!err) { if (!rec->timestamp_filename) { @@ -3277,7 +3304,7 @@ int cmd_record(int argc, const char **argv) rec->no_buildid = true; } - if (rec->opts.kcore) + if (rec->opts.kcore || record__threads_enabled(rec)) rec->data.is_dir = true; if (rec->opts.comp_level != 0) { diff --git a/tools/perf/util/record.h b/tools/perf/util/record.h index 68f471d9a88b..4d68b7e27272 100644 --- a/tools/perf/util/record.h +++ b/tools/perf/util/record.h @@ -77,6 +77,7 @@ struct record_opts { int ctl_fd; int ctl_fd_ack; bool ctl_fd_close; + int threads_spec; }; extern const char * const *record_usage; -- 2.19.0