[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 4/4] contrib/vhost-user-blk: introduce a vhos
From: |
Liu, Changpeng |
Subject: |
Re: [Qemu-devel] [PATCH v4 4/4] contrib/vhost-user-blk: introduce a vhost-user-blk sample application |
Date: |
Fri, 20 Oct 2017 01:39:38 +0000 |
> -----Original Message-----
> From: Paolo Bonzini [mailto:address@hidden
> Sent: Thursday, October 19, 2017 7:44 PM
> To: Liu, Changpeng <address@hidden>; address@hidden
> Cc: address@hidden; address@hidden; address@hidden;
> address@hidden; Harris, James R <address@hidden>
> Subject: Re: [PATCH v4 4/4] contrib/vhost-user-blk: introduce a vhost-user-blk
> sample application
>
> On 19/10/2017 07:24, Changpeng Liu wrote:
> >
> > +}
> > +
> > +static void
> > +vub_flush(VubReq *req)
> > +{
> > + VubDev *vdev_blk = req->vdev_blk;
> > +
> > + if (vdev_blk->blk_fd) {
> > + fsync(vdev_blk->blk_fd);
> > + }
> > +}
> > +
>
> No need to check the file descriptor---vub_readv and vub_writev aren't
> checking it either. Also please use fdatasync instead of fsync.
Ok.
>
> > +static uint64_t
> > +vub_get_features(VuDev *dev)
> > +{
> > + return 1ull << VIRTIO_BLK_F_SIZE_MAX |
> > + 1ull << VIRTIO_BLK_F_SEG_MAX |
> > + 1ull << VIRTIO_BLK_F_TOPOLOGY |
> > + 1ull << VIRTIO_BLK_F_BLK_SIZE |
> > + 1ull << VIRTIO_F_VERSION_1 |
> > + 1ull << VHOST_USER_F_PROTOCOL_FEATURES;
> > +}
>
> VIRTIO_BLK_F_FLUSH is missing.
Yes, will add.
>
> Thanks,
>
> Paolo
- Re: [Qemu-devel] [PATCH v4 2/4] vhost-user-blk: introduce a new vhost-user-blk host device, (continued)
- Re: [Qemu-devel] [PATCH v4 2/4] vhost-user-blk: introduce a new vhost-user-blk host device, Stefan Hajnoczi, 2017/10/23
- Re: [Qemu-devel] [PATCH v4 2/4] vhost-user-blk: introduce a new vhost-user-blk host device, Liu, Changpeng, 2017/10/23
- Re: [Qemu-devel] [PATCH v4 2/4] vhost-user-blk: introduce a new vhost-user-blk host device, Stefan Hajnoczi, 2017/10/24
- Re: [Qemu-devel] [PATCH v4 2/4] vhost-user-blk: introduce a new vhost-user-blk host device, Liu, Changpeng, 2017/10/24
[Qemu-devel] [PATCH v4 3/4] contrib/libvhost-user: enable virtio config space messages, Changpeng Liu, 2017/10/19
[Qemu-devel] [PATCH v4 4/4] contrib/vhost-user-blk: introduce a vhost-user-blk sample application, Changpeng Liu, 2017/10/19