Re: [PATCH] riscv: fix bugon.cocci warnings

CC: Guo Ren > Reported-by: kernel test robot > Signed-off-by: kernel test robot > Signed-off-by: Julia Lawall Reviewed-by: Pekka Enberg

Re: [PATCH] riscv: Fixup CONFIG_GENERIC_TIME_VSYSCALL

meofday won't work. > > Signed-off-by: Guo Ren > Cc: Atish Patra > Cc: Palmer Dabbelt > Cc: Vincent Chen Reviewed-by: Pekka Enberg

Re: [PATCH] riscv: mm: abort uaccess retries upon fatal signal

arm.com/ > Signed-off-by: Guo Ren > Cc: Mark Rutland > Cc: Pekka Enberg > Cc: Palmer Dabbelt Reviewed-by: Pekka Enberg

Re: [PATCH v2 0/2] Let illegal access to user-space memory die

) helper > -Split one long line code into two Please also make no_context() use the new helper. Other than that: Reviewed-by: Pekka Enberg

Re: [PATCH] riscv/mm: Prevent kernel module access user-space memory without uaccess routines

On Mon, Nov 30, 2020 at 7:33 AM Eric Lin wrote: > > In the page fault handler, an access to user-space memory > without get/put_user() or copy_from/to_user() routines is > not resolved properly. Like arm and other architectures, > we need to let it die earlier in page fault handler. Fix looks

Re: [PATCH v3] riscv: fix pfn_to_virt err in do_page_fault().

On Mon, Oct 26, 2020 at 08:26:54PM +0800, liush wrote: > From: Liu Shaohua > > The argument to pfn_to_virt() should be pfn not the value of CSR_SATP. > > Reviewed-by: Palmer Dabbelt > Reviewed-by: Anup Patel > Signed-off-by: liush Reviewed-by: Pekka Enberg > ---

Re: [PATCH -next] riscv/mm/fault: fix old-style-declaration warning

Hi, On Wed, Sep 9, 2020 at 2:20 PM Wei Yongjun wrote: > > gcc report build warning as follows: > > arch/riscv/mm/fault.c:81:1: warning: > 'inline' is not at beginning of declaration [-Wold-style-declaration] >81 | static void inline vmalloc_fault(struct pt_regs *regs, int code, > unsigned

Re: [PATCH v4 0/3] Get cache information from userland

For the series: Reviewed-by: Pekka Enberg - Pekka

Re: [PATCH v3 1/3] riscv: Set more data to cacheinfo

Hi, On Mon, Aug 31, 2020 at 9:15 AM Zong Li wrote: > If the sets is one, it means that the cache is fully associative, then > we don't need to fill the ways number, just keep way number as zero, > so here we want to find the fully associative case first and make the > if expression fail at the

Re: [PATCH v3 3/3] riscv: Add cache information in AUX vector

On Fri, Aug 28, 2020 at 10:09 AM Zong Li wrote: > +uintptr_t get_cache_geometry(u32 level, enum cache_type type) > +{ > + struct cacheinfo *this_leaf = get_cacheinfo(level, type); > + uintptr_t ret = (this_leaf->ways_of_associativity << 16 | > +

Re: [PATCH v3 1/3] riscv: Set more data to cacheinfo

Hi, On Fri, Aug 28, 2020 at 10:09 AM Zong Li wrote: > > Set cacheinfo.{size,sets,line_size} for each cache node, then we can > get these information from userland through auxiliary vector. > > Signed-off-by: Zong Li > --- > arch/riscv/kernel/cacheinfo.c | 59 ++-

Re: [PATCH v2 17/23] riscv: use asm-generic/mmu_context.h for no-op implementations

On Wed, Aug 26, 2020 at 5:54 PM Nicholas Piggin wrote: > > Cc: Paul Walmsley > Cc: Palmer Dabbelt > Cc: Albert Ou > Cc: linux-ri...@lists.infradead.org > Acked-by: Palmer Dabbelt > Signed-off-by: Nicholas Piggin Reviewed-by: Pekka Enberg

[RFC PATCH] x86/mm/fault: Inline page fault paths to reduce kernel text size

From: Pekka Enberg The commit 92181f190b649f7ef2b79cbf5c00f26ccc66da2a ("x86: optimise x86's do_page_fault (C entry point for the page fault path)") from 2009 shows significant stack savings when infrequent page fault handling paths are moved out of line with the "noinline" a

Re: [PATCH v2 0/3] mm/slub: Fix count_partial() problem

ative approach for this patch would be to somehow make the lock in count_partial() more granular, but I don't know how feasible that actually is. Anyway, I am OK with this approach: Reviewed-by: Pekka Enberg You still need to convince Christoph, though, because he had objections over this approach. - Pekka

Re: [PATCH 1/2] mm/slub: Introduce two counters for the partial objects

Hi Christopher, On Tue, Aug 11, 2020 at 3:52 PM Christopher Lameter wrote: > > On Fri, 7 Aug 2020, Pekka Enberg wrote: > > > Why do you consider this to be a fast path? This is all partial list > > accounting when we allocate/deallocate a slab, no? Just like > > ___s

Re: [PATCH] mm: sort freelist by rank number

Hi KyongHo and David, On 07.08.20 09:08, Pekka Enberg wrote: > > > I think having more knowledge of DRAM controller details in the OS > > > would be potentially beneficial for better page allocation policy, so > > > maybe try come up with something more ge

Re: [PATCH -next] x86/mpparse: remove duplicate include

On Wed, Aug 19, 2020 at 07:29:10PM +0800, Wang Hai wrote: > Remove asm/io_apic.h which is included more than once > > Reported-by: Hulk Robot > Signed-off-by: Wang Hai Reviewed-by: Pekka Enberg

Re: [PATCH -next] mm: slab: Remove duplicate include

On Tue, Aug 18, 2020 at 2:43 PM YueHaibing wrote: > > Remove duplicate header which is included twice. > > Signed-off-by: YueHaibing Reviewed-by: Pekka Enberg

Re: [PATCH] mm/slub: fix missing ALLOC_SLOWPATH stat when bulk alloc

On Tue, Aug 11, 2020 at 5:25 AM wrote: > > From: Abel Wu > > The ALLOC_SLOWPATH statistics is missing in bulk allocation now. > Fix it by doing statistics in alloc slow path. > > Signed-off-by: Abel Wu Reviewed-by: Pekka Enberg

Re: Odd-sized kmem_cache_alloc and slub_debug=Z

Hi Marco and Kees, On Fri, Aug 07, 2020 at 08:06PM +0300, Pekka Enberg wrote: > > Anything interesting in your .config? The fault does not reproduce > > with 5.8.0 + x86-64 defconfig. On Fri, Aug 7, 2020 at 8:18 PM Marco Elver wrote: > It's quite close to defconfig, just som

Re: [PATCH 1/2] mm/slub: Introduce two counters for the partial objects

Hi Christopher, On Fri, 7 Aug 2020, Pekka Enberg wrote: > > I think we can just default to the counters. After all, if I > > understood correctly, we're talking about up to 100 ms time period > > with IRQs disabled when count_partial() is called. As this is > > triggerable

Re: Odd-sized kmem_cache_alloc and slub_debug=Z

Hi Marco, On Fri, Aug 7, 2020 at 7:07 PM Marco Elver wrote: > I found that the below debug-code using kmem_cache_alloc(), when using > slub_debug=Z, results in the following crash: > > general protection fault, probably for non-canonical address > 0xcca41caea170: [#1] PREEMPT

Re: [PATCH 1/2] mm/slub: Introduce two counters for the partial objects

time period with IRQs disabled when count_partial() is called. As this is triggerable from user space, that's a performance bug whatever way you look at it. Whoever needs to eliminate these counters from fast-path, can wrap them in a CONFIG_MAKE_SLABINFO_EXTREMELY_SLOW option. So for this patch,

Re: [PATCH] mm: sort freelist by rank number

Hi Cho and David, On Mon, Aug 3, 2020 at 10:57 AM David Hildenbrand wrote: > > On 03.08.20 08:10, pullip@samsung.com wrote: > > From: Cho KyongHo > > > > LPDDR5 introduces rank switch delay. If three successive DRAM accesses > > happens and the first and the second ones access one rank and

Re: [PATCH v2 1/2] riscv: Fix build warning for mm/init

esource_init' [-Wmissing-prototypes] > > Signed-off-by: Zong Li Reviewed-by: Pekka Enberg - Pekka

Re: [PATCH 2/2] riscv: fix build warning of mm/pageattr

On Thu, Jul 16, 2020 at 10:11 AM Pekka Enberg wrote: > > On Thu, Jul 16, 2020 at 9:16 AM Zong Li wrote: > > > > Add hearder for missing prototype. Also, static keyword should be at > > beginning of declaration. > > > > Signed-off-by: Zong Li > > Which p

Re: [PATCH 2/2] riscv: fix build warning of mm/pageattr

On Thu, Jul 16, 2020 at 9:16 AM Zong Li wrote: > > Add hearder for missing prototype. Also, static keyword should be at > beginning of declaration. > > Signed-off-by: Zong Li Which prototype is missing? - Pekka

Re: [PATCH 17/26] mm/riscv: Use general page fault accounting

Hi Palmer, On Sat, Jul 11, 2020 at 10:43 PM Palmer Dabbelt wrote: > This still slightly changes the accounting numbers, but I don't think it does > so in a way that's meaningful enough to care about. SIGBUS is the only one > that might happen frequently enough to notice, I doubt anyone cares

Re: [PATCH 1/2] mm/slub: Introduce two counters for the partial objects

Hi! (Sorry for the delay, I missed your response.) On Fri, Jul 3, 2020 at 12:38 PM xunlei wrote: > > On 2020/7/2 PM 7:59, Pekka Enberg wrote: > > On Thu, Jul 2, 2020 at 11:32 AM Xunlei Pang > > wrote: > >> The node list_lock in count_partial() spend long time iter

Re: [PATCH v2] mm/page_alloc: skip setting nodemask when we are in interrupt

success. > > Signed-off-by: Muchun Song Reviewed-by: Pekka Enberg

Re: [PATCH V1 0/5] riscv: Add k/uprobe supported

t all. > > > So we give out a reject list and simulate list in decode-insn.c. On Sat, Jul 4, 2020 at 2:40 PM Pekka Enberg wrote: > > Can you elaborate on what you mean by this? Why would you need a > > single-step facility for kprobes? Is it for executing the instruction > &

Re: [PATCH V1 0/5] riscv: Add k/uprobe supported

On Sat, Jul 4, 2020 at 6:34 AM wrote: > The patchset includes kprobe/uprobe support and some related fixups. Nice! On Sat, Jul 4, 2020 at 6:34 AM wrote: > There is no single step exception in riscv ISA, so utilize ebreak to > simulate. Some pc related instructions couldn't be executed out of

Re: [PATCH RESEND] mm/page_alloc: skip setting nodemask when we are in interrupt

On 03.07.20 08:34, Pekka Enberg wrote: > > if (cpusets_enabled()) { > > *alloc_mask |= __GFP_HARDWALL; > > if (!in_interrupt() && !ac->nodemask) > > ac->nodemask = _current_

Re: [PATCH RESEND] mm/page_alloc: skip setting nodemask when we are in interrupt

On Fri, Jul 3, 2020 at 9:14 AM Muchun Song wrote: > > When we are in the interrupt context, it is irrelevant to the > current task context. If we use current task's mems_allowed, we > can fair to alloc pages in the fast path and fall back to slow > path memory allocation when the current

Re: [PATCH 1/2] mm/slub: Introduce two counters for the partial objects

On Thu, Jul 2, 2020 at 11:32 AM Xunlei Pang wrote: > The node list_lock in count_partial() spend long time iterating > in case of large amount of partial page lists, which can cause > thunder herd effect to the list_lock contention, e.g. it cause > business response-time jitters when accessing

Re: [PATCH v3] mm, slab: Check GFP_SLAB_BUG_MASK before alloc_pages in kmalloc_order

eck out of slab & slub, and call it from > kmalloc_order() as well. In order to make the code clear, the warning > message is put in one place. > > Signed-off-by: Long Li Reviewed-by: Pekka Enberg

Re: [PATCH 17/26] mm/riscv: Use general page fault accounting

ting some days ago: http://lists.infradead.org/pipermail/linux-riscv/2020-June/000775.html However, your fix is obviously even better. For the generic and riscv parts: Reviewed-by: Pekka Enberg - Pekka

Re: [PATCH 0/8] mm: cleanup usage of

ctions where > appropriate. Very nice cleanup series to the page table code! FWIW: Reviewed-by: Pekka Enberg

Re: [PATCH] mm: Fix kobject memleak in SLUB

-by: Pekka Enberg

Re: [PATCH 0/1] mm: Remove the SLAB allocator

Hi, On 4/11/19 10:55 AM, Michal Hocko wrote: Please please have it more rigorous then what happened when SLUB was forced to become a default This is the hard part. Even if you are able to show that SLUB is as fast as SLAB for all the benchmarks you run, there's bound to be that one workload

Re: [PATCH v4] kmemleak: survive in a low-memory situation

hackiness into the existing code for now. On Thu, Mar 28, 2019 at 08:05:31AM +0200, Pekka Enberg wrote: Unfortunately I am not that brave soul, but I'm wondering what the complication here is? It shouldn't be too hard to teach calculate_sizes() in SLUB about a new SLAB_KMEMLEAK flag that reserves

Re: [PATCH v4] kmemleak: survive in a low-memory situation

Hi, On 27/03/2019 2.59, Qian Cai wrote: Unless there is a brave soul to reimplement the kmemleak to embed it's metadata into the tracked memory itself in a foreseeable future, this provides a good balance between enabling kmemleak in a low-memory situation and not introducing too much hackiness

Re: [PATCH] slub: untag object before slab end

below, so untag the object before checking for a NULL object there. Reviewed-by: Pekka Enberg

Re: [PATCH] mm/slab: Increase width of first /proc/slabinfo column

Hi, On 01/02/2019 4.34, Christopher Lameter wrote: On Fri, 1 Feb 2019, Tobin C. Harding wrote: Currently when displaying /proc/slabinfo if any cache names are too long then the output columns are not aligned. We could do something fancy to get the maximum length of any cache name in the

Re: [PATCH 0/3] slub: Do trivial comments fixes

the series: Reviewed-by: Pekka Enberg

Re: [PATCH] mm: Prevent mapping slab pages to userspace

On 25/01/2019 19.38, Matthew Wilcox wrote: It's never appropriate to map a page allocated by SLAB into userspace. A buggy device driver might try this, or an attacker might be able to find a way to make it happen. Signed-off-by: Matthew Wilcox Acked-by: Pekka Enberg A WARN_ON_ONCE() would

Re: [PATCH] mm/slub.c: freelist is ensured to be NULL when new_slab() fails

On 29/12/2018 8.25, Peng Wang wrote: new_slab_objects() will return immediately if freelist is not NULL. if (freelist) return freelist; One more assignment operation could be avoided. Signed-off-by: Peng Wang Reviewed-by: Pekka Enberg --- mm/slub.c | 3

Re: [PATCH] mm, slab: reschedule cache_reap() on the same CPU

t;workqueue: schedule WORK_CPU_UNBOUND work on wq_unbound_cpumask CPUs") CC: <sta...@vger.kernel.org> Cc: Joonsoo Kim <iamjoonsoo@lge.com> Cc: David Rientjes <rient...@google.com> Cc: Pekka Enberg <penb...@kernel.org> Cc: Christoph Lameter <c...@linux.com> Cc:

Re: [PATCH] mm, slab: reschedule cache_reap() on the same CPU

ORK_CPU_UNBOUND work on wq_unbound_cpumask CPUs") CC: Cc: Joonsoo Kim Cc: David Rientjes Cc: Pekka Enberg Cc: Christoph Lameter Cc: Tejun Heo Cc: Lai Jiangshan Cc: John Stultz Cc: Thomas Gleixner Cc: Stephen Boyd Acked-by: Pekka Enberg --- mm/slab.c | 3 ++- 1 file changed

Re: [PATCH 0/4] zswap: Optimize compressed pool memory utilization

On Wed, Aug 17, 2016 at 1:03 PM, Srividya Desireddy wrote: > This series of patches optimize the memory utilized by zswap for storing > the swapped out pages. > > Zswap is a cache which compresses the pages that are being swapped out > and stores them into a dynamically

Re: [PATCH 0/4] zswap: Optimize compressed pool memory utilization

On Wed, Aug 17, 2016 at 1:03 PM, Srividya Desireddy wrote: > This series of patches optimize the memory utilized by zswap for storing > the swapped out pages. > > Zswap is a cache which compresses the pages that are being swapped out > and stores them into a dynamically allocated RAM-based memory

Re: [PATCH 3/4] zswap: Zero-filled pages handling

overhead caused due to zero-filled page check is very minimal >> when compared to the time saved by avoiding compression and allocation in >> case of zero-filled pages. The load time of a zero-filled page is reduced >> by 80% when compared to baseline. On Wed, Aug 17, 2016 at 3:25

Re: [PATCH 3/4] zswap: Zero-filled pages handling

lled page check is very minimal >> when compared to the time saved by avoiding compression and allocation in >> case of zero-filled pages. The load time of a zero-filled page is reduced >> by 80% when compared to baseline. On Wed, Aug 17, 2016 at 3:25 PM, Pekka Enberg wrot

Re: [PATCH 3/4] zswap: Zero-filled pages handling

On Wed, Aug 17, 2016 at 1:18 PM, Srividya Desireddy wrote: > This patch adds a check in zswap_frontswap_store() to identify zero-filled > page before compression of the page. If the page is a zero-filled page, set > zswap_entry.zeroflag and skip the compression of the

Re: [PATCH 3/4] zswap: Zero-filled pages handling

On Wed, Aug 17, 2016 at 1:18 PM, Srividya Desireddy wrote: > This patch adds a check in zswap_frontswap_store() to identify zero-filled > page before compression of the page. If the page is a zero-filled page, set > zswap_entry.zeroflag and skip the compression of the page and alloction > of

Re: [PATCH, REGRESSION v3] mm: make apply_to_page_range more robust

On 01/22/2016 01:12 AM, David Rientjes wrote: NACK to your patch as it is just covering up buggy code silently. The problem needs to be addressed in change_memory_common() to return if there is no size to change (numpages == 0). It's a two line fix to that function. So add a WARN_ON there

Re: [PATCH, REGRESSION v3] mm: make apply_to_page_range more robust

On 01/22/2016 01:12 AM, David Rientjes wrote: NACK to your patch as it is just covering up buggy code silently. The problem needs to be addressed in change_memory_common() to return if there is no size to change (numpages == 0). It's a two line fix to that function. So add a WARN_ON there

Re: [PATCH] mm: slab: convert slab_is_available to boolean

On 9/15/15 8:50 PM, Denis Kirjanov wrote: A good one candidate to return a boolean result Signed-off-by: Denis Kirjanov Reviewed-by: Pekka Enberg -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More

Re: [PATCH] mm: slab: convert slab_is_available to boolean

On 9/15/15 8:50 PM, Denis Kirjanov wrote: A good one candidate to return a boolean result Signed-off-by: Denis Kirjanov <k...@linux-powerpc.org> Reviewed-by: Pekka Enberg <penb...@kernel.org> -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in

[PATCH 04/14] cxgb4: Use kvfree() in t4_free_mem()

Use kvfree() instead of open-coding it. Cc: Hariprasad S Signed-off-by: Pekka Enberg --- drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c b/drivers/net/ethernet/chelsio/cxgb4

[PATCH 07/14] drm/nouveau/gem: Use kvfree() in u_free()

Use kvfree() instead of open-coding it. Cc: David Airlie Signed-off-by: Pekka Enberg --- drivers/gpu/drm/nouveau/nouveau_gem.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_gem.c b/drivers/gpu/drm/nouveau/nouveau_gem.c index 0e690bf

[PATCH 03/14] cxgb3: Use kvfree() in cxgb_free_mem()

Use kvfree() instead of open-coding it. Cc: Santosh Raspatur Signed-off-by: Pekka Enberg --- drivers/net/ethernet/chelsio/cxgb3/cxgb3_offload.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/net/ethernet/chelsio/cxgb3/cxgb3_offload.c b/drivers/net/ethernet

[PATCH 09/14] IB/ehca: Use kvfree() in ipz_queue_{cd}tor()

Use kvfree() instead of open-coding it. Cc: Hoang-Nam Nguyen Cc: Christoph Raisch Signed-off-by: Pekka Enberg --- drivers/infiniband/hw/ehca/ipz_pt_fn.c | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/drivers/infiniband/hw/ehca/ipz_pt_fn.c b/drivers/infiniband

[PATCH 08/14] drivers/input/evdev.c: Use kvfree() in evdev_release()

Use kvfree() instead of open-coding it. Cc: Dmitry Torokhov Signed-off-by: Pekka Enberg --- drivers/input/evdev.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/input/evdev.c b/drivers/input/evdev.c index a18f41b..9d35499 100644 --- a/drivers/input/evdev.c

[PATCH 01/14] ipc/util.c: Use kvfree() in ipc_rcu_free()

Use kvfree() instead of open-coding it. Signed-off-by: Pekka Enberg --- ipc/util.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/ipc/util.c b/ipc/util.c index ff3323e..537a41c 100644 --- a/ipc/util.c +++ b/ipc/util.c @@ -467,10 +467,7 @@ void ipc_rcu_free(struct

[PATCH 10/14] target: Use kvfree() in session alloc and free

Use kvfree() instead of open-coding it. Cc: "Nicholas A. Bellinger" Signed-off-by: Pekka Enberg --- drivers/target/target_core_transport.c | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/drivers/target/target_core_transport.c b/driv

[PATCH 12/14] dm: Use kvfree() in dm_kvfree()

Use kvfree() instead of open-coding it. Cc: Alasdair Kergon Cc: Mike Snitzer Signed-off-by: Pekka Enberg --- drivers/md/dm-stats.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/md/dm-stats.c b/drivers/md/dm-stats.c index f478a4c..492fe6a 100644 --- a/drivers

[PATCH 11/14] libcxgbi: Use kvfree() in cxgbi_free_big_mem()

Use kvfree() instead of open-coding it. Cc: "James E.J. Bottomley" Signed-off-by: Pekka Enberg --- drivers/scsi/cxgbi/libcxgbi.h | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/scsi/cxgbi/libcxgbi.h b/drivers/scsi/cxgbi/libcxgbi.h index aba1af7..c2eb

[PATCH 13/14] bcache: Use kvfree() in various places

Use kvfree() instead of open-coding it. Cc: Kent Overstreet Signed-off-by: Pekka Enberg --- drivers/md/bcache/super.c | 10 ++ drivers/md/bcache/util.h | 10 ++ 2 files changed, 4 insertions(+), 16 deletions(-) diff --git a/drivers/md/bcache/super.c b/drivers/md/bcache

[PATCH 14/14] NTFS: Use kvfree() in ntfs_free()

Use kvfree() instead of open-coding it. Cc: Anton Altaparmakov Signed-off-by: Pekka Enberg --- fs/ntfs/malloc.h | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/fs/ntfs/malloc.h b/fs/ntfs/malloc.h index a44b14c..ab172e5 100644 --- a/fs/ntfs/malloc.h +++ b/fs/ntfs

[PATCH 06/14] drm: Use kvfree() in drm_free_large()

Use kvfree() instead of open-coding it. Cc: David Airlie Signed-off-by: Pekka Enberg --- include/drm/drm_mem_util.h | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/include/drm/drm_mem_util.h b/include/drm/drm_mem_util.h index 19a2404..e42495a 100644 --- a/include/drm

[PATCH 02/14] kernel/relay.c: Use kvfree() in relay_free_page_array()

Use kvfree() instead of open-coding it. Signed-off-by: Pekka Enberg --- kernel/relay.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/kernel/relay.c b/kernel/relay.c index e9dbaeb..0b4570c 100644 --- a/kernel/relay.c +++ b/kernel/relay.c @@ -81,10 +81,7 @@ static struct

[PATCH 05/14] ceph: Use kvfree() in ceph_put_page_vector()

Use kvfree instead of open-coding it. Cc: "Yan, Zheng" Cc: Sage Weil Signed-off-by: Pekka Enberg --- net/ceph/pagevec.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/net/ceph/pagevec.c b/net/ceph/pagevec.c index 096d914..d4f5f22 100644 --- a/net/ceph/pagev

[PATCH 06/14] drm: Use kvfree() in drm_free_large()

Use kvfree() instead of open-coding it. Cc: David Airlie airl...@linux.ie Signed-off-by: Pekka Enberg penb...@kernel.org --- include/drm/drm_mem_util.h | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/include/drm/drm_mem_util.h b/include/drm/drm_mem_util.h index 19a2404

[PATCH 12/14] dm: Use kvfree() in dm_kvfree()

Use kvfree() instead of open-coding it. Cc: Alasdair Kergon a...@redhat.com Cc: Mike Snitzer snit...@redhat.com Signed-off-by: Pekka Enberg penb...@kernel.org --- drivers/md/dm-stats.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/md/dm-stats.c b/drivers/md/dm

[PATCH 03/14] cxgb3: Use kvfree() in cxgb_free_mem()

Use kvfree() instead of open-coding it. Cc: Santosh Raspatur sant...@chelsio.com Signed-off-by: Pekka Enberg penb...@kernel.org --- drivers/net/ethernet/chelsio/cxgb3/cxgb3_offload.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/net/ethernet/chelsio/cxgb3

[PATCH 09/14] IB/ehca: Use kvfree() in ipz_queue_{cd}tor()

Use kvfree() instead of open-coding it. Cc: Hoang-Nam Nguyen hngu...@de.ibm.com Cc: Christoph Raisch rai...@de.ibm.com Signed-off-by: Pekka Enberg penb...@kernel.org --- drivers/infiniband/hw/ehca/ipz_pt_fn.c | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/drivers

[PATCH 08/14] drivers/input/evdev.c: Use kvfree() in evdev_release()

Use kvfree() instead of open-coding it. Cc: Dmitry Torokhov dmitry.torok...@gmail.com Signed-off-by: Pekka Enberg penb...@kernel.org --- drivers/input/evdev.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/input/evdev.c b/drivers/input/evdev.c index a18f41b

[PATCH 01/14] ipc/util.c: Use kvfree() in ipc_rcu_free()

Use kvfree() instead of open-coding it. Signed-off-by: Pekka Enberg penb...@kernel.org --- ipc/util.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/ipc/util.c b/ipc/util.c index ff3323e..537a41c 100644 --- a/ipc/util.c +++ b/ipc/util.c @@ -467,10 +467,7 @@ void

[PATCH 02/14] kernel/relay.c: Use kvfree() in relay_free_page_array()

Use kvfree() instead of open-coding it. Signed-off-by: Pekka Enberg penb...@kernel.org --- kernel/relay.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/kernel/relay.c b/kernel/relay.c index e9dbaeb..0b4570c 100644 --- a/kernel/relay.c +++ b/kernel/relay.c @@ -81,10

[PATCH 05/14] ceph: Use kvfree() in ceph_put_page_vector()

Use kvfree instead of open-coding it. Cc: Yan, Zheng z...@redhat.com Cc: Sage Weil s...@redhat.com Signed-off-by: Pekka Enberg penb...@kernel.org --- net/ceph/pagevec.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/net/ceph/pagevec.c b/net/ceph/pagevec.c index 096d914

[PATCH 11/14] libcxgbi: Use kvfree() in cxgbi_free_big_mem()

Use kvfree() instead of open-coding it. Cc: James E.J. Bottomley jbottom...@odin.com Signed-off-by: Pekka Enberg penb...@kernel.org --- drivers/scsi/cxgbi/libcxgbi.h | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/scsi/cxgbi/libcxgbi.h b/drivers/scsi/cxgbi

[PATCH 13/14] bcache: Use kvfree() in various places

Use kvfree() instead of open-coding it. Cc: Kent Overstreet k...@daterainc.com Signed-off-by: Pekka Enberg penb...@kernel.org --- drivers/md/bcache/super.c | 10 ++ drivers/md/bcache/util.h | 10 ++ 2 files changed, 4 insertions(+), 16 deletions(-) diff --git a/drivers/md

[PATCH 14/14] NTFS: Use kvfree() in ntfs_free()

Use kvfree() instead of open-coding it. Cc: Anton Altaparmakov an...@tuxera.com Signed-off-by: Pekka Enberg penb...@kernel.org --- fs/ntfs/malloc.h | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/fs/ntfs/malloc.h b/fs/ntfs/malloc.h index a44b14c..ab172e5 100644 --- a/fs

[PATCH 07/14] drm/nouveau/gem: Use kvfree() in u_free()

Use kvfree() instead of open-coding it. Cc: David Airlie airl...@linux.ie Signed-off-by: Pekka Enberg penb...@kernel.org --- drivers/gpu/drm/nouveau/nouveau_gem.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_gem.c b/drivers/gpu/drm

[PATCH 04/14] cxgb4: Use kvfree() in t4_free_mem()

Use kvfree() instead of open-coding it. Cc: Hariprasad S haripra...@chelsio.com Signed-off-by: Pekka Enberg penb...@kernel.org --- drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/net/ethernet/chelsio/cxgb4

[PATCH 10/14] target: Use kvfree() in session alloc and free

Use kvfree() instead of open-coding it. Cc: Nicholas A. Bellinger n...@linux-iscsi.org Signed-off-by: Pekka Enberg penb...@kernel.org --- drivers/target/target_core_transport.c | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/drivers/target/target_core_transport.c

Re: [PATCH v2] perf kmem: Show warning when trying to run stat without record

below to inform user: # perf kmem stat --page --caller Not found page events. Have you run 'perf kmem record --page' before? Acked-by: Pekka Enberg Signed-off-by: Namhyung Kim Thanks, applied. I just found the messages a bit odd souding, perhaps: # perf kmem stat --page --caller

Re: [PATCH v2] perf kmem: Show warning when trying to run stat without record

below to inform user: # perf kmem stat --page --caller Not found page events. Have you run 'perf kmem record --page' before? Acked-by: Pekka Enberg penb...@kernel.org Signed-off-by: Namhyung Kim namhy...@kernel.org Thanks, applied. I just found the messages a bit odd souding, perhaps

Re: [PATCH 0/13] Parallel struct page initialisation v4

16] kswapd 2 initialised deferred memory in 1148ms > > Once booted the machine appears to work as normal. Boot times were measured > from the time shutdown was called until ssh was available again. In the > 64G case, the boot time savings are negligible. On the 1TB machine, the &g

Re: [PATCH 0/13] Parallel struct page initialisation v4

the machine appears to work as normal. Boot times were measured from the time shutdown was called until ssh was available again. In the 64G case, the boot time savings are negligible. On the 1TB machine, the savings were 16 seconds. FWIW, Acked-by: Pekka Enberg penb...@kernel.org for the whole series

Re: [PATCHSET 0/6] perf kmem: Implement page allocation analysis (v7)

it does slab allocation analysis for backward compatibility. Nice addition! Acked-by: Pekka Enberg for the whole series. - Pekka -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at

Re: [PATCHSET 0/6] perf kmem: Implement page allocation analysis (v7)

, it does slab allocation analysis for backward compatibility. Nice addition! Acked-by: Pekka Enberg penb...@kernel.org for the whole series. - Pekka -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info

Re: [PATCH v6 2/4] perf tools: add Java demangling support

Hi Stephane, On Tue, Mar 31, 2015 at 1:19 AM, Stephane Eranian wrote: > +#define BASE_ENT(c, n) [c-'A']=n > +static const char *base_types['Z'-'A' + 1]={ > + BASE_ENT('B', "byte" ), > + BASE_ENT('C', "char" ), > + BASE_ENT('D', "double" ), > + BASE_ENT('F', "float" ), > +

Re: [PATCH v6 2/4] perf tools: add Java demangling support

Hi Stephane, On Tue, Mar 31, 2015 at 1:19 AM, Stephane Eranian eran...@google.com wrote: +#define BASE_ENT(c, n) [c-'A']=n +static const char *base_types['Z'-'A' + 1]={ + BASE_ENT('B', byte ), + BASE_ENT('C', char ), + BASE_ENT('D', double ), + BASE_ENT('F', float ),

Re: stand-alone kvmtool

On 2/26/15 1:02 PM, Alex Bennée wrote: If you can have it all it would be nice to preserve buildability all through your history for bisecting (and the moon on a stick please ;-) Is the dependency on the kernel sources something that has been stable over the projects history or something that's

Re: stand-alone kvmtool

On 2/26/15 1:02 PM, Alex Bennée wrote: If you can have it all it would be nice to preserve buildability all through your history for bisecting (and the moon on a stick please ;-) Is the dependency on the kernel sources something that has been stable over the projects history or something that's

Re: stand-alone kvmtool

Hi, On 2/18/15 5:50 PM, Will Deacon wrote: Thanks for doing this. Since it looks unlikely that kvmtool will ever be merged back into the kernel tree, it makes sense to cut the dependency in my opinion. I am certainly OK with a standalone repository which preserves the history. Will, would

Re: stand-alone kvmtool

Hi, On 2/18/15 5:50 PM, Will Deacon wrote: Thanks for doing this. Since it looks unlikely that kvmtool will ever be merged back into the kernel tree, it makes sense to cut the dependency in my opinion. I am certainly OK with a standalone repository which preserves the history. Will, would

Re: [PATCH v3 1/5] slab: Correct size_index table before replacing the bootstrap kmem_cache_node.

Cc: Pekka Enberg Cc: David Rientjes Cc: Joonsoo Kim Cc: Andrew Morton Cc: linux...@kvack.org Cc: linux-kernel@vger.kernel.org Acked-by: Pekka Enberg -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More

  1   2   3   4   5   6   7   8   9   10   >