From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8E84F3CF21F; Thu, 1 Oct 2026 18:54:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790880875; cv=none; b=B78N7uioHHmfg3cFaOjEV5l+iY+vjP+Hhg2UBBj6MzwoM1id/yVFQVvVUyVGX9FSnY2Vca3bGh98ZVOAA3RsDNfoYBD8GgX+Ri2JTun6kkwO7k9L29Vu/Pjkino/6ruWX2LB6QbFN3T4ENi1zefl0T6NFIoNhzgTfNHWtvIpUUo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1790880875; c=relaxed/simple; bh=9bPn1koXs4obEqqoVQM+nyg3XG1ggs5RMO89AgdHXHw=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=hVT1OF1WrG+vZZs2UOghpNoO0k2RK+Tm851DuHTmU6FU0wxt3kpIbMPbcN6FTw9/74Gh8iUCi3XPEB1kY3Af8+khVbVpHqJ0sATYAWlFzqP6HXuUzgXqfqh1vprnzz8GBuadtOW9WhTqTpYU//x8ya7ATGL0VYHDe+7fEqKW1L8= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=FnJXf2J+; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="FnJXf2J+" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 261031F00899; Thu, 1 Oct 2026 18:54:30 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1790880874; bh=NcY1S+W15PLa/BGdDFIaPoZN8W/RM1PdImAaQGCLNns=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=FnJXf2J+srHaAq6jLCPL6uL5m+gRDpD5HL3xa5JHSgqNL9th0tnbvrB7MrwCswy93 i78FCnciwYzoTx3QdvYqdyvZ1YBz3x2lzLb0lcST9U8j1dmIfAxPcuTL01/pYynwZU 7/7MLvUQEMEXhLga8Xh+XzjHY4FnlpcJD6SsvOxwKwUPasjm04vXd6CQNop2Eq0+3v 66w1/VeHRULqrNVg7FSFe03pPo/SPB5xGxbSD3HvHuQ6EQf9tyurEuugjlz22F69Yz 5kLSfuLD6YFP6XSU/IMTpn/thZx8ssIk+i/MH4CkEcxgGPqLHz0eHz6E5LPgRsjRt1 M6M6k5gnEQOiA== From: Arnaldo Carvalho de Melo To: Namhyung Kim Cc: Ingo Molnar , Thomas Gleixner , James Clark , Jiri Olsa , Ian Rogers , Adrian Hunter , Clark Williams , linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org, Arnaldo Carvalho de Melo Subject: [PATCH v7 7/9] perf report: Add --no-progress option Date: Thu, 1 Oct 2026 20:53:58 +0200 Message-ID: <20261001185400.2754753-8-acme@kernel.org> X-Mailer: git-send-email 2.55.0 In-Reply-To: <20261001185400.2754753-1-acme@kernel.org> References: <20261001185400.2754753-1-acme@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit From: Arnaldo Carvalho de Melo Now that --progress is being added for the stdio case, wire up its counterpart for the browsers: the TUI and GTK ones present progress of their own and there is no way to turn it off. Install the no-op ui_progress ops, the ones already used until a backend sets theirs, after setup_browser() installed the ones of the browser in use. The phases are still counted, nothing is shown for them, and no second option is needed for it: parse-options provides --no-progress as the negation of --progress, report.progress_set saying that it was asked for, report.progress being false both when nothing was asked for and when --no-progress was. Suggested-by: Namhyung Kim Assisted-by: LLM Signed-off-by: Arnaldo Carvalho de Melo --- tools/perf/Documentation/perf-report.txt | 5 +++++ tools/perf/builtin-report.c | 9 ++++++--- tools/perf/ui/progress.c | 6 ++++++ tools/perf/ui/progress.h | 2 ++ 4 files changed, 19 insertions(+), 3 deletions(-) diff --git a/tools/perf/Documentation/perf-report.txt b/tools/perf/Documentation/perf-report.txt index a7429a30ec28f903..e145124d6c9a1097 100644 --- a/tools/perf/Documentation/perf-report.txt +++ b/tools/perf/Documentation/perf-report.txt @@ -43,6 +43,11 @@ OPTIONS present progress information, or when --quiet is used, that asks for no messages at all. +--no-progress:: + Do not show progress while processing the perf.data file. It + also turns off the progress the TUI and GTK browsers present, + which is their own. + -n:: --show-nr-samples:: Show the number of samples for each symbol diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c index 81e13fb8194976c9..29085fdc91e58884 100644 --- a/tools/perf/builtin-report.c +++ b/tools/perf/builtin-report.c @@ -88,6 +88,7 @@ struct report { #endif bool use_stdio; bool progress; + bool progress_set; bool show_full_info; bool show_threads; bool inverted_callchain; @@ -1385,8 +1386,8 @@ int cmd_report(int argc, const char **argv) "Use the stdio interface"), OPT_BOOLEAN(0, "weights", &symbol_conf.annotate_weight, "Show or hide weight columns in annotation. Default show if non-zero."), - OPT_BOOLEAN(0, "progress", &report.progress, - "Show progress while processing the perf.data file"), + OPT_BOOLEAN_SET(0, "progress", &report.progress, &report.progress_set, + "Show progress while processing the perf.data file"), OPT_BOOLEAN(0, "header", &report.header, "Show data header."), OPT_BOOLEAN(0, "header-only", &report.header_only, "Show only data header."), @@ -1793,7 +1794,9 @@ int cmd_report(int argc, const char **argv) else use_browser = 0; - if (report.progress && !quiet && use_browser == 0) + if (report.progress_set && !report.progress) + ui_progress__noop_init(); + else if (report.progress && !quiet && use_browser == 0) stdio_progress__init(); if (report.data_type && use_browser == 1) { diff --git a/tools/perf/ui/progress.c b/tools/perf/ui/progress.c index 99d60223c74b2957..362680989ace606a 100644 --- a/tools/perf/ui/progress.c +++ b/tools/perf/ui/progress.c @@ -13,6 +13,12 @@ static struct ui_progress_ops null_progress__ops = struct ui_progress_ops *ui_progress__ops = &null_progress__ops; +/* Everything counts but nothing is shown, the way it starts out. */ +void ui_progress__noop_init(void) +{ + ui_progress__ops = &null_progress__ops; +} + void ui_progress__update(struct ui_progress *p, u64 adv) { u64 last = p->curr; diff --git a/tools/perf/ui/progress.h b/tools/perf/ui/progress.h index 03f1a8bb260ba076..e8c4f9f768aaf12b 100644 --- a/tools/perf/ui/progress.h +++ b/tools/perf/ui/progress.h @@ -25,6 +25,8 @@ void ui_progress__update(struct ui_progress *p, u64 adv); void stdio_progress__init(void); +void ui_progress__noop_init(void); + struct ui_progress_ops { void (*init)(struct ui_progress *p); void (*update)(struct ui_progress *p); -- 2.55.0