RFR: 8295099: vmTestbase/nsk/stress/strace/strace013.java failed with "TestFailure: wrong lengths of stack traces: strace013Thread0: NNN strace013Thread83: MMM" [v3]

[David Holmes]

> The test is incorrectly synchronized and may take the stack trace of a thread before it has entered the wait() and thus be three frames short of the correct stack. So I cleaned that up to ensure things are correctly synchronized.
> 
> A number of comments in the test description etc were wrong so I also fixed those.
> 
> These tests are stylistically awful but I refrained from embarking on a major cleanup and adopted the prevailing styles.
> 
> Testing: local plus 50x on each platform (in progress)
> 
> Thanks.

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

  Revert sophisticated fix and just correct the "slack" allownace from 2 to 3.

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/11011/files
  - new: https://git.openjdk.org/jdk/pull/11011/files/fc1378fa..42412393

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

  Stats: 28 lines in 1 file changed: 12 ins; 10 del; 6 mod
  Patch: https://git.openjdk.org/jdk/pull/11011.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/11011/head:pull/11011

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