Fix for test/hotspot/jtreg/runtime/8176717/TestInheritFD.java on Alpine Linux

[Alexander Scherbatiy]

On 13.02.2020 01:18, Mikael Vidstedt wrote:

> I’m not completely sold on the concept of storing a boolean as a string in the command array. How about creating an actual class with an explicit field for that instead?

Here is the updated webrev where Command class is used instead of an 
array of strings:

http://cr.openjdk.java.net/~alexsch/portola/TestInheritFD/webrev.01/

Thanks,

Alexander.

>
> Cheers,
> Mikael
>
>> On Feb 10, 2020, at 5:22 AM, Alexander Scherbatiy <alexander.scherbatiy at bell-sw.com> wrote:
>>
>> Hello,
>>
>> The test test/hotspot/jtreg/runtime/8176717/TestInheritFD.java fails on Alpine Linux because "lsof" command from busybox does not support "-p" option and just list all open files for all process (see discussion [1])
>>
>> The proposed fix checks not only that result of "lsof -f" command contains the GC log file but also that it has the child pid:
>>
>> http://cr.openjdk.java.net/~alexsch/portola/TestInheritFD/webrev.00/
>>
>> This does not work work for Solaris "pfiles -F" so the additional flag is added that the pid is only checked for "lsof".
>>
>> Another proposed options was to check open files for a process by "ls -l /proc/$$/fd". It seems it also is not directly applied for Solaris.
>>
>> [1] https://www.openwall.com/lists/musl/2020/02/08/3
>>
>> Thanks,
>>
>> Alexandr.
>>
>>