Re: [PATCH v3 4/4] generic/563: Increase the iosize to to cover for btrfs

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]




On 8/22/25 20:07, Disha Goel wrote:
On 20/08/25 1:45 pm, Nirjhar Roy (IBM) wrote:
When tested with block size/node size 64K on btrfs, then the test fails
with the folllowing error:
      QA output created by 563
      read/write
      read is in range
     -write is in range
     +write has value of 8855552
     +write is NOT in range 7969177.6 .. 8808038.4
      write -> read/write
     ...
The slight increase in the amount of bytes that are written is because
of the increase in the the nodesize(metadata) and hence it exceeds
the tolerance limit slightly. Fix this by increasing the iosize.
Increasing the iosize increases the tolerance range and covers the
tolerance for btrfs higher node sizes.
A very detailed explanation is given by Qu Wenruo in [1]

[1] https://lore.kernel.org/all/fa0dc9e3-2025-49f2-9f20-71190382fce5@xxxxxxx/

Reviewed-by: Qu Wenruo <wqu@xxxxxxxx>
Reported-by: Disha Goel <disgoel@xxxxxxxxxxxxx>
Signed-off-by: Nirjhar Roy (IBM) <nirjhar.roy.lists@xxxxxxxxx>

The patch looks good. However, the subject line seems incorrect, could you please fix it.

Thanks. Added the RBs, fixed some typos and sent the final [v4].


[v4] https://lore.kernel.org/all/cover.1756101620.git.nirjhar.roy.lists@xxxxxxxxx/

--NR


I tested it on Power, and the generic/563 test passes with both 4k & 64k block sizes.

Tested-by: Disha Goel <disgoel@xxxxxxxxxxxxx>

---
  tests/generic/563 | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/generic/563 b/tests/generic/563
index 89a71aa4..6cb9ddb0 100755
--- a/tests/generic/563
+++ b/tests/generic/563
@@ -43,7 +43,7 @@ _require_block_device $SCRATCH_DEV
  _require_non_zoned_device ${SCRATCH_DEV}
    cgdir=$CGROUP2_PATH
-iosize=$((1024 * 1024 * 8))
+iosize=$((1024 * 1024 * 16))
    # Check cgroup read/write charges against expected values. Allow for some   # tolerance as different filesystems seem to account slightly differently.

--
Nirjhar Roy
Linux Kernel Developer
IBM, Bangalore





[Index of Archives]     [Linux Filesystems Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux