Web lists-archives.com

Re: [PATCH perf/urgent] perf tools: Fix the code to strip command name






On 04/20/2017 07:35 PM, Jiri Olsa wrote:
On Thu, Apr 20, 2017 at 07:17:34PM +0900, Taeung Song wrote:
Hi Jiri,

On 04/20/2017 06:24 PM, Jiri Olsa wrote:
Recent commit broke command name strip in perf_event__get_comm_ids
function. It replaced left to right search for '\n' with rtrim,
which actually does right to left search. It occasionally caught
earlier '\n' and kept trash in the command name.

Sorry for my commit that have failings.

Could I know the command name in the above case ?
The command name can have two '\n' ?

it's the next line in the status file.. parts of the Umask string
and 1 newline

Name:   systemd
Umask:  0000
State:  S (sleeping)
...

I've already posted it in here:
  http://marc.info/?l=linuxppc-embedded&m=149200723316270&w=2

jirka


I understood it.
Sorry for my mistake..

Thanks,
Taeung