Searched hist:"3 bb0e2980a96db6276e5032dcb2ccbf41f699b59" (Results 1 – 2 of 2) sorted by relevance
/openbmc/qemu/tests/unit/ |
H A D | test-bdrv-graph-mod.c | diff 3bb0e2980a96db6276e5032dcb2ccbf41f699b59 Wed Apr 28 10:17:45 CDT 2021 Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> block: fix bdrv_replace_node_common
inore_children thing doesn't help to track all propagated permissions of children we want to ignore. The simplest way to correctly update permissions is update graph first and then do permission update. In this case we just referesh permissions for the whole subgraph (in topological-sort defined order) and everything is correctly calculated automatically without any ignore_children.
So, refactor bdrv_replace_node_common to first do graph update and then refresh the permissions.
Test test_parallel_exclusive_write() now pass, so move it out of debugging "if".
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> Reviewed-by: Kevin Wolf <kwolf@redhat.com> Message-Id: <20210428151804.439460-18-vsementsov@virtuozzo.com> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
|
/openbmc/qemu/ |
H A D | block.c | diff 3bb0e2980a96db6276e5032dcb2ccbf41f699b59 Wed Apr 28 10:17:45 CDT 2021 Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> block: fix bdrv_replace_node_common
inore_children thing doesn't help to track all propagated permissions of children we want to ignore. The simplest way to correctly update permissions is update graph first and then do permission update. In this case we just referesh permissions for the whole subgraph (in topological-sort defined order) and everything is correctly calculated automatically without any ignore_children.
So, refactor bdrv_replace_node_common to first do graph update and then refresh the permissions.
Test test_parallel_exclusive_write() now pass, so move it out of debugging "if".
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> Reviewed-by: Kevin Wolf <kwolf@redhat.com> Message-Id: <20210428151804.439460-18-vsementsov@virtuozzo.com> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
|