2009-11-20 21:31:21

by Peter Zijlstra

[permalink] [raw]
Subject: [PATCH 00/15] perf_event patches

Some perf patches, most resulting from chasing Corey's scale issue.

There's still one issue open with that and that is that we're reading
->total_time_{enabled,running} without serialization.

Please have a thorough look at 7-14.

It also removes all the system_state == STATE_RUNNING muck from
everywhere and it still boots, which leaves me wondering what changed to
not make things go boom.


2009-11-23 05:52:45

by Paul Mackerras

[permalink] [raw]
Subject: Re: [PATCH 00/15] perf_event patches

Peter Zijlstra writes:

> Some perf patches, most resulting from chasing Corey's scale issue.
>
> There's still one issue open with that and that is that we're reading
> ->total_time_{enabled,running} without serialization.
>
> Please have a thorough look at 7-14.
>
> It also removes all the system_state == STATE_RUNNING muck from
> everywhere and it still boots, which leaves me wondering what changed to
> not make things go boom.

Nice series! At a quickish look it all looks fine to me. I'd give
you acked-bys, but Ingo has already committed them all.

Paul.