diff options
author | Alberto Garcia <berto@igalia.com> | 2015-06-08 18:17:45 +0200 |
---|---|---|
committer | Stefan Hajnoczi <stefanha@redhat.com> | 2015-06-12 14:00:00 +0100 |
commit | db6283385cb708b9d589e5b57e96eab4afd0269e (patch) | |
tree | 4c50c7acecd3d79c6efb6beee4a359bfdb3322f1 /include | |
parent | 76f4afb40fa076ed23fe0ab42c7a768ddb71123f (diff) |
throttle: acquire the ThrottleGroup lock in bdrv_swap()
bdrv_swap() touches the fields of a BlockDriverState that are
protected by the ThrottleGroup lock. Although those fields end up in
their original place, they are temporarily swapped in the process,
so there's a chance that an operation on a member of the same group
happening on a different thread can try to use them.
Signed-off-by: Alberto Garcia <berto@igalia.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Message-id: d92dc40d7c4f1fc5cda5cbbf4ffb7a4670b79d17.1433779731.git.berto@igalia.com
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Diffstat (limited to 'include')
-rw-r--r-- | include/block/throttle-groups.h | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/include/block/throttle-groups.h b/include/block/throttle-groups.h index 322139a7a2..fab113f6d1 100644 --- a/include/block/throttle-groups.h +++ b/include/block/throttle-groups.h @@ -40,4 +40,7 @@ void coroutine_fn throttle_group_co_io_limits_intercept(BlockDriverState *bs, unsigned int bytes, bool is_write); +void throttle_group_lock(BlockDriverState *bs); +void throttle_group_unlock(BlockDriverState *bs); + #endif |