Arun,

Several remarks about this new version:
  - my compiler was generating some warnings about prototypes in
perfmon_int.i. I fixed
    them easily.

  - you cannot hardcode event names into self.py because each PMU is
different. For
    instance, CPU_CLK_UNHALTED does not quite exist as is on Core 2.
The only way
    to make this portable is to either use pfm_get_cycle_event(),
pfm_get_inst_retired()
    or use an extra indirection layer, like PAPI

  - When you are not attaching to self, you need to use ptrace() to
control the other
    thread, otherwise perfmon is going to reject syscalls. In other
words, you need to
    have a way to stop/start another thread. Ptrace is currently used
but at the user level,
    the kernel does not do the work for you. With utrace, it may be
easier to let the kernel
    do this but this is to be investigated (soon).

Thanks.



On Tue, Mar 25, 2008 at 8:42 AM, Arun Sharma <[EMAIL PROTECTED]> wrote:
> On Mon, Mar 24, 2008 at 8:21 PM, Arun Sharma <[EMAIL PROTECTED]> wrote:
>
>  >
>  >  Stephane, your changes look good to me. But I'm almost done
>  >  refactoring the examples roughly as follows:
>
>  Updated patch attached. I think I captured most of your comments in
>  this patch - but I might have missed a couple.
>
>   -Arun
>

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
perfmon2-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/perfmon2-devel

Reply via email to