Messages in this thread | | | Subject | Re: [RFC PATCH 1/2] perf: Enable group siblings on exec if attr::enable_on_exec set | From | Peter Zijlstra <> | Date | Wed, 10 Nov 2010 15:54:42 +0100 |
| |
On Wed, 2010-11-10 at 22:49 +0800, Lin Ming wrote: > > I don't mind the patch too much (although I would like to avoid that > > double sibling iteration in both group_enable_on_exec() and > > __perf_event_mark_enabled()), but I think it can already work as is. > > I got it now! > > Another stupid patch, sorry.
Well, I wouldn't call it a stupid patch, it does fix something that is evidently non-obvious so it might be worth considering it on that merit.
I'm simply pointing out that what you're trying to achieve might be possible without changing code.
| |