[Top][All Lists]

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

Re: [PATCH 2/7] block: Manipulate children list in .attach/.detach

From: Vladimir Sementsov-Ogievskiy
Subject: Re: [PATCH 2/7] block: Manipulate children list in .attach/.detach
Date: Wed, 10 Nov 2021 15:51:49 +0300
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.1.0

04.11.2021 13:38, Hanna Reitz wrote:
The children list is specific to BDS parents.  We should not modify it
in the general children modification code, but let BDS parents deal with
it in their .attach() and .detach() methods.

This also has the advantage that a BdrvChild is removed from the
children list before its .bs pointer can become NULL.  BDS parents
generally assume that their children's .bs pointer is never NULL, so
this is actually a bug fix.

Signed-off-by: Hanna Reitz<hreitz@redhat.com>

Interesting that nor child_root neither child_job do similar things in .attach 
/ .detach ... Should we do something with it?

Best regards,

reply via email to

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