[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20160221233521.GA23122@thunk.org>
Date: Sun, 21 Feb 2016 18:35:21 -0500
From: Theodore Ts'o <tytso@....edu>
To: Daeho Jeong <daeho.jeong@...sung.com>
Cc: jack@...e.cz, linux-ext4@...r.kernel.org
Subject: Re: [PATCH v2] ext4: make sure to revoke all the freeable blocks in
ext4_free_blocks
On Thu, Dec 17, 2015 at 10:49:00AM +0900, Daeho Jeong wrote:
> Now, ext4_free_blocks() doesn't revoke data blocks of per-file data
> journalled inode and it can cause file data inconsistency problems.
> Even though data blocks of per-file data journalled inode are already
> forgotten by jbd2_journal_invalidatepage() in advance of invoking
> ext4_free_blocks(), we still need to revoke the data blocks here.
> Moreover some of the metadata blocks, which are not found by
> sb_find_get_block(), are still needed to be revoked, but this is also
> missing here.
>
> Signed-off-by: Daeho Jeong <daeho.jeong@...sung.com>
> Reviewed-by: Jan Kara <jack@...e.cz>
Thanks, applied.
Apologies for the delay.
- Ted
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists