On Tue, 25 Jul 2023 06:28:16 GMT, Thomas Stuefe <stu...@openjdk.org> wrote:
> Trivial patch. > > The test just fires `jcmd System.trim_native_heap`, but without preparing any > special memory allocation conditions. Typically, we end up firing the command > when VM initialization is done, and therefore end up trimming a lot of > memory. But once in a blue moon concurrent allocations may be ongoing, so we > may not trim anything, or even see increased RSS. > > The patch just loosens the success condition to succeed if the trim happened, > irregardless of how much memory had been trimmed. Looks good to me. ------------- Marked as reviewed by kevinw (Committer). PR Review: https://git.openjdk.org/jdk/pull/15009#pullrequestreview-1545405008