Commit f87bcc88 authored by Linus Torvalds's avatar Linus Torvalds

Merge branch 'akpm' (patches from Andrew)

Merge misc mm fixes from Andrew Morton:
 "2 patches.

  Subsystems affected by this patch series: mm (userfaultfd and damon)"

* akpm:
  mm/damon/dbgfs: fix 'struct pid' leaks in 'dbgfs_target_ids_write()'
  userfaultfd/selftests: fix hugetlb area allocations
parents e46227bf ebb3f994
...@@ -353,6 +353,7 @@ static ssize_t dbgfs_target_ids_write(struct file *file, ...@@ -353,6 +353,7 @@ static ssize_t dbgfs_target_ids_write(struct file *file,
const char __user *buf, size_t count, loff_t *ppos) const char __user *buf, size_t count, loff_t *ppos)
{ {
struct damon_ctx *ctx = file->private_data; struct damon_ctx *ctx = file->private_data;
struct damon_target *t, *next_t;
bool id_is_pid = true; bool id_is_pid = true;
char *kbuf, *nrs; char *kbuf, *nrs;
unsigned long *targets; unsigned long *targets;
...@@ -397,8 +398,12 @@ static ssize_t dbgfs_target_ids_write(struct file *file, ...@@ -397,8 +398,12 @@ static ssize_t dbgfs_target_ids_write(struct file *file,
goto unlock_out; goto unlock_out;
} }
/* remove targets with previously-set primitive */ /* remove previously set targets */
damon_set_targets(ctx, NULL, 0); damon_for_each_target_safe(t, next_t, ctx) {
if (targetid_is_pid(ctx))
put_pid((struct pid *)t->id);
damon_destroy_target(t);
}
/* Configure the context for the address space type */ /* Configure the context for the address space type */
if (id_is_pid) if (id_is_pid)
......
...@@ -87,7 +87,7 @@ static bool test_uffdio_minor = false; ...@@ -87,7 +87,7 @@ static bool test_uffdio_minor = false;
static bool map_shared; static bool map_shared;
static int shm_fd; static int shm_fd;
static int huge_fd; static int huge_fd = -1; /* only used for hugetlb_shared test */
static char *huge_fd_off0; static char *huge_fd_off0;
static unsigned long long *count_verify; static unsigned long long *count_verify;
static int uffd = -1; static int uffd = -1;
...@@ -223,6 +223,9 @@ static void noop_alias_mapping(__u64 *start, size_t len, unsigned long offset) ...@@ -223,6 +223,9 @@ static void noop_alias_mapping(__u64 *start, size_t len, unsigned long offset)
static void hugetlb_release_pages(char *rel_area) static void hugetlb_release_pages(char *rel_area)
{ {
if (huge_fd == -1)
return;
if (fallocate(huge_fd, FALLOC_FL_PUNCH_HOLE | FALLOC_FL_KEEP_SIZE, if (fallocate(huge_fd, FALLOC_FL_PUNCH_HOLE | FALLOC_FL_KEEP_SIZE,
rel_area == huge_fd_off0 ? 0 : nr_pages * page_size, rel_area == huge_fd_off0 ? 0 : nr_pages * page_size,
nr_pages * page_size)) nr_pages * page_size))
...@@ -235,16 +238,17 @@ static void hugetlb_allocate_area(void **alloc_area) ...@@ -235,16 +238,17 @@ static void hugetlb_allocate_area(void **alloc_area)
char **alloc_area_alias; char **alloc_area_alias;
*alloc_area = mmap(NULL, nr_pages * page_size, PROT_READ | PROT_WRITE, *alloc_area = mmap(NULL, nr_pages * page_size, PROT_READ | PROT_WRITE,
(map_shared ? MAP_SHARED : MAP_PRIVATE) | map_shared ? MAP_SHARED :
MAP_HUGETLB, MAP_PRIVATE | MAP_HUGETLB |
huge_fd, *alloc_area == area_src ? 0 : (*alloc_area == area_src ? 0 : MAP_NORESERVE),
nr_pages * page_size); huge_fd,
*alloc_area == area_src ? 0 : nr_pages * page_size);
if (*alloc_area == MAP_FAILED) if (*alloc_area == MAP_FAILED)
err("mmap of hugetlbfs file failed"); err("mmap of hugetlbfs file failed");
if (map_shared) { if (map_shared) {
area_alias = mmap(NULL, nr_pages * page_size, PROT_READ | PROT_WRITE, area_alias = mmap(NULL, nr_pages * page_size, PROT_READ | PROT_WRITE,
MAP_SHARED | MAP_HUGETLB, MAP_SHARED,
huge_fd, *alloc_area == area_src ? 0 : huge_fd, *alloc_area == area_src ? 0 :
nr_pages * page_size); nr_pages * page_size);
if (area_alias == MAP_FAILED) if (area_alias == MAP_FAILED)
......
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