This is the mail archive of the systemtap@sourceware.org mailing list for the systemtap project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]