From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id F1EACC43333 for ; Sun, 22 Mar 2020 01:00:45 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 92D272070A for ; Sun, 22 Mar 2020 01:00:45 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 92D272070A Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=arm.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 015666B0003; Sat, 21 Mar 2020 21:00:45 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id F08676B0006; Sat, 21 Mar 2020 21:00:44 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id DD0116B0007; Sat, 21 Mar 2020 21:00:44 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0153.hostedemail.com [216.40.44.153]) by kanga.kvack.org (Postfix) with ESMTP id C2CC46B0003 for ; Sat, 21 Mar 2020 21:00:44 -0400 (EDT) Received: from smtpin04.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id 946658248068 for ; Sun, 22 Mar 2020 01:00:44 +0000 (UTC) X-FDA: 76621193208.04.yard55_7ad82cd02010b X-HE-Tag: yard55_7ad82cd02010b X-Filterd-Recvd-Size: 2687 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by imf26.hostedemail.com (Postfix) with ESMTP for ; Sun, 22 Mar 2020 01:00:44 +0000 (UTC) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id E0B3131B; Sat, 21 Mar 2020 18:00:42 -0700 (PDT) Received: from [192.168.0.129] (unknown [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id D8DE63F305; Sat, 21 Mar 2020 18:00:38 -0700 (PDT) Subject: Re: [PATCH] x86/mm: Make pud_present() check _PAGE_PROTNONE and _PAGE_PSE as well To: John Hubbard , "Kirill A. Shutemov" Cc: linux-mm@kvack.org, Thomas Gleixner , Ingo Molnar , Borislav Petkov , "H. Peter Anvin" , Dave Hansen , Andrew Morton , x86@kernel.org, linux-kernel@vger.kernel.org, Dan Williams References: <1584507679-11976-1-git-send-email-anshuman.khandual@arm.com> <20200320114741.c62iolt2yzltnscf@box> <2e7a04cf-80cb-58c1-7344-2f8422ed7d31@arm.com> <082aae4a-b190-7b54-eda9-0bbc28c8a6b3@nvidia.com> From: Anshuman Khandual Message-ID: <0a3ac234-303c-26b7-abb7-de42025b3e0d@arm.com> Date: Sun, 22 Mar 2020 06:30:35 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: <082aae4a-b190-7b54-eda9-0bbc28c8a6b3@nvidia.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On 03/21/2020 01:20 AM, John Hubbard wrote: > On 3/20/20 6:22 AM, Anshuman Khandual wrote: > ... >>>> +Cc: Kirill A. Shutemov >>>> +Cc: Dan Williams >>> >>> Or we can just drop the pud_mknotpresent(). There's no users AFAICS and >>> only x86 provides it. > > +1 > >> >> Yes that will be an option but IMHO fixing pud_present() here might be >> a better choice because, >> >> (1) pud_mknotpresent() with fixed pud_present() might be required later > > > It might. Or it might not. Let's wait until it's actually used, and see. > Dead code is an avoidable expense (adds size, space on the screen, email > traffic and other wasted time), so let's avoid it here. Sure, will resend with pud_mknotpresent() dropped. > > >> (2) PMD & PUD will be exact same (THP is supported on either level) >> >> Nonetheless, I am happy to go either way. >> > > > thanks,