From patchwork Tue Nov 6 12:33:38 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Christoffer Dall X-Patchwork-Id: 10670263 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 54F2414BD for ; Tue, 6 Nov 2018 12:34:07 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 410A82A435 for ; Tue, 6 Nov 2018 12:34:07 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 34AF12A45C; Tue, 6 Nov 2018 12:34:07 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-3.6 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,MAILING_LIST_MULTI,RCVD_IN_DNSWL_LOW autolearn=unavailable version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id D67DA2A435 for ; Tue, 6 Nov 2018 12:34:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:MIME-Version:Cc:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:Message-Id:Date: Subject:To:From:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To: References:List-Owner; bh=iRComq7coi+WvyB7WwGit0WcGaDV+T/YTj5HF2f1Joc=; b=cmh y/v3Cj7AgZbjJI60CKp54RqKvussoR4R3mqO01LLOCj59Ane8s02f/7nwz3fb+1b7mEQNCUA/LYNq rnQnP7y9jkUXprvrPM9SgJXyLEp7MambLDa10eVnQKBRDMixyC1ObpoQ49e31z8EiVj9mrE3jm/1y Y9aDSImhPP53ZQApTJZyXXGwwKQ9X4aqkag1fN2jMtVnMi8A9ZjdSRfg/vA2sQd+gmFboYMPvymbM 5iXWHUzSxgsNvtvdkYcIqs6ACAbng8Wa83F0BjckGsqmhbEZnDjnGSVfalD/7uPKZleySy1usXcDE 20TxzSwhvmfkUiR2QFIbs48RcmjjQOA==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1gK0YR-0004di-EV; Tue, 06 Nov 2018 12:33:59 +0000 Received: from merlin.infradead.org ([2001:8b0:10b:1231::1]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1gK0YQ-0004db-Cs for linux-arm-kernel@bombadil.infradead.org; Tue, 06 Nov 2018 12:33:58 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=merlin.20170209; h=Message-Id:Date:Subject:Cc:To:From: Sender:Reply-To:MIME-Version:Content-Type:Content-Transfer-Encoding: Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender: Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=uzBislbL2BawQvdydmQ3YS0Kasc/uX1CEedkzeyOur8=; b=mb4hXRV41HPsUjuogz+YQJ3W0 1dY1qdzhQ7AaDyDxgypa8L7qEZw6LcqyLs2kS+4HAaeT0u0wxPH2OvOQskZ98Z8Lx0XxDvSOP1QfW b2F5uThn7Qy5FN6ygz6/PTgaIsSgoS0fhs7Yl9mehXGmb8iLToWD75MZJ/D3w4T+hXjVgE/OdQjGb XumusUpc2vTBKVicLe106+tKvf+JgAayQbPc8CzqtxK/Mn93lmYumPUtA5VBnqBm46kV7moiqcXPw KVVV1zpZKx7M+ljnxRxTlXMbV3eqvHq9/x3CzMCwEOGJdpNKONF1zfCFeY7zCvzoAb8unR8C42ELo +06GdpflQ==; Received: from foss.arm.com ([217.140.101.70]) by merlin.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1gK0YN-0006Ax-Nv for linux-arm-kernel@lists.infradead.org; Tue, 06 Nov 2018 12:33:56 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id BA44CA78; Tue, 6 Nov 2018 04:33:44 -0800 (PST) Received: from localhost (e113682-lin.copenhagen.arm.com [10.32.144.41]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 225353F5CF; Tue, 6 Nov 2018 04:33:43 -0800 (PST) From: Christoffer Dall To: kvmarm@lists.cs.columbia.edu, linux-arm-kernel@lists.infradead.org Subject: [PATCH v2] KVM: arm64: Clarify explanation of STAGE2_PGTABLE_LEVELS Date: Tue, 6 Nov 2018 13:33:38 +0100 Message-Id: <20181106123338.3786-1-christoffer.dall@arm.com> X-Mailer: git-send-email 2.18.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20181106_073355_877435_83B66BA7 X-CRM114-Status: GOOD ( 17.49 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Marc Zyngier , Christoffer Dall , kvm@vger.kernel.org, Suzuki K Poulose MIME-Version: 1.0 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP In attempting to re-construct the logic for our stage 2 page table layout I found the reaoning in the comment explaining how we calculate the number of levels used for stage 2 page tables a bit backwards. This commit attempts to clarify the comment, to make it slightly easier to read without having the Arm ARM open on the right page. While we're at it, fixup a typo in a comment that was recently changed. Signed-off-by: Christoffer Dall Reviewed-by: Suzuki K Poulose --- Changes since v1: - Dropped note about (PAGE_SHIFT - 3) > 4 arch/arm64/include/asm/stage2_pgtable.h | 14 ++++++-------- virt/kvm/arm/mmu.c | 2 +- 2 files changed, 7 insertions(+), 9 deletions(-) diff --git a/arch/arm64/include/asm/stage2_pgtable.h b/arch/arm64/include/asm/stage2_pgtable.h index d352f6df8d2c..72ecf8ac3668 100644 --- a/arch/arm64/include/asm/stage2_pgtable.h +++ b/arch/arm64/include/asm/stage2_pgtable.h @@ -31,15 +31,13 @@ /* * The hardware supports concatenation of up to 16 tables at stage2 entry level - * and we use the feature whenever possible. + * and we use the feature whenever possible, which means we resolve 4 bits of + * address at the entry level. * - * Now, the minimum number of bits resolved at any level is (PAGE_SHIFT - 3). - * On arm64, the smallest PAGE_SIZE supported is 4k, which means - * (PAGE_SHIFT - 3) > 4 holds for all page sizes. - * This implies, the total number of page table levels at stage2 expected - * by the hardware is actually the number of levels required for (IPA_SHIFT - 4) - * in normal translations(e.g, stage1), since we cannot have another level in - * the range (IPA_SHIFT, IPA_SHIFT - 4). + * This implies, the total number of page table levels required for + * IPA_SHIFT at stage2 expected by the hardware can be calculated using + * the same logic used for the (non-collapsable) stage1 page tables but for + * (IPA_SHIFT - 4). */ #define stage2_pgtable_levels(ipa) ARM64_HW_PGTABLE_LEVELS((ipa) - 4) #define kvm_stage2_levels(kvm) VTCR_EL2_LVLS(kvm->arch.vtcr) diff --git a/virt/kvm/arm/mmu.c b/virt/kvm/arm/mmu.c index 4e7572656b5c..78d8020df4a4 100644 --- a/virt/kvm/arm/mmu.c +++ b/virt/kvm/arm/mmu.c @@ -1234,7 +1234,7 @@ static bool transparent_hugepage_adjust(kvm_pfn_t *pfnp, phys_addr_t *ipap) struct page *page = pfn_to_page(pfn); /* - * PageTransCompoungMap() returns true for THP and + * PageTransCompoundMap() returns true for THP and * hugetlbfs. Make sure the adjustment is done only for THP * pages. */