vitalif
/
e2fsprogs
mirror of https://github.com/vitalif/e2fsprogs
1
0
Fork 0
Code Issues Releases Wiki Activity

libext2fs: Make ext2fs_extent_set_bmap() more robust against ENOSPC 

In the case where we ext2fs_extent_set_bmap() is replacing the block
mapping at the beginning of an already-existing extent, insert a new
extent if necessary before shrinking an existing extent, to avoid data
loss if the disk is full.

This mostly addresses the problem described in Red Hat Bugzilla's
statistics are still wrong, but at least the files on the filesystem
are not corrupted.  If there is a failure during the
inode_scan_and_fix pass, the simplest thing to do may be to tell the
user to run e2fsck -fy.

Addresses-Red-Hat-Bug: #510379

Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
bitmap-optimize
Theodore Ts'o 2009-07-10 00:19:28 -04:00
parent ef1a526dfb
commit 0dc2916112
1 changed files with 33 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

46
lib/ext2fs/extent.c
View File

@ -1173,7 +1173,6 @@ errcode_t ext2fs_extent_set_bmap(ext2_extent_handle_t handle,
orig_height = info.max_depth - info.curr_level;
orig_lblk = extent.e_lblk;
again:
/* go to the logical spot we want to (re/un)map */
retval = ext2fs_extent_goto(handle, logical);
if (retval) {
@ -1270,24 +1269,44 @@ again:
#ifdef DEBUG
printf("(re/un)mapping first block in extent\n");
#endif
if (physical) {
retval = ext2fs_extent_get(handle,
EXT2_EXTENT_PREV_LEAF,
&extent);
if (extent.e_flags & EXT2_EXTENT_FLAGS_UNINIT)
extent_uninit = 1;
if (retval == EXT2_ET_EXTENT_NO_PREV) {
retval = ext2fs_extent_goto(handle, logical);
if (retval)
goto done;
retval = ext2fs_extent_insert(handle,
0, &newextent);
} else if (retval)
goto done;
else if ((logical == extent.e_lblk + extent.e_len) &&
(physical == extent.e_pblk + extent.e_len) &&
(new_uninit == extent_uninit) &&
((int) extent.e_len < max_len-1)) {
extent.e_len++;
retval = ext2fs_extent_replace(handle, 0,
&extent);
} else {
retval = ext2fs_extent_insert(handle,
EXT2_EXTENT_INSERT_AFTER, &newextent);
}
if (retval)
goto done;
}
retval = ext2fs_extent_get(handle, EXT2_EXTENT_NEXT_LEAF,
&extent);
if (retval)
goto done;
extent.e_pblk++;
extent.e_lblk++;
extent.e_len--;
retval = ext2fs_extent_replace(handle, 0, &extent);
if (retval)
goto done;
if (physical) {
/*
* We've removed the old block, now rely on
* the optimized hueristics for adding a new
* mapping with appropriate merging if necessary.
*/
goto again;
} else {
retval = ext2fs_extent_fix_parents(handle);
if (retval)
goto done;
}
} else {
__u32 orig_length;
@ -1335,6 +1354,7 @@ errcode_t ext2fs_extent_delete(ext2_extent_handle_t handle, int flags)
char *cp;
struct ext3_extent_header *eh;
errcode_t retval = 0;
struct ext2fs_extent extent;
EXT2_CHECK_MAGIC(handle, EXT2_ET_MAGIC_EXTENT_HANDLE);