From: Vincent Fu Date: Fri, 29 Dec 2023 03:12:47 +0000 (-0500) Subject: t/strided: check_result() has no return value X-Git-Tag: fio-3.37~70 X-Git-Url: https://git.kernel.dk/?a=commitdiff_plain;h=06c40418f97811092c0aece1760487400bcdd506;p=fio.git t/strided: check_result() has no return value check_result() erroneously returned True or False. Drop the return value to be consistent with the parent and related classes. Signed-off-by: Vincent Fu --- diff --git a/t/strided.py b/t/strided.py index b8396aef..75c429e4 100755 --- a/t/strided.py +++ b/t/strided.py @@ -99,7 +99,7 @@ class StridedTest(FioJobCmdTest): offset = int(tokens[4]) if offset < zonestart or offset >= zonestart + self.fio_opts['zonerange']: print(f"Offset {offset} outside of zone starting at {zonestart}") - return False + return # skip next section if norandommap is enabled with no # random_generator or with a random_generator != lfsr @@ -117,17 +117,15 @@ class StridedTest(FioJobCmdTest): block = (offset - zonestart) / self.fio_opts['bs'] if block in zoneset: print(f"Offset {offset} in zone already touched") - return False + return zoneset.add(block) if iosperzone % iosperrange == 0: if len(zoneset) != iosperrange: print(f"Expected {iosperrange} blocks in zone but only saw {len(zoneset)}") - return False + return zoneset = set() - return True - TEST_LIST = [ # randommap enabled {