Re: [f2fs-dev] [PATCH v5 2/2] f2fs: fix missing inplace count in overwrite with direct io

2021-10-27 Thread 常凤楠
> -Original Message- > From: changfeng...@vivo.com On Behalf Of > Chao Yu > Sent: Wednesday, October 27, 2021 11:31 AM > To: fengnan chang > Cc: 常凤楠 ; Jaegeuk Kim ; > linux-f2fs-devel@lists.sourceforge.net > Subject: Re: [f2fs-dev] [PATCH v5 2/2] f2fs: fix missing inplace count in > ove

Re: [f2fs-dev] [PATCH] f2fs: remove circular locking between sb_internal and fs_reclaim

2021-10-27 Thread Daeho Jeong
On Tue, Oct 26, 2021 at 10:56 AM Daeho Jeong wrote: > > > Yes, I doubt the problem may also happen in f2fs_evict_inode() with below > > callpath: > > > > - evict_inode > > - dquot_initialize failed > > - sb_start_intwrite > > - f2fs_truncate > >- dquot_initialize lock dqio_sem > > > > Ho

Re: [f2fs-dev] [PATCH] f2fs: fix incorrect return value in f2fs_sanity_check_ckpt()

2021-10-27 Thread Jaegeuk Kim
Could you post the patch again? I don't see this in my box. On 10/27, Chao Yu wrote: > Jaegeuk, > > Missed to apply this patch? > > Thanks, > > On 2021/9/24 17:50, Pavel Machek wrote: > > Hi! > > > > > This code looks quite confused: part of function returns 1 on > > > corruption, part returns

[f2fs-dev] [PATCH 1/2] f2fs-tools: remove false failure alarm when fixing quota

2021-10-27 Thread Daeho Jeong
From: Daeho Jeong Even though fsck fixed quota issues, fsck returns failure status and makes the user code misunderstand like errors are still there. Signed-off-by: Daeho Jeong --- fsck/fsck.c | 6 -- include/f2fs_fs.h | 1 + 2 files changed, 5 insertions(+), 2 deletions(-) diff --g

[f2fs-dev] [PATCH 2/2] f2fs-tools: separate other bugs in fsck_verify

2021-10-27 Thread Daeho Jeong
From: Daeho Jeong When we check other corrupted bugs in fsck, we use bug_on. So, if major bugs are already detected in fsck_verify(), fsck will false alarm that there are other bugs, even if there are no other bugs. So, let's fix this. Signed-off-by: Daeho Jeong --- fsck/fsck.c | 35 ++

[f2fs-dev] Track unicode tree in linux-next (was Re: [PATCH 10/11] unicode: Add utf8-data module)

2021-10-27 Thread Gabriel Krisman Bertazi
Stephen Rothwell writes: > You just need to send me a git URL for your tree/branch (not a cgit or > gitweb URL, please), plus some idea of what the tree include and how it > is sent to Linus (directly or via another tree). The branch should > have a generic name (i.e. not including a version) as