From patchwork Tue Sep 24 07:58:22 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Guo Ren X-Patchwork-Id: 11158195 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id AB20613B1 for ; Tue, 24 Sep 2019 07:59:44 +0000 (UTC) Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 82588207FD for ; Tue, 24 Sep 2019 07:59:44 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="Nh76Lfqq" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 82588207FD Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=qemu-devel-bounces+patchwork-qemu-devel=patchwork.kernel.org@nongnu.org Received: from localhost ([::1]:42210 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iCfjb-0005d1-Nk for patchwork-qemu-devel@patchwork.kernel.org; Tue, 24 Sep 2019 03:59:43 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:43549) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iCfif-0004z4-OZ for qemu-devel@nongnu.org; Tue, 24 Sep 2019 03:58:50 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1iCfie-0003ju-9P for qemu-devel@nongnu.org; Tue, 24 Sep 2019 03:58:45 -0400 Received: from mail.kernel.org ([198.145.29.99]:35674) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1iCfie-0003jc-4I; Tue, 24 Sep 2019 03:58:44 -0400 Received: from guoren-Inspiron-7460.lan (unknown [223.93.147.148]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 34E00207FD; Tue, 24 Sep 2019 07:58:37 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1569311923; bh=xBBJFJZyvPw2TXI53lVQHlWDrYQbfyT0JcrFKZjyoxk=; h=From:To:Cc:Subject:Date:From; b=Nh76LfqqMXKTUrc0IhJ7gOxK8e+1ronLrY84lXt1tOzK3IDqYQMtX61sRMBcscy9J V+9Pcz1I7IaBrx40sBNfA9nuXfhTTDlQCe0eOpS+EQyOGt9ZceGh0DOXitcEjskMef +825zRRvd1JTh4LxTFVXBJuU8hURcCfm3GXnOHw8= From: guoren@kernel.org To: qemu-devel@nongnu.org, qemu-riscv@nongnu.org Subject: [PATCH V2] target/riscv: Bugfix reserved bits in PTE for RV64 Date: Tue, 24 Sep 2019 15:58:22 +0800 Message-Id: <1569311902-12173-1-git-send-email-guoren@kernel.org> X-Mailer: git-send-email 2.7.4 X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 198.145.29.99 X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: alistair23@gmail.com, palmer@sifive.com, alistair.francis@wdc.com, Guo Ren Errors-To: qemu-devel-bounces+patchwork-qemu-devel=patchwork.kernel.org@nongnu.org Sender: "Qemu-devel" From: Guo Ren Highest 10 bits of PTE are reserved in riscv-privileged, ref: [1], so we need to ignore them. They can not be a part of ppn. 1: The RISC-V Instruction Set Manual, Volume II: Privileged Architecture 4.4 Sv39: Page-Based 39-bit Virtual-Memory System 4.5 Sv48: Page-Based 48-bit Virtual-Memory System Changelog V2: - Bugfix pte destroyed cause boot fail - Change to AND with a mask instead of shifting both directions Signed-off-by: Guo Ren Reviewed-by: Liu Zhiwei --- target/riscv/cpu_bits.h | 3 +++ target/riscv/cpu_helper.c | 3 ++- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/target/riscv/cpu_bits.h b/target/riscv/cpu_bits.h index e998348..ae8aa0f 100644 --- a/target/riscv/cpu_bits.h +++ b/target/riscv/cpu_bits.h @@ -470,6 +470,9 @@ #define PTE_D 0x080 /* Dirty */ #define PTE_SOFT 0x300 /* Reserved for Software */ +/* Reserved highest 10 bits in PTE */ +#define PTE_RESERVED ((target_ulong)0x3ff << 54) + /* Page table PPN shift amount */ #define PTE_PPN_SHIFT 10 diff --git a/target/riscv/cpu_helper.c b/target/riscv/cpu_helper.c index 87dd6a6..7a540cc 100644 --- a/target/riscv/cpu_helper.c +++ b/target/riscv/cpu_helper.c @@ -258,10 +258,11 @@ restart: } #if defined(TARGET_RISCV32) target_ulong pte = ldl_phys(cs->as, pte_addr); + hwaddr ppn = pte >> PTE_PPN_SHIFT; #elif defined(TARGET_RISCV64) target_ulong pte = ldq_phys(cs->as, pte_addr); + hwaddr ppn = (pte & ~PTE_RESERVED) >> PTE_PPN_SHIFT; #endif - hwaddr ppn = pte >> PTE_PPN_SHIFT; if (!(pte & PTE_V)) { /* Invalid PTE */