aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNeilBrown <neilb@suse.de>2009-06-17 18:48:19 -0400
committerNeilBrown <neilb@suse.de>2009-06-17 18:48:19 -0400
commit1b57f132231593923cb4ab99943ddd777e8745bc (patch)
treeb76f936d08e5ef72430c6fbcf9352f53721481ec
parent8c6ac868b107ed50a46204f6d14e2ad9443ff146 (diff)
md: move assignment of ->utime so that it never gets skipped.
Currently the assignment to utime gets skipped for 'external' metadata. So move it to the top of the function so that it always gets effected. This is of largely cosmetic interest. Nothing actually depends on ->utime being right for external arrays. "mdadm --monitor" does use it for 0.90 and 1.x arrays, but with mdadm-3.0, this is not important for external metadata. Signed-off-by: NeilBrown <neilb@suse.de>
-rw-r--r--drivers/md/md.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/md/md.c b/drivers/md/md.c
index 90147370bfd7..d9cce54cc2d5 100644
--- a/drivers/md/md.c
+++ b/drivers/md/md.c
@@ -1904,6 +1904,7 @@ static void md_update_sb(mddev_t * mddev, int force_change)
1904 int sync_req; 1904 int sync_req;
1905 int nospares = 0; 1905 int nospares = 0;
1906 1906
1907 mddev->utime = get_seconds();
1907 if (mddev->external) 1908 if (mddev->external)
1908 return; 1909 return;
1909repeat: 1910repeat:
@@ -1933,7 +1934,6 @@ repeat:
1933 nospares = 0; 1934 nospares = 0;
1934 1935
1935 sync_req = mddev->in_sync; 1936 sync_req = mddev->in_sync;
1936 mddev->utime = get_seconds();
1937 1937
1938 /* If this is just a dirty<->clean transition, and the array is clean 1938 /* If this is just a dirty<->clean transition, and the array is clean
1939 * and 'events' is odd, we can roll back to the previous clean state */ 1939 * and 'events' is odd, we can roll back to the previous clean state */