Re: [rvm-research] Approach for fixing the RVM-1017(Implicit unlock in synchronized method relies on local variable 0 )

David P Grove <[email protected]>
Newsgroups gmane.comp.java.jikes.rvm.devel
Message-ID <OF6B914776.E646CF3E-ON85257B66.0006EFFA-85257B66.00079246@us.ibm.com>


Pushparaj Motamari <[email protected]> wrote on 05/07/2013 09:26:35 PM:
>
> I have fixed the issue for optimizing compiler

Nice.  The fixes are independent, so if you wanted to contribute (
http://jikesrvm.org/Contributions) the opt compiler patch by attaching it
to RVM-1017 we could merge that in without waiting for the baseline
compiler fix.

> , now I started
> working on BaseLine Compiler. Please find the patch attached with
> the mail. The build getting successful , but when I try to run the
program  I am getting the gollowing exceptions
> JikesRVM: TROUBLE.  Got a signal (Segmentation fault; #11) from outside
the VM's address space in thread 0xb7404940.

I suspect you may also need to change STACKFRAME_HEADER_SIZE in
org.jikesrvm.ia32.StackframeLayoutConstants and then trace through the code
to see where else constants in StackframeLayoutConstants are referenced to
see what else needs to be adjusted.  My guess is that adding the extra slot
to a baseline compiled stackframe is going to have small, but rippling
changes through various parts of the baseline and JNI compilers and runtime
that do low-level stack manipulation.

--dave

------------------------------------------------------------------------------
Learn Graph Databases - Download FREE O'Reilly Book
"Graph Databases" is the definitive new guide to graph databases and 
their applications. This 200-page book is written by three acclaimed 
leaders in the field. The early access version is available now. 
Download your free book today! http://p.sf.net/sfu/neotech_d2d_may

_______________________________________________
Jikesrvm-researchers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/jikesrvm-researchers
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.