Received: by 2002:a05:6602:18e:0:0:0:0 with SMTP id m14csp4597699ioo; Tue, 31 May 2022 07:44:58 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxnhYVUGfyvH+I0g6T1DDUqlDhGoIqI2ymCGgK3RavuekEU46RDT9IjORwtVBpNUwBszHOR X-Received: by 2002:a63:e458:0:b0:3fc:5145:356a with SMTP id i24-20020a63e458000000b003fc5145356amr3752156pgk.320.1654008297815; Tue, 31 May 2022 07:44:57 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1654008297; cv=none; d=google.com; s=arc-20160816; b=0ABF614wHFy5OyaT1Q2h1hPdEG8I6PuQj/Iwyv5HTXURDD6M73Qy7jjbAL0wFsJuNK MZ/rBuPaV82q7vxmK4QPRTObHEiLVDqQHXbsLReyIqUCI/jJaHdGGuGPOazqVVwLe7Zv EdWsorhv7BoiS8dXOWw0YZvKnK/U2i+ejla8x6oP4jhyPkiWHT/1I7Bl7fIETPrccPgK lsd0rzAIumnvVdPTeGuS3KJ7wJq+C3JWlLz6mUtoUzKLA+bxNTOxfoJDRoZyh69YzekO PxgJ03RhjqiHB5WX4yrRH2tLnk79fWdT5SQb6KOmvRWBIuZSzkm6jqafXqQa9NZFJpvI 9fsQ== 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 :dkim-signature; bh=jMrBxZYPbeFUurfukQ/2U4FIRA07iFNb/YD2s1OiXOs=; b=QxMBwolwxrmYNxRycNkPXqKh40InDSpDJaj11LrzjCAcNsT6Eqrq4vaO+xRkwxDZIv nb713u9IOkOGrWmk+5RJHprmwPyvsZaL10BLItRKc/nIcjAiTmjSUKgecj+nolESeeSx x/cVlti9Hf7DItg16LZsMNnE7c6KOAEsVjZr7NPcsPV3VMR/vdriwWSKpWc8WZYc92QW 5H9QjeUHnLIhbn+2Ai7HVlU8w6wUB31BKR3mD83701chwylmyp+rTgz0Xt9+pSeNwYW3 sj2JzRUxKMpTxixxnnXK5JtwuaN2keyQZsDFt1Gu1K/DLH5RttZx5ryGg4tCrCF/DTn7 r9pA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=kM6JNT2q; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id mq2-20020a17090b380200b001c72275b573si3311221pjb.173.2022.05.31.07.44.43; Tue, 31 May 2022 07:44:57 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=kM6JNT2q; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235982AbiE3LmG (ORCPT + 99 others); Mon, 30 May 2022 07:42:06 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42748 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235980AbiE3Llv (ORCPT ); Mon, 30 May 2022 07:41:51 -0400 Received: from mail-pl1-x62b.google.com (mail-pl1-x62b.google.com [IPv6:2607:f8b0:4864:20::62b]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C20B2B57 for ; Mon, 30 May 2022 04:41:34 -0700 (PDT) Received: by mail-pl1-x62b.google.com with SMTP id n8so10114642plh.1 for ; Mon, 30 May 2022 04:41:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=jMrBxZYPbeFUurfukQ/2U4FIRA07iFNb/YD2s1OiXOs=; b=kM6JNT2qmeWTwEt34yIFbrqnzAHYZBlZElPZAixACO+F0lhiA9WJC6YwLSVQGnHa3T KVZJZFtLuIIiGYnWQXYIukbCJgJy1svErJQzmmNi6JhZ6uWGgNCHqNXjUD6Fo/xpq5eF 525wDmx18xKKhHOSr+M/UZkG/dU+FBmTb4UuDvIJF0oLA+Ewaho4XLrKZpH/ArfQTXfa MnLYClQ4lkhYc9BSC7SKpHpNerelkkeXnyhz3Hls2AVB2N/9MH08dA+t02mBgA+1ZJp8 RhFrOWQn2jMM+GCFexw0K7hioRgPW/N7kRkriy+jgzjcuB6KppKLabOeIpQwPE6jVHCe UWng== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=jMrBxZYPbeFUurfukQ/2U4FIRA07iFNb/YD2s1OiXOs=; b=1cmWk9Wcds4uysThARQLWPhW/cRmZI1ZhOkxO716XjID5+jMWLzVRb/hVNE/70KRZj sV2LCmAI9AQKIwH+FWknSk/mf8IFzn5XRb1SrwT4gJg/0/K16K+54+IYl+mh7fqRzFSa UkhHFxCc1qpsacFXnzdwDdBtjBBFiIO9vHbCRelCwEuS3G+oxnCMZ3QNgFDKE4BMlRAs WzkrHt9oY2AjEcEjSUPaLOLogUUpHbC3cqHOhV7gXetjz3EMed3t0fzX0HF+3BUijYsQ otxNHRR7Sz8gHq1O5TdKfnPtb6Sf+T+H+BZe+dRiORnsgvWZx8ikCvT/oWgsrIqVgB6n Tl8w== X-Gm-Message-State: AOAM5303LPXsz56HoWBHRCHIxgDqXT4i0bc5THn+Qw8DoJ7WobgYgUNb Jy5OL0/sjJmNFhjQRK5yre7HsBA11wTJZZ30cf1cfw== X-Received: by 2002:a17:902:8645:b0:15f:2eb5:3d49 with SMTP id y5-20020a170902864500b0015f2eb53d49mr55910710plt.42.1653910893477; Mon, 30 May 2022 04:41:33 -0700 (PDT) Received: from leo-build-box.lan (n058152077182.netvigator.com. [58.152.77.182]) by smtp.gmail.com with ESMTPSA id c7-20020a170902724700b00161a9df4de8sm8846194pll.145.2022.05.30.04.41.28 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 30 May 2022 04:41:33 -0700 (PDT) From: Leo Yan To: Arnaldo Carvalho de Melo , Peter Zijlstra , Ingo Molnar , Mark Rutland , Alexander Shishkin , Jiri Olsa , Namhyung Kim , Alyssa Ross , Ian Rogers , Like Xu , Kajol Jain , Li Huafei , Joe Mario , Adam Li , German Gomez , James Clark , Ali Saidi , linux-perf-users@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Leo Yan Subject: [PATCH v4 07/12] perf c2c: Rename dimension from 'percent_hitm' to 'percent_costly_snoop' Date: Mon, 30 May 2022 19:40:31 +0800 Message-Id: <20220530114036.3225544-8-leo.yan@linaro.org> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20220530114036.3225544-1-leo.yan@linaro.org> References: <20220530114036.3225544-1-leo.yan@linaro.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Use more general naming for the main sort dimension, this can allow us not to sort only on HITM snoop type, so it can be extended to support other costly snooping operations. So rename the dimension to the prefix 'percent_costly_". Signed-off-by: Leo Yan Tested-by: Ali Saidi --- tools/perf/builtin-c2c.c | 40 ++++++++++++++++++++-------------------- 1 file changed, 20 insertions(+), 20 deletions(-) diff --git a/tools/perf/builtin-c2c.c b/tools/perf/builtin-c2c.c index b50b23ea28ec..d3608d24bea4 100644 --- a/tools/perf/builtin-c2c.c +++ b/tools/perf/builtin-c2c.c @@ -798,7 +798,7 @@ percent_color(struct perf_hpp_fmt *fmt, struct perf_hpp *hpp, return hpp_color_scnprintf(hpp, "%*.2f%%", width - 1, per); } -static double percent_hitm(struct c2c_hist_entry *c2c_he) +static double percent_costly_snoop(struct c2c_hist_entry *c2c_he) { struct c2c_hists *hists; struct c2c_stats *stats; @@ -838,8 +838,8 @@ static double percent_hitm(struct c2c_hist_entry *c2c_he) }) static int -percent_hitm_entry(struct perf_hpp_fmt *fmt, struct perf_hpp *hpp, - struct hist_entry *he) +percent_costly_snoop_entry(struct perf_hpp_fmt *fmt, struct perf_hpp *hpp, + struct hist_entry *he) { struct c2c_hist_entry *c2c_he; int width = c2c_width(fmt, hpp, he->hists); @@ -847,20 +847,20 @@ percent_hitm_entry(struct perf_hpp_fmt *fmt, struct perf_hpp *hpp, double per; c2c_he = container_of(he, struct c2c_hist_entry, he); - per = percent_hitm(c2c_he); + per = percent_costly_snoop(c2c_he); return scnprintf(hpp->buf, hpp->size, "%*s", width, PERC_STR(buf, per)); } static int -percent_hitm_color(struct perf_hpp_fmt *fmt, struct perf_hpp *hpp, - struct hist_entry *he) +percent_costly_snoop_color(struct perf_hpp_fmt *fmt, struct perf_hpp *hpp, + struct hist_entry *he) { - return percent_color(fmt, hpp, he, percent_hitm); + return percent_color(fmt, hpp, he, percent_costly_snoop); } static int64_t -percent_hitm_cmp(struct perf_hpp_fmt *fmt __maybe_unused, - struct hist_entry *left, struct hist_entry *right) +percent_costly_snoop_cmp(struct perf_hpp_fmt *fmt __maybe_unused, + struct hist_entry *left, struct hist_entry *right) { struct c2c_hist_entry *c2c_left; struct c2c_hist_entry *c2c_right; @@ -870,8 +870,8 @@ percent_hitm_cmp(struct perf_hpp_fmt *fmt __maybe_unused, c2c_left = container_of(left, struct c2c_hist_entry, he); c2c_right = container_of(right, struct c2c_hist_entry, he); - per_left = percent_hitm(c2c_left); - per_right = percent_hitm(c2c_right); + per_left = percent_costly_snoop(c2c_left); + per_right = percent_costly_snoop(c2c_right); return per_left - per_right; } @@ -1605,17 +1605,17 @@ static struct c2c_dimension dim_tot_loads = { .width = 7, }; -static struct c2c_header percent_hitm_header[] = { +static struct c2c_header percent_costly_snoop_header[] = { [DISPLAY_LCL_HITM] = HEADER_BOTH("Lcl", "Hitm"), [DISPLAY_RMT_HITM] = HEADER_BOTH("Rmt", "Hitm"), [DISPLAY_TOT_HITM] = HEADER_BOTH("Tot", "Hitm"), }; -static struct c2c_dimension dim_percent_hitm = { - .name = "percent_hitm", - .cmp = percent_hitm_cmp, - .entry = percent_hitm_entry, - .color = percent_hitm_color, +static struct c2c_dimension dim_percent_costly_snoop = { + .name = "percent_costly_snoop", + .cmp = percent_costly_snoop_cmp, + .entry = percent_costly_snoop_entry, + .color = percent_costly_snoop_color, .width = 7, }; @@ -1844,7 +1844,7 @@ static struct c2c_dimension *dimensions[] = { &dim_ld_rmthit, &dim_tot_recs, &dim_tot_loads, - &dim_percent_hitm, + &dim_percent_costly_snoop, &dim_percent_rmt_hitm, &dim_percent_lcl_hitm, &dim_percent_rmt_peer, @@ -2748,7 +2748,7 @@ static int ui_quirks(void) nodestr = "CL"; } - dim_percent_hitm.header = percent_hitm_header[c2c.display]; + dim_percent_costly_snoop.header = percent_costly_snoop_header[c2c.display]; /* Fix the zero line for dcacheline column. */ buf = fill_line("Cacheline", dim_dcacheline.width + @@ -3074,7 +3074,7 @@ static int perf_c2c__report(int argc, const char **argv) "dcacheline," "dcacheline_node," "dcacheline_count," - "percent_hitm," + "percent_costly_snoop," "tot_hitm,lcl_hitm,rmt_hitm," "tot_recs," "tot_loads," -- 2.25.1