Messages in this thread | | | Date | Mon, 7 Sep 2015 14:28:14 +0200 | From | Jiri Olsa <> | Subject | Re: [PATCH 2/3] perf tests: Add arch tests |
| |
On Sat, Sep 05, 2015 at 08:02:21PM +0100, Matt Fleming wrote:
SNIP
> }; > > +static struct test *tests[] = { > + generic_tests, > + arch_tests, > +}; > + > static bool perf_test__matches(struct test *test, int curr, int argc, const char *argv[]) > { > int i; > @@ -237,7 +229,11 @@ static int run_test(struct test *test) > return err; > } > > -#define for_each_test(t) for (t = &tests[0]; t->func; t++) > +static unsigned int ___j; /* This is obviously not thread-safe */ > + > +#define for_each_test(t) \ > + for (___j = 0; ___j < ARRAY_SIZE(tests); ___j++) \ > + for (t = &tests[___j][0]; t->func; t++)
why not have ____j on stack and pas it into for_each_test
for_each_test(j, t) ...
jirka
| |