diff options
| author | Eric Sandeen <sandeen@redhat.com> | 2009-10-03 03:20:55 +0200 |
|---|---|---|
| committer | Theodore Ts'o <tytso@mit.edu> | 2009-10-03 03:20:55 +0200 |
| commit | fbbf69456619de5d251cb9f1df609069178c62d5 (patch) | |
| tree | e0a5bc75a02dda7f46ba4d0e838c54f39b1fe4f6 | |
| parent | ext4: Fix build warning in ext4_dirty_inode() (diff) | |
| download | linux-fbbf69456619de5d251cb9f1df609069178c62d5.tar.xz linux-fbbf69456619de5d251cb9f1df609069178c62d5.zip | |
[PATCH] ext4: retry failed direct IO allocations
On a 256M filesystem, doing this in a loop:
xfs_io -F -f -d -c 'pwrite 0 64m' test
rm -f test
eventually leads to ENOSPC. (the xfs_io command does a
64m direct IO write to the file "test")
As with other block allocation callers, it looks like we need to
potentially retry the allocations on the initial ENOSPC.
Signed-off-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
| -rw-r--r-- | fs/ext4/inode.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c index 635f8ec6ebc6..5c5bc5dafff8 100644 --- a/fs/ext4/inode.c +++ b/fs/ext4/inode.c @@ -3378,6 +3378,7 @@ static ssize_t ext4_ind_direct_IO(int rw, struct kiocb *iocb, ssize_t ret; int orphan = 0; size_t count = iov_length(iov, nr_segs); + int retries = 0; if (rw == WRITE) { loff_t final_size = offset + count; @@ -3400,9 +3401,12 @@ static ssize_t ext4_ind_direct_IO(int rw, struct kiocb *iocb, } } +retry: ret = blockdev_direct_IO(rw, iocb, inode, inode->i_sb->s_bdev, iov, offset, nr_segs, ext4_get_block, NULL); + if (ret == -ENOSPC && ext4_should_retry_alloc(inode->i_sb, &retries)) + goto retry; if (orphan) { int err; |