[prev in list] [next in list] [prev in thread] [next in thread] 

List:       oprofile-list
Subject:    Re: [patch 0/7] Oprofile AMD IBS support
From:       "Jason Yeh" <jason.yeh () amd ! com>
Date:       2008-01-30 16:23:49
Message-ID: 47A0A495.2050501 () amd ! com
[Download RAW message or body]

John Levon wrote:
> On Wed, Jan 30, 2008 at 09:54:35AM -0600, Jason Yeh wrote:
> 
> > Sorry about that. I attached that tar.gz of the patches.
> 
> Is there any way you can send it inline but not mangled? It's harder for
> me to review in a tar file. If not, that's fine.

I do not know way to prevent it from happening. My client has line wrap disabled and \
the mail is being edited as text. I search around to see if I can find a way to do \
that.

 
> > > Why are we using this form instead of using --event somehow? I'd like to
> > > see a clearer explanation of this. CPU-specific command lines make me
> > > nervous.
> > The main reason is that IBS has no equivalent of kernel and user bit, no 
> > unit mask. I felt that using separate switched made it more clear on using 
> > IBS. If "--event" is to be used, which format of the switch would be 
> > preferred: 
> > 1. Something similar to "--event=ibs_op:count" and 
> > "--event=ibs_fetch:count" and modify the parsing code.
> 
> We can do this. Make it so that the user/kernel/unit mask cause an error
> if specified for these special events.

Sounds good to me. I will make changes accordingly.

> So what about stuff like linear addresses you say you store?

Unfortunately, the linear address is currently not used. 

Jason




-------------------------------------------------------------------------
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/
_______________________________________________
oprofile-list mailing list
oprofile-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/oprofile-list


[prev in list] [next in list] [prev in thread] [next in thread]