On 09/09, Greg KH wrote: > On Wed, Sep 08, 2021 at 10:28:37PM -0700, Jaegeuk Kim wrote: > > On 09/09, Greg KH wrote: > > > On Wed, Sep 08, 2021 at 03:00:20PM -0700, Jaegeuk Kim wrote: > > > > From: Jaegeuk Kim <jaegeuk@xxxxxxxxxx> > > > > > > > > commit dddd3d65293a52c2c3850c19b1e5115712e534d8 upstream. > > > > > > > > We must flush all the dirty data when enabling checkpoint back. Let's guarantee > > > > that first by adding a retry logic on sync_inodes_sb(). In addition to that, > > > > this patch adds to flush data in fsync when checkpoint is disabled, which can > > > > mitigate the sync_inodes_sb() failures in advance. > > > > > > > > Reviewed-by: Chao Yu <chao@xxxxxxxxxx> > > > > Signed-off-by: Jaegeuk Kim <jaegeuk@xxxxxxxxxx> > > > > --- > > > > fs/f2fs/file.c | 5 ++--- > > > > fs/f2fs/super.c | 11 ++++++++++- > > > > 2 files changed, 12 insertions(+), 4 deletions(-) > > > > > > What stable kernel(s) are you wanting to have this backported to? > > > > 5.10 please. > > Why would you want to skip 5.14.y and 5.13.y? You never want anyone to > upgrade stable kernel releases and have a regression. I was just looking at the essential kernel version, since the fix is only related to checkpoint=disable feature used in android only. Feel free to merge it into any stable kernels if you want. > > thanks, > > greg k-h
