blockjob: Don't set iostatus of target
authorKevin Wolf <kwolf@redhat.com>
Mon, 18 Apr 2016 09:36:38 +0000 (11:36 +0200)
committerKevin Wolf <kwolf@redhat.com>
Thu, 19 May 2016 14:45:31 +0000 (16:45 +0200)
commit81e254dc8349795d17c585a997f9076b77a863fc
treec3f3b3668627b4600426f46850282f856072b940
parent4c265bf9f434d4b47f42e3c079adc205b7625ad6
blockjob: Don't set iostatus of target

When block job errors were introduced, we assigned the iostatus of the
target BDS "just in case". The field has never been accessible for the
user because the target isn't listed in query-block.

Before we can allow the user to have a second BlockBackend on the
target, we need to clean this up. If anything, we would want to set the
iostatus for the internal BB of the job (which we can always do later),
but certainly not for a separate BB which the job doesn't even use.

As a nice side effect, this gets us rid of another bs->blk use.

Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Reviewed-by: Max Reitz <mreitz@redhat.com>
block/backup.c
block/mirror.c
block/stream.c
blockjob.c
include/block/blockjob.h