[PATCH 15/19] perf metric: Add recursion check when processing nested metrics

2020-07-29 Thread Jiri Olsa
Keeping the stack of nested metrics via 'struct expr_id' objects and checking if we are in recursion via already processed metric. The stack is implemented as static array within the struct egroup with 100 entries, which should be enough nesting depth for any metric we have or plan to have at the

Re: [PATCH 15/19] perf metric: Add recursion check when processing nested metrics

2020-07-28 Thread Arnaldo Carvalho de Melo
Em Sun, Jul 19, 2020 at 03:40:26PM -0700, Ian Rogers escreveu: > On Sun, Jul 19, 2020 at 11:14 AM Jiri Olsa wrote: > > > > Keeping the stack of nested metrics via 'struct expr_id' objects > > and checking if we are in recursion via already processed metric. > > > > The stack is implemented as stat

Re: [PATCH 15/19] perf metric: Add recursion check when processing nested metrics

2020-07-26 Thread kajoljain
On 7/20/20 4:10 AM, Ian Rogers wrote: > On Sun, Jul 19, 2020 at 11:14 AM Jiri Olsa wrote: >> >> Keeping the stack of nested metrics via 'struct expr_id' objects >> and checking if we are in recursion via already processed metric. >> >> The stack is implemented as static array within the struct

Re: [PATCH 15/19] perf metric: Add recursion check when processing nested metrics

2020-07-19 Thread Ian Rogers
On Sun, Jul 19, 2020 at 11:14 AM Jiri Olsa wrote: > > Keeping the stack of nested metrics via 'struct expr_id' objects > and checking if we are in recursion via already processed metric. > > The stack is implemented as static array within the struct egroup > with 100 entries, which should be enoug

[PATCH 15/19] perf metric: Add recursion check when processing nested metrics

2020-07-19 Thread Jiri Olsa
Keeping the stack of nested metrics via 'struct expr_id' objects and checking if we are in recursion via already processed metric. The stack is implemented as static array within the struct egroup with 100 entries, which should be enough nesting depth for any metric we have or plan to have at the