RFR: 8291106: ZPlatformGranuleSizeShift is redundant due to it can not be modified

Leslie Zhai lzhai at openjdk.org
Wed Jul 27 01:25:15 UTC 2022


Hi @stefank 

When I am tunning `ZPageSizeSmall` to 16 MB from 2MB:


const size_t ZPlatformGranuleSizeShift          = 24; // 16MB

// Granule shift/size
const size_t      ZGranuleSizeShift             = ZPlatformGranuleSizeShift;

// Page size shifts
const size_t      ZPageSizeSmallShift           = ZGranuleSizeShift;

// Page sizes
const size_t      ZPageSizeSmall                = (size_t)1 << ZPageSizeSmallShift;


`zBitField` failed to work:


Internal Error
 (/home/zhaixiang/jdk/src/hotspot/share/gc/z/zBitField.hpp:76), pid=923047, tid=923069
#  assert(((ContainerType)value & (FieldMask << ValueShift)) == (ContainerType)value) failed: Invalid value
#
# JRE version: OpenJDK Runtime Environment (20.0) (fastdebug build 20-internal-adhoc.root.jdk)
# Java VM: OpenJDK 64-Bit Server VM (fastdebug 20-internal-adhoc.root.jdk, mixed mode, sharing, tiered, compressed class ptrs, z gc, linux-aarch64)


Perhaps `mask` also need to be adjusted:


static size_t object_index(oop obj) {
  const uintptr_t addr = ZOop::to_address(obj);
  const uintptr_t offset = ZAddress::offset(addr);
  const uintptr_t mask = ZGranuleSize - 1;
  return (offset & mask) >> ZObjectAlignmentSmallShift;
}


Back to the point: ZPlatformGranuleSizeShift is redundant due to it can not be modified, for example, `24`, so I removed the `ZPlatformGranuleSizeShift` from cpu backends, just keep it for debugging AArch64 to see the issue.

Please review my patch.

Thanks,
Leslie Zhai

-------------

Commit messages:
 - 8291106: ZPlatformGranuleSizeShift is redundant due to it can not be modified
 - ZPlatformGranuleSizeShift is redundant due to it can not be modified

Changes: https://git.openjdk.org/jdk/pull/9582/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=9582&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8291106
  Stats: 20 lines in 6 files changed: 15 ins; 3 del; 2 mod
  Patch: https://git.openjdk.org/jdk/pull/9582.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/9582/head:pull/9582

PR: https://git.openjdk.org/jdk/pull/9582


More information about the hotspot-dev mailing list