diff options
author | Simon Rettberg | 2018-03-16 10:07:07 +0100 |
---|---|---|
committer | Simon Rettberg | 2018-03-16 10:07:07 +0100 |
commit | 1f3de3f0e20d20e8f388f07339630090edaba56c (patch) | |
tree | 9bf99629a878019b55fa907852b19eedebc2fd37 /src/server/image.c | |
parent | [SERVER] Experimental support for sparse files in proxy mode (diff) | |
download | dnbd3-1f3de3f0e20d20e8f388f07339630090edaba56c.tar.gz dnbd3-1f3de3f0e20d20e8f388f07339630090edaba56c.tar.xz dnbd3-1f3de3f0e20d20e8f388f07339630090edaba56c.zip |
[SERVER] Make sparse file mode actually work
Diffstat (limited to 'src/server/image.c')
-rw-r--r-- | src/server/image.c | 17 |
1 files changed, 12 insertions, 5 deletions
diff --git a/src/server/image.c b/src/server/image.c index 78091f1..cb01181 100644 --- a/src/server/image.c +++ b/src/server/image.c @@ -1123,19 +1123,22 @@ bool image_create(char *image, int revision, uint64_t size) goto failure_cleanup; } // Try cache map first - if ( !file_alloc( fdCache, 0, mapsize ) ) { + if ( !file_alloc( fdCache, 0, mapsize ) && !file_setSize( fdCache, mapsize ) ) { const int err = errno; logadd( LOG_DEBUG1, "Could not allocate %d bytes for %s (errno=%d)", mapsize, cache, err ); } // Now write image if ( !_sparseFiles && !file_alloc( fdImage, 0, size ) ) { - const int err = errno; - logadd( LOG_ERROR, "Could not allocate %" PRIu64 " bytes for %s (errno=%d)", size, path, err ); + logadd( LOG_ERROR, "Could not allocate %" PRIu64 " bytes for %s (errno=%d)", size, path, errno ); logadd( LOG_ERROR, "It is highly recommended to use a file system that supports preallocating disk" " space without actually writing all zeroes to the block device." ); logadd( LOG_ERROR, "If you cannot fix this, try setting sparseFiles=true, but don't expect" " divine performance during replication." ); goto failure_cleanup; + } else if ( _sparseFiles && !file_setSize( fdImage, size ) ) { + logadd( LOG_ERROR, "Could not create sparse file of %" PRIu64 " bytes for %s (errno=%d)", size, path, errno ); + logadd( LOG_ERROR, "Make sure you have enough disk space, check directory permissions, fs errors etc." ); + goto failure_cleanup; } close( fdImage ); close( fdCache ); @@ -1741,8 +1744,12 @@ static bool image_ensureDiskSpace(uint64_t size, bool force) current = image_release( current ); } declare_now; - if ( oldest == NULL || timing_diff( &oldest->atime, &now ) < 86400 ) { - logadd( LOG_DEBUG1, "No image is old enough (all have been in use in past 24h) :-(\n" ); + if ( oldest == NULL || ( !_sparseFiles && timing_diff( &oldest->atime, &now ) < 86400 ) ) { + if ( oldest == NULL ) { + logadd( LOG_INFO, "All images are currently in use :-(" ); + } else { + logadd( LOG_INFO, "Won't free any image, all have been in use in the past 24 hours :-(" ); + } return false; } oldest = image_lock( oldest ); |