--- 99-raid-check.orig	2011-12-17 13:30:32.000000000 +1100
+++ /etc/cron.weekly/99-raid-check	2011-12-17 13:31:53.000000000 +1100
@@ -18,37 +18,25 @@
 active_list=`grep "^md.*: active" /proc/mdstat | cut -f 1 -d ' '`
 [ -z "$active_list" ] && exit 0
 
-dev_list=""
 check_list=""
-devnum=0
 for dev in $active_list; do
     echo $SKIP_DEVS | grep -w $dev >/dev/null 2>&1 && continue
     if [ -f /sys/block/$dev/md/sync_action ]; then
-	# Only perform the checks on idle, healthy arrays, but delay
-	# actually writing the check field until the next loop so we
-	# don't switch currently idle arrays to active, which happens
-	# when two or more arrays are on the same physical disk
+	# Only perform actions on healthy arrays that aren't already 
+	# undergoing some resync operation.
 	array_state=`cat /sys/block/$dev/md/array_state`
 	sync_action=`cat /sys/block/$dev/md/sync_action`
-	if [ "$array_state" = clean -a "$sync_action" = idle ]; then
+	if [ "$array_state" = clean -o "$array_state" = active ] && [ "$sync_action" = idle ]; then
 	    ck=""
 	    echo $REPAIR_DEVS | grep -w $dev >/dev/null 2>&1 && ck="repair"
 	    echo $CHECK_DEVS | grep -w $dev >/dev/null 2>&1 && ck="check"
 	    [ -z "$ck" ] && ck=$CHECK
-	    dev_list="$dev_list $dev"
-	    check[$devnum]=$ck
-	    let devnum++
+	    echo "$ck" > /sys/block/$dev/md/sync_action
 	    [ "$ck" = "check" ] && check_list="$check_list $dev"
 	fi
     fi
 done
-[ -z "$dev_list" ] && exit 0
 
-devnum=0
-for dev in $dev_list; do
-    echo "${check[$devnum]}" > /sys/block/$dev/md/sync_action
-    let devnum++
-done
 [ -z "$check_list" ] && exit 0
 
 checking=1