linux-kernel - Re: [PATCH v2 0/2] Fix __GFP

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-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20180412192424.GB21205@bombadil.infradead.org>
Date:   Thu, 12 Apr 2018 12:24:24 -0700
From:   Matthew Wilcox <willy@...radead.org>
To:     Minchan Kim <minchan@...nel.org>
Cc:     linux-mm@...ck.org, Matthew Wilcox <mawilcox@...rosoft.com>,
        Christoph Lameter <cl@...ux.com>,
        Pekka Enberg <penberg@...nel.org>,
        David Rientjes <rientjes@...gle.com>,
        Joonsoo Kim <iamjoonsoo.kim@....com>,
        Andrew Morton <akpm@...ux-foundation.org>,
        linux-kernel@...r.kernel.org, Jan Kara <jack@...e.cz>,
        Jeff Layton <jlayton@...hat.com>,
        Mel Gorman <mgorman@...hsingularity.net>,
        Chris Fries <cfries@...gle.com>, jaegeuk@...nel.org
Subject: Re: [PATCH v2 0/2] Fix __GFP_ZERO vs constructor

On Thu, Apr 12, 2018 at 09:54:51AM +0900, Minchan Kim wrote:
> Matthew,
> 
> Please Cced relevant people so they know what's going on the problem
> they spent on much time. Everyone doesn't keep an eye on mailing list.

My apologies; I assumed that git send-email would pick up the people
named in the changelog.  I have now read the source code and discovered
it only picks up the people listed in Signed-off-by: and Cc:.  That
surprises me; I'll submit a patch.

> On Tue, Apr 10, 2018 at 11:03:18PM -0700, Matthew Wilcox wrote:
> > From: Matthew Wilcox <mawilcox@...rosoft.com>
> > 
> > v1->v2:
> >  - Added review/ack tags (thanks!)
> >  - Switched the order of the patches
> >  - Reworded commit message of the patch which actually fixes the bug
> >  - Moved slab debug patches under CONFIG_DEBUG_VM to check _every_
> >    allocation and added checks in the slowpath for the allocations which
> >    end up allocating a page.
> > 
> > Matthew Wilcox (2):
> >   Fix NULL pointer in page_cache_tree_insert
> >   slab: __GFP_ZERO is incompatible with a constructor
> > 
> >  mm/filemap.c | 9 ++++-----
> >  mm/slab.c    | 7 ++++---
> >  mm/slab.h    | 7 +++++++
> >  mm/slob.c    | 4 +++-
> >  mm/slub.c    | 5 +++--
> >  5 files changed, 21 insertions(+), 11 deletions(-)
> > 
> > -- 
> > 2.16.3
> > 
>