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