On 09.06.2016 10:20, Alberto Garcia wrote: > Block jobs are identified by the name of the BlockBackend of the BDS > where the job was started. > > We want block jobs to have unique, arbitrary identifiers that are not > tied to a block device, so this patch decouples the ID from the device > name in the BlockJob structure. > > The ID is generated automatically for the moment, in later patches > we'll allow the user to set it. > > Signed-off-by: Alberto Garcia <be...@igalia.com> > --- > blockjob.c | 15 +++++++++------ > include/block/blockjob.h | 12 ++++++++---- > include/qemu/id.h | 1 + > util/id.c | 1 + > 4 files changed, 19 insertions(+), 10 deletions(-)
I suppose this patch tries to "silently" (i.e. not visibly) introduce this new ID for now? If so, there is one instance of job->id left that should probably be changed to job->device (in block_job_complete()). Max
signature.asc
Description: OpenPGP digital signature
