diff options
author | Tejun Heo <tj@kernel.org> | 2014-09-07 20:51:29 -0400 |
---|---|---|
committer | Tejun Heo <tj@kernel.org> | 2014-09-07 20:51:29 -0400 |
commit | 908c7f1949cb7cc6e92ba8f18f2998e87e265b8e (patch) | |
tree | af885c65c6fe794cab7b7ad37bd811531a2a2ac5 /mm | |
parent | ebd8fef304f99da84d4a52ad056f6137ac9652d4 (diff) |
percpu_counter: add @gfp to percpu_counter_init()
Percpu allocator now supports allocation mask. Add @gfp to
percpu_counter_init() so that !GFP_KERNEL allocation masks can be used
with percpu_counters too.
We could have left percpu_counter_init() alone and added
percpu_counter_init_gfp(); however, the number of users isn't that
high and introducing _gfp variants to all percpu data structures would
be quite ugly, so let's just do the conversion. This is the one with
the most users. Other percpu data structures are a lot easier to
convert.
This patch doesn't make any functional difference.
Signed-off-by: Tejun Heo <tj@kernel.org>
Acked-by: Jan Kara <jack@suse.cz>
Acked-by: "David S. Miller" <davem@davemloft.net>
Cc: x86@kernel.org
Cc: Jens Axboe <axboe@kernel.dk>
Cc: "Theodore Ts'o" <tytso@mit.edu>
Cc: Alexander Viro <viro@zeniv.linux.org.uk>
Cc: Andrew Morton <akpm@linux-foundation.org>
Diffstat (limited to 'mm')
-rw-r--r-- | mm/backing-dev.c | 2 | ||||
-rw-r--r-- | mm/mmap.c | 2 | ||||
-rw-r--r-- | mm/nommu.c | 2 | ||||
-rw-r--r-- | mm/shmem.c | 2 |
4 files changed, 4 insertions, 4 deletions
diff --git a/mm/backing-dev.c b/mm/backing-dev.c index 1706cbbdf5f0..f19a818be2d3 100644 --- a/mm/backing-dev.c +++ b/mm/backing-dev.c | |||
@@ -455,7 +455,7 @@ int bdi_init(struct backing_dev_info *bdi) | |||
455 | bdi_wb_init(&bdi->wb, bdi); | 455 | bdi_wb_init(&bdi->wb, bdi); |
456 | 456 | ||
457 | for (i = 0; i < NR_BDI_STAT_ITEMS; i++) { | 457 | for (i = 0; i < NR_BDI_STAT_ITEMS; i++) { |
458 | err = percpu_counter_init(&bdi->bdi_stat[i], 0); | 458 | err = percpu_counter_init(&bdi->bdi_stat[i], 0, GFP_KERNEL); |
459 | if (err) | 459 | if (err) |
460 | goto err; | 460 | goto err; |
461 | } | 461 | } |
@@ -3196,7 +3196,7 @@ void __init mmap_init(void) | |||
3196 | { | 3196 | { |
3197 | int ret; | 3197 | int ret; |
3198 | 3198 | ||
3199 | ret = percpu_counter_init(&vm_committed_as, 0); | 3199 | ret = percpu_counter_init(&vm_committed_as, 0, GFP_KERNEL); |
3200 | VM_BUG_ON(ret); | 3200 | VM_BUG_ON(ret); |
3201 | } | 3201 | } |
3202 | 3202 | ||
diff --git a/mm/nommu.c b/mm/nommu.c index a881d9673c6b..bd1808e194a7 100644 --- a/mm/nommu.c +++ b/mm/nommu.c | |||
@@ -539,7 +539,7 @@ void __init mmap_init(void) | |||
539 | { | 539 | { |
540 | int ret; | 540 | int ret; |
541 | 541 | ||
542 | ret = percpu_counter_init(&vm_committed_as, 0); | 542 | ret = percpu_counter_init(&vm_committed_as, 0, GFP_KERNEL); |
543 | VM_BUG_ON(ret); | 543 | VM_BUG_ON(ret); |
544 | vm_region_jar = KMEM_CACHE(vm_region, SLAB_PANIC); | 544 | vm_region_jar = KMEM_CACHE(vm_region, SLAB_PANIC); |
545 | } | 545 | } |
diff --git a/mm/shmem.c b/mm/shmem.c index 0e5fb225007c..d4bc55d3f107 100644 --- a/mm/shmem.c +++ b/mm/shmem.c | |||
@@ -2993,7 +2993,7 @@ int shmem_fill_super(struct super_block *sb, void *data, int silent) | |||
2993 | #endif | 2993 | #endif |
2994 | 2994 | ||
2995 | spin_lock_init(&sbinfo->stat_lock); | 2995 | spin_lock_init(&sbinfo->stat_lock); |
2996 | if (percpu_counter_init(&sbinfo->used_blocks, 0)) | 2996 | if (percpu_counter_init(&sbinfo->used_blocks, 0, GFP_KERNEL)) |
2997 | goto failed; | 2997 | goto failed; |
2998 | sbinfo->free_inodes = sbinfo->max_inodes; | 2998 | sbinfo->free_inodes = sbinfo->max_inodes; |
2999 | 2999 | ||