On Mon, Apr 18, 2022 at 01:45:06PM +0200, Greg KH wrote:
> On Fri, Apr 15, 2022 at 02:10:46PM +0200, Holger Hoffstätte wrote:
> >
> > I noticed that our autobot recently backported parts of a series which
> > fixed XFS syncfs error handling [1]. Unfortunately - due to missing
> > requirements - it only managed to merge those patches which do not
> > actually fix anything.
> >
> > This can be repaired by applying the prerequisites and then the missing
> > parts of the original series, namely in order:
> >
> > 9a208ba5c9af fs: remove __sync_filesystem
> > 70164eb6ccb7 block: remove __sync_blockdev
> > 1e03a36bdff4 block: simplify the block device syncing code
> > 5679897eb104 vfs: make sync_filesystem return errors from ->sync_fs
> > 2d86293c7075 xfs: return errors in xfs_fs_sync_fs
> >
> > With all that we could also put a cherry on top and merge:
> >
> > b97cca3ba909 xfs: only bother with sync_filesystem during readonly remount
> >
> > but that's just a touchup and not a real bugfix, so probably optional.
>
> Can we get an ack from the XFS developers that this is ok to do?
> Without that, we can't apply xfs patches to the stable trees.
You might consider adding these two other patches:
2719c7160dcf ("vfs: make freeze_super abort when sync_filesystem returns error")
dd5532a4994b ("quota: make dquot_quota_sync return errors from ->sync_fs")
to the pile, but otherwise that looks ok to me.
--D
> thanks,
>
> greg k-h
