[GEDI] [PATCH v6 12/13] blkio: implement BDRV_REQ_REGISTERED_BUF optimization

Stefan Hajnoczi stefanha at redhat.com
Thu Oct 13 18:42:18 UTC 2022


On Mon, Oct 10, 2022 at 02:57:47PM +0100, Alberto Faria wrote:
> On Thu, Oct 6, 2022 at 10:35 PM Stefan Hajnoczi <stefanha at redhat.com> wrote:
> > @@ -620,15 +767,39 @@ static int blkio_file_open(BlockDriverState *bs, QDict *options, int flags,
> >          return ret;
> >      }
> >
> > +    ret = blkio_get_bool(s->blkio,
> > +                         "mem-regions-pinned",
> 
> Should the property be named "may-pin-mem-regions" or similar? For
> drivers like vhost-user we may not be able to determine for sure
> whether memory regions will be pinned, and making that uncertainty
> explicit in the name may be a good idea, for instance to ensure users
> don't decide to rely on memory regions being pinned when the property
> is true.

Sounds good to me. I'll change the property name in this patch and then
send a libblkio patch that adds the property.

Stefan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://lists.gluster.org/pipermail/integration/attachments/20221013/8d58b623/attachment-0001.sig>


More information about the integration mailing list