[RFC v6 04/10] selftests/damon/_chk_record: Do not check number of gaps

From: SeongJae Park
Date: Wed Aug 05 2020 - 03:02:05 EST


From: SeongJae Park <sjpark@xxxxxxxxx>

Now the regions can be explicitly set as users want. Therefore checking
the number of gaps doesn't make sense. Remove the condition.

Signed-off-by: SeongJae Park <sjpark@xxxxxxxxx>
---
tools/testing/selftests/damon/_chk_record.py | 6 ------
1 file changed, 6 deletions(-)

diff --git a/tools/testing/selftests/damon/_chk_record.py b/tools/testing/selftests/damon/_chk_record.py
index 73e128904319..5f11be64abed 100644
--- a/tools/testing/selftests/damon/_chk_record.py
+++ b/tools/testing/selftests/damon/_chk_record.py
@@ -37,12 +37,9 @@ def chk_task_info(f):
print('too many regions: %d > %d' % (nr_regions, max_nr_regions))
exit(1)

- nr_gaps = 0
eaddr = 0
for r in range(nr_regions):
saddr = struct.unpack('L', f.read(8))[0]
- if eaddr and saddr != eaddr:
- nr_gaps += 1
eaddr = struct.unpack('L', f.read(8))[0]
nr_accesses = struct.unpack('I', f.read(4))[0]

@@ -56,9 +53,6 @@ def chk_task_info(f):
print('too high nr_access: expected %d but %d' %
(max_nr_accesses, nr_accesses))
exit(1)
- if nr_gaps != 2:
- print('number of gaps are not two but %d' % nr_gaps)
- exit(1)

def parse_time_us(bindat):
sec = struct.unpack('l', bindat[0:8])[0]
--
2.17.1