Re: [PATCH 2/9] cachefiles: drop direct usage of ->bmap method.

[Christoph Hellwig <hch@xxxxxx>]

On Thu, Aug 08, 2019 at 10:27:37AM +0200, Carlos Maiolino wrote:
> +	block = page->index;
> +	block <<= shift;

Can't this cause overflows?

> +
> +	ret = bmap(inode, &block);
> +	ASSERT(!ret);

I think we want some real error handling here instead of just an
assert..