This is the mail archive of the
systemtap@sourceware.org
mailing list for the systemtap project.
Re: [RFC] Enhanced Garbage Collection Probe Points
- From: Jon VanAlten <jvanalte at redhat dot com>
- To: Lukas Berk <lberk at redhat dot com>
- Cc: systemtap at sourceware dot org, distro-pkg-dev at openjdk dot java dot net
- Date: Thu, 25 Oct 2012 01:02:09 -0400 (EDT)
- Subject: Re: [RFC] Enhanced Garbage Collection Probe Points
----- Original Message -----
> From: "Lukas Berk" <lberk@redhat.com>
> To: "Jon VanAlten" <jvanalte@redhat.com>
> Cc: systemtap@sourceware.org, distro-pkg-dev@openjdk.java.net
> Sent: Tuesday, October 23, 2012 12:53:30 PM
> Subject: Re: [RFC] Enhanced Garbage Collection Probe Points
>
> Hey,
>
> * Jon VanAlten <jvanalte@redhat.com> [2012-09-24 14:16]:
> >
> > >
> > > > > Is this ok to commit?
> > > > >
> > > >
> > > > With caveats noted above, I say yes :)
> > > >
> > >
> > > Thanks! Please let me know if there is anything else you'd like
> > > to
> > > know,
> > > updated patch is attached.
> > >
> >
> > Hi,
> >
> > This has been in my TODO for some time, so I am very sorry not to
> > have responded earlier. From looking at the patch, it seems okay
> > now. I intend to push this to HEAD on your behalf, unless someone
> > else has some reason why not. (Now is your chance to speak up
> > about that, if you have such a reason!). But, I do feel
> > responsible
> > to build with it myself and verify that things seem to be working
> > as
> > intended. Various other things have been jumping my work queue,
> > but
> > I may have time this week to give this a sanity check and finally
> > get it into hg. Thanks for persisting!
> >
> > cheers,
> > Jon
>
> I've updated the patch slightly to reflect a correction in
> psScavenge.cpp that was causing a build error when icedtea-debug
> wasn't
> specified. The rest of the patch is unchanged from before.
>
And I've now pushed this. I know it's preferred that new work
goes to forest in general, but this was discussed earlier when
patch was originally posted, and afaict the feeling was it
would be better to move all systemtap related patches there in
one step some time later[1].
Thanks Lukas for this contribution!
cheers,
jon
[1] http://mail.openjdk.java.net/pipermail/distro-pkg-dev/2012-August/020066.html