RFR: JDK-8315026: ProcessHandle implementation listing processes on AIX should use getprocs64 [v5]
Thomas Stuefe
stuefe at openjdk.org
Thu Oct 12 16:02:35 UTC 2023
On Thu, 12 Oct 2023 09:30:09 GMT, Joachim Kern <jkern at openjdk.org> wrote:
>> We see rather often failures in java/lang/ProcessHandle/TreeTest.java on AIX in TreeTest.test5.
>> The reason is: Previously the implementation based on the /proc file system lead to double pids in the child list; at least intermittent. Using the API getprocs64() instead I was able to eliminate those double pids (and increase the performance by a factor of 4). Otherwise we would have to add an algorithm to filter out the doubles after creating the raw list.
>
> Joachim Kern has updated the pull request incrementally with one additional commit since the last revision:
>
> revise of the function comment and some other proposals
ok
-------------
Marked as reviewed by stuefe (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/16051#pullrequestreview-1674558899
More information about the core-libs-dev
mailing list