lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sun, 19 Dec 2021 12:45:09 +0800 From: Xin Yin <yinxin.x@...edance.com> To: Harshad Shirwadkar <harshadshirwadkar@...il.com> Cc: adilger.kernel@...ger.ca, linux-ext4@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [External] Re: [PATCH] ext4: fix fast commit may miss tracking range for FALLOC_FL_ZERO_RANGE On Sat, Dec 18, 2021 at 4:43 AM Harshad Shirwadkar <harshadshirwadkar@...il.com> wrote: > > > fs/ext4/extents.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > >diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c > >index 9229ab1f99c5..4108896d471b 100644 > >--- a/fs/ext4/extents.c > >+++ b/fs/ext4/extents.c > >@@ -4433,6 +4433,8 @@ static int ext4_alloc_file_blocks(struct file *file, ext4_lblk_t offset, > > ret2 = ext4_journal_stop(handle); > > break; > > } > >+ ext4_fc_track_range(handle, inode, map.m_lblk, > >+ map.m_lblk + map.m_len - 1); > > ext4_alloc_file_blocks() calls ext4_map_blocks(), inside which we do > call ext4_fc_track_range(). However, we are doing that only if > map.m_flags & EXT4_MAP_MAPPED is true. So, unwritten flag is set we > are not calling track_range there. Perhaps the right fix is to call > ext4_fc_track_range() from ext4_map_blocks() if MAPPED or UNWRITTEN > flag is set? Thanks, you are right, this should be better. I will test this way and resend a v2 patch for this issue. > > > map.m_lblk += ret; > > map.m_len = len = len - ret; > > epos = (loff_t)map.m_lblk << inode->i_blkbits; > >@@ -4599,8 +4601,6 @@ static long ext4_zero_range(struct file *file, loff_t offset, > > ret = ext4_mark_inode_dirty(handle, inode); > > if (unlikely(ret)) > > goto out_handle; > >- ext4_fc_track_range(handle, inode, offset >> inode->i_sb->s_blocksize_bits, > >- (offset + len - 1) >> inode->i_sb->s_blocksize_bits); > > /* Zero out partial block at the edges of the range */ > > ret = ext4_zero_partial_blocks(handle, inode, offset, len); > > if (ret >= 0) > >--
Powered by blists - more mailing lists