RFR: 8078193: BACKOUT: Rename and clean up the ParGCAllocBuffer class
Thomas Schatzl
thomas.schatzl at oracle.com
Tue Apr 21 12:51:03 UTC 2015
Hi,
On Tue, 2015-04-21 at 10:09 +0200, Bengt Rutisson wrote:
> Hi Kim,
>
> On 2015-04-21 00:14, Kim Barrett wrote:
> > Please review this change to back out a changeset.
> >
> > The intended fix for JDK-8074546 was pushed with a commit message
> > containing the wrong bug id. To simplify tracking the changeset
> > through repository promotions to release, we're going to hg backout
> > the problem changeset and re-push with a corrected commit message.
> >
> > The hg backout applied cleanly:
> > Changeset: hs-gc/hotspot: 8148:dafb0a98e749
> >
> > CR:
> > https://bugs.openjdk.java.net/browse/JDK-8078193
> >
> > Webrev:
> > http://cr.openjdk.java.net/~kbarrett/8078193/webrev.00/
> >
> > Testing:
> > JPRT
>
>
> Doing hg backout does not apply cleanly for me.
>
> $ hg backout -r 8148
[...]
> merging src/share/vm/runtime/globals.hpp
> merging src/share/vm/runtime/globals.hpp failed!
> 142 files updated, 0 files merged, 1 files removed, 1 files unresolved
> use 'hg resolve' to retry unresolved file merges
Backout works here.
> There is a conflict in globals.hpp. But the patch file provided in the
> webrev:
> http://cr.openjdk.java.net/~kbarrett/8078193/webrev.00/hotspot.changeset
>
> applies cleanly for me and does the right thing to globals.hpp.
>
> So, I think the webrev looks fine. Reviewed.
Think so too. Please push this backout and the good change at the same
time as Bengt suggested.
Thanks,
Thomas
More information about the hotspot-gc-dev
mailing list