From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pd0-f169.google.com (mail-pd0-f169.google.com [209.85.192.169]) by kanga.kvack.org (Postfix) with ESMTP id B4D096B0037 for ; Thu, 14 Nov 2013 21:12:31 -0500 (EST) Received: by mail-pd0-f169.google.com with SMTP id y13so2823063pdi.14 for ; Thu, 14 Nov 2013 18:12:31 -0800 (PST) Received: from psmtp.com ([74.125.245.128]) by mx.google.com with SMTP id pl8si448839pbb.194.2013.11.14.18.12.28 for ; Thu, 14 Nov 2013 18:12:30 -0800 (PST) Message-ID: <5285838C.6070508@asianux.com> Date: Fri, 15 Nov 2013 10:14:36 +0800 From: Chen Gang MIME-Version: 1.0 Subject: Re: [PATCH] arch: um: kernel: skas: mmu: remove pmd_free() and pud_free() for failure processing in init_stub_pte() References: <528308E8.8040203@asianux.com> <52847237.5030405@asianux.com> <52847CD5.1030105@asianux.com> In-Reply-To: <52847CD5.1030105@asianux.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: Hugh Dickins Cc: Jeff Dike , Richard Weinberger , Andrew Morton , linux-kernel@vger.kernel.org, linux-mm@kvack.org, uml-devel , uml-user On 11/14/2013 03:33 PM, Chen Gang wrote: > On 11/14/2013 02:48 PM, Chen Gang wrote: >>> >From the look of it, if an error did occur in init_stub_pte(), >>>> then the special mapping of STUB_CODE and STUB_DATA would not >>>> be installed, so this area would be invisible to munmap and exit, >>>> and with your patch then the pages allocated likely to be leaked. >>>> >> It sounds reasonable to me: "although 'pgd' related with 'mm', but they >> are not installed". But just like you said originally: "better get ACK >> from some mm guys". >> >> >> Hmm... is it another issue: "after STUB_CODE succeeds, but STUB_DATA >> fails, the STUB_CODE will be leaked". >> >> >>>> Which is not to say that the existing code is actually correct: >>>> you're probably right that it's technically wrong. But it would >>>> be very hard to get init_stub_pte() to fail, and has anyone >>>> reported a problem with it? My guess is not, and my own >>>> inclination to dabble here is zero. >>>> >> Yeah. >> > > If we can not get ACK from any mm guys, and we have no enough time > resource to read related source code, for me, I still recommend to > remove p?d_free() in failure processing. > Oh, I am very sorry to Hugh and Richard, I make a mistake in common sense: I recognized incorrect members (I treated Hugh as Richard), Hugh is "mm guys". Next time, I should see the mail carefully, not only for contents, but also for senders. Sorry again to both of you. Thanks. > In the worst cases, we will leak a little memory, and no any other > negative effect, it is an executable way which is no any risks. > > For current mm implementation, it seems we can not assume any thing, > although they sounds (or should be) reasonable (include what you said > about mm). > > > Thanks. > -- Chen Gang -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org