Re: [Qemu-devel] [PATCH] vmdk: Leave bdi intact if -ENOTSUP in vmdk_get_info

2014-11-16 Thread Fam Zheng
On Fri, 11/14 09:29, Stefan Hajnoczi wrote: > On Fri, Nov 14, 2014 at 12:09:21PM +0800, Fam Zheng wrote: > > When extent types don't match, we return -ENOTSUP. In this case, be > > polite to the caller and don't modify bdi. > > > > Signed-off-by: Fam Zheng > > --- > > block/vmdk.c | 20 +

Re: [Qemu-devel] [PATCH] vmdk: Leave bdi intact if -ENOTSUP in vmdk_get_info

2014-11-14 Thread Stefan Hajnoczi
On Fri, Nov 14, 2014 at 12:09:21PM +0800, Fam Zheng wrote: > When extent types don't match, we return -ENOTSUP. In this case, be > polite to the caller and don't modify bdi. > > Signed-off-by: Fam Zheng > --- > block/vmdk.c | 20 +--- > 1 file changed, 13 insertions(+), 7 deletio

Re: [Qemu-devel] [PATCH] vmdk: Leave bdi intact if -ENOTSUP in vmdk_get_info

2014-11-14 Thread Max Reitz
On 2014-11-14 at 05:09, Fam Zheng wrote: When extent types don't match, we return -ENOTSUP. In this case, be polite to the caller and don't modify bdi. Signed-off-by: Fam Zheng --- block/vmdk.c | 20 +--- 1 file changed, 13 insertions(+), 7 deletions(-) Reviewed-by: Max Rei

[Qemu-devel] [PATCH] vmdk: Leave bdi intact if -ENOTSUP in vmdk_get_info

2014-11-13 Thread Fam Zheng
When extent types don't match, we return -ENOTSUP. In this case, be polite to the caller and don't modify bdi. Signed-off-by: Fam Zheng --- block/vmdk.c | 20 +--- 1 file changed, 13 insertions(+), 7 deletions(-) diff --git a/block/vmdk.c b/block/vmdk.c index 673d3f5..2cbfd3e 10