diff mbox

[v2] common/xfs: fix unexpected failure on real 4k sector device

Message ID 1488183783-15194-1-git-send-email-zlang@redhat.com (mailing list archive)
State New, archived
Headers show

Commit Message

Zorro Lang Feb. 27, 2017, 8:23 a.m. UTC
_xfs_mkfs_validation_check() run "$cmd -s size=2s" to be sure if
xfsprogs support stricter input checks. But there's an unexpected
failure on 4k sector device:

  # blockdev --getbsz --getpbsz --getss $dev
  4096
  4096
  4096
  # mkfs.xfs -f -N -d file,name=$tmpfile,size=1g -s size=2s
  illegal sector size 1024; hw sector is 4096
  ....

So change the sector size from '2s' to '8s' to keep away this
failure.

Signed-off-by: Zorro Lang <zlang@redhat.com>
---

This's the V2 of "[PATCH] common/xfs: change _xfs_mkfs_validation_check return logic",
change '2s' to '8s'.

Thanks,
Zorro

 common/xfs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox

Patch

diff --git a/common/xfs b/common/xfs
index 767a481..4bd8465 100644
--- a/common/xfs
+++ b/common/xfs
@@ -542,7 +542,7 @@  _xfs_mkfs_validation_check()
 	local tmpfile=`mktemp`
 	local cmd="$MKFS_XFS_PROG -f -N -d file,name=$tmpfile,size=1g"
 
-	$cmd -s size=2s >/dev/null 2>&1
+	$cmd -s size=8s >/dev/null 2>&1
 	local sum=$?
 
 	$cmd -l version=2,su=260k >/dev/null 2>&1