Re: [PATCH 3/6] block/file-posix: Do not force-cap *pnum

[Max Reitz]

On 18.06.21 22:16, Eric Blake wrote:
> On Thu, Jun 17, 2021 at 05:52:44PM +0200, Max Reitz wrote:
> > bdrv_co_block_status() does it for us, we do not need to do it here.
> >
> > The advantage of not capping *pnum is that bdrv_co_block_status() can
> > cache larger data regions than requested by its caller.
> We should update the documentation in include/block/block_int.h to
> mention that the driver's block_status callback may treat *pnum as a
> soft cap, and that returning a larger value is fine.

Oh, sure.

Max

> But I agree with this change in the individual drivers, as long as we
> remember to make our global contract explicit that we can now rely on
> it ;)
>
> Reviewed-by: Eric Blake <eblake@redhat.com>