diff options
author | NeilBrown <neilb@suse.de> | 2009-06-17 18:48:19 -0400 |
---|---|---|
committer | NeilBrown <neilb@suse.de> | 2009-06-17 18:48:19 -0400 |
commit | 1b57f132231593923cb4ab99943ddd777e8745bc (patch) | |
tree | b76f936d08e5ef72430c6fbcf9352f53721481ec /drivers/md | |
parent | 8c6ac868b107ed50a46204f6d14e2ad9443ff146 (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>
Diffstat (limited to 'drivers/md')
-rw-r--r-- | drivers/md/md.c | 2 |
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; |
1909 | repeat: | 1910 | repeat: |
@@ -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 */ |