Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752017AbdFUGLG (ORCPT ); Wed, 21 Jun 2017 02:11:06 -0400 Received: from mail-pf0-f173.google.com ([209.85.192.173]:35937 "EHLO mail-pf0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751150AbdFUGLF (ORCPT ); Wed, 21 Jun 2017 02:11:05 -0400 Date: Tue, 20 Jun 2017 23:10:56 -0700 (PDT) From: Hugh Dickins X-X-Sender: hugh@eggly.anvils To: Willy Tarreau cc: Hugh Dickins , Greg Kroah-Hartman , linux-kernel@vger.kernel.org, stable@vger.kernel.org, Michal Hocko , Linus Torvalds Subject: Re: [PATCH 3.18 32/32] mm: larger stack guard gap, between vmas In-Reply-To: <20170621055902.GF8079@1wt.eu> Message-ID: References: <20170619152035.750974520@linuxfoundation.org> <20170619152037.416644984@linuxfoundation.org> <20170621055902.GF8079@1wt.eu> User-Agent: Alpine 2.11 (LSU 23 2013-08-11) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2043 Lines: 48 On Wed, 21 Jun 2017, Willy Tarreau wrote: > On Tue, Jun 20, 2017 at 10:49:16PM -0700, Hugh Dickins wrote: > > On Mon, 19 Jun 2017, Greg Kroah-Hartman wrote: > > > > > 3.18-stable review patch. If anyone has any objections, please let me know. > > > > > > ------------------ > > > > > > From: Hugh Dickins > > > > > > commit 1be7107fbe18eed3e319a6c3e83c78254b693acb upstream. > > > > Here's a few adjustments to the 3.18 patch: no doubt you'll have > > already sorted out any build errors (and I have to confess that > > I haven't even tried to build this); and the VM_WARN_ON line (as > > in 4.4) only fixes a highly unlikely error; but those FOLL_MLOCK > > lines in mm/gup.c were mistaken, and do need to be deleted. > > Are you sure ? The test on the FOLL_MLOCK flag remains present in > 4.11 and mainline : > > /* mlock all present pages, but do not fault in new pages */ > if ((*flags & (FOLL_POPULATE | FOLL_MLOCK)) == FOLL_MLOCK) > return -ENOENT; > > And this test was present although different in 3.18 as well : > > /* For mlock, just skip the stack guard page. */ > if ((*flags & FOLL_MLOCK) && > (stack_guard_page_start(vma, address) || > stack_guard_page_end(vma, address + PAGE_SIZE))) > > So by removing it we're totally removing any test on FOLL_MLOCK. That > might be the correct fix, but I'm just a bit surprized since the mainline > patch doesn't remove it, and only removes the test on FOLL_POPULATE. I think I'm sure :) Please take another look, the intention of those two FOLL_MLOCK tests is completely different. One of them is about the mlock2() syscall (I think), which wants not to fault in every page at syscall time; and the other is about stack guard pages bogusly included in the vma extents, which must not be faulted in. The stack guard pages are no longer included in the vma extents, so we can just delete those lines (note the "&&" in the condition); but we don't want mlock() to stop faulting its pages in. Makes sense now, or am I tired and confused? Hugh