From: Haifeng Xu Date: Tue, 30 Jul 2024 04:20:08 +0000 (+0800) Subject: ovl: don't set the superblock's errseq_t manually X-Git-Tag: v6.12-rc1~142^2~3 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=34b4540e6646577d02afacfcbc08e563b7d69a9f;p=thirdparty%2Fkernel%2Flinux.git ovl: don't set the superblock's errseq_t manually Since commit 5679897eb104 ("vfs: make sync_filesystem return errors from ->sync_fs"), the return value from sync_fs callback can be seen in sync_filesystem(). Thus the errseq_set opreation can be removed here. Depends-on: commit 5679897eb104 ("vfs: make sync_filesystem return errors from ->sync_fs") Signed-off-by: Haifeng Xu Reviewed-by: Amir Goldstein Signed-off-by: Amir Goldstein --- diff --git a/fs/overlayfs/super.c b/fs/overlayfs/super.c index 06a231970cb57..fe511192f83cb 100644 --- a/fs/overlayfs/super.c +++ b/fs/overlayfs/super.c @@ -202,15 +202,9 @@ static int ovl_sync_fs(struct super_block *sb, int wait) int ret; ret = ovl_sync_status(ofs); - /* - * We have to always set the err, because the return value isn't - * checked in syncfs, and instead indirectly return an error via - * the sb's writeback errseq, which VFS inspects after this call. - */ - if (ret < 0) { - errseq_set(&sb->s_wb_err, -EIO); + + if (ret < 0) return -EIO; - } if (!ret) return ret;