Re: [patch v2 4/4] mm, mempool: poison elements backed by page allocator

2015-04-02 Thread David Rientjes
On Tue, 31 Mar 2015, Andrey Ryabinin wrote: > > We don't have a need to set PAGE_EXT_DEBUG_POISON on these pages sitting > > in the reserved pool, nor do we have a need to do kmap_atomic() since it's > > already mapped and must be mapped to be on the reserved pool, which is > > handled by

Re: [patch v2 4/4] mm, mempool: poison elements backed by page allocator

2015-04-02 Thread David Rientjes
On Tue, 31 Mar 2015, Andrey Ryabinin wrote: We don't have a need to set PAGE_EXT_DEBUG_POISON on these pages sitting in the reserved pool, nor do we have a need to do kmap_atomic() since it's already mapped and must be mapped to be on the reserved pool, which is handled by

Re: [patch v2 4/4] mm, mempool: poison elements backed by page allocator

2015-03-31 Thread Andrey Ryabinin
On 03/27/2015 01:50 AM, David Rientjes wrote: > We don't have a need to set PAGE_EXT_DEBUG_POISON on these pages sitting > in the reserved pool, nor do we have a need to do kmap_atomic() since it's > already mapped and must be mapped to be on the reserved pool, which is > handled by

Re: [patch v2 4/4] mm, mempool: poison elements backed by page allocator

2015-03-31 Thread Andrey Ryabinin
On 03/27/2015 01:50 AM, David Rientjes wrote: We don't have a need to set PAGE_EXT_DEBUG_POISON on these pages sitting in the reserved pool, nor do we have a need to do kmap_atomic() since it's already mapped and must be mapped to be on the reserved pool, which is handled by mempool_free().

Re: [patch v2 4/4] mm, mempool: poison elements backed by page allocator

2015-03-30 Thread Andrey Ryabinin
On 03/27/2015 01:50 AM, David Rientjes wrote: > On Thu, 26 Mar 2015, Andrey Ryabinin wrote: > >>> +static void check_element(mempool_t *pool, void *element) >>> +{ >>> + /* Mempools backed by slab allocator */ >>> + if (pool->free == mempool_free_slab || pool->free == mempool_kfree)

Re: [patch v2 4/4] mm, mempool: poison elements backed by page allocator

2015-03-30 Thread Andrey Ryabinin
On 03/27/2015 01:50 AM, David Rientjes wrote: On Thu, 26 Mar 2015, Andrey Ryabinin wrote: +static void check_element(mempool_t *pool, void *element) +{ + /* Mempools backed by slab allocator */ + if (pool-free == mempool_free_slab || pool-free == mempool_kfree) +

Re: [patch v2 4/4] mm, mempool: poison elements backed by page allocator

2015-03-26 Thread David Rientjes
On Thu, 26 Mar 2015, Andrey Ryabinin wrote: > > +static void check_element(mempool_t *pool, void *element) > > +{ > > + /* Mempools backed by slab allocator */ > > + if (pool->free == mempool_free_slab || pool->free == mempool_kfree) > > + __check_element(pool, element,

Re: [patch v2 4/4] mm, mempool: poison elements backed by page allocator

2015-03-26 Thread Andrey Ryabinin
2015-03-25 2:10 GMT+03:00 David Rientjes : ... > > + > +static void check_element(mempool_t *pool, void *element) > +{ > + /* Mempools backed by slab allocator */ > + if (pool->free == mempool_free_slab || pool->free == mempool_kfree) > + __check_element(pool, element,

Re: [patch v2 4/4] mm, mempool: poison elements backed by page allocator

2015-03-26 Thread Andrey Ryabinin
On 03/26/2015 12:55 AM, Andrew Morton wrote: > On Tue, 24 Mar 2015 16:10:01 -0700 (PDT) David Rientjes > wrote: > >> Elements backed by the slab allocator are poisoned when added to a >> mempool's reserved pool. >> >> It is also possible to poison elements backed by the page allocator >>

Re: [patch v2 4/4] mm, mempool: poison elements backed by page allocator

2015-03-26 Thread Andrey Ryabinin
On 03/26/2015 12:55 AM, Andrew Morton wrote: On Tue, 24 Mar 2015 16:10:01 -0700 (PDT) David Rientjes rient...@google.com wrote: Elements backed by the slab allocator are poisoned when added to a mempool's reserved pool. It is also possible to poison elements backed by the page allocator

Re: [patch v2 4/4] mm, mempool: poison elements backed by page allocator

2015-03-26 Thread Andrey Ryabinin
2015-03-25 2:10 GMT+03:00 David Rientjes rient...@google.com: ... + +static void check_element(mempool_t *pool, void *element) +{ + /* Mempools backed by slab allocator */ + if (pool-free == mempool_free_slab || pool-free == mempool_kfree) + __check_element(pool,

Re: [patch v2 4/4] mm, mempool: poison elements backed by page allocator

2015-03-26 Thread David Rientjes
On Thu, 26 Mar 2015, Andrey Ryabinin wrote: +static void check_element(mempool_t *pool, void *element) +{ + /* Mempools backed by slab allocator */ + if (pool-free == mempool_free_slab || pool-free == mempool_kfree) + __check_element(pool, element,

Re: [patch v2 4/4] mm, mempool: poison elements backed by page allocator

2015-03-25 Thread Andrew Morton
On Tue, 24 Mar 2015 16:10:01 -0700 (PDT) David Rientjes wrote: > Elements backed by the slab allocator are poisoned when added to a > mempool's reserved pool. > > It is also possible to poison elements backed by the page allocator > because the mempool layer knows the allocation order. > >

Re: [patch v2 4/4] mm, mempool: poison elements backed by page allocator

2015-03-25 Thread Andrew Morton
On Tue, 24 Mar 2015 16:10:01 -0700 (PDT) David Rientjes rient...@google.com wrote: Elements backed by the slab allocator are poisoned when added to a mempool's reserved pool. It is also possible to poison elements backed by the page allocator because the mempool layer knows the allocation

[patch v2 4/4] mm, mempool: poison elements backed by page allocator

2015-03-24 Thread David Rientjes
Elements backed by the slab allocator are poisoned when added to a mempool's reserved pool. It is also possible to poison elements backed by the page allocator because the mempool layer knows the allocation order. This patch extends mempool element poisoning to include memory backed by the page

[patch v2 4/4] mm, mempool: poison elements backed by page allocator

2015-03-24 Thread David Rientjes
Elements backed by the slab allocator are poisoned when added to a mempool's reserved pool. It is also possible to poison elements backed by the page allocator because the mempool layer knows the allocation order. This patch extends mempool element poisoning to include memory backed by the page