Review Request: Zero JNI handles fix
Christian Thalinger
Christian.Thalinger at Sun.COM
Fri Apr 30 04:28:49 PDT 2010
On Thu, 2010-04-22 at 12:14 +0100, Gary Benson wrote:
> > I've reworked the fix:
> >
> > http://cr.openjdk.java.net/~gbenson/zero-jnihandles-fix-2/
> >
> > last_Java_sp() now returns the actual Zero stack pointer. The frame
> > anchor has a new last_Java_fp field that stores the value previously
> > stored in last_Java_sp, in much the same way as the x86 port does.
> >
> > Note that the code in frame_zero.* already stored both these values,
> > but with the names confusingly reversed. I have unreversed them in
> > this webrev. I have also changed the type of the frame pointer be a
> > ZeroFrame* -- leaving the stack pointer as an intptr_t* -- to help
> > ensure that everything gets the arguments in the right order.
>
> A minor update:
>
> http://cr.openjdk.java.net/~gbenson/zero-jnihandles-fix-3/
I assume the changes work as you have (hopefully) tested them. Since it
only touches Zero files, I'd say the changes are OK. I will push it.
I also add Keith as reviewer.
-- Christian
More information about the hotspot-compiler-dev
mailing list