Am 17/08/2023 um 14:50 schrieb Kevin Wolf: > The function reads the parents list, so it needs to hold the graph lock. > > This happens to result in BlockDriver.bdrv_set_perm() to be called with > the graph lock held. For consistency, make it the same for all of the > BlockDriver callbacks for updating permissions and annotate the function > pointers with GRAPH_RDLOCK_PTR. > > Signed-off-by: Kevin Wolf <kw...@redhat.com> Reviewed-by: Emanuele Giuseppe Esposito <eespo...@redhat.com>
- [PATCH 00/21] Graph locking part 4 (node manage... Kevin Wolf
- [PATCH 20/21] block: Mark bdrv_unref_child... Kevin Wolf
- Re: [PATCH 20/21] block: Mark bdrv_unr... Emanuele Giuseppe Esposito
- Re: [PATCH 20/21] block: Mark bdrv_unr... Stefan Hajnoczi
- [PATCH 03/21] preallocate: Don't poll duri... Kevin Wolf
- Re: [PATCH 03/21] preallocate: Don't p... Eric Blake
- Re: [PATCH 03/21] preallocate: Don't p... Stefan Hajnoczi
- [PATCH 10/21] block: Mark bdrv_attach_chil... Kevin Wolf
- Re: [PATCH 10/21] block: Mark bdrv_att... Stefan Hajnoczi
- [PATCH 14/21] block: Mark bdrv_get_cumulat... Kevin Wolf
- Re: [PATCH 14/21] block: Mark bdrv_get... Emanuele Giuseppe Esposito
- Re: [PATCH 14/21] block: Mark bdrv_get... Stefan Hajnoczi
- [PATCH 15/21] block: Mark bdrv_child_perm(... Kevin Wolf
- Re: [PATCH 15/21] block: Mark bdrv_chi... Emanuele Giuseppe Esposito
- Re: [PATCH 15/21] block: Mark bdrv_chi... Stefan Hajnoczi
- [PATCH 08/21] block: Mark bdrv_replace_chi... Kevin Wolf
- Re: [PATCH 08/21] block: Mark bdrv_rep... Emanuele Giuseppe Esposito
- Re: [PATCH 08/21] block: Mark bdrv_rep... Stefan Hajnoczi
- [PATCH 05/21] block: Introduce bdrv_schedu... Kevin Wolf
- Re: [PATCH 05/21] block: Introduce bdr... Eric Blake
- Re: [PATCH 05/21] block: Introduce... Eric Blake