DAMON_SYSFS validates user inputs for monitoring target regions to disallow negative size regions. DAMON core assumes only positive size regions, though. The validation is incomplete. Fortunately damon_set_regions(), which is eventually used by DAMON_SYSFS, does the complete validation. Remove the incomplete and unnecessary validation. Signed-off-by: SJ Park --- mm/damon/sysfs.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/mm/damon/sysfs.c b/mm/damon/sysfs.c index e666dddf1feba..b65651498e0d1 100644 --- a/mm/damon/sysfs.c +++ b/mm/damon/sysfs.c @@ -1987,9 +1987,6 @@ static int damon_sysfs_set_regions(struct damon_target *t, struct damon_sysfs_region *sys_region = sysfs_regions->regions_arr[i]; - if (sys_region->ar.start > sys_region->ar.end) - goto out; - ranges[i].start = sys_region->ar.start; ranges[i].end = sys_region->ar.end; if (i == 0) -- 2.47.3