[Devel] [PATCH v2] fuse: fuse_flush must check mapping->flags for errors

Miklos Szeredi miklos at szeredi.hu
Thu Jul 21 03:28:41 PDT 2016


On Wed, Jul 20, 2016 at 3:12 AM, Maxim Patlasov <mpatlasov at virtuozzo.com> wrote:
> fuse_flush() calls write_inode_now() that triggers writeback, but actual
> writeback will happen later, on fuse_sync_writes(). If an error happens,
> fuse_writepage_end() will set error bit in mapping->flags. So, we have to
> check mapping->flags after fuse_sync_writes().
>
> Changed in v2:
>  - fixed silly type: check must be *after* fuse_sync_writes()

I applied both with a cleanup patch at the end to use the
filemap_check_errors() helper, which does exactly this.

Thanks,
Miklos


More information about the Devel mailing list