linux-kernel - Re: [PATCH] perf tool: Add event name to error message for filters

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <20150324222318.GD12599@kernel.org>
Date:	Tue, 24 Mar 2015 19:23:18 -0300
From:	Arnaldo Carvalho de Melo <acme@...nel.org>
To:	David Ahern <david.ahern@...cle.com>
Cc:	linux-kernel@...r.kernel.org, Jiri Olsa <jolsa@...nel.org>,
	Namhyung Kim <namhyung@...nel.org>
Subject: Re: [PATCH] perf tool: Add event name to error message for filters

Em Tue, Mar 24, 2015 at 04:00:36PM -0600, David Ahern escreveu:
> On 3/24/15 3:57 PM, Arnaldo Carvalho de Melo wrote:
> >With the patch below, that leaves printing something, using whatever UI, to the
> >tool, acceptable?
> 
> I keep forgetting about the python bindings.

Oh, its not just about the python bindings, but kinda layering
violation, i.e. we should leave the error reporting to the actual tool,
that may be using GTK, Qt, some custom made TUI, whatever UI.

The python bindings, were not a problem somehow, at least 'perf test
python' hasn't failed when I tried.

> Your patch serves the purpose. Thanks, Arnaldo.

Thanks,

Applying with your Acked-by and Based-on-a-patch-by.

- Arnaldo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/