lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <eltfh7yi5ccwdcxewcqgu56plrpr2uvlq6r3elzsswmp67i2x2@j3absnlmj6r2>
Date: Sat, 21 Jun 2025 22:48:34 -0400
From: Aaron Tomlin <atomlin@...mlin.com>
To: Namhyung Kim <namhyung@...nel.org>
Cc: arnaldo.melo@...il.com, peterz@...radead.org, mingo@...hat.com, 
	acme@...nel.org, mark.rutland@....com, alexander.shishkin@...ux.intel.com, 
	jolsa@...nel.org, irogers@...gle.com, adrian.hunter@...el.com, 
	kan.liang@...ux.intel.com, linux-perf-users@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [RESEND PATCH] perf probe: Extend the quiet option

On Mon, Jun 09, 2025 at 09:59:14PM -0700, Namhyung Kim wrote:
> Hello,

Hi Namhyung,

> Sorry for the long delay.

No problem.

> On Sun, May 04, 2025 at 11:43:51AM +0100, Aaron Tomlin wrote:
> > To provide a consistent and complete quiet mode, this patch expands the
> > scope of the [-q|--quiet] option to suppress all informational messages,
> > including those indicating successful event creation, in addition to the
> > currently suppressed warnings and messages. For example:
> > 
> >     ❯ sudo ./perf probe --quiet --add proc_sys_open
> >     ❯ echo $?
> >     0
> 
> I don't think this change is needed.  I also see no messages before this
> change.

Fair enough.

> Indeed, using -q/--quiet option makes verbose to be -1 which disables
> any pr_info() messages.

Understood.


Regards,

-- 
Aaron Tomlin

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ