[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-serviceability-dev
Subject: Re: RFR: 8264411: serviceability/jvmti/HeapMonitor tests intermittently fail due to large TLAB size
From: Hui Shi <hshi () openjdk ! java ! net>
Date: 2021-03-30 23:51:41
Message-ID: 653aJgBTTTvZWbRDrROT_5QzhkI6HO074qhVBkM-3ik=.aacec9d5-fd2f-407e-9d44-605664d30689 () github ! com
[Download RAW message or body]
> …ue to large TLAB size
>
> serviceability/jvmti/HeapMonitor tests intermittently fail when using PS/Serial GC, \
> original test has implicit assumptions on TLAB size and depends on allocate fix \
> amount of objects to consume TLAB and trigger object sampling. These tests will \
> fail if TLAB is above 20M (this can easily happen when using PS/Serial GC and heap \
> is large), when allocation can not consume current TLAB and _byte_until_sample.
> Fix in tests is adding an explicit GC to consume current TLAB.
> Running on 256G memory machine, make run-test CONF=release \
> TEST="test/hotspot/jtreg/serviceability/jvmti/HeapMonitor/MyPackage/" \
> 'JTREG=JOBS=12;VM_OPTIONS=-XX:ActiveProcessorCount=1'
> before fix: 6 or 7 tests in 20 tests intermittently fail
> after fix: no failure in 100 runs release/fastdebug
>
> This might also fix https://bugs.openjdk.java.net/browse/JDK-8225313
Hui Shi has updated the pull request incrementally with one additional commit since \
the last revision:
update copyright year
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/3265/files
- new: https://git.openjdk.java.net/jdk/pull/3265/files/0b098656..3dd108d5
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=3265&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=3265&range=00-01
Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
Patch: https://git.openjdk.java.net/jdk/pull/3265.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/3265/head:pull/3265
PR: https://git.openjdk.java.net/jdk/pull/3265
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic