RFR: 8298173: GarbageCollectionNotificationContentTest test failed: no decrease in Eden usage

[Daniel D. Daugherty]

On Wed, 7 Dec 2022 21:50:11 GMT, Kevin Walls <kevinw at openjdk.org> wrote:

> Test should not check that Eden has decreased in used size if the GCCause is "No GC".
> 
> These happen with the message "end of concurrent GC pause" which is logged by the GCMemoryManager _conc_gc_memory_manager (in G1MonitoringSupport).  This GCMemoryManager only interacts with the old gen memory pool, so it is expected that eden usage does not change.
> 
> Updated a few variable names to make it more obvious what count and number are recording, and some formatting.  The key  part of the change is at line 137 onwards.

The variable renaming and code clean ups were a bit distracting.
However, I'm okay with the changes to skip verification when
"No GC" is the GCCause.

What kind of testing was done?

The bug is targeted at '21'. If you plan to fix this in '20' before
the cutoff, then please change the target to '20'. If you don't
plan to fix this in '20', then you'll need to ProblemList the test
in '20' because this failure is rather noisy as we already have
6 sightings in the JDK20 CI.

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

Marked as reviewed by dcubed (Reviewer).

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