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.ri...@oracle.com
<mailto:roger.ri...@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