Re: [PATCH 5/5] perf tool: Add basic event modifier sanity check

2012-11-12 Thread Namhyung Kim
Hi Jiri, On Mon, 12 Nov 2012 18:34:04 +0100, Jiri Olsa wrote: > Updating event parser to allow any non zero string containing > [ukhpGH] characters for event modifier. > > The modifier sanity is checked later in parse-event object logic. > The check validates modifier to contain only one instance

[PATCH 5/5] perf tool: Add basic event modifier sanity check

2012-11-12 Thread Jiri Olsa
Updating event parser to allow any non zero string containing [ukhpGH] characters for event modifier. The modifier sanity is checked later in parse-event object logic. The check validates modifier to contain only one instance of any modifier (apart from 'p') present. Signed-off-by: Jiri Olsa

[PATCH 5/5] perf tool: Add basic event modifier sanity check

2012-11-12 Thread Jiri Olsa
Updating event parser to allow any non zero string containing [ukhpGH] characters for event modifier. The modifier sanity is checked later in parse-event object logic. The check validates modifier to contain only one instance of any modifier (apart from 'p') present. Signed-off-by: Jiri Olsa

Re: [PATCH 5/5] perf tool: Add basic event modifier sanity check

2012-11-12 Thread Namhyung Kim
Hi Jiri, On Mon, 12 Nov 2012 18:34:04 +0100, Jiri Olsa wrote: Updating event parser to allow any non zero string containing [ukhpGH] characters for event modifier. The modifier sanity is checked later in parse-event object logic. The check validates modifier to contain only one instance of