block: Handle multiwrite errors only when all requests have completed
authorKevin Wolf <kwolf@redhat.com>
Thu, 1 Jul 2010 14:08:51 +0000 (16:08 +0200)
committerKevin Wolf <kwolf@redhat.com>
Fri, 2 Jul 2010 13:44:12 +0000 (15:44 +0200)
commit31525020cc16f2e750bfa4542f16284ffc4f0aed
treef03df0cfa7f24a156d289ce556cb76902a0a318a
parent20626c0b2b302312fc7099f0e62eba85597a3ec4
block: Handle multiwrite errors only when all requests have completed

Don't try to be clever by freeing all temporary data and calling all callbacks
when the return value (an error) is certain. Doing so has at least two
important problems:

* The temporary data that is freed (qiov, possibly zero buffer) is still used
  by the requests that have not yet completed.
* Calling the callbacks for all requests in the multiwrite means for the caller
  that it may free buffers etc. which are still in use.

Just remember the error value and do the cleanup when all requests have
completed.

Signed-off-by: Kevin Wolf <kwolf@redhat.com>
block.c