Integrated: JDK-8275440: Remove VirtualSpaceList::is_full()

Thomas Stuefe stuefe at openjdk.java.net
Fri Oct 29 03:51:22 UTC 2021


On Thu, 28 Oct 2021 06:18:39 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

> This function is only used from a single gtest. Ioi remarked that it is not threadsafe. The context in which it is used is not multithreaded, but nevertheless I opted to remove the function and modified the test to make the condition somewhat more generic.
> 
> This trades a slight decrease in complexity with a very minor decrease in test precision, but that is fine.
> 
> Test: 
> - manual gtests
> - GHAs
> - ran the metaspace jtreg gtests, which execute gtests with various metaspace modes

This pull request has now been integrated.

Changeset: c9e65f8e
Author:    Thomas Stuefe <stuefe at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/c9e65f8ef926c3796867558afa536770eed71cd6
Stats:     15 lines in 3 files changed: 0 ins; 13 del; 2 mod

8275440: Remove VirtualSpaceList::is_full()

Reviewed-by: coleenp

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

PR: https://git.openjdk.java.net/jdk/pull/6147


More information about the hotspot-runtime-dev mailing list