Re: [Qemu-block] [Qemu-devel] [PULL 05/33] virtio-blk: fix comment for virtio_blk_rw_complete
On Tue, Nov 06, 2018 at 11:17:03AM +0800, Dongli Zhang wrote: > > > On 11/06/2018 02:15 AM, Michael S. Tsirkin wrote: > > From: Yaowei Bai > > > > Here should be submit_requests, there is no submit_merged_requests > > function. > > > > Signed-off-by: Yaowei Bai > > Reviewed-by: Michael S. Tsirkin > > Signed-off-by: Michael S. Tsirkin > > --- > > hw/block/virtio-blk.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/hw/block/virtio-blk.c b/hw/block/virtio-blk.c > > index 225fe44b7a..83cf5c01f9 100644 > > --- a/hw/block/virtio-blk.c > > +++ b/hw/block/virtio-blk.c > > @@ -97,8 +97,8 @@ static void virtio_blk_rw_complete(void *opaque, int ret) > > > > if (req->qiov.nalloc != -1) { > > /* If nalloc is != 1 req->qiov is a local copy of the original > > Should it be "If nalloc is != -1" in the comment? Seems the initial state is > -1. Makes sense. Patch? > > - * external iovec. It was allocated in submit_merged_requests > > - * to be able to merge requests. */ > > + * external iovec. It was allocated in submit_requests to be > > + * able to merge requests. */ > > qemu_iovec_destroy(>qiov); > > } > > > > > > Dongli Zhang
Re: [Qemu-block] [Qemu-devel] [PULL 05/33] virtio-blk: fix comment for virtio_blk_rw_complete
On 11/06/2018 02:15 AM, Michael S. Tsirkin wrote: > From: Yaowei Bai > > Here should be submit_requests, there is no submit_merged_requests > function. > > Signed-off-by: Yaowei Bai > Reviewed-by: Michael S. Tsirkin > Signed-off-by: Michael S. Tsirkin > --- > hw/block/virtio-blk.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/hw/block/virtio-blk.c b/hw/block/virtio-blk.c > index 225fe44b7a..83cf5c01f9 100644 > --- a/hw/block/virtio-blk.c > +++ b/hw/block/virtio-blk.c > @@ -97,8 +97,8 @@ static void virtio_blk_rw_complete(void *opaque, int ret) > > if (req->qiov.nalloc != -1) { > /* If nalloc is != 1 req->qiov is a local copy of the original Should it be "If nalloc is != -1" in the comment? Seems the initial state is -1. > - * external iovec. It was allocated in submit_merged_requests > - * to be able to merge requests. */ > + * external iovec. It was allocated in submit_requests to be > + * able to merge requests. */ > qemu_iovec_destroy(>qiov); > } > > Dongli Zhang