[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