Steve Poole wrote:
Hi all - I attach a hg export file with a fix for 6927816 for JDK7. Since
this is my first submission please be gentle.
There is no testcase for this problem as it manifests only when using our
J9 JVM. I've build and run the change on Linux and Solaris using B121 and
there doesn't seem to be any side effects.
I'd appreciate a review of the change to make sure I've got it right - also
some guidance on what to expect in seeing this fix progress through the
process to the main jdk7 repository.
(See attached file: 6927816.export)
Regards
Steve Poole
Looks good to me too.
In case you haven't seen them, the test cases for the jvmti demos are in
jdk/test/demo/jvmti/ directory. You can use jtreg to run them.
I would suggest pushing it the TL repo [1] once your ssh key is setup
and you have a username. Once your changes are in the TL repo then they
will find their way into the master. It can take a few weeks, depending
on when you push. At this time, the changes accumulated in TL are pushed
to master every two weeks.
-Alan.
[1] http://hg.openjdk.java.net/jdk7/tl/jdk
