On Fri, Sep 14, 2018 at 03:23:53PM +0200, Christoph Hellwig wrote: > On Wed, Sep 12, 2018 at 02:25:35PM +0200, Carlos Maiolino wrote: > > Replace the direct usage of ->bmap method by a bmap() call. > > Would be nice to have some real error handling instead of the assert > on an error return, but otherwise this looks fine: > Thanks. I'll see if I can do something about it and write a patch for it. /me doesn't know much about cachefiles yet. Thanks for the review. Cheers > Reviewed-by: Christoph Hellwig <hch@xxxxxx> -- Carlos