Re: [PATCH v2 30/36] block: bdrv_reopen_multiple: refresh permissions on updated graph

2021-02-10 Thread Kevin Wolf
Am 08.02.2021 um 12:21 hat Vladimir Sementsov-Ogievskiy geschrieben: > > Come to think of it, the AioContext handling is probably wrong already > > before your series. reopen_commit for one node could move the whole tree > > to a different context and then the later nodes would all be processed >

Re: [PATCH v2 30/36] block: bdrv_reopen_multiple: refresh permissions on updated graph

2021-02-10 Thread Kevin Wolf
Am 08.02.2021 um 12:21 hat Vladimir Sementsov-Ogievskiy geschrieben: > 05.02.2021 20:57, Kevin Wolf wrote: > > Am 27.11.2020 um 15:45 hat Vladimir Sementsov-Ogievskiy geschrieben: > > > Move bdrv_reopen_multiple to new paradigm of permission update: > > > first update graph relations, then do

Re: [PATCH v2 30/36] block: bdrv_reopen_multiple: refresh permissions on updated graph

2021-02-08 Thread Vladimir Sementsov-Ogievskiy
05.02.2021 20:57, Kevin Wolf wrote: Am 27.11.2020 um 15:45 hat Vladimir Sementsov-Ogievskiy geschrieben: Move bdrv_reopen_multiple to new paradigm of permission update: first update graph relations, then do refresh the permissions. We have to modify reopen process in file-posix driver: with

Re: [PATCH v2 30/36] block: bdrv_reopen_multiple: refresh permissions on updated graph

2021-02-05 Thread Kevin Wolf
Am 27.11.2020 um 15:45 hat Vladimir Sementsov-Ogievskiy geschrieben: > Move bdrv_reopen_multiple to new paradigm of permission update: > first update graph relations, then do refresh the permissions. > > We have to modify reopen process in file-posix driver: with new scheme > we don't have

[PATCH v2 30/36] block: bdrv_reopen_multiple: refresh permissions on updated graph

2020-11-27 Thread Vladimir Sementsov-Ogievskiy
Move bdrv_reopen_multiple to new paradigm of permission update: first update graph relations, then do refresh the permissions. We have to modify reopen process in file-posix driver: with new scheme we don't have prepared permissions in raw_reopen_prepare(), so we should reconfigure fd in