[aarch64-port-dev ] Freeze aarch64/jdk8

Andrew Hughes gnu.andrew at redhat.com
Mon Feb 29 16:37:47 UTC 2016


----- Original Message -----
> On 29/02/16 16:14, Andrew Hughes wrote:
> > snip...
> > 
> >>
> >>>> largecodecache.patch
> >>>> revid: 8576
> >>>> Add support for large code cache
> >>>>
> >>>> this makes changes to two files.
> >>>
> >>> The non-AArch64-specific parts of this patch are not used by anything
> >>> so should not be included.  The rest is OK.
> >>
> >> Ok, I will rework this to include only the AArch64-specific code.
> >>
> > 
> > The other changes appear to have been included and the build on s390 is
> > now broken as a result (mismatch between the types of
> > CODE_CACHE_DEFAULT_LIMIT and ReservedCodeCacheSize * 5 in the min2 macro).
> > Can I revert the changes to these two files?
> > (src/share/vm/utilities/globalDefinitions.hpp &
> > src/share/vm/runtime/arguments.cpp)
> 
> Andrew Haley followed up with a note explaining that these changes are
> needed on AArch64 (which is why I put them back in again). I think it
> might be better to fix the breakage to the PPC code. Perhaps Andrew
> Haley can comment?
> 

* s390.

The change was there for AArch64 only before.

-    FLAG_SET_DEFAULT(ReservedCodeCacheSize,
-                     MIN2(CODE_CACHE_DEFAULT_LIMIT, ReservedCodeCacheSize * 5));
+    FLAG_SET_DEFAULT(ReservedCodeCacheSize, ReservedCodeCacheSize * 5);
+    // The maximum B/BL offset range on AArch64 is 128MB
+    AARCH64_ONLY(FLAG_SET_DEFAULT(ReservedCodeCacheSize, MIN2(ReservedCodeCacheSize, 128*M)));

Maybe it's just a case that the first setting needs to be not AArch64
and the AArch64 one needs updating to the new version? e.g.

-    FLAG_SET_DEFAULT(ReservedCodeCacheSize,
-                     MIN2(CODE_CACHE_DEFAULT_LIMIT, ReservedCodeCacheSize * 5));
+    NOT_AARCH64(FLAG_SET_DEFAULT(ReservedCodeCacheSize, ReservedCodeCacheSize * 5));
+    AARCH64_ONLY(FLAG_SET_DEFAULT(ReservedCodeCacheSize,
+				  MIN2(CODE_CACHE_DEFAULT_LIMIT, ReservedCodeCacheSize * 5)));

We then don't change the behaviour on other architectures.

> regards,
> 
> 
> Andrew Dinn
> -----------
> 
> 

-- 
Andrew :)

Senior Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

PGP Key: ed25519/35964222 (hkp://keys.gnupg.net)
Fingerprint = 5132 579D D154 0ED2 3E04  C5A0 CFDA 0F9B 3596 4222




More information about the aarch64-port-dev mailing list