> Hi all,
> 
> This PR make 2 changes which make test more rubustness:
> 
> 1. Add max 10 times tries which removed by JDK-8342659, this will make test 
> more steady after JDK-8373945. 
> vmTestbase/nsk/jvmti/CompiledMethodUnload/compmethunload001/TestDescription.java
>  fails before this PR, run passed after this PR with fastdebug build.
> 2. 
> test/hotspot/jtreg/vmTestbase/nsk/jvmti/scenarios/events/EM02/em02t003.java 
> try to invoke thread and do not wailt the thread run finish unload the class 
> immediately. This PR wait util the thread is non alive, and wait more 2 
> senonds to make sure the thread exit completely. This will make test more 
> robustness, before this PR 
> vmTestbase/nsk/jvmti/scenarios/events/EM02/em02t003/TestDescription.java 
> fails probability 1/50, after this PR test run all passed 10k times.
> 
> Additional testing:
> 
> - [ ] All tests touched test by JDK-8373945

SendaoYan has updated the pull request incrementally with one additional commit 
since the last revision:

  1. Add small delay after check thrd.isAlive; 2. Add small delay between 
classloader reclaimed retries.

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/29285/files
  - new: https://git.openjdk.org/jdk/pull/29285/files/7b6daf35..4f3d03a8

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=29285&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=29285&range=00-01

  Stats: 11 lines in 2 files changed: 11 ins; 0 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/29285.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/29285/head:pull/29285

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

Reply via email to