Integrated: 8314759: VirtualThread.parkNanos timeout adjustment when pinned should be replaced

Alan Bateman alanb at openjdk.org
Thu Aug 24 16:54:36 UTC 2023


On Wed, 23 Aug 2023 16:41:23 GMT, Alan Bateman <alanb at openjdk.org> wrote:

> If yielding fails due to the pinning then VirtualThread.parkNanos parks on the carrier thread with the remaining time. The calculation of the remaining time needs to be replaced so that it obviously uses the difference between the start and end time in the calculation. The current code isn't correct for cases where System.nanoTimes return a negative value or when parking for durations close to Long.MAX_VALUE (292 years). The change isn't really testable so there aren't any test changes included.

This pull request has now been integrated.

Changeset: d9fc6b69
Author:    Alan Bateman <alanb at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/d9fc6b6933ad733d1b2a4d2bd91ecb3824970ea8
Stats:     4 lines in 1 file changed: 0 ins; 2 del; 2 mod

8314759: VirtualThread.parkNanos timeout adjustment when pinned should be replaced

Reviewed-by: aturbanov, shade, dfuchs

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

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


More information about the core-libs-dev mailing list