[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 3/7] block: require job-id when device is a n
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH v3 3/7] block: require job-id when device is a node name |
Date: |
Thu, 7 Sep 2017 14:24:53 +0200 |
User-agent: |
Mutt/1.8.3 (2017-05-23) |
Am 25.08.2017 um 15:23 hat Manos Pitsidianakis geschrieben:
> With implicit filter nodes on the top of the graph it is not possible to
> generate job-ids with the name of the device in block_job_create()
> anymore, since the job's bs will not be a child_root.
>
> Instead we can require that job-id is not NULL in block_job_create(),
> and check that a job-id has been set in the callers of
> block_job_create() in blockdev.c. It is more consistent to require an
> explicit job-id when the device parameter in the job creation command,
> eg
>
> { "execute": "drive-backup",
> "arguments": { "device": "drive0",
> "sync": "full",
> "target": "backup.img" } }
>
> is not a BlockBackend name, instead of automatically getting it from the
> root BS if device is a node name. That information is lost after calling
> block_job_create(), so we can do it in its caller instead.
>
> Signed-off-by: Manos Pitsidianakis <address@hidden>
Reviewed-by: Kevin Wolf <address@hidden>
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH v3 3/7] block: require job-id when device is a node name,
Kevin Wolf <=