Re: [Qemu-devel] [PATCH 1/3] vhost: simple clean up

2017-04-07 Thread Stefan Hajnoczi
On Fri, Apr 07, 2017 at 02:58:46PM +0800, weiqi wrote: > From: Wei Qi > > It is unnecessary to assign 'hdev->nvqs' to 'i', because 'hdev->nvqs' unused > after assignment, so remove it. > > Signed-off-by: Wei Qi > --- > hw/virtio/vhost.c | 1 - > 1 file changed, 1 deletion(-) Reviewed-by: Ste

Re: [Qemu-devel] [PATCH 1/3] vhost: simple clean up

2017-04-07 Thread Eric Blake
On 04/07/2017 03:38 AM, Sahid Orentino Ferdjaoui wrote: > On Fri, Apr 07, 2017 at 02:58:46PM +0800, weiqi wrote: >> From: Wei Qi >> >> It is unnecessary to assign 'hdev->nvqs' to 'i', because 'hdev->nvqs' unused >> after assignment, so remove it. > > nit: The line should not be longer than 76 ch

Re: [Qemu-devel] [PATCH 1/3] vhost: simple clean up

2017-04-07 Thread Sahid Orentino Ferdjaoui
On Fri, Apr 07, 2017 at 02:58:46PM +0800, weiqi wrote: > From: Wei Qi > > It is unnecessary to assign 'hdev->nvqs' to 'i', because 'hdev->nvqs' unused > after assignment, so remove it. nit: The line should not be longer than 76 characters [0]. [0] http://wiki.qemu-project.org/Contribute/Submi

[Qemu-devel] [PATCH 1/3] vhost: simple clean up

2017-04-07 Thread weiqi
From: Wei Qi It is unnecessary to assign 'hdev->nvqs' to 'i', because 'hdev->nvqs' unused after assignment, so remove it. Signed-off-by: Wei Qi --- hw/virtio/vhost.c | 1 - 1 file changed, 1 deletion(-) diff --git a/hw/virtio/vhost.c b/hw/virtio/vhost.c index 613494d..5c57983 100644 --- a/hw