On Mon, 09/18 14:11, Kevin Wolf wrote: > But how does it determine the desired new perms? Either you duplicate > the logic of the .bdrv_child_perm implementation into a new set of > functions that does the same thing, but based on the new state; or you > extend the existing function with a BlockReopenQueue parameter. The > latter is basically this series, except with an additional unnecessary > detour through the driver code instead of doing it in common code. > > Also note that storing it in BDRVReopenState would have to involve a new > list of an additional data structure because permissions are per > BdrvChild, not per BlockDriverState.
Indeed, this is not going to remove any complexity. :( Fam