qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH v6 12/15] blockdev: Keep track of monitor-owned


From: Alberto Garcia
Subject: Re: [Qemu-devel] [PATCH v6 12/15] blockdev: Keep track of monitor-owned BDS
Date: Mon, 09 Nov 2015 17:38:30 +0100
User-agent: Notmuch/0.13.2 (http://notmuchmail.org) Emacs/23.2.1 (i486-pc-linux-gnu)

On Mon 09 Nov 2015 05:26:38 PM CET, Kevin Wolf wrote:

>> >      if (blk) {
>> >          blk_unref(blk);
>> >      } else {
>> > +        QTAILQ_REMOVE(&monitor_bdrv_states, bs, monitor_list);
>> >          bdrv_unref(bs);
>> >      }
>> 
>> blk_unref(blk) will also unref the BDS (if there's any), so you also
>> need to update monitor_bdrv_states in that case, don't you?
>
> No, in that case the BDS referenced wasn't owned by the monitor in the
> first place. It was owned by the BB.

I see, I hadn't noticed that the BDS is added to monitor_bdrv_states
only if it is created without a BB.

You're right then, thanks!

Berto



reply via email to

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