Integrated: 8339661: ZGC: Move some page resets and verification to callsites
Joel Sikström
duke at openjdk.org
Wed Sep 11 08:11:17 UTC 2024
On Fri, 6 Sep 2024 12:43:28 GMT, Joel Sikström <duke at openjdk.org> wrote:
> Currently, `ZPage::reset()` does different things depending on where the page is being reset and what it will be used for. This leads to checks being somewhat hard to understand and to follow the edge-cases of.
>
> By moving some of the reset logic that is now part of `ZPage::reset()` to the callsite, some operations can be made easier to understand the reason behind when reading the code.
>
> Main highlights:
> - Clear logic behind initializing remsets at callsites, now guarded by asserts in `ZPage::remset_initialize()`.
> - `ZPage::clone_limited()` retains the value of the top-pointer.
> - The kind of verification for remsets are now at callsites:
> - Allocations from the page cache, and only if the page got a remset
> - Old-to-old in-place relocations, where only the inactive remset is checked
This pull request has now been integrated.
Changeset: ceef161e
Author: Joel Sikström <joel.sikstrom at oracle.com>
Committer: Stefan Karlsson <stefank at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/ceef161eea51578160b71b20826a9328f9a87a88
Stats: 127 lines in 6 files changed: 34 ins; 64 del; 29 mod
8339661: ZGC: Move some page resets and verification to callsites
Reviewed-by: stefank, eosterlund
-------------
PR: https://git.openjdk.org/jdk/pull/20890
More information about the hotspot-gc-dev
mailing list