* xenoprof support patch issue for oprofile 0.9.3
@ 2007-07-17 18:46 William Cohen
2007-07-19 19:02 ` [Xen-devel] " Santos, Jose Renato G
0 siblings, 1 reply; 6+ messages in thread
From: William Cohen @ 2007-07-17 18:46 UTC (permalink / raw)
To: xen-devel
Oprofile 0.9.3 released yesterday. The patches for xenoprof are not currently in
the upstream oprofile. I noticed that one of the defines in the xenoprof patches
for oprofile has a define DOMAIN_SWITCH_CODE in daemon/opd_interface.h which
conflicts with SPU_PROFILING_CODE. This conflict will need to be worked out.
-Will
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [Xen-devel] xenoprof support patch issue for oprofile 0.9.3
2007-07-17 18:46 xenoprof support patch issue for oprofile 0.9.3 William Cohen
@ 2007-07-19 19:02 ` Santos, Jose Renato G
2007-07-20 14:44 ` William Cohen
0 siblings, 1 reply; 6+ messages in thread
From: Santos, Jose Renato G @ 2007-07-19 19:02 UTC (permalink / raw)
To: William Cohen, xen-devel; +Cc: oprofile-list
Will,
Thanks for pointing this out.
I will work on a patch to fix this
By the way, it would be nice to have the xenoprof patches merged into
Oprofile CVS tree.
John Levon is not happy with the way passive domain samples are stored
now and I don't have the bandwidth (and knowdledge of user level
Oprofile code) to do the right fixes. This has prevented the patches to
be merged. It would be nice if we can find a volunteer to do the right
fixes and have the code merged. I would be happy to assist anyone who
volunteers for the task. What we need is a way of separating samples
from different domains and change to the tools to understand this new
information.
Thanks
Renato
>> -----Original Message-----
>> From: xen-devel-bounces@lists.xensource.com
>> [mailto:xen-devel-bounces@lists.xensource.com] On Behalf Of
>> William Cohen
>> Sent: Tuesday, July 17, 2007 11:47 AM
>> To: xen-devel@lists.xensource.com
>> Subject: [Xen-devel] xenoprof support patch issue for oprofile 0.9.3
>>
>> Oprofile 0.9.3 released yesterday. The patches for xenoprof
>> are not currently in the upstream oprofile. I noticed that
>> one of the defines in the xenoprof patches for oprofile has
>> a define DOMAIN_SWITCH_CODE in daemon/opd_interface.h which
>> conflicts with SPU_PROFILING_CODE. This conflict will need
>> to be worked out.
>>
>>
>>
>> -Will
>>
>> _______________________________________________
>> Xen-devel mailing list
>> Xen-devel@lists.xensource.com
>> http://lists.xensource.com/xen-devel
>>
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Xen-devel] xenoprof support patch issue for oprofile 0.9.3
2007-07-19 19:02 ` [Xen-devel] " Santos, Jose Renato G
@ 2007-07-20 14:44 ` William Cohen
2007-07-20 16:42 ` Santos, Jose Renato G
0 siblings, 1 reply; 6+ messages in thread
From: William Cohen @ 2007-07-20 14:44 UTC (permalink / raw)
To: Santos, Jose Renato G; +Cc: xen-devel, oprofile-list
Santos, Jose Renato G wrote:
> Will,
>
> Thanks for pointing this out.
> I will work on a patch to fix this
> By the way, it would be nice to have the xenoprof patches merged into
> Oprofile CVS tree.
> John Levon is not happy with the way passive domain samples are stored
> now and I don't have the bandwidth (and knowdledge of user level
> Oprofile code) to do the right fixes. This has prevented the patches to
> be merged. It would be nice if we can find a volunteer to do the right
> fixes and have the code merged. I would be happy to assist anyone who
> volunteers for the task. What we need is a way of separating samples
> from different domains and change to the tools to understand this new
> information.
>
> Thanks
>
> Renato
Hi Renato,
Definitely want to avoid having separate patches to avoid just this type of
problem. Getting patches upstream is the way to go on that.
Do you have a pointer to the email discussion about the passive domain sample
storage? Passive means that the domain is not taking the samples and storing
them in the domain, right?
One of the things that I have noticed on my previous experiments with the
xenoprof is there are quite a few steps to setup xenoprof. The user has to do
setup on each domain. Ideally it would be nice to be able to just do one setup
in the host domain and then have that information be propagated to the guest
domains. Similarly the analysis would be nice to get one over all picture of all
the domains. Clusters have a similar issue; one doesn't want to manage machine
individually. However clusters may have different hardware for the various nodes
in the cluster. With xen the only difference could be the software.
-Will
-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: xenoprof support patch issue for oprofile 0.9.3
2007-07-20 14:44 ` William Cohen
@ 2007-07-20 16:42 ` Santos, Jose Renato G
2007-07-26 14:29 ` [Xen-devel] " William Cohen
0 siblings, 1 reply; 6+ messages in thread
From: Santos, Jose Renato G @ 2007-07-20 16:42 UTC (permalink / raw)
To: William Cohen; +Cc: xen-devel, oprofile-list
>> -----Original Message-----
>> From: xen-devel-bounces@lists.xensource.com
>> [mailto:xen-devel-bounces@lists.xensource.com] On Behalf Of
>> William Cohen
>> Sent: Friday, July 20, 2007 7:44 AM
>> To: Santos, Jose Renato G
>> Cc: xen-devel@lists.xensource.com;
>> oprofile-list@lists.sourceforge.net
>> Subject: Re: [Xen-devel] xenoprof support patch issue for
>> oprofile 0.9.3
>>
>> Santos, Jose Renato G wrote:
>> > Will,
>> >
>> > Thanks for pointing this out.
>> > I will work on a patch to fix this
>> > By the way, it would be nice to have the xenoprof patches
>> merged into
>> > Oprofile CVS tree.
>> > John Levon is not happy with the way passive domain
>> samples are stored
>> > now and I don't have the bandwidth (and knowdledge of user level
>> > Oprofile code) to do the right fixes. This has prevented
>> the patches
>> > to be merged. It would be nice if we can find a volunteer
>> to do the
>> > right fixes and have the code merged. I would be happy to assist
>> > anyone who volunteers for the task. What we need is a way of
>> > separating samples from different domains and change to
>> the tools to
>> > understand this new information.
>> >
>> > Thanks
>> >
>> > Renato
>>
>> Hi Renato,
>>
>> Definitely want to avoid having separate patches to avoid
>> just this type of problem. Getting patches upstream is the
>> way to go on that.
>>
>> Do you have a pointer to the email discussion about the
>> passive domain sample storage?
I will forward the email discussion I had with John on a private
message
>> Passive means that the domain
>> is not taking the samples and storing them in the domain, right?
>>
Right! This makes profiling guests in kernel and xen mode much
simpler than with active domains and avoids the complexities you
mention below. With passive domains there is no need to run
oprofile on guests as dom0 process all guest samples.
The disadvantage is that you cannot assign
user level samples to the right binaries in the guest.
>> One of the things that I have noticed on my previous
>> experiments with the xenoprof is there are quite a few steps
>> to setup xenoprof. The user has to do setup on each domain.
>> Ideally it would be nice to be able to just do one setup in
>> the host domain and then have that information be propagated
>> to the guest domains. Similarly the analysis would be nice
>> to get one over all picture of all the domains. Clusters
>> have a similar issue; one doesn't want to manage machine
>> individually. However clusters may have different hardware
>> for the various nodes in the cluster. With xen the only
>> difference could be the software.
>>
Yes, profiling multiple guests (as active domains) is somewhat
complex. But, it should not be difficult to have some wrapper scripts
to coordinate oprofile execution on multiple domains, using
possibly ssh, hiding the complexities from the user.
Renato
>> -Will
>>
>> _______________________________________________
>> Xen-devel mailing list
>> Xen-devel@lists.xensource.com
>> http://lists.xensource.com/xen-devel
>>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Xen-devel] xenoprof support patch issue for oprofile 0.9.3
2007-07-20 16:42 ` Santos, Jose Renato G
@ 2007-07-26 14:29 ` William Cohen
2007-07-30 23:11 ` Santos, Jose Renato G
0 siblings, 1 reply; 6+ messages in thread
From: William Cohen @ 2007-07-26 14:29 UTC (permalink / raw)
To: Santos, Jose Renato G; +Cc: xen-devel, oprofile-list
Hi Renato,
I have munged the xenoprof patch to apply cleanly to the user-space oprofile.
The patch don't address the larger changes with handling passive domains. The
new oprofile-0.9.3-2.fc8 rpm built with this patch can be found at:
http://koji.fedoraproject.org/koji/buildinfo?buildID=11990
The xen kernel space will need to change the defines to correctly match.
DOMAIN_SWITCH_CODE and LAST_CODE have changed to:
+#define DOMAIN_SWITCH_CODE 13
+#define LAST_CODE 14
-Will
-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems? Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [Xen-devel] xenoprof support patch issue for oprofile 0.9.3
2007-07-26 14:29 ` [Xen-devel] " William Cohen
@ 2007-07-30 23:11 ` Santos, Jose Renato G
0 siblings, 0 replies; 6+ messages in thread
From: Santos, Jose Renato G @ 2007-07-30 23:11 UTC (permalink / raw)
To: William Cohen; +Cc: xen-devel, oprofile-list
Will
Xenoprof patches for oprofile 0.9.3 (to enable passive domain profiling)
are now available at http://xenoprof.sourceforge.net.
A required patch for the kernel is also available at the same location.
I think the kernel patch should NOT be included into the public Xen tree
at this time, as this will break compatibility with oprofile 0.9.2 that
users are currently using. I think it is better to have users wanting to
upgrade to oprofile 0.9.3 to apply an extra patch to the kernel rather
than breaking compatibility for all current users. I think we should
wait until Xen dom0 support gets into mainline linux to do that (or at
least until xenoprof patches are included in an official oprofile
release).
Regards
Renato
>> -----Original Message-----
>> From: William Cohen [mailto:wcohen@redhat.com]
>> Sent: Thursday, July 26, 2007 7:29 AM
>> To: Santos, Jose Renato G
>> Cc: xen-devel@lists.xensource.com;
>> oprofile-list@lists.sourceforge.net
>> Subject: Re: [Xen-devel] xenoprof support patch issue for
>> oprofile 0.9.3
>>
>> Hi Renato,
>>
>> I have munged the xenoprof patch to apply cleanly to the
>> user-space oprofile.
>> The patch don't address the larger changes with handling
>> passive domains. The new oprofile-0.9.3-2.fc8 rpm built with
>> this patch can be found at:
>>
>> http://koji.fedoraproject.org/koji/buildinfo?buildID=11990
>>
>> The xen kernel space will need to change the defines to
>> correctly match.
>> DOMAIN_SWITCH_CODE and LAST_CODE have changed to:
>>
>>
>> +#define DOMAIN_SWITCH_CODE 13
>> +#define LAST_CODE 14
>>
>>
>> -Will
>>
-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems? Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2007-07-30 23:11 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-07-17 18:46 xenoprof support patch issue for oprofile 0.9.3 William Cohen
2007-07-19 19:02 ` [Xen-devel] " Santos, Jose Renato G
2007-07-20 14:44 ` William Cohen
2007-07-20 16:42 ` Santos, Jose Renato G
2007-07-26 14:29 ` [Xen-devel] " William Cohen
2007-07-30 23:11 ` Santos, Jose Renato G
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.