Re: [PATCH 02/14] block: use return status of bdrv_append()

2020-09-17 Thread Vladimir Sementsov-Ogievskiy
10.09.2020 19:10, Greg Kurz wrote: On Wed, 9 Sep 2020 21:59:18 +0300 Vladimir Sementsov-Ogievskiy wrote: Now bdrv_append returns status and we can drop all the local_err things around it. Signed-off-by: Vladimir Sementsov-Ogievskiy --- Reviewed-by: Greg Kurz Just one suggestion for a fo

Re: [PATCH 02/14] block: use return status of bdrv_append()

2020-09-17 Thread Alberto Garcia
On Wed 09 Sep 2020 08:59:18 PM CEST, Vladimir Sementsov-Ogievskiy wrote: > Now bdrv_append returns status and we can drop all the local_err things > around it. > > Signed-off-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Alberto Garcia Berto

Re: [PATCH 02/14] block: use return status of bdrv_append()

2020-09-10 Thread Greg Kurz
On Wed, 9 Sep 2020 21:59:18 +0300 Vladimir Sementsov-Ogievskiy wrote: > Now bdrv_append returns status and we can drop all the local_err things > around it. > > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- Reviewed-by: Greg Kurz Just one suggestion for a follow-up below... > block.c

[PATCH 02/14] block: use return status of bdrv_append()

2020-09-09 Thread Vladimir Sementsov-Ogievskiy
Now bdrv_append returns status and we can drop all the local_err things around it. Signed-off-by: Vladimir Sementsov-Ogievskiy --- block.c | 5 + block/backup-top.c | 20 block/commit.c | 5 + block/mirror.c |