Re: [PATCH 2/6] perf tools: Introduce struct perf_log

Namhyung Kim
2013-12-20T01:28:13Z

Hi Arnaldo,

On Thu, 19 Dec 2013 10:28:06 -0300, Arnaldo Carvalho de Melo wrote:
> Em Thu, Dec 19, 2013 at 04:00:07PM +0900, Namhyung Kim escreveu:
>> From: Namhyung Kim <namhyung.kim@lge.com>
>>
>> Add new functions to save error messages in a temp file. It'll be
>> used by some UI front-ends to see the messages.
>
> Do we really have to grow a logging facility like this? Can't we somehow
> openlog/syslog/closelog? Just a first gut reaction, continuing to read
> the series...

I'm not sure. It's not really a logging facility - I just wanted a way to
save log messages somewhere in order to show them in a TUI window. My
feeling is that it doesn't need to grow since perf is not a service or so..

Thanks,
Namhyung