Code Review - CR# 7030573 insufficient disk space for large file test
Alan Bateman
Alan.Bateman at oracle.com
Thu Jan 5 13:30:31 UTC 2012
On 23/12/2011 16:19, Gary Adams wrote:
> The LargeFileAvailable regression test had intermittent failures
> when there was not sufficient space available to create
> a 7G temp file. This webrev presents a simple check to
> see if the available usable space is less than 7G and
> scales the test back accordingly.
>
> The original bug report suggests that the test be switched
> to use the current working directory rather than a temp
> file. I think that could be the wrong choice for an embedded
> system that might have the tests mounted from a remote
> file system. In that scenario, using the local temp file
> space provides a better solution for what this test is designed
> to check.
>
> http://cr.openjdk.java.net/~gadams/7030573/
The only thing is that when the test is scaled back too much then it no
longer tests the original issue. This test will create a sparse file on
file systems that support it and I suspect the reason it fails on
Solaris is that tmp is backed by swap. It might be better if we changed
the test to create the file in the current directory (or a sub-directory
of). It will be removed by jtreg if the test doesn't delete it.
-Alan
More information about the core-libs-dev
mailing list