> …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

Reply via email to