RFR: JDK-8315026: java/lang/ProcessHandle/TreeTest.java fails intermittent on AIX in TreeTest.test5 [v2]

[Matthias Baesken]

On Mon, 9 Oct 2023 15:00:18 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.
>> 
>> Here is a java callstack of the error situation:
>> test TreeTest.test5(): failure
>> java.lang.AssertionError: expected direct children expected [2] but found [3]
>>         at org.testng.Assert.fail(Assert.java:99)
>>         at org.testng.Assert.failNotEquals(Assert.java:1037)
>>         at org.testng.Assert.assertEqualsImpl(Assert.java:140)
>>         at org.testng.Assert.assertEquals(Assert.java:122)
>>         at org.testng.Assert.assertEquals(Assert.java:907)
>>         at TreeTest.test5(TreeTest.java:447)
>>         at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:103)
>>         at java.base/java.lang.reflect.Method.invoke(Method.java:580)
>>         at org.testng.internal.MethodInvocationHelper.invokeMethod(MethodInvocationHelper.java:132)
>>         at org.testng.internal.TestInvoker.invokeMethod(TestInvoker.java:599)
>>         at org.testng.internal.TestInvoker.invokeTestMethod(TestInvoker.java:174)
>>         at org.testng.internal.MethodRunner.runInSequence(MethodRunner.java:46)
>>         at org.testng.internal.TestInvoker$MethodInvocationAgent.invoke(TestInvoker.java:822)
>>         at org.testng.internal.TestInvoker.invokeTestMethods(TestInvoker.java:147)
>>         at org.testng.internal.TestMethodWorker.invokeTestMethods(TestMethodWorker.java:146)
>>         at org.testng.internal.TestMethodWorker.run(TestMethodWorker.java:128)
>>         at java.base/java.util.ArrayList.forEach(ArrayList.java:1597)
>>         at org.testng.TestRunner.privateRun(TestRunner.java:764)
>>         at org.testng.TestRunner.run(TestRunner.java:585)
>>         at org.testng.SuiteRunner.runTest(SuiteRunner.java:384)
>>         at org.testng.SuiteRunner.runSequentially(SuiteRunner.java:378)
>>         at org.testng.SuiteRunner.privateRun(SuiteRunner.java:337)
>>         at org.testng.SuiteRunner.run(SuiteRunner.java:286)
>>         at org.testng.SuiteRunnerWorker.runSuite(SuiteRunnerWorker.java:53)
>>         at org.testng.SuiteRunnerWorker.run(SuiteRunnerWorker.java:96)
>>       ...
>
> Joachim Kern has updated the pull request incrementally with one additional commit since the last revision:
> 
>   add comment

struct procentry64 ProcessBuffer[100];
    pid_t IndexPointer = 0;
    int Chunk = 100;


Hi, do you always want the ProcessBuffer size ==  Chunk  ? If so, why not use a const and also use this const for the size of ProcessBuffer ?

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

PR Comment: https://git.openjdk.org/jdk/pull/16051#issuecomment-1754650903