Commit b808f629 authored by Muhammad Usama Anjum's avatar Muhammad Usama Anjum Committed by Andrew Morton

selftests: mm: fix build errors on armhf

The __NR_mmap isn't found on armhf.  The mmap() is commonly available
system call and its wrapper is present on all architectures.  So it should
be used directly.  It solves problem for armhf and doesn't create problem
for other architectures.

Remove sys_mmap() functions as they aren't doing anything else other than
calling mmap().  There is no need to set errno = 0 manually as glibc
always resets it.

For reference errors are as following:

  CC       seal_elf
seal_elf.c: In function 'sys_mmap':
seal_elf.c:39:33: error: '__NR_mmap' undeclared (first use in this function)
   39 |         sret = (void *) syscall(__NR_mmap, addr, len, prot,
      |                                 ^~~~~~~~~

mseal_test.c: In function 'sys_mmap':
mseal_test.c:90:33: error: '__NR_mmap' undeclared (first use in this function)
   90 |         sret = (void *) syscall(__NR_mmap, addr, len, prot,
      |                                 ^~~~~~~~~

Link: https://lkml.kernel.org/r/20240809082511.497266-1-usama.anjum@collabora.com
Fixes: 4926c7a5 ("selftest mm/mseal memory sealing")
Signed-off-by: default avatarMuhammad Usama Anjum <usama.anjum@collabora.com>
Cc: Jeff Xu <jeffxu@chromium.org>
Cc: Kees Cook <kees@kernel.org>
Cc: Liam R. Howlett <Liam.Howlett@oracle.com>
Cc: Shuah Khan <shuah@kernel.org>
Cc: <stable@vger.kernel.org>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
parent 431c1646
...@@ -81,17 +81,6 @@ static int sys_mprotect_pkey(void *ptr, size_t size, unsigned long orig_prot, ...@@ -81,17 +81,6 @@ static int sys_mprotect_pkey(void *ptr, size_t size, unsigned long orig_prot,
return sret; return sret;
} }
static void *sys_mmap(void *addr, unsigned long len, unsigned long prot,
unsigned long flags, unsigned long fd, unsigned long offset)
{
void *sret;
errno = 0;
sret = (void *) syscall(__NR_mmap, addr, len, prot,
flags, fd, offset);
return sret;
}
static int sys_munmap(void *ptr, size_t size) static int sys_munmap(void *ptr, size_t size)
{ {
int sret; int sret;
...@@ -172,7 +161,7 @@ static void setup_single_address(int size, void **ptrOut) ...@@ -172,7 +161,7 @@ static void setup_single_address(int size, void **ptrOut)
{ {
void *ptr; void *ptr;
ptr = sys_mmap(NULL, size, PROT_READ, MAP_ANONYMOUS | MAP_PRIVATE, -1, 0); ptr = mmap(NULL, size, PROT_READ, MAP_ANONYMOUS | MAP_PRIVATE, -1, 0);
*ptrOut = ptr; *ptrOut = ptr;
} }
...@@ -181,7 +170,7 @@ static void setup_single_address_rw(int size, void **ptrOut) ...@@ -181,7 +170,7 @@ static void setup_single_address_rw(int size, void **ptrOut)
void *ptr; void *ptr;
unsigned long mapflags = MAP_ANONYMOUS | MAP_PRIVATE; unsigned long mapflags = MAP_ANONYMOUS | MAP_PRIVATE;
ptr = sys_mmap(NULL, size, PROT_READ | PROT_WRITE, mapflags, -1, 0); ptr = mmap(NULL, size, PROT_READ | PROT_WRITE, mapflags, -1, 0);
*ptrOut = ptr; *ptrOut = ptr;
} }
...@@ -205,7 +194,7 @@ bool seal_support(void) ...@@ -205,7 +194,7 @@ bool seal_support(void)
void *ptr; void *ptr;
unsigned long page_size = getpagesize(); unsigned long page_size = getpagesize();
ptr = sys_mmap(NULL, page_size, PROT_READ, MAP_ANONYMOUS | MAP_PRIVATE, -1, 0); ptr = mmap(NULL, page_size, PROT_READ, MAP_ANONYMOUS | MAP_PRIVATE, -1, 0);
if (ptr == (void *) -1) if (ptr == (void *) -1)
return false; return false;
...@@ -481,7 +470,7 @@ static void test_seal_zero_address(void) ...@@ -481,7 +470,7 @@ static void test_seal_zero_address(void)
int prot; int prot;
/* use mmap to change protection. */ /* use mmap to change protection. */
ptr = sys_mmap(0, size, PROT_NONE, ptr = mmap(0, size, PROT_NONE,
MAP_ANONYMOUS | MAP_PRIVATE | MAP_FIXED, -1, 0); MAP_ANONYMOUS | MAP_PRIVATE | MAP_FIXED, -1, 0);
FAIL_TEST_IF_FALSE(ptr == 0); FAIL_TEST_IF_FALSE(ptr == 0);
...@@ -1209,7 +1198,7 @@ static void test_seal_mmap_overwrite_prot(bool seal) ...@@ -1209,7 +1198,7 @@ static void test_seal_mmap_overwrite_prot(bool seal)
} }
/* use mmap to change protection. */ /* use mmap to change protection. */
ret2 = sys_mmap(ptr, size, PROT_NONE, ret2 = mmap(ptr, size, PROT_NONE,
MAP_ANONYMOUS | MAP_PRIVATE | MAP_FIXED, -1, 0); MAP_ANONYMOUS | MAP_PRIVATE | MAP_FIXED, -1, 0);
if (seal) { if (seal) {
FAIL_TEST_IF_FALSE(ret2 == MAP_FAILED); FAIL_TEST_IF_FALSE(ret2 == MAP_FAILED);
...@@ -1240,7 +1229,7 @@ static void test_seal_mmap_expand(bool seal) ...@@ -1240,7 +1229,7 @@ static void test_seal_mmap_expand(bool seal)
} }
/* use mmap to expand. */ /* use mmap to expand. */
ret2 = sys_mmap(ptr, size, PROT_READ, ret2 = mmap(ptr, size, PROT_READ,
MAP_ANONYMOUS | MAP_PRIVATE | MAP_FIXED, -1, 0); MAP_ANONYMOUS | MAP_PRIVATE | MAP_FIXED, -1, 0);
if (seal) { if (seal) {
FAIL_TEST_IF_FALSE(ret2 == MAP_FAILED); FAIL_TEST_IF_FALSE(ret2 == MAP_FAILED);
...@@ -1268,7 +1257,7 @@ static void test_seal_mmap_shrink(bool seal) ...@@ -1268,7 +1257,7 @@ static void test_seal_mmap_shrink(bool seal)
} }
/* use mmap to shrink. */ /* use mmap to shrink. */
ret2 = sys_mmap(ptr, 8 * page_size, PROT_READ, ret2 = mmap(ptr, 8 * page_size, PROT_READ,
MAP_ANONYMOUS | MAP_PRIVATE | MAP_FIXED, -1, 0); MAP_ANONYMOUS | MAP_PRIVATE | MAP_FIXED, -1, 0);
if (seal) { if (seal) {
FAIL_TEST_IF_FALSE(ret2 == MAP_FAILED); FAIL_TEST_IF_FALSE(ret2 == MAP_FAILED);
...@@ -1650,7 +1639,7 @@ static void test_seal_discard_ro_anon_on_filebacked(bool seal) ...@@ -1650,7 +1639,7 @@ static void test_seal_discard_ro_anon_on_filebacked(bool seal)
ret = fallocate(fd, 0, 0, size); ret = fallocate(fd, 0, 0, size);
FAIL_TEST_IF_FALSE(!ret); FAIL_TEST_IF_FALSE(!ret);
ptr = sys_mmap(NULL, size, PROT_READ, mapflags, fd, 0); ptr = mmap(NULL, size, PROT_READ, mapflags, fd, 0);
FAIL_TEST_IF_FALSE(ptr != MAP_FAILED); FAIL_TEST_IF_FALSE(ptr != MAP_FAILED);
if (seal) { if (seal) {
...@@ -1680,7 +1669,7 @@ static void test_seal_discard_ro_anon_on_shared(bool seal) ...@@ -1680,7 +1669,7 @@ static void test_seal_discard_ro_anon_on_shared(bool seal)
int ret; int ret;
unsigned long mapflags = MAP_ANONYMOUS | MAP_SHARED; unsigned long mapflags = MAP_ANONYMOUS | MAP_SHARED;
ptr = sys_mmap(NULL, size, PROT_READ, mapflags, -1, 0); ptr = mmap(NULL, size, PROT_READ, mapflags, -1, 0);
FAIL_TEST_IF_FALSE(ptr != (void *)-1); FAIL_TEST_IF_FALSE(ptr != (void *)-1);
if (seal) { if (seal) {
......
...@@ -30,17 +30,6 @@ static int sys_mseal(void *start, size_t len) ...@@ -30,17 +30,6 @@ static int sys_mseal(void *start, size_t len)
return sret; return sret;
} }
static void *sys_mmap(void *addr, unsigned long len, unsigned long prot,
unsigned long flags, unsigned long fd, unsigned long offset)
{
void *sret;
errno = 0;
sret = (void *) syscall(__NR_mmap, addr, len, prot,
flags, fd, offset);
return sret;
}
static inline int sys_mprotect(void *ptr, size_t size, unsigned long prot) static inline int sys_mprotect(void *ptr, size_t size, unsigned long prot)
{ {
int sret; int sret;
...@@ -56,7 +45,7 @@ static bool seal_support(void) ...@@ -56,7 +45,7 @@ static bool seal_support(void)
void *ptr; void *ptr;
unsigned long page_size = getpagesize(); unsigned long page_size = getpagesize();
ptr = sys_mmap(NULL, page_size, PROT_READ, MAP_ANONYMOUS | MAP_PRIVATE, -1, 0); ptr = mmap(NULL, page_size, PROT_READ, MAP_ANONYMOUS | MAP_PRIVATE, -1, 0);
if (ptr == (void *) -1) if (ptr == (void *) -1)
return false; return false;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment