diff options
| author | Mariusz Tkaczyk <mariusz.tkaczyk@linux.intel.com> | 2024-02-29 12:52:05 +0100 |
|---|---|---|
| committer | Mariusz Tkaczyk <mariusz.tkaczyk@linux.intel.com> | 2024-03-11 11:06:01 +0100 |
| commit | 0c0f09cb035b6a27a1d11c54836742a9945a5014 (patch) | |
| tree | b9437d6c55abdb7a0ed37fdbfeb4aef3111f6ce4 /Incremental.c | |
| parent | Detail: remove duplicated code (diff) | |
| download | mdadm-0c0f09cb035b6a27a1d11c54836742a9945a5014.tar.xz mdadm-0c0f09cb035b6a27a1d11c54836742a9945a5014.zip | |
mdadm: Add functions for spare criteria verification
It is done similar way in few places. As a result, two almost identical
functions (dev_size_from_id() and dev_sector_size_from_id()) are
removed. Now, it uses same file descriptor to send two ioctls.
Two extern functions are added, in next patches
disk_fd_matches_criteria() is used.
Next optimization is inline zeroing struct spare_criteria. With that,
we don't need to reset values in get_spare_criteria_imsm().
Dedicated boolean field for checking if criteria are filled is added.
We don't need to execute the code if it is not set.
Signed-off-by: Mariusz Tkaczyk <mariusz.tkaczyk@linux.intel.com>
Diffstat (limited to 'Incremental.c')
| -rw-r--r-- | Incremental.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Incremental.c b/Incremental.c index 30c07c03..2b5a5859 100644 --- a/Incremental.c +++ b/Incremental.c @@ -874,7 +874,7 @@ static int array_try_spare(char *devname, int *dfdp, struct dev_policy *pol, struct domainlist *dl = NULL; struct mdinfo *sra; unsigned long long devsize, freesize = 0; - struct spare_criteria sc = {0, 0}; + struct spare_criteria sc = {0}; if (is_subarray(mp->metadata)) continue; |