On Wed, 05/24 10:52, Fam Zheng wrote: > What's done in the source's context change notifier is moving the > target's context to follow the new one, so we request this permission > here. > > Signed-off-by: Fam Zheng <f...@redhat.com> > --- > block/backup.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/block/backup.c b/block/backup.c > index a4fb288..546c5c5 100644 > --- a/block/backup.c > +++ b/block/backup.c > @@ -636,7 +636,7 @@ BlockJob *backup_job_create(const char *job_id, > BlockDriverState *bs, > } > > /* The target must match the source in size, so no resize here either */ > - job->target = blk_new(BLK_PERM_WRITE, > + job->target = blk_new(BLK_PERM_WRITE | BLK_PERM_AIO_CONTEXT_CHANGE, > BLK_PERM_CONSISTENT_READ | BLK_PERM_WRITE | > BLK_PERM_WRITE_UNCHANGED | BLK_PERM_GRAPH_MOD); > ret = blk_insert_bs(job->target, target, errp);
Since fc0932fdc we reuse @bs as the backing of @target if sync=none, and this hunk causes problem: the device's BB probably doesn't permit BLK_PERM_AIO_CONTEXT_CHANGE. Still scratching my head on this. Fam
