On Wed, 23 Aug 2023 16:41:23 GMT, Alan Bateman <al...@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 <al...@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

Reply via email to