The patch titled
Subject: selftests-vm-add-tests-for-lock-on-fault-v9
has been removed from the -mm tree. Its filename was
selftests-vm-add-tests-for-lock-on-fault-v9.patch
This patch was dropped because it was folded into selftests-vm-add-tests-for-lock-on-fault.patch
------------------------------------------------------
From: Eric B Munson <emunson@xxxxxxxxxx>
Subject: selftests-vm-add-tests-for-lock-on-fault-v9
Signed-off-by: Eric B Munson <emunson@xxxxxxxxxx>
Cc: Shuah Khan <shuahkh@xxxxxxxxxxxxxxx>
Cc: Michal Hocko <mhocko@xxxxxxx>
Cc: Vlastimil Babka <vbabka@xxxxxxx>
Cc: Jonathan Corbet <corbet@xxxxxxx>
Cc: Catalin Marinas <catalin.marinas@xxxxxxx>
Cc: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
Cc: Guenter Roeck <linux@xxxxxxxxxxxx>
Cc: Heiko Carstens <heiko.carstens@xxxxxxxxxx>
Cc: Kirill A. Shutemov <kirill.shutemov@xxxxxxxxxxxxxxx>
Cc: Michael Kerrisk <mtk.manpages@xxxxxxxxx>
Cc: Ralf Baechle <ralf@xxxxxxxxxxxxxx>
Cc: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx>
Cc: Thierry Reding <treding@xxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
---
tools/testing/selftests/vm/mlock2-tests.c | 170 ++++++++++++++------
1 file changed, 125 insertions(+), 45 deletions(-)
diff -puN tools/testing/selftests/vm/mlock2-tests.c~selftests-vm-add-tests-for-lock-on-fault-v9 tools/testing/selftests/vm/mlock2-tests.c
--- a/tools/testing/selftests/vm/mlock2-tests.c~selftests-vm-add-tests-for-lock-on-fault-v9
+++ a/tools/testing/selftests/vm/mlock2-tests.c
@@ -112,7 +112,7 @@ static uint64_t get_pageflags(unsigned l
return pfn;
}
-static unsigned long get_kpageflags(unsigned long pfn)
+static uint64_t get_kpageflags(unsigned long pfn)
{
uint64_t flags;
FILE *file;
@@ -137,16 +137,63 @@ static unsigned long get_kpageflags(unsi
return flags;
}
+static FILE *seek_to_smaps_entry(unsigned long addr)
+{
+ FILE *file;
+ char *line = NULL;
+ size_t size = 0;
+ unsigned long start, end;
+ char perms[5];
+ unsigned long offset;
+ char dev[32];
+ unsigned long inode;
+ char path[BUFSIZ];
+
+ file = fopen("/proc/self/smaps", "r");
+ if (!file) {
+ perror("fopen smaps");
+ _exit(1);
+ }
+
+ while (getline(&line, &size, file) > 0) {
+ if (sscanf(line, "%lx-%lx %s %lx %s %lu %s\n",
+ &start, &end, perms, &offset, dev, &inode, path) < 6)
+ goto next;
+
+ if (start <= addr && addr < end)
+ goto out;
+
+next:
+ free(line);
+ line = NULL;
+ size = 0;
+ }
+
+ fclose(file);
+ file = NULL;
+
+out:
+ free(line);
+ return file;
+}
+
#define VMFLAGS "VmFlags:"
-static bool find_flag(FILE *file, const char *vmflag)
+static bool is_vmflag_set(unsigned long addr, const char *vmflag)
{
char *line = NULL;
char *flags;
size_t size = 0;
bool ret = false;
+ FILE *smaps;
- while (getline(&line, &size, file) > 0) {
+ smaps = seek_to_smaps_entry(addr);
+ if (!smaps) {
+ printf("Unable to parse /proc/self/smaps\n");
+ goto out;
+ }
+
+ while (getline(&line, &size, smaps) > 0) {
if (!strstr(line, VMFLAGS)) {
free(line);
line = NULL;
@@ -161,47 +208,71 @@ static bool find_flag(FILE *file, const
out:
free(line);
+ fclose(smaps);
return ret;
}
-static bool is_vmflag_set(unsigned long addr, const char *vmflag)
+#define SIZE "Size:"
+#define RSS "Rss:"
+#define LOCKED "lo"
+
+static bool is_vma_lock_on_fault(unsigned long addr)
{
- FILE *file;
+ bool ret = false;
+ bool locked;
+ FILE *smaps = NULL;
+ unsigned long vma_size, vma_rss;
char *line = NULL;
+ char *value;
size_t size = 0;
- bool ret = false;
- unsigned long start, end;
- char perms[5];
- unsigned long offset;
- char dev[32];
- unsigned long inode;
- char path[BUFSIZ];
- file = fopen("/proc/self/smaps", "r");
- if (!file) {
- perror("fopen smaps");
- _exit(1);
+ locked = is_vmflag_set(addr, LOCKED);
+ if (!locked)
+ goto out;
+
+ smaps = seek_to_smaps_entry(addr);
+ if (!smaps) {
+ printf("Unable to parse /proc/self/smaps\n");
+ goto out;
}
- while (getline(&line, &size, file) > 0) {
- if (sscanf(line, "%lx-%lx %s %lx %s %lu %s\n",
- &start, &end, perms, &offset, dev, &inode, path) < 6)
- goto next;
+ while (getline(&line, &size, smaps) > 0) {
+ if (!strstr(line, SIZE)) {
+ free(line);
+ line = NULL;
+ size = 0;
+ continue;
+ }
- if (start <= addr && addr < end) {
- ret = find_flag(file, vmflag);
+ value = line + strlen(SIZE);
+ if (sscanf(value, "%lu kB", &vma_size) < 1) {
+ printf("Unable to parse smaps entry for Size\n");
goto out;
}
+ break;
+ }
-next:
- free(line);
- line = NULL;
- size = 0;
+ while (getline(&line, &size, smaps) > 0) {
+ if (!strstr(line, RSS)) {
+ free(line);
+ line = NULL;
+ size = 0;
+ continue;
+ }
+
+ value = line + strlen(RSS);
+ if (sscanf(value, "%lu kB", &vma_rss) < 1) {
+ printf("Unable to parse smaps entry for Rss\n");
+ goto out;
+ }
+ break;
}
+ ret = locked && (vma_rss < vma_size);
out:
free(line);
- fclose(file);
+ if (smaps)
+ fclose(smaps);
return ret;
}
@@ -209,9 +280,6 @@ out:
#define PFN_MASK 0x007FFFFFFFFFFFFF
#define UNEVICTABLE_BIT (1UL << 18)
-#define LOCKED "lo"
-#define LOCKEDONFAULT "lf"
-
static int lock_check(char *map)
{
unsigned long page_size = getpagesize();
@@ -237,9 +305,13 @@ static int lock_check(char *map)
return 1;
}
- if (!is_vmflag_set((unsigned long)map, LOCKED) ||
- !is_vmflag_set((unsigned long)map + page_size, LOCKED)) {
- printf("VMA flag %s is missing\n", LOCKED);
+ if (!is_vmflag_set((unsigned long)map, LOCKED)) {
+ printf("VMA flag %s is missing on page 1\n", LOCKED);
+ return 1;
+ }
+
+ if (!is_vmflag_set((unsigned long)map + page_size, LOCKED)) {
+ printf("VMA flag %s is missing on page 2\n", LOCKED);
return 1;
}
@@ -261,9 +333,13 @@ static int unlock_lock_check(char *map)
return 1;
}
- if (is_vmflag_set((unsigned long)map, LOCKED) ||
- is_vmflag_set((unsigned long)map + page_size, LOCKED)) {
- printf("VMA flag %s is still set after unlock\n", LOCKED);
+ if (is_vmflag_set((unsigned long)map, LOCKED)) {
+ printf("VMA flag %s is present on page 1 after unlock\n", LOCKED);
+ return 1;
+ }
+
+ if (is_vmflag_set((unsigned long)map + page_size, LOCKED)) {
+ printf("VMA flag %s is present on page 2 after unlock\n", LOCKED);
return 1;
}
@@ -344,9 +420,13 @@ static int onfault_check(char *map)
return 1;
}
- if (!is_vmflag_set((unsigned long)map, LOCKEDONFAULT) ||
- !is_vmflag_set((unsigned long)map + page_size, LOCKEDONFAULT)) {
- printf("VMA flag %s is missing\n", LOCKEDONFAULT);
+ if (!is_vma_lock_on_fault((unsigned long)map)) {
+ printf("VMA is not marked for lock on fault\n");
+ return 1;
+ }
+
+ if (!is_vma_lock_on_fault((unsigned long)map + page_size)) {
+ printf("VMA is not marked for lock on fault\n");
return 1;
}
@@ -366,9 +446,9 @@ static int unlock_onfault_check(char *ma
return 1;
}
- if (is_vmflag_set((unsigned long)map, LOCKEDONFAULT) ||
- is_vmflag_set((unsigned long)map + page_size, LOCKEDONFAULT)) {
- printf("VMA flag %s is still set after unlock\n", LOCKEDONFAULT);
+ if (is_vma_lock_on_fault((unsigned long)map) ||
+ is_vma_lock_on_fault((unsigned long)map + page_size)) {
+ printf("VMA is still lock on fault after unlock\n");
return 1;
}
@@ -453,9 +533,9 @@ static int test_lock_onfault_of_present(
goto unmap;
}
- if (!is_vmflag_set((unsigned long)map, LOCKEDONFAULT) ||
- !is_vmflag_set((unsigned long)map + page_size, LOCKEDONFAULT)) {
- printf("VMA flag %s is missing for one of the pages\n", LOCKEDONFAULT);
+ if (!is_vma_lock_on_fault((unsigned long)map) ||
+ !is_vma_lock_on_fault((unsigned long)map + page_size)) {
+ printf("VMA with present pages is not marked lock on fault\n");
goto unmap;
}
ret = 0;
_
Patches currently in -mm which might be from emunson@xxxxxxxxxx are
mm-mlock-refactor-mlock-munlock-and-munlockall-code.patch
mm-mlock-add-new-mlock-system-call.patch
mm-introduce-vm_lockonfault.patch
mm-mlock-add-mlock-flags-to-enable-vm_lockonfault-usage.patch
selftests-vm-add-tests-for-lock-on-fault.patch
--
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
