Commit 02d7d31a authored by Dev Jain's avatar Dev Jain Committed by Andrew Morton
Browse files

selftests/mm: parse VMA range in one go 

Use sscanf() to directly parse the VMA range. No functional change is intended.

Link: https://lkml.kernel.org/r/20240322120551.818764-1-dev.jain@arm.com


Signed-off-by: default avatarDev Jain <dev.jain@arm.com>
Cc: Anshuman Khandual <anshuman.khandual@arm.com>
Cc: Shuah Khan <shuah@kernel.org>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
parent 353dc187

tools/testing/selftests/mm/mlock2-tests.c

+1 −14
Original line number Diff line number Diff line
@@ -20,8 +20,6 @@ static int get_vm_area(unsigned long addr, struct vm_boundaries *area) 
	FILE *file;

	int ret = 1;

	char line[1024] = {0};

	char *end_addr;

	char *stop;

	unsigned long start;

	unsigned long end;
@@ -37,21 +35,10 @@ static int get_vm_area(unsigned long addr, struct vm_boundaries *area) 
	memset(area, 0, sizeof(struct vm_boundaries));



	while(fgets(line, 1024, file)) {

		end_addr = strchr(line, '-');

		if (!end_addr) {

		if (sscanf(line, "%lx-%lx", &start, &end) != 2) {

			ksft_print_msg("cannot parse /proc/self/maps\n");

			goto out;

		}

		*end_addr = '\0';

		end_addr++;

		stop = strchr(end_addr, ' ');

		if (!stop) {

			ksft_print_msg("cannot parse /proc/self/maps\n");

			goto out;

		}



		sscanf(line, "%lx", &start);

		sscanf(end_addr, "%lx", &end);



		if (start <= addr && end > addr) {

			area->start = start;