Martchus
/
mdadm
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Grow: don't forget_backup when length of backup is zero. 

This is just a waste of IO

Signed-off-by: NeilBrown <neilb@suse.de>
This commit is contained in:
NeilBrown 2011-03-10 15:43:04 +11:00
parent 8ff6d0946c
commit b3cf095a94
1 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
Grow.c
View File

@ -2990,19 +2990,23 @@ int child_monitor(int afd, struct mdinfo *sra, struct reshape *reshape,
/* Clear any backup region that is before 'here' */
if (increasing) {
if (reshape_completed >= (__le64_to_cpu(bsb.arraystart) +
if (__le64_to_cpu(bsb.length) > 0 &&
reshape_completed >= (__le64_to_cpu(bsb.arraystart) +
__le64_to_cpu(bsb.length)))
forget_backup(dests, destfd,
destoffsets, 0);
if (reshape_completed >= (__le64_to_cpu(bsb.arraystart2) +
if (__le64_to_cpu(bsb.length2) > 0 &&
reshape_completed >= (__le64_to_cpu(bsb.arraystart2) +
__le64_to_cpu(bsb.length2)))
forget_backup(dests, destfd,
destoffsets, 1);
} else {
if (reshape_completed <= (__le64_to_cpu(bsb.arraystart)))
if (__le64_to_cpu(bsb.length) > 0 &&
reshape_completed <= (__le64_to_cpu(bsb.arraystart)))
forget_backup(dests, destfd,
destoffsets, 0);
if (reshape_completed <= (__le64_to_cpu(bsb.arraystart2)))
if (__le64_to_cpu(bsb.length2) > 0 &&
reshape_completed <= (__le64_to_cpu(bsb.arraystart2)))
forget_backup(dests, destfd,
destoffsets, 1);
}