12b913a4e3
- Update mdadm-4.3 to latest status (jsc#PED-7542)
- Remove hardcoded checkpoint interval checking
0001-Remove-hardcoded-checkpoint-interval-checking.patch
- monitor: refactor checkpoint update
0002-monitor-refactor-checkpoint-update.patch
- Super-intel: Fix first checkpoint restart
0003-Super-intel-Fix-first-checkpoint-restart.patch
- Grow: Move update_tail assign to Grow_reshape()
0004-Grow-Move-update_tail-assign-to-Grow_reshape.patch
- Add understanding output section in man
0005-Add-understanding-output-section-in-man.patch
- Upgrade to mdadm-4.3 (jsc#PED-7542). Beside previous already back
ported patches, mdadm-4.3 has the following extra changes since
last update upto commit 582945c2d3bb,
- Fix null pointer for incremental in mdadm.
- Super1: fix truncation check for journal device.
- Fix some cases eyesore formatting.
- Bump minimum kernel version to 2.6.32.
- Remove the config files in mdcheck_start|continue service.
- Define DEV_MD_DIR, DEV_NUM_PREF, is_devname_ignore(),
ident_set_devname().
- Enable RAID for SATA under VMD.
- Imsm: Fix possible segfault in check_no_platform()
- Imsm refactor on imsm_get_free_size(), merge_extents().
- Imsm: return free space after volume for expand.
- Imsm: fix free space calculations.
- Add secure gethostname() wrapper.
- mdadm: Stop mdcheck_continue timer when mdcheck_start service can
finish check.
OBS-URL: https://build.opensuse.org/request/show/1149803
OBS-URL: https://build.opensuse.org/package/show/Base:System/mdadm?expand=0&rev=233
69 lines
2.6 KiB
Diff
69 lines
2.6 KiB
Diff
From aec3b907de48be54106600a1ecb69d1231f4801d Mon Sep 17 00:00:00 2001
|
|
From: Mateusz Kusiak <mateusz.kusiak@intel.com>
|
|
Date: Thu, 18 Jan 2024 11:30:15 +0100
|
|
Subject: [PATCH 1/5] Remove hardcoded checkpoint interval checking
|
|
Git-commit: aec3b907de48be54106600a1ecb69d1231f4801d
|
|
Patch-mainline: mdadm-4.3+
|
|
References: jsc#PED-7542
|
|
|
|
Mdmon assumes that kernel marks checkpoint every 1/16 of the volume size
|
|
and that the checkpoints are equal in size. This is not true, kernel may
|
|
mark checkpoints more frequently depending on several factors, including
|
|
sync speed. This results in checkpoints reported by mdadm --examine
|
|
falling behind the one reported by kernel.
|
|
|
|
Remove hardcoded checkpoint interval checking.
|
|
|
|
Signed-off-by: Mateusz Kusiak <mateusz.kusiak@intel.com>
|
|
Signed-off-by: Mariusz Tkaczyk <mariusz.tkaczyk@linux.intel.com>
|
|
Signed-off-by: Coly Li <colyli@suse.de>
|
|
---
|
|
monitor.c | 22 ++++++----------------
|
|
1 file changed, 6 insertions(+), 16 deletions(-)
|
|
|
|
diff --git a/monitor.c b/monitor.c
|
|
index 4acec67..b8d9e88 100644
|
|
--- a/monitor.c
|
|
+++ b/monitor.c
|
|
@@ -564,22 +564,10 @@ static int read_and_act(struct active_array *a, fd_set *fds)
|
|
}
|
|
}
|
|
|
|
- /* Check for recovery checkpoint notifications. We need to be a
|
|
- * minimum distance away from the last checkpoint to prevent
|
|
- * over checkpointing. Note reshape checkpointing is handled
|
|
- * in the second branch.
|
|
+ /* Handle reshape checkpointing
|
|
*/
|
|
- if (sync_completed > a->last_checkpoint &&
|
|
- sync_completed - a->last_checkpoint > a->info.component_size >> 4 &&
|
|
- a->curr_action > reshape) {
|
|
- /* A (non-reshape) sync_action has reached a checkpoint.
|
|
- * Record the updated position in the metadata
|
|
- */
|
|
- a->last_checkpoint = sync_completed;
|
|
- a->container->ss->set_array_state(a, a->curr_state <= clean);
|
|
- } else if ((a->curr_action == idle && a->prev_action == reshape) ||
|
|
- (a->curr_action == reshape &&
|
|
- sync_completed > a->last_checkpoint)) {
|
|
+ if ((a->curr_action == idle && a->prev_action == reshape) ||
|
|
+ (a->curr_action == reshape && sync_completed > a->last_checkpoint)) {
|
|
/* Reshape has progressed or completed so we need to
|
|
* update the array state - and possibly the array size
|
|
*/
|
|
@@ -607,8 +595,10 @@ static int read_and_act(struct active_array *a, fd_set *fds)
|
|
a->last_checkpoint = sync_completed;
|
|
}
|
|
|
|
- if (sync_completed > a->last_checkpoint)
|
|
+ if (sync_completed > a->last_checkpoint) {
|
|
a->last_checkpoint = sync_completed;
|
|
+ a->container->ss->set_array_state(a, a->curr_state <= clean);
|
|
+ }
|
|
|
|
if (sync_completed >= a->info.component_size)
|
|
a->last_checkpoint = 0;
|
|
--
|
|
2.35.3
|
|
|