From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-m2458.qiye.163.com (mail-m2458.qiye.163.com [220.194.24.58]) by mail19.linbit.com (LINBIT Mail Daemon) with ESMTP id A4B324201D4 for ; Mon, 11 Apr 2022 14:10:28 +0200 (CEST) From: Rui Xu To: philipp.reisner@linbit.com, joel.colledge@linbit.com, drbd-dev@lists.linbit.com Date: Mon, 11 Apr 2022 20:02:04 +0800 Message-Id: <20220411120204.3683999-1-rui.xu@easystack.cn> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Cc: Rui Xu , dongsheng.yang@easystack.cn Subject: [Drbd-dev] [PATCH] drbd:clear NEW_CUR_UUID when uuid was actually created List-Id: "*Coordination* of development, patches, contributions -- *Questions* \(even to developers\) go to drbd-user, please." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , If there is an IO blocked by creating new uuid, but we found we dont have quorum, then NEW_CUR_UUID will be cleared but new uuid was not actually created. After one peer online, we have enough quorum to continue IO, then this IO will reach primary and the online peer. But if there is another peer join later, this peer will found same uuid, and will lose data. So don't clear NEW_CUR_UUID unless uuid was actually created. Signed-off-by: Rui Xu --- drbd/drbd_sender.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drbd/drbd_sender.c b/drbd/drbd_sender.c index 7238730..b0e3701 100644 --- a/drbd/drbd_sender.c +++ b/drbd/drbd_sender.c @@ -2360,15 +2360,17 @@ void drbd_check_peers_new_current_uuid(struct drbd_device *device) drbd_check_peers(resource); - if (device->have_quorum[NOW] && drbd_data_accessible(device, NOW)) + if (device->have_quorum[NOW] && drbd_data_accessible(device, NOW)) { drbd_uuid_new_current(device, false); + clear_bit(NEW_CUR_UUID, &device->flags); + } } static void make_new_current_uuid(struct drbd_device *device) { drbd_check_peers_new_current_uuid(device); - get_work_bits(1UL << NEW_CUR_UUID | 1UL << WRITING_NEW_CUR_UUID, &device->flags); + clear_bit(WRITING_NEW_CUR_UUID, &device->flags); wake_up(&device->misc_wait); } -- 1.8.3.1