Am 14.09.2016 um 17:52 hat Alberto Garcia geschrieben: > bdrv_reopen_queue_child() assumes that a BlockDriverState is never > added twice to BlockReopenQueue. > > That's however not the case: commit_start() adds 'base' (and its > children) to a new reopen queue, and then 'overlay_bs' (and its > children, which include 'base') to the same queue. The effect of this > is that the first set of options is ignored and overriden by the > second. > > We fixed this by swapping the order in which both BDSs were added to > the queue in 3db2bd5508c86a1605258bc77c9672d93b5c350e. This patch > checks if a BDS is already in the reopen queue and keeps its options. > > Signed-off-by: Alberto Garcia <be...@igalia.com>
Reviewed-by: Kevin Wolf <kw...@redhat.com>
