Hi all,

  please review this change that removes the pinned tag from `HeapRegion`.

So that "pinned" tag for G1 heap regions indicates that the region should not 
move during (young) gc. This applies to now removed archive regions and 
humongous objects/regions.

With "real" g1 region pinning to deal with gclocker in g1 once and for all 
upcoming we need a refcount, a single bit is not sufficient anymore. Further 
there will be a naming conflict as this kind of "pinning" is different to g1 
region pinning "pinning". The former indicates "contents can not be moved, but 
can be reclaimed", while the latter means "contents can not be moved and not 
reclaimed".

The (current) pinned flag is surprisingly little used, only for policy 
decisions.

The suggestion this change implements is to remove the "pinned" tag as it is, 
and reserve it for future g1 region pinning (that needs to store the pinning 
attribute differently anyway).

Testing: tier1-3, gha

Thanks,
  Thomas

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

Commit messages:
 - Fix hsdb
 - compilation fixes
 - Initial implementation

Changes: https://git.openjdk.org/jdk/pull/13643/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=13643&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8306836
  Stats: 66 lines in 20 files changed: 16 ins; 22 del; 28 mod
  Patch: https://git.openjdk.org/jdk/pull/13643.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/13643/head:pull/13643

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

Reply via email to