Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1326026

Re: [PATCH] mm/debug-pagealloc: add missing debug_pagealloc_enabled

From Laura Abbott <labbott@redhat.com>
Newsgroups linux.kernel
Subject Re: [PATCH] mm/debug-pagealloc: add missing debug_pagealloc_enabled
Date 2016-02-03 23:40 +0100
Message-ID <qYefF-86A-37@gated-at.bofh.it> (permalink)
References <qYdWj-7Sg-27@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On 02/03/2016 02:15 PM, Arnd Bergmann wrote:
> The change to move the pagealloc logic broke the slab allocator
> check when it's disabled at compile time:
>
> mm/slab.c: In function 'is_debug_pagealloc_cache':
> mm/slab.c:1608:29: error: implicit declaration of function 'debug_pagealloc_enabled' [-Werror=implicit-function-declaration]
>
> This adds an inline helper to get it to work again.
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> Fixes: 0a244aea1a61 ("mm/slab: clean up DEBUG_PAGEALLOC processing code")
> ---
>   include/linux/mm.h | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/include/linux/mm.h b/include/linux/mm.h
> index 5d86eb2e8584..90d600ce56ad 100644
> --- a/include/linux/mm.h
> +++ b/include/linux/mm.h
> @@ -2242,6 +2242,7 @@ kernel_map_pages(struct page *page, int numpages, int enable)
>   extern bool kernel_page_present(struct page *page);
>   #endif /* CONFIG_HIBERNATION */
>   #else
> +static inline bool debug_pagealloc_enabled(void) { return 0; }
>   static inline void
>   kernel_map_pages(struct page *page, int numpages, int enable) {}
>   #ifdef CONFIG_HIBERNATION
>

I believe this should be fixed with http://article.gmane.org/gmane.linux.kernel.mm/145655

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH] mm/debug-pagealloc: add missing debug_pagealloc_enabled Arnd Bergmann <arnd@arndb.de> - 2016-02-03 23:20 +0100
  Re: [PATCH] mm/debug-pagealloc: add missing debug_pagealloc_enabled Laura Abbott <labbott@redhat.com> - 2016-02-03 23:40 +0100
    Re: [PATCH] mm/debug-pagealloc: add missing debug_pagealloc_enabled Arnd Bergmann <arnd@arndb.de> - 2016-02-04 00:00 +0100

csiph-web