OSDN Git Service

job: Convert block_job_cancel_async() to Job
authorKevin Wolf <kwolf@redhat.com>
Fri, 20 Apr 2018 12:56:08 +0000 (14:56 +0200)
committerKevin Wolf <kwolf@redhat.com>
Wed, 23 May 2018 12:30:50 +0000 (14:30 +0200)
commit004e95df98266da33e08c9f1731aca71b6d6d7c4
treefb978b083b796cca2c0fad011557841d6f330ede
parent4ad351819b974d724e926fd23cdd66bec3c9768e
job: Convert block_job_cancel_async() to Job

block_job_cancel_async() did two things that were still block job
specific:

* Setting job->force. This field makes sense on the Job level, so we can
  just move it. While at it, rename it to job->force_cancel to make its
  purpose more obvious.

* Resetting the I/O status. This can't be moved because generic Jobs
  don't have an I/O status. What the function really implements is a
  user resume, except without entering the coroutine. Consequently, it
  makes sense to call the .user_resume driver callback here which
  already resets the I/O status.

  The old block_job_cancel_async() has two separate if statements that
  check job->iostatus != BLOCK_DEVICE_IO_STATUS_OK and job->user_paused.
  However, the former condition always implies the latter (as is
  asserted in block_job_iostatus_reset()), so changing the explicit call
  of block_job_iostatus_reset() on the former condition with the
  .user_resume callback on the latter condition is equivalent and
  doesn't need to access any BlockJob specific state.

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