Received: by 2002:a25:ad19:0:0:0:0:0 with SMTP id y25csp3429648ybi; Mon, 29 Jul 2019 06:27:26 -0700 (PDT) X-Google-Smtp-Source: APXvYqxw/faMk/hpa3AiO6nRgzw7O8/PyokSgJBqrwv2lb8PgLb5RXsY0feLWe0jY/eNmM4fUJCo X-Received: by 2002:a63:5a4d:: with SMTP id k13mr101270196pgm.174.1564406846313; Mon, 29 Jul 2019 06:27:26 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1564406846; cv=none; d=google.com; s=arc-20160816; b=H/9Mh3wJ2eKpEliU+MjX03pqTB2sf25GLshpNLHhy92Ub5OUxe/BgurrN3CFGLO/eU yJmSpVWKxMYcz6E2JvKr6WvjgVR2WU8vEz5pINHJWckH0xb/Z5SFcwu7Husv2bFNbkn2 n91wfcX8tH8SLHUaMu3hNBtBXylle3Rp50xW6ZxnTZayzZb1hK4Nxs9BJXVb6Ne+4Bso BCfBDppDQBZxDIdolMvjS/iIdIzQI9sXz40hrWayTuw1C+LJR7/pfr2+gE7t9oL2+e9r r5fCvLXcQcVqTcfZwjFcYIDXQ45LXfXzW//VAKbLzM2SHhVQV92uGZB+vloeYftbKYsu rRLw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject; bh=WopCjjspgIawBvVbPd70+X5gbjHpqmrzPaq1C0u4lO8=; b=gYYIseVeJAuySRH7dj4hNYHGjogolQSFXBasuxvxfyCDXsr+HNb06j6k4aNXWb3cBg boOdGdgUd8E8tn/9Qsj6M3LyimHy192dngVggjdrzNOVYuVW9LasC18MPh81ZKEF6iqa tkiDhY37N5Uwb6zDt4Jvrioa+4YL90SFfptnsGfsXH4foGGoyzf53WBljAbL5RkgrMEO xeGsxSXxQFtscYegUCFhIe3MqT+AJItjsiRMHV9r5CBakiVLrmK2uNFa5gYfUZXbzf7e czIk8m+0T0qGe7tyH3ZfQyB/PC/pE7S7FKbmVJVQ2rp90ntLB2aEnphhquUOvasVooSz +Nzg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id f4si26939772pgg.334.2019.07.29.06.27.11; Mon, 29 Jul 2019 06:27:26 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2387734AbfG2Lca (ORCPT + 99 others); Mon, 29 Jul 2019 07:32:30 -0400 Received: from foss.arm.com ([217.140.110.172]:42402 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2387483AbfG2Lc3 (ORCPT ); Mon, 29 Jul 2019 07:32:29 -0400 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 EF9BD28; Mon, 29 Jul 2019 04:32:28 -0700 (PDT) Received: from [10.1.196.133] (e112269-lin.cambridge.arm.com [10.1.196.133]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 609A73F694; Mon, 29 Jul 2019 04:32:26 -0700 (PDT) Subject: Re: [PATCH v9 00/21] Generic page walk and ptdump To: Anshuman Khandual , linux-mm@kvack.org, Helge Deller Cc: Mark Rutland , x86@kernel.org, Arnd Bergmann , Ard Biesheuvel , Peter Zijlstra , Catalin Marinas , Dave Hansen , linux-kernel@vger.kernel.org, =?UTF-8?B?SsOpcsO0bWUgR2xpc3Nl?= , Ingo Molnar , Borislav Petkov , Andy Lutomirski , "H. Peter Anvin" , James Morse , Thomas Gleixner , Will Deacon , Andrew Morton , linux-arm-kernel@lists.infradead.org, "Liang, Kan" References: <20190722154210.42799-1-steven.price@arm.com> <794fb469-00c8-af10-92a8-cb7c0c83378b@arm.com> From: Steven Price Message-ID: <270ce719-49f9-7c61-8b25-bc9548a2f478@arm.com> Date: Mon, 29 Jul 2019 12:32:25 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 MIME-Version: 1.0 In-Reply-To: <794fb469-00c8-af10-92a8-cb7c0c83378b@arm.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-GB Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 28/07/2019 12:20, Anshuman Khandual wrote: > On 07/22/2019 09:11 PM, Steven Price wrote: >> Steven Price (21): >> arc: mm: Add p?d_leaf() definitions >> arm: mm: Add p?d_leaf() definitions >> arm64: mm: Add p?d_leaf() definitions >> mips: mm: Add p?d_leaf() definitions >> powerpc: mm: Add p?d_leaf() definitions >> riscv: mm: Add p?d_leaf() definitions >> s390: mm: Add p?d_leaf() definitions >> sparc: mm: Add p?d_leaf() definitions >> x86: mm: Add p?d_leaf() definitions > > The set of architectures here is neither complete (e.g ia64, parisc missing) > nor does it only include architectures which had previously enabled PTDUMP > like arm, arm64, powerpc, s390 and x86. Is there any reason for this set of > archs to be on the list and not the others which are currently falling back > on generic p?d_leaf() defined later in the series ? Are the missing archs > do not have huge page support in the MMU ? If there is a direct dependency > for these symbols with CONFIG_HUGETLB_PAGE then it must be checked before > falling back on the generic ones. The list of architectures here is what I believe to be the list of architectures which can have leaf entries further up the tree than normal. I'm by no means an expert on all these architectures so I'm hoping someone will chime in if they notice something amiss. Obviously all the NO_MMU ia64 as far as I can tell doesn't implement leaf entries further up - it has an interesting hybrid hardware/software walk mechanism and as I understand it the hardware never walks the page table tree that the p?d_xxx() operations operate on. So this is a software implementation detail - but the existance of p?d_huge functions which always return 0 were my first clue that leaf entries are only at the bottom of the tree. parisc is more interesting and I'm not sure if this is necessarily correct. I originally proposed a patch with the line "For parisc, we don't support large pages, so add stubs returning 0" which got Acked by Helge Deller. However going back to look at that again I see there was a follow up thread[2] which possibly suggests I was wrong? Can anyone shed some light on whether parisc does support leaf entries of the page table tree at a higher than the normal depth? [1] https://lkml.org/lkml/2019/2/27/572 [2] https://lkml.org/lkml/2019/3/5/610 The intention is that the page table walker would be available for all architectures so that it can be used in any generic code - PTDUMP simply seemed like a good place to start. > Now that pmd_leaf() and pud_leaf() are getting used in walk_page_range() these > functions need to be defined on all arch irrespective if they use PTDUMP or not > or otherwise just define it for archs which need them now for sure i.e x86 and > arm64 (which are moving to new generic PTDUMP framework). Other archs can > implement these later. The intention was to have valid definitions for all architectures, but obviously I need help from those familiar with those other architectures to check whether I've understood them correctly. Thanks, Steve