linux-kernel - Re: + mm-kasan-switch-slub-to-stackdepot-enable-memory-quarantine-for-slub-fix-2.patch added to -mm tree

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <57908C79.2020105@virtuozzo.com>
Date:	Thu, 21 Jul 2016 11:48:57 +0300
From:	Andrey Ryabinin <aryabinin@...tuozzo.com>
To:	<akpm@...ux-foundation.org>, <kirill@...temov.name>,
	<glider@...gle.com>, <mgorman@...hsingularity.net>,
	<mhocko@...e.cz>, <riel@...hat.com>, <rientjes@...gle.com>,
	<mm-commits@...r.kernel.org>, LKML <linux-kernel@...r.kernel.org>
Subject: Re: +
 mm-kasan-switch-slub-to-stackdepot-enable-memory-quarantine-for-slub-fix-2.patch
 added to -mm tree



On 07/20/2016 09:43 PM, akpm@...ux-foundation.org wrote:
> 
> The patch titled
>      Subject: mm-kasan-switch-slub-to-stackdepot-enable-memory-quarantine-for-slub-fix
> has been added to the -mm tree.  Its filename is
>      mm-kasan-switch-slub-to-stackdepot-enable-memory-quarantine-for-slub-fix-2.patch
> 
> This patch should soon appear at
>     http://ozlabs.org/~akpm/mmots/broken-out/mm-kasan-switch-slub-to-stackdepot-enable-memory-quarantine-for-slub-fix-2.patch
> and later at
>     http://ozlabs.org/~akpm/mmotm/broken-out/mm-kasan-switch-slub-to-stackdepot-enable-memory-quarantine-for-slub-fix-2.patch
> 
> Before you just go and hit "reply", please:
>    a) Consider who else should be cc'ed
>    b) Prefer to cc a suitable mailing list as well
>    c) Ideally: find the original patch on the mailing list and do a
>       reply-to-all to that, adding suitable additional cc's
> 
> *** Remember to use Documentation/SubmitChecklist when testing your code ***
> 
> The -mm tree is included into linux-next and is updated
> there every 3-4 working days
> 
> ------------------------------------------------------
> From: "Kirill A. Shutemov" <kirill@...temov.name>
> Subject: mm-kasan-switch-slub-to-stackdepot-enable-memory-quarantine-for-slub-fix
> 

This should be a separate patch. Stackdepot was added in cd11016e5f5212c13c0cec7384a525edc93b4921
which is in v4.6.

> use __GFP_NOWARN for stack
> 
> Link: http://lkml.kernel.org/r/20160720151905.GB19146@node.shutemov.name
> Cc: Andrey Ryabinin <aryabinin@...tuozzo.com>
> Cc: Alexander Potapenko <glider@...gle.com>
> Cc: Michal Hocko <mhocko@...e.cz>
> Cc: Rik van Riel <riel@...hat.com>
> Cc: David Rientjes <rientjes@...gle.com>
> Cc: Mel Gorman <mgorman@...hsingularity.net>
> Signed-off-by: Andrew Morton <akpm@...ux-foundation.org>
> ---
> 
>  lib/stackdepot.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff -puN lib/stackdepot.c~mm-kasan-switch-slub-to-stackdepot-enable-memory-quarantine-for-slub-fix-2 lib/stackdepot.c
> --- a/lib/stackdepot.c~mm-kasan-switch-slub-to-stackdepot-enable-memory-quarantine-for-slub-fix-2
> +++ a/lib/stackdepot.c
> @@ -242,6 +242,7 @@ depot_stack_handle_t depot_save_stack(st
>  		 */
>  		alloc_flags &= ~GFP_ZONEMASK;
>  		alloc_flags &= (GFP_ATOMIC | GFP_KERNEL);
> +		alloc_flags |= __GFP_NOWARN;
>  		page = alloc_pages(alloc_flags, STACK_ALLOC_ORDER);
>  		if (page)
>  			prealloc = page_address(page);
> _
> 
> Patches currently in -mm which might be from kirill@...temov.name are
> 
> mm-kasan-switch-slub-to-stackdepot-enable-memory-quarantine-for-slub-fix-2.patch
>