RFR (S) - 8065155 Refactor Hotspot mapfiles
Magnus Ihse Bursie
magnus.ihse.bursie at oracle.com
Fri Sep 18 07:06:03 UTC 2015
On 2015-09-18 00:44, Ioi Lam wrote:
> Please review a small fix:
>
> http://cr.openjdk.java.net/~iklam/8065155-refactor-hotspot-mapfiles.00/
Looks good to me.
/Magnus
>
> Bug: Refactor Hotspot mapfiles
>
> https://bugs.openjdk.java.net/browse/JDK-8065155
>
> Summary of fix:
>
> After this fix, when you're adding a new JVM_XXX or JNI_XXX function,
> you no longer need to edit *9* files!!
>
> I checked the mapfiles of all platforms that are in the jdk9/hs-rt
> repo:
>
> [a] linux, solaris, bsd, bsd-darwin all have an identical common
> part that
> contains many JNI_ and JVM_ functions.
> [b] the common parts of linux, solaris and bsd are sorted in the
> same order.
> bsd-darwin is sorted differently but the contents are the same.
> [c] aix is slightly differently than the other platforms, probably
> due to
> code rot.
>
> So the fix is to move the common part into a separate file. I
> changed the makefiles of all platforms to concatenate the common
> part into
> the main mapfile.
>
> As part of the fix, I also addressed:
> JDK-8134783 - libjvm.so is not rebuilt after mapfile-ext is
> modified
>
> Note that I have not tested AIX since I don't have a build
> environment. However,
> it should be no worse than before.
>
> Tests:
>
> JPRT (passed)
> RBT (will do it before pushing)
>
> Thanks
> - Ioi
More information about the build-dev
mailing list