[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