Messages in this thread Patch in this message | | | From | SeongJae Park <> | Subject | [RFC v9 04/10] selftests/damon/_chk_record: Do not check number of gaps | Date | Wed, 7 Oct 2020 09:14:03 +0200 |
| |
From: SeongJae Park <sjpark@amazon.de>
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@amazon.de> --- 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
| |