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 F17283F65E6; Sun, 4 Oct 2026 06:05:32 +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=1791093934; cv=none; b=EkLCXZjbU7vlr3vMAQNFuv5ILElEMdarbuSPh3rjJtBgW0GTxKq6rdp2rErLa2JMq55bFVLao9W5mKnQXQjr9lyDf9A4TultzcFua+EnjjX9IHVPc3GzG+3X5CgsgyhEWmIYSqcb/3HXAPjSgGr0RshnxMvryrNClPPqSLnNIXs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791093934; c=relaxed/simple; bh=9bPn1koXs4obEqqoVQM+nyg3XG1ggs5RMO89AgdHXHw=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=sWHwzdWmRfK0amx7BYCbYWiBjT/4EaEDIeNdUv74Wy5qBdvCPx4CkpEwq91FWGRkTWhQmVSmUD5/VgyOjwjdCPITdiPHE424+GbHqAKMX1zhKMLLK4VbO4pEFDxNiICvliOLL+bZn8hcu9RHP3HtG8muDVW7DiVNf5rueHPJ8ok= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=RPec58uq; 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="RPec58uq" Received: by smtp.kernel.org (Postfix) with ESMTPSA id BFEF71F00893; Sun, 4 Oct 2026 06:05:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1791093932; bh=NcY1S+W15PLa/BGdDFIaPoZN8W/RM1PdImAaQGCLNns=; h=From:To:Cc:Subject:Date:In-Reply-To:References; b=RPec58uqxzreO02AbH9RO7ubOAtiQrzi+Bu6MEUzYIJsMcxQ/kzwPDKj3h1xmDs54 OUAGbQ+2+diaSZaeF8ZZc+Pf5hy3NmN6v398FaJgyyVCz2cnZcGwxSEfvBOGFflfz3 UwlIxEFjpMLNloLNIbdepK5XSo56W3+SK5Llg8Ouba+UimyxXd/8oEf+hfwmfUwBuK IzKhvJhKvJ1RP3SriVFm7VHpASzz1fe0DBnSrMiBKfvUmnZ2eKTiyp+cNLwV84ui2l KibOwHMV+Wu7PcVZJTlqeReOvEN5FwFTeYb1eCUiIxIRDB7IgpAvEbypgYR5iYmM13 LoKnA6vZcwdKQ== 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 v9 7/9] perf report: Add --no-progress option Date: Sun, 4 Oct 2026 08:04:54 +0200 Message-ID: <20261004060457.3353-8-acme@kernel.org> X-Mailer: git-send-email 2.54.0 In-Reply-To: <20261004060457.3353-1-acme@kernel.org> References: <20261004060457.3353-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