Skip to content

Commit

Permalink
madvise11: ignore EBUSY for MADV_SOFT_OFFLINE
Browse files Browse the repository at this point in the history
The EBUSY error could be easily triggered on small system with
kernel-debug. By not treating EBUSY as a failure, the test can
avoid false positives where the test environment itself might
frequently cause resource contention.

Test output:
  madvise11.c:409: TINFO: Spawning 4 threads, with a total of 640 memory pages
  madvise11.c:163: TINFO: Thread [0] returned 0, succeeded.
  madvise11.c:163: TINFO: Thread [2] returned 0, succeeded.
  madvise11.c:163: TINFO: Thread [1] returned 0, succeeded.
  madvise11.c:163: TINFO: Thread [3] returned 0, succeeded.
  madvise11.c:198: TPASS: soft-offline / mmap race still clean     <--- end of 1st test
  madvise11.c:132: TFAIL: madvise failed: EBUSY (16)               <--- ERROR!
  ...

From kernel log:
  [ 431.590511] soft offline: 0xbfa8f: page migration failed 1, type
  0x800000008002e(referenced|uptodate|dirty|active|swapbacked|node=0|zone=1)
  ...
  [ 435.510819] soft offline: 0x98fb6: page migration failed 1, type
  0x800000008000e(referenced|uptodate|dirty|swapbacked|node=0|zone=1)
  ...

Kernel callpath:
  do_madvise()
  ---
  #ifdef CONFIG_MEMORY_FAILURE
      if (behavior == MADV_HWPOISON || behavior == MADV_SOFT_OFFLINE)
          return madvise_inject_error(behavior, start, start + len_in);
  #endif
  ---
    madvise_inject_error()
      soft_offline_page()
        soft_offline_in_use_page()
 	  ...
           2727	pr_info("soft offline: %#lx: %s migration failed %ld, type %pGp\n",
           2728		pfn, msg_page[huge], ret, &page->flags);
           2729	if (ret > 0)
           2730		ret = -EBUSY;   <--- Here

Debugged-by: Luis Goncalves <lgoncalv@redhat.com>
Signed-off-by: Li Wang <liwang@redhat.com>
Reviewed-by: Petr Vorel <pvorel@suse.cz>
  • Loading branch information
wangli5665 committed May 10, 2024
1 parent 8c9ecdf commit 1bddece
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions testcases/kernel/syscalls/madvise/madvise11.c
Original file line number Diff line number Diff line change
Expand Up @@ -128,6 +128,8 @@ static int allocate_offline(int tnum)
return -1;

if (madvise(ptrs[num_alloc], pagesize, MADV_SOFT_OFFLINE) == -1) {
if (errno == EBUSY)
continue;
if (errno != EINVAL)
tst_res(TFAIL | TERRNO, "madvise failed");
if (errno == EINVAL)
Expand Down

0 comments on commit 1bddece

Please sign in to comment.