diff options
| author | Christoph Hellwig <hch@lst.de> | 2010-06-04 11:29:58 +0200 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2010-08-09 16:47:33 -0400 |
| commit | 155130a4f7848b1aac439cab6bda1a175507c71c (patch) | |
| tree | 5019b14b0a9221b08959196e01e3f9326957d678 /fs/bfs | |
| parent | 6e1db88d536adcbbfe562b2d4b7d6425784fff12 (diff) | |
| download | lwn-155130a4f7848b1aac439cab6bda1a175507c71c.tar.gz lwn-155130a4f7848b1aac439cab6bda1a175507c71c.zip | |
get rid of block_write_begin_newtrunc
Move the call to vmtruncate to get rid of accessive blocks to the callers
in preparation of the new truncate sequence and rename the non-truncating
version to block_write_begin.
While we're at it also remove several unused arguments to block_write_begin.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/bfs')
| -rw-r--r-- | fs/bfs/file.c | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/fs/bfs/file.c b/fs/bfs/file.c index 88b9a3ff44e4..8fc2e9c9739d 100644 --- a/fs/bfs/file.c +++ b/fs/bfs/file.c @@ -168,9 +168,17 @@ static int bfs_write_begin(struct file *file, struct address_space *mapping, loff_t pos, unsigned len, unsigned flags, struct page **pagep, void **fsdata) { - *pagep = NULL; - return block_write_begin(file, mapping, pos, len, flags, - pagep, fsdata, bfs_get_block); + int ret; + + ret = block_write_begin(mapping, pos, len, flags, pagep, + bfs_get_block); + if (unlikely(ret)) { + loff_t isize = mapping->host->i_size; + if (pos + len > isize) + vmtruncate(mapping->host, isize); + } + + return ret; } static sector_t bfs_bmap(struct address_space *mapping, sector_t block) |