Integrated: 8252713: jtreg time out of CtrlASCII.java seems to hang the Xserver.

Alexander Zvegintsev azvegint at openjdk.org
Tue Nov 22 09:23:35 UTC 2022


On Thu, 17 Nov 2022 15:51:50 GMT, Alexander Zvegintsev <azvegint at openjdk.org> wrote:

> This test needs more stabilization:
> 
> - The test can be easily hang by replacing <br> iteration on `keycharHash.elements()` with `keycharHash.forEach((k, v) -> punchCtrlKey(robot, v));` 
> I do not see any reason why we need a synchronized `Hashtable`, so switched to `HashMap`
> Looks like it might be a reason of original test hanging.
> - `keyRelease`, `dispose` calls wrapped with finally block
> - Just to be safe added timeout `@run main/timeout=600` ( none of the CI test runs took too long thought)
> 
> CI testing looks good on all platforms.

This pull request has now been integrated.

Changeset: 6d6046b3
Author:    Alexander Zvegintsev <azvegint at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/6d6046b3799217c281d077f12bce1ec590149849
Stats:     229 lines in 2 files changed: 56 ins; 55 del; 118 mod

8252713: jtreg time out of CtrlASCII.java seems to hang the Xserver.

Reviewed-by: prr

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

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



More information about the client-libs-dev mailing list