From: Kaixu Xia <kaixuxia@xxxxxxxxxxx>
There are two places that set the configured sector sizes in validate_sectorsize,
actually we can simplify them and combine into one if statement.
Signed-off-by: Kaixu Xia <kaixuxia@xxxxxxxxxxx>
---
mkfs/xfs_mkfs.c | 14 ++++----------
1 file changed, 4 insertions(+), 10 deletions(-)
diff --git a/mkfs/xfs_mkfs.c b/mkfs/xfs_mkfs.c
index 039b1dcc..e1904d57 100644
--- a/mkfs/xfs_mkfs.c
+++ b/mkfs/xfs_mkfs.c
@@ -1696,14 +1696,6 @@ validate_sectorsize(
int dry_run,
int force_overwrite)
{
- /* set configured sector sizes in preparation for checks */
- if (!cli->sectorsize) {
- cfg->sectorsize = dft->sectorsize;
- } else {
- cfg->sectorsize = cli->sectorsize;
- }
- cfg->sectorlog = libxfs_highbit32(cfg->sectorsize);
-
/*
* Before anything else, verify that we are correctly operating on
* files or block devices and set the control parameters correctly.
@@ -1730,6 +1722,7 @@ validate_sectorsize(
memset(ft, 0, sizeof(*ft));
get_topology(cli->xi, ft, force_overwrite);
+ /* set configured sector sizes in preparation for checks */
if (!cli->sectorsize) {
/*
* Unless specified manually on the command line use the
@@ -1759,9 +1752,10 @@ _("specified blocksize %d is less than device physical sector size %d\n"
ft->lsectorsize);
cfg->sectorsize = ft->lsectorsize;
}
+ } else
+ cfg->sectorsize = cli->sectorsize;
- cfg->sectorlog = libxfs_highbit32(cfg->sectorsize);
- }
+ cfg->sectorlog = libxfs_highbit32(cfg->sectorsize);
/* validate specified/probed sector size */
if (cfg->sectorsize < XFS_MIN_SECTORSIZE ||
--
2.20.0
