qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v6 2/5] block/io: bdrv_common_block_status_above: support inc


From: Alberto Garcia
Subject: Re: [PATCH v6 2/5] block/io: bdrv_common_block_status_above: support include_base
Date: Wed, 23 Sep 2020 18:18:43 +0200
User-agent: Notmuch/0.18.2 (http://notmuchmail.org) Emacs/24.4.1 (i586-pc-linux-gnu)

On Wed 16 Sep 2020 02:20:05 PM CEST, Vladimir Sementsov-Ogievskiy wrote:
> -    for (p = backing_bs(bs); p != base; p = backing_bs(p)) {
> +    for (p = backing_bs(bs); include_base || p != base; p = backing_bs(p)) {
>          ret = bdrv_co_block_status(p, want_zero, offset, bytes, pnum, map,
>                                     file);
>          if (ret < 0) {
> @@ -2420,6 +2422,11 @@ bdrv_co_common_block_status_above(BlockDriverState *bs,
>              break;
>          }
>  
> +        if (p == base) {
> +            assert(include_base);
> +            break;
> +        }
> +

Another option is something like:

   BlockDriverState *last_bs = include_base ? base : backing_bs(base);

and you get a simpler 'for' loop.

But why do we need include_base at all? Can't the caller just pass
backing_bs(base) instead? I'm talking also about the existing case of
bdrv_is_allocated_above().

Berto



reply via email to

[Prev in Thread] Current Thread [Next in Thread]