On Tue, May 19, 2015 at 03:24:33PM +0300, Alberto Garcia wrote: > 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 <be...@igalia.com> > --- > block.c | 16 ++++++++++++++++ > block/throttle-groups.c | 31 ++++++++++++++++++++++++++++++- > include/block/throttle-groups.h | 3 +++ > 3 files changed, 49 insertions(+), 1 deletion(-)
Reviewed-by: Stefan Hajnoczi <stefa...@redhat.com>
pgpDtDdyMeKXh.pgp
Description: PGP signature
