Attachment 587768 Details for Bug 819943 – Patch to add MAXCONCURRENT to raid-check

[patch] Patch to add MAXCONCURRENT to raid-check

raid-check-maxconcurrent.patch (text/plain), 2.14 KB, created by Orion Poplawski on 2012-05-30 15:19:26 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Orion Poplawski

Created: 2012-05-30 15:19:26 UTC

Size: 2.14 KB

patch

obsolete

>diff --git a/mdadm-raid-check-sysconfig b/mdadm-raid-check-sysconfig
>index ecb4005..365278e 100644
>--- a/mdadm-raid-check-sysconfig
>+++ b/mdadm-raid-check-sysconfig
>@@ -34,6 +34,8 @@
> #       NICE - Change the raid check CPU and IO priority in order to make
> #		the system more responsive during lengthy checks.  Valid
> #		values are high, normal, low, idle.
>+#	MAXCONCURENT - Limit the number of devices to be checked at a time.
>+#		By default all devices will be checked at the same time.
> #
> # Note: the raid-check script is run by the /etc/cron.d/raid-check cron job.
> # Users may modify the frequency and timing at which raid-check is run by
>@@ -51,3 +53,4 @@ NICE=low
> CHECK_DEVS=""
> REPAIR_DEVS=""
> SKIP_DEVS=""
>+MAXCONCURRENT=
>diff --git a/raid-check b/raid-check
>index 829dc58..7b2fb59 100644
>--- a/raid-check
>+++ b/raid-check
>@@ -11,6 +11,25 @@
> [ -f /etc/sysconfig/raid-check ] || exit 0
> . /etc/sysconfig/raid-check
> 
>+# Wait until no more than arg1 arrays in arg2 list are busy
>+waitbusy() {
>+    local threshold=$(($1 + 1))
>+    local dev_list="$2"
>+    while true
>+    do
>+	local busy=0
>+	local dev=""
>+	for dev in $dev_list; do
>+	    local sync_action=`cat /sys/block/$dev/md/sync_action`
>+	    if [ "$sync_action" != "idle" ]; then
>+		let busy++
>+	    fi
>+	done
>+        [ $busy -lt $threshold ] && break
>+	sleep 60
>+    done
>+}
>+
> [ "$ENABLED" != "yes" ] && exit 0
> 
> case "$CHECK" in
>@@ -70,6 +89,10 @@ done
> [ -z "$dev_list" ] && exit 0
> 
> for dev in $dev_list; do
>+    #Only run $MAXCONCURRENT checks at a time
>+    if [ -n "$MAXCONCURRENT" ]; then
>+	waitbusy $((MAXCONCURRENT - 1)) "$dev_list"
>+    fi
>     echo "${check[$dev]}" > /sys/block/$dev/md/sync_action
> 
>     resync_pid=""
>@@ -86,18 +109,8 @@ for dev in $dev_list; do
> done
> [ -z "$check_list" ] && exit 0
> 
>-checking=1
>-while [ $checking -ne 0 ]
>-do
>-	sleep 60
>-	checking=0
>-	for dev in $check_list; do
>-	sync_action=`cat /sys/block/$dev/md/sync_action`
>-		if [ "$sync_action" != "idle" ]; then
>-			checking=1
>-		fi
>-	done
>-done
>+waitbusy 0 "$check_list"
>+
> for dev in $check_list; do
> 	mismatch_cnt=`cat /sys/block/$dev/md/mismatch_cnt`
> 	# Due to the fact that raid1/10 writes in the kernel are unbuffered,

Actions: View | Diff

Attachments on bug 819943: 583043 | 587768