summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStefan Brüns <stefan.bruens@rwth-aachen.de>2016-10-09 18:15:28 (GMT)
committerTom Rini <trini@konsulko.com>2016-10-24 12:04:37 (GMT)
commit805e3e00f24cf5d7f922a0e5a0cd0e08ce9a8c7b (patch)
treefdc1a2be767a9149cba5ba4bb0eb86403ac501fc
parentd1bdf22461e9f0638d4ddca0ccccfe61f8a653cf (diff)
downloadu-boot-805e3e00f24cf5d7f922a0e5a0cd0e08ce9a8c7b.tar.xz
ext4: Only write journal entries for modified blocks in unlink_filename
Instead of creating a journal entry for each directory block, even if the block is unmodified, only log the modified block. Signed-off-by: Stefan Brüns <stefan.bruens@rwth-aachen.de> Reviewed-by: Tom Rini <trini@konsulko.com> Reviewed-by: Lukasz Majewski <l.majewski@samsung.com>
-rw-r--r--fs/ext4/ext4_common.c23
1 files changed, 13 insertions, 10 deletions
diff --git a/fs/ext4/ext4_common.c b/fs/ext4/ext4_common.c
index 0c11ddb..4248ac1 100644
--- a/fs/ext4/ext4_common.c
+++ b/fs/ext4/ext4_common.c
@@ -874,8 +874,6 @@ static int unlink_filename(char *filename, unsigned int blknr)
if (status == 0)
goto fail;
- if (ext4fs_log_journal(block_buffer, blknr))
- goto fail;
offset = 0;
do {
previous_dir = dir;
@@ -889,14 +887,6 @@ static int unlink_filename(char *filename, unsigned int blknr)
if (dir->inode && (strlen(filename) == dir->namelen) &&
(strncmp(direntname, filename, dir->namelen) == 0)) {
inodeno = le32_to_cpu(dir->inode);
- if (previous_dir) {
- uint16_t new_len;
- new_len = le16_to_cpu(previous_dir->direntlen);
- new_len += le16_to_cpu(dir->direntlen);
- previous_dir->direntlen = cpu_to_le16(new_len);
- } else {
- dir->inode = 0;
- }
break;
}
@@ -905,7 +895,20 @@ static int unlink_filename(char *filename, unsigned int blknr)
} while (offset < fs->blksz);
if (inodeno > 0) {
+ printf("file found, deleting\n");
+ if (ext4fs_log_journal(block_buffer, blknr))
+ goto fail;
+ if (previous_dir) {
+ /* merge dir entry with predecessor */
+ uint16_t new_len;
+ new_len = le16_to_cpu(previous_dir->direntlen);
+ new_len += le16_to_cpu(dir->direntlen);
+ previous_dir->direntlen = cpu_to_le16(new_len);
+ } else {
+ /* invalidate dir entry */
+ dir->inode = 0;
+ }
if (ext4fs_put_metadata(block_buffer, blknr))
goto fail;
ret = inodeno;