On 12.11.2015 08:12, Fam Zheng wrote: > On Mon, 11/09 23:39, Max Reitz wrote: >> We need this list so that bdrv_close_all() can keep track of which BDSs >> are still open after having removed the BDSs from all of the BBs and >> having released all monitor BDS references. >> >> Signed-off-by: Max Reitz >> Reviewed-by: Kevin Wolf > > Reviewed-by: Fam Zheng > > Is graph_bdrv_states becoming a subset of this list, just with non-empty > node_name? Yes, that is correct. There shouldn't be any BDS that isn't in all_bdrv_states. Max