Re: [Qemu-devel] [PATCH] virtio-blk: dataplane cleanup

2016-07-18 Thread Stefan Hajnoczi
On Mon, Jul 18, 2016 at 12:05:49PM +0800, Cao jin wrote: > No need duplicate the judgment, there is one in function entry. > > Cc: Stefan Hajnoczi > Cc: Kevin Wolf > Cc: Max Reitz > Signed-off-by: Cao jin > --- > hw/block/dataplane/virtio-blk.c | 6 ++ > 1 file changed, 2 insertions(+), 4

Re: [Qemu-devel] [PATCH] virtio-blk: dataplane cleanup

2016-07-17 Thread Fam Zheng
On Mon, 07/18 12:05, Cao jin wrote: > No need duplicate the judgment, there is one in function entry. > > Cc: Stefan Hajnoczi > Cc: Kevin Wolf > Cc: Max Reitz > Signed-off-by: Cao jin > --- > hw/block/dataplane/virtio-blk.c | 6 ++ > 1 file changed, 2 insertions(+), 4 deletions(-) > > di

[Qemu-devel] [PATCH] virtio-blk: dataplane cleanup

2016-07-17 Thread Cao jin
No need duplicate the judgment, there is one in function entry. Cc: Stefan Hajnoczi Cc: Kevin Wolf Cc: Max Reitz Signed-off-by: Cao jin --- hw/block/dataplane/virtio-blk.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/hw/block/dataplane/virtio-blk.c b/hw/block/data