Review Request: 6909153: Fix broken options on Zero
Christian Thalinger
Christian.Thalinger at Sun.COM
Tue Dec 22 03:16:57 PST 2009
On Fri, 2009-12-18 at 08:38 +0000, Gary Benson wrote:
> Hi,
>
> Christian Thalinger wrote:
> > On Wed, 2009-12-09 at 16:30 +0000, Gary Benson wrote:
> > > I've been doing a mini-audit of Zero to ensure it still works with
> > > non-standard options. This webrev:
> > >
> > > http://cr.openjdk.java.net/~gbenson/zero-options-fixes/
> > >
> > > fixes the following issues:
> > >
> > > 1. HotSpot would segfault with -XX:+CITime if the VM is built
> > > with no compilers. The patch fixes this.
> > >
> > > 2. The C++ interpreter doesn't support -XX:+ProfileInterpreter
> > > and -XX:+UseCompressedOops, and Zero doesn't support
> > > -XX:+TaggedStackInterpreter. The patch disables these
> > > options in these configurations.
> >
> > Changes look good to me.
> >
> > > I haven't filed a bug for this, so it has no bug id yet.
> >
> > The CR is:
> >
> > 6909153: Fix broken options on Zero
>
> Would it be possible for someone to push this for me?
Ohh, sorry, I forgot this one. I'll do it when I find some time
(currently having vacation).
-- Christian
More information about the hotspot-dev
mailing list