78f6e472f9
- DDF-compare_super_ddf-fix-sequence-number-check.patch - Monitor-don-t-set-arrays-dirty-after-transition-to-r.patch - DDF-add_to_super_ddf-be-careful-with-workspace_lba.patch - Monitor-write-meta-data-in-readonly-state-sometimes.patch - Assembe-fix-bug-in-force_array-it-wasn-t-forcing-pro.patch Various bugfixes from upstream - several to improve DDF support, one which fixes a nasty bug in "--assemble --force". - DDF-brief_examine_subarrays_ddf-print-array-name.patch DDF-factor-out-array-name-generation.patch Include name in "-Db" output for DDF FATE#316007 OBS-URL: https://build.opensuse.org/package/show/Base:System/mdadm?expand=0&rev=96
43 lines
1.5 KiB
Diff
43 lines
1.5 KiB
Diff
From a4921f30ec2ecc2d685573899cb86064b0cd78e9 Mon Sep 17 00:00:00 2001
|
|
From: "mwilck@arcor.de" <mwilck@arcor.de>
|
|
Date: Tue, 24 Sep 2013 20:53:18 +0200
|
|
Subject: [PATCH] Monitor: write meta data in readonly state, sometimes
|
|
|
|
This patch reverts 24a216bf:
|
|
"Monitor: Don't write metadata in inactive array state".
|
|
|
|
While it's true that writing meta data is usually not necessary
|
|
in readonly state, there is one important exception: if a
|
|
disk goes faulty, we want to record that, even if the array is
|
|
inactive.
|
|
|
|
We might as well just revert 24a216bf, because with the recently
|
|
submitted patch
|
|
"Monitor: don't set arrays dirty after transition to read-only"
|
|
those meta data writes that really annoying (for a clean, readonly,
|
|
healthy array during startup) are gone anyway.
|
|
|
|
Signed-off-by: Martin Wilck <mwilck@arcor.de>
|
|
Signed-off-by: NeilBrown <neilb@suse.de>
|
|
---
|
|
monitor.c | 3 +--
|
|
1 file changed, 1 insertion(+), 2 deletions(-)
|
|
|
|
diff --git a/monitor.c b/monitor.c
|
|
index 3b57ed05ec9b..f81e707532f6 100644
|
|
--- a/monitor.c
|
|
+++ b/monitor.c
|
|
@@ -421,8 +421,7 @@ static int read_and_act(struct active_array *a)
|
|
if (sync_completed > a->last_checkpoint)
|
|
a->last_checkpoint = sync_completed;
|
|
|
|
- if (deactivate || a->curr_state >= clean)
|
|
- a->container->ss->sync_metadata(a->container);
|
|
+ a->container->ss->sync_metadata(a->container);
|
|
dprintf("%s(%d): state:%s action:%s next(", __func__, a->info.container_member,
|
|
array_states[a->curr_state], sync_actions[a->curr_action]);
|
|
|
|
--
|
|
1.8.3.1.487.g3e7a5b4
|
|
|