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=-26.3 required=3.0 tests=BAYES_00,DKIMWL_WL_MED, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_CR_TRAILER,INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, USER_AGENT_GIT,USER_IN_DEF_DKIM_WL autolearn=ham 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 151B3C64E69 for ; Wed, 18 Nov 2020 05:04:03 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id BDBED246B5 for ; Wed, 18 Nov 2020 05:04:02 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="PbNwL3SN" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1725834AbgKRFDq (ORCPT ); Wed, 18 Nov 2020 00:03:46 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36552 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726195AbgKRFDp (ORCPT ); Wed, 18 Nov 2020 00:03:45 -0500 Received: from mail-pj1-x104a.google.com (mail-pj1-x104a.google.com [IPv6:2607:f8b0:4864:20::104a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C0561C0613D4 for ; Tue, 17 Nov 2020 21:03:45 -0800 (PST) Received: by mail-pj1-x104a.google.com with SMTP id o10so553549pjr.1 for ; Tue, 17 Nov 2020 21:03:45 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=sender:date:in-reply-to:message-id:mime-version:references:subject :from:to:cc; bh=vGdxQHqtpriKkfthftM5p2GKc/XM2K+zaCfAHe4o9VE=; b=PbNwL3SN8jkxJU6aTvWM84Y/M/KjekWLP4CucXxZuWNxCc8mUNUAXKeQkwoSCbHtrC rr7AcCp9YAHX27bi77k1nCn43v7l0oy6QdQBvbliqq+A/21tAm9L+MXcGAyL8SNt8IuQ u4L11Dt9dvLI5ZmwjC8lEFuFQzNR4+kWstmuDCvraHhi0FZ6B26pBbpFCpbp0pZciz4m LXJYkAkk0LShtZ5M66hHxYRIKUyjsajuQ070du91ZjNxzhjMisFtDedBVLLcl+5Mc6L+ dqXyE8wdJx4som03r3CeXsakFPh6mgFvZVlLgLTHJ3TxsRnZ1qiVnIy8lmfTmnBOsGMR 9wAw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:date:in-reply-to:message-id:mime-version :references:subject:from:to:cc; bh=vGdxQHqtpriKkfthftM5p2GKc/XM2K+zaCfAHe4o9VE=; b=afKUGs+Wlp1IcPQVhOnAyf8Wab++UM/MUYSXPsPtCNlZUyNYEUIbU6J7Jk9sMg8fic mL8llVczzQOddhbyeWFAHisZ/LBRfMwYe+gHucuX3T+BW1uA4I0KG5gJtREKnsHBcTxc kmDTzMum3fyLRroZzdcf2dhXnITaQxEnJ1C0qyW5D5wtzJVEn1Qi1ohSnJOIMpY3aqlg K1fcj4iPPUPCUpMkq+wk3NcPGSoKmrvSJS8vE6Cev/QOgPHBKhXtJ0Uqa+GoInD0fTSY t013cppY3qJBnl3QAohd7C1f8BuHBlR4L8aZJfj3yhuHz0oIkr7ZK+T6IHzC3EXws/Pk azBQ== X-Gm-Message-State: AOAM533oPRfLI6gCDHWKhioztNHWxXKcmtU8wjnXDXpdkqtf4nGbiAZ9 +nj6+HoPDZZszsLzp8MkD3op+I6pLKU3 X-Google-Smtp-Source: ABdhPJx32TWQrQpd0xiLh2PfwIkaB41Fzsl4VTfOYFyyGG3VEFnW7yAIyELeyu41MOF6WpWqdX5aWnLdKHrA Sender: "irogers via sendgmr" X-Received: from irogers.svl.corp.google.com ([2620:15c:2cd:2:f693:9fff:fef4:4583]) (user=irogers job=sendgmr) by 2002:a17:90a:d250:: with SMTP id o16mr2339649pjw.25.1605675825201; Tue, 17 Nov 2020 21:03:45 -0800 (PST) Date: Tue, 17 Nov 2020 21:03:33 -0800 In-Reply-To: <20201118050335.2347358-1-irogers@google.com> Message-Id: <20201118050335.2347358-4-irogers@google.com> Mime-Version: 1.0 References: <20201118050335.2347358-1-irogers@google.com> X-Mailer: git-send-email 2.29.2.299.gdc1121823c-goog Subject: [PATCH v2 3/5] perf metric: Rename expr__find_other. From: Ian Rogers To: Peter Zijlstra , Ingo Molnar , Arnaldo Carvalho de Melo , Mark Rutland , Alexander Shishkin , Jiri Olsa , Namhyung Kim , linux-kernel@vger.kernel.org, Andi Kleen , Jin Yao , John Garry , Paul Clarke , kajoljain Cc: Stephane Eranian , Sandeep Dasgupta , linux-perf-users@vger.kernel.org, Ian Rogers Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-perf-users@vger.kernel.org Archived-At: List-Archive: List-Post: A later change will remove the notion of other, rename the function to expr__find_ids as this is what it populated. Signed-off-by: Ian Rogers --- tools/perf/tests/expr.c | 8 ++++---- tools/perf/tests/pmu-events.c | 9 ++++----- tools/perf/util/expr.c | 4 ++-- tools/perf/util/expr.h | 2 +- tools/perf/util/metricgroup.c | 2 +- tools/perf/util/stat-shadow.c | 6 +++--- 6 files changed, 15 insertions(+), 16 deletions(-) diff --git a/tools/perf/tests/expr.c b/tools/perf/tests/expr.c index b0a3b5fd0c00..9d7032041318 100644 --- a/tools/perf/tests/expr.c +++ b/tools/perf/tests/expr.c @@ -65,8 +65,8 @@ int test__expr(struct test *t __maybe_unused, int subtest __maybe_unused) expr__ctx_clear(ctx); TEST_ASSERT_VAL("find other", - expr__find_other("FOO + BAR + BAZ + BOZO", "FOO", - ctx, 1) == 0); + expr__find_ids("FOO + BAR + BAZ + BOZO", "FOO", + ctx, 1) == 0); TEST_ASSERT_VAL("find other", hashmap__size(ctx->ids) == 3); TEST_ASSERT_VAL("find other", hashmap__find(ctx->ids, "BAR", (void **)&val_ptr)); @@ -77,8 +77,8 @@ int test__expr(struct test *t __maybe_unused, int subtest __maybe_unused) expr__ctx_clear(ctx); TEST_ASSERT_VAL("find other", - expr__find_other("EVENT1\\,param\\=?@ + EVENT2\\,param\\=?@", - NULL, ctx, 3) == 0); + expr__find_ids("EVENT1\\,param\\=?@ + EVENT2\\,param\\=?@", + NULL, ctx, 3) == 0); TEST_ASSERT_VAL("find other", hashmap__size(ctx->ids) == 2); TEST_ASSERT_VAL("find other", hashmap__find(ctx->ids, "EVENT1,param=3/", (void **)&val_ptr)); diff --git a/tools/perf/tests/pmu-events.c b/tools/perf/tests/pmu-events.c index 294daf568bb6..3ac70fa31379 100644 --- a/tools/perf/tests/pmu-events.c +++ b/tools/perf/tests/pmu-events.c @@ -502,9 +502,8 @@ static int test_parsing(void) if (!pe->metric_expr) continue; expr__ctx_clear(ctx); - if (expr__find_other(pe->metric_expr, NULL, ctx, 0) - < 0) { - expr_failure("Parse other failed", map, pe); + if (expr__find_ids(pe->metric_expr, NULL, ctx, 0) < 0) { + expr_failure("Parse find ids failed", map, pe); ret++; continue; } @@ -559,8 +558,8 @@ static int metric_parse_fake(const char *str) pr_debug("parsing '%s'\n", str); ctx = expr__ctx_new(); - if (expr__find_other(str, NULL, ctx, 0) < 0) { - pr_err("expr__find_other failed\n"); + if (expr__find_ids(str, NULL, ctx, 0) < 0) { + pr_err("expr__find_ids failed\n"); return -1; } diff --git a/tools/perf/util/expr.c b/tools/perf/util/expr.c index e0623d38e6ee..a248d14882cc 100644 --- a/tools/perf/util/expr.c +++ b/tools/perf/util/expr.c @@ -287,8 +287,8 @@ int expr__parse(double *final_val, struct expr_parse_ctx *ctx, return __expr__parse(final_val, ctx, expr, EXPR_PARSE, runtime) ? -1 : 0; } -int expr__find_other(const char *expr, const char *one, - struct expr_parse_ctx *ctx, int runtime) +int expr__find_ids(const char *expr, const char *one, + struct expr_parse_ctx *ctx, int runtime) { int ret = __expr__parse(NULL, ctx, expr, EXPR_OTHER, runtime); diff --git a/tools/perf/util/expr.h b/tools/perf/util/expr.h index 00b941cfe6a6..955d5adb7ca4 100644 --- a/tools/perf/util/expr.h +++ b/tools/perf/util/expr.h @@ -43,7 +43,7 @@ int expr__resolve_id(struct expr_parse_ctx *ctx, const char *id, struct expr_id_data **datap); int expr__parse(double *final_val, struct expr_parse_ctx *ctx, const char *expr, int runtime); -int expr__find_other(const char *expr, const char *one, +int expr__find_ids(const char *expr, const char *one, struct expr_parse_ctx *ids, int runtime); double expr_id_data__value(const struct expr_id_data *data); diff --git a/tools/perf/util/metricgroup.c b/tools/perf/util/metricgroup.c index 342dcccb860f..0be684bb020f 100644 --- a/tools/perf/util/metricgroup.c +++ b/tools/perf/util/metricgroup.c @@ -761,7 +761,7 @@ static int __add_metric(struct list_head *metric_list, * For both the parent and referenced metrics, we parse * all the metric's IDs and add it to the parent context. */ - if (expr__find_other(pe->metric_expr, NULL, m->pctx, runtime) < 0) { + if (expr__find_ids(pe->metric_expr, NULL, m->pctx, runtime) < 0) { if (m->metric_refs_cnt == 0) { expr__ctx_free(m->pctx); free(m); diff --git a/tools/perf/util/stat-shadow.c b/tools/perf/util/stat-shadow.c index bea7b5c6b1c0..91bb7245f8b1 100644 --- a/tools/perf/util/stat-shadow.c +++ b/tools/perf/util/stat-shadow.c @@ -357,9 +357,9 @@ void perf_stat__collect_metric_expr(struct evlist *evsel_list) expr__ctx_clear(ctx); metric_events = counter->metric_events; if (!metric_events) { - if (expr__find_other(counter->metric_expr, - counter->name, - ctx, 1) < 0) + if (expr__find_ids(counter->metric_expr, + counter->name, + ctx, 1) < 0) continue; metric_events = calloc(sizeof(struct evsel *), -- 2.29.2.299.gdc1121823c-goog