Re: [PATCH v2 32/33] block: Pass BdrvChildRole in remaining cases

2020-02-18 Thread Eric Blake
On 2/18/20 6:01 AM, Max Reitz wrote: Is it worth an assert(role) somewhere now that you've converted all callers to pass at least one role? Well, as the commit message states, block_job_add_bdrv() in blockjob.c still passes BdrvChildRole=0 to bdrv_root_attach_child(). So it depends on what f

Re: [PATCH v2 32/33] block: Pass BdrvChildRole in remaining cases

2020-02-18 Thread Max Reitz
On 11.02.20 16:53, Eric Blake wrote: > On 2/4/20 11:08 AM, Max Reitz wrote: >> These calls have no real use for the child role yet, but it will not >> harm to give one. >> >> Notably, the bdrv_root_attach_child() call in blockjob.c is left >> unmodified because there is not much the generic BlockJo

Re: [PATCH v2 32/33] block: Pass BdrvChildRole in remaining cases

2020-02-11 Thread Eric Blake
On 2/4/20 11:08 AM, Max Reitz wrote: These calls have no real use for the child role yet, but it will not harm to give one. Notably, the bdrv_root_attach_child() call in blockjob.c is left unmodified because there is not much the generic BlockJob object wants from its children. Signed-off-by: M

[PATCH v2 32/33] block: Pass BdrvChildRole in remaining cases

2020-02-04 Thread Max Reitz
These calls have no real use for the child role yet, but it will not harm to give one. Notably, the bdrv_root_attach_child() call in blockjob.c is left unmodified because there is not much the generic BlockJob object wants from its children. Signed-off-by: Max Reitz --- block/block-backend.c |