Messages in this thread | | | Date | Sat, 19 Dec 2009 19:58:34 -0200 | From | Arnaldo Carvalho de Melo <> | Subject | Re: [PATCH 1/1 v2] perf record: We should fork only if a program was specified to run |
| |
Em Sat, Dec 19, 2009 at 10:39:01PM +0100, Peter Zijlstra escreveu: > On Sat, 2009-12-19 at 19:22 -0200, Arnaldo Carvalho de Melo wrote: > > @@ -550,7 +551,7 @@ static int __cmd_record(int argc __used, const char **argv) > > return err; > > } > > > > - if (!system_wide) > > + if (!system_wide && profile_cpu == -1) > > event__synthesize_thread(pid, process_synthesized_event, > > session); > > Not actually sure about this bit though, since we now have > per-task-per-cpu things, the profile_cpu is also used to !system_wide.
this actually means: if it is not system wide and not for an specific cpu, it must be for an specific pid.
Not clear, right, you may want to profile an specific pid but only when it is running on an specific CPU... But at least this patch allows for specifiying either an specific CPU or an specific pid or systemwide.
- Arnaldo
| |