diff options
| author | Paolo Bonzini | 2016-10-27 12:48:49 +0200 |
|---|---|---|
| committer | Fam Zheng | 2016-10-28 15:50:18 +0200 |
| commit | 50ab0e0908d592b8bda56c2d7495e1190d734b0b (patch) | |
| tree | ff0a7cbe4910852f1a7a709e2cdfeda3a20ed49c /include | |
| parent | Merge remote-tracking branch 'remotes/kevin/tags/for-upstream' into staging (diff) | |
| download | qemu-50ab0e0908d592b8bda56c2d7495e1190d734b0b.tar.gz qemu-50ab0e0908d592b8bda56c2d7495e1190d734b0b.tar.xz qemu-50ab0e0908d592b8bda56c2d7495e1190d734b0b.zip | |
replication: interrupt failover if the main device is closed
Without this change, there is a race condition in tests/test-replication.
Depending on how fast the failover job (active commit) runs, there is a
chance of two bad things happening:
1) replication_done can be called after the secondary has been closed
and hence when the BDRVReplicationState is not valid anymore.
2) two copies of the active disk are present during the
/replication/secondary/stop test (that test runs immediately after
/replication/secondary/start, which tests failover). This causes the
corruption detector to fire.
Reviewed-by: Wen Congyang <wency@cn.fujitsu.com>
Reviewed-by: Changlong Xie <xiecl.fnst@cn.fujitsu.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Fam Zheng <famz@redhat.com>
Message-Id: <1477565348-5458-2-git-send-email-pbonzini@redhat.com>
Signed-off-by: Fam Zheng <famz@redhat.com>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions