diff options
author | Mikolaj Golub <trociny@FreeBSD.org> | 2011-03-22 20:27:26 +0000 |
---|---|---|
committer | Mikolaj Golub <trociny@FreeBSD.org> | 2011-03-22 20:27:26 +0000 |
commit | 9237aa3fa5273c3debe2dcbd32e84d9e0ef703c5 (patch) | |
tree | 8376a860e9b6021ee72ce005783c54e926cdeebc /sbin | |
parent | 999d680c925b6e953b95fd5b9d8999b09e21aba5 (diff) | |
download | src-9237aa3fa5273c3debe2dcbd32e84d9e0ef703c5.tar.gz src-9237aa3fa5273c3debe2dcbd32e84d9e0ef703c5.zip |
After synchronization is complete we should make primary counters be
equal to secondary counters:
primary_localcnt = secondary_remotecnt
primary_remotecnt = secondary_localcnt
Previously it was done wrong and split-brain was observed after
primary had synchronized up-to-date data from secondary.
Approved by: pjd (mentor)
MFC after: 1 week
Notes
Notes:
svn path=/head/; revision=219882
Diffstat (limited to 'sbin')
-rw-r--r-- | sbin/hastd/primary.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/sbin/hastd/primary.c b/sbin/hastd/primary.c index b943077dfe34..966973bee45f 100644 --- a/sbin/hastd/primary.c +++ b/sbin/hastd/primary.c @@ -1757,13 +1757,13 @@ sync_thread(void *arg __unused) mtx_lock(&metadata_lock); res->hr_syncsrc = HAST_SYNCSRC_UNDEF; res->hr_primary_localcnt = - res->hr_secondary_localcnt; - res->hr_primary_remotecnt = res->hr_secondary_remotecnt; + res->hr_primary_remotecnt = + res->hr_secondary_localcnt; pjdlog_debug(1, "Setting localcnt to %ju and remotecnt to %ju.", (uintmax_t)res->hr_primary_localcnt, - (uintmax_t)res->hr_secondary_localcnt); + (uintmax_t)res->hr_primary_remotecnt); (void)metadata_write(res); mtx_unlock(&metadata_lock); } |