RFR 8202292 : test/jdk/java/io/FileOutputStream/UnreferencedFOSClosesFd.java fails with "raw fd count wrong"
David Holmes
david.holmes at oracle.com
Mon Jun 25 05:00:51 UTC 2018
Hi Roger,
On 23/06/2018 12:49 AM, Roger Riggs wrote:
> Hi Brian, Mandy,
>
> Listing the open file descriptors can be a useful utility. Originally I
> thought it was a temporary and throw away code.
> To be a utility it needed to be a bit more general (sending the output
> to a particular stream).
> Thanks for the suggestions.
You might want to look at the code Leo used for a hotspot test recently.
It covers more locations for lsof and also supports pfiles for Solaris:
http://hg.openjdk.java.net/jdk/jdk/rev/9ba6f5dfbe56
http://hg.openjdk.java.net/jdk/jdk/rev/1372f66e0a17
But there can be other locations like /usr/local/bin for lsof on Solaris
too.
David
> Please review:
> http://cr.openjdk.java.net/~rriggs/webrev-count-wrong-8202292/
>
> Thanks, Roger
>
> On 6/21/2018 3:25 PM, Brian Burkhalter wrote:
>> Hi Roger,
>>
>> Are the three versions of listProcFD() identical? Is so, could this
>> method be put in test/lib/jdk/test/lib/util/FileUtils.java instead?
>>
>> Thanks,
>>
>> Brian
>>
>> On Jun 21, 2018, at 11:59 AM, Roger Riggs <Roger.Riggs at Oracle.com
>> <mailto:Roger.Riggs at Oracle.com>> wrote:
>>
>>> Please review a test improvement to avoid spurious errors caused by
>>> concurrent changes in open files during the test.
>>> The other existing conditions in the test are sufficient to verify
>>> the correct finalize behavior.
>>> Some additional diagnostics are added and the tests removed from the
>>> problem list.
>>>
>>> Webrev:
>>> http://cr.openjdk.java.net/~rriggs/webrev-count-wrong-8202292/index.html
>>> <http://cr.openjdk.java.net/%7Erriggs/webrev-count-wrong-8202292/index.html>
>>>
>>>
>>> Issue:
>>> https://bugs.openjdk.java.net/browse/JDK-8202292
>>
>
More information about the core-libs-dev
mailing list