From nobody Sun May 5 16:04:21 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; dkim=fail header.i=@wdc.com; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=wdc.com ARC-Seal: i=1; a=rsa-sha256; t=1585263229; cv=none; d=zohomail.com; s=zohoarc; b=P87FG4pl9Hyq3Z5rXAbJEpoit0mZzDr0WOKeaqg6lJkaUfJy43y/EAGeNSWQaZ2i+SMSGehSnuldSGJWnRcYvxZhchxFoZqxvH2koRtRgqSYyHsToVLjKnkxFmPlpN7zRQ27dx3YpmGAgYclKnGN3o/YUq2oHlk6O2sT1NROlS8= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1585263229; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=p8T9EQyn6kvSiHzkQo7gC7rpNKkPiGo2fzSpMvLwzm4=; b=j5GaPKakzFLYO3mmq3khMUXXAYgYGQlcwg7KMtKACjQz9ZffvktdDsD2ADeZF7V5G9VSu3a0/ibP4PCRpDyMuUsOgQfbjgzH96AINKNtH4Fh8vUZ1TpFRvV0SqqhgKeCntCYisNe0XVVFEcnu5B0EmVEMECrgwBKpeq0Tq3/rHQ= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=fail header.i=@wdc.com; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1585263229039360.7491528998248; Thu, 26 Mar 2020 15:53:49 -0700 (PDT) Received: from localhost ([::1]:33906 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jHbNj-0002yl-Rb for importer@patchew.org; Thu, 26 Mar 2020 18:53:47 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:59282) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jHbLk-000194-V9 for qemu-devel@nongnu.org; Thu, 26 Mar 2020 18:51:45 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1jHbLj-0001Hk-Ot for qemu-devel@nongnu.org; Thu, 26 Mar 2020 18:51:44 -0400 Received: from esa2.hgst.iphmx.com ([68.232.143.124]:23231) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1jHbLj-0001Fk-Gh; Thu, 26 Mar 2020 18:51:43 -0400 Received: from uls-op-cesaip02.wdc.com (HELO uls-op-cesaep02.wdc.com) ([199.255.45.15]) by ob1.hgst.iphmx.com with ESMTP; 27 Mar 2020 06:51:56 +0800 Received: from uls-op-cesaip01.wdc.com ([10.248.3.36]) by uls-op-cesaep02.wdc.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Mar 2020 15:42:44 -0700 Received: from 2j0d3g2.ad.shared (HELO risc6-mainframe.hgst.com) ([10.86.54.250]) by uls-op-cesaip01.wdc.com with ESMTP; 26 Mar 2020 15:51:42 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=wdc.com; i=@wdc.com; q=dns/txt; s=dkim.wdc.com; t=1585263118; x=1616799118; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=RsqcTNcToJ/y6Hxiu4n41D32DgYFNh/mBQdoR/mbUYY=; b=q3yOTTJoRcOTBxECfd0kTr/ip2u03/HaexG8iaACy646Rc5cEb4KfIMs wFyujyjsDH6kbmD718pwzxfyyjZ8AAb0a4S3iNeS4sLDk9FFjjNSBp4Ml k+B7PReiiavsDnSwtLlNjDgXgjV6uAzje1UaJb5VArs8iIno2LKAWAtpK SmG+A8RPi5GelorW76IWXMUlRQket0AYiPaI47qmJUmdWu5aZPMtzOQ8Y lf/zyLlSH33q8Iv8eUg47KJfimFZSpAO8K2Go9eW6QAzhvJf6wjB8RyTG Tt3lyT+9d34yCdj5wtCW/FHmHX76YQlrOLkb2a5G+8BNaQ75jKCAlT+TH A==; IronPort-SDR: f34d++HvcLIRmUHqsFT9Ghy0aRFJYLzlNmahrxlLbfBDbwKORnwbsAgJ9SQ3Y4gCRXpHQLwXBs ApTkN0eO8vVnqgunmm6TZDusioZ5IaLAuZDj7I/okz32401fksuPaXHv1WTAm1nkuFype3ZmSF CdUEy5gjdQvD+ewPUQfy9mTwM7+auL3+y1bTLxrl+tH4Ygah+4vlQJkRGLz4QTx10zCMeuR6DI iAyBVS4Z5lq7uhWKxzXaK/gVo395uSEBHIMQ6jp0Z4MCEaWaZoOoS/Fr9WNZc8kHaOl75LnRBn b8E= X-IronPort-AV: E=Sophos;i="5.72,310,1580745600"; d="scan'208";a="235858178" IronPort-SDR: KtkiX+beHzw1Zp+2mwcl61ptewZ614TTaHIE7oJl/ikyRi+Elh8pMkp5qyMjld1yas3DyZjeDi gXKOdhAehFxzzZzS+PXqap0TfuCfuvNh8GwaCudzDZi8dPeG9NYjSQe6saUQzzNl1dZzLt7kV0 JYFdrS0jxq/yHuxXEHl8a3BGuNnQmBAoOaXoZNG8puPuMsGNxRC6GjQf9zkwXDPPwVyu6QChhz r5fa5IRpIjDWHviIpMxs/BaNbd7wUUFd3h0hLKHcjqcGbxGF/T+rQTjXK8gIJuR8oLI7pmGuIT gGGWdljj2AzZHiqxGxGQuoSt IronPort-SDR: huc3OBaOUGQKMU+9+nZAPWpeqkuCpsETEST1duowVDM00LD1pSDeGRyZpm/7y6Znv3Qzqy/Azs +j5IC1zdHERdsWkgo3IzL5uTrKNQszgnpIxNWJ+OjIA+q+Tr1dwoYzYbNfhY7nX9DqNHYJMspI wNjSdUZvFPMfasP1skANb3UMoJu1a8L7LXrrt8y3uR313w5L1DZ31ilVDpFnKeFeM5Jarj2Nri WYdQBH6tWu0I4qEGEwaqXkM4MXYcniVi5QwNzBW1FAbSbDVl66bBgrn9U9FqJHGCT5igHZaHfJ PqQ= WDCIronportException: Internal From: Alistair Francis To: qemu-devel@nongnu.org, qemu-riscv@nongnu.org Subject: [PATCH for 5.0 v1 1/2] riscv: Don't use stage-2 PTE lookup protection flags Date: Thu, 26 Mar 2020 15:44:07 -0700 Message-Id: <931db85d6890ed4bc2b527fd1011197cd28299aa.1585262586.git.alistair.francis@wdc.com> X-Mailer: git-send-email 2.26.0 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: FreeBSD 9.x [fuzzy] X-Received-From: 68.232.143.124 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: alistair.francis@wdc.com, palmer@dabbelt.com, alistair23@gmail.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) Content-Type: text/plain; charset="utf-8" When doing the fist of a two stage lookup (Hypervisor extensions) don't set the current protection flags from the second stage lookup of the base address PTE. Signed-off-by: Alistair Francis Reviewed-by: Richard Henderson Tested-by: Anup Patel --- target/riscv/cpu_helper.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/target/riscv/cpu_helper.c b/target/riscv/cpu_helper.c index d3ba9efb02..f36d184b7b 100644 --- a/target/riscv/cpu_helper.c +++ b/target/riscv/cpu_helper.c @@ -452,10 +452,11 @@ restart: hwaddr pte_addr; =20 if (two_stage && first_stage) { + int vbase_prot; hwaddr vbase; =20 /* Do the second stage translation on the base PTE address. */ - get_physical_address(env, &vbase, prot, base, access_type, + get_physical_address(env, &vbase, &vbase_prot, base, access_ty= pe, mmu_idx, false, true); =20 pte_addr =3D vbase + idx * ptesize; --=20 2.26.0 From nobody Sun May 5 16:04:21 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; dkim=fail header.i=@wdc.com; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=wdc.com ARC-Seal: i=1; a=rsa-sha256; t=1585263361; cv=none; d=zohomail.com; s=zohoarc; b=guwp573h60K7HSw39PzXbvJ5WtBIz6S9xJhmdV2P7WvWaqdt9i/mOCMpwcIn1EkYL/6k50qrebHqF8ne8PamGEZmVnudYMlwd9ZnqsBOuvFjJOzZXvtvaR1Vf/NHP2QTDM9fdWSlTFCcoyEAcynAuamphtIZ/xkJHAoLPsPTksA= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1585263361; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=U6FG1NQs151p5xTnMW6lEIVTOqdEnrchOEfyx7ACw/0=; b=Ao8G2fXuN/z1EQVeQ8Z8ghN8Q01Ot2f20piu1zUK8DdqNAB1piw5lD/zNhykUWajCqoiBsZclUMemD/rg0aWSafWMMiIWkL8hUoNVVE+RmhJfbFnx0NpOhNO4sCYFQGcFwAPBgktk0m5ALckN9TXdqPvzYgDX9FbxPJVBOPpedg= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=fail header.i=@wdc.com; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 1585263361745901.7576388575396; Thu, 26 Mar 2020 15:56:01 -0700 (PDT) Received: from localhost ([::1]:33952 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jHbPs-0005AF-BD for importer@patchew.org; Thu, 26 Mar 2020 18:56:00 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:59428) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jHbLn-0001Dy-S9 for qemu-devel@nongnu.org; Thu, 26 Mar 2020 18:51:48 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1jHbLm-0001Mo-PX for qemu-devel@nongnu.org; Thu, 26 Mar 2020 18:51:47 -0400 Received: from esa2.hgst.iphmx.com ([68.232.143.124]:23243) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1jHbLm-0001L2-GS; Thu, 26 Mar 2020 18:51:46 -0400 Received: from uls-op-cesaip02.wdc.com (HELO uls-op-cesaep02.wdc.com) ([199.255.45.15]) by ob1.hgst.iphmx.com with ESMTP; 27 Mar 2020 06:52:01 +0800 Received: from uls-op-cesaip01.wdc.com ([10.248.3.36]) by uls-op-cesaep02.wdc.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Mar 2020 15:42:47 -0700 Received: from 2j0d3g2.ad.shared (HELO risc6-mainframe.hgst.com) ([10.86.54.250]) by uls-op-cesaip01.wdc.com with ESMTP; 26 Mar 2020 15:51:45 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=wdc.com; i=@wdc.com; q=dns/txt; s=dkim.wdc.com; t=1585263122; x=1616799122; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=nu1k2YInD1z4OmhzuT9vF/dekBAG0pZjrUjpjyIB2mY=; b=Eqp9KWGJuBYNvSAd8Hce5eDGIPcINlnwpiwTESaVSv29HHVxd6ASJTxe gWtrK6mYtrsA1ge+ArAok5IbDBQdePBa8RJTOyjgMH8euSITm+RYM3AYM J03cZ+h5cQQKB3pukFnmZM2WhZiWubE1NrK9P9eSbW/WcRW8PBFM8Y8QD RChidVIdfsqdYRhhsGL1Fa0cn+JJThqsfo9JNoE0lVR1FDOY8d9dZAU4J 1IL0lnwxd2a/POM0sFmDuUHNIuusZQKIbmMbFL+X5nMluLLQh4Xzzf2Bf vNyIE5eu2NauyGclfSy6VTv/nR9vIZhKUXbKtEPLHxKw7UoZWqP02hncx g==; IronPort-SDR: Zd6M6TVWNoaTEOz+zH+ZPgbocRJFqeyy3KIbJgddv0EjODR9tCMBEN8dgXkBoi4gjm643470jH cWsG0DykkBKRwW/2l4psMyIozvYv8fdTONB6rSx1Kf5E9ZtFLgOCKgYJc0nG7zHYBHis3JDu52 hCUjUZykdy8GOOGfw+PyQ70u4PGkZi3uKfZH7vb26qLD6fI+adfNldx+GVRosho8xNoLse0QSB q3BiZL4rY/5MqOTaTDIyntortfQRIgeGXjIA8EMiK5lbilG/OZPtP1qSFU5Sh6/kj2PVERduUo CyU= X-IronPort-AV: E=Sophos;i="5.72,310,1580745600"; d="scan'208";a="235858180" IronPort-SDR: Gl8/aWlWVWvUq5YR6iPwTzWhM6y6Q1iQy/emjgdRNi7P73uTLivSPWfJNL3iuDINdmEvFF07Fh 6fDB/bGhrn5HrfMLPAFWP7A8BFETFiDaCz85FAgWB0ABtJRii01IAOXFjwfz2aJr95hBgwze2Y evHno/l4F/tIjb+7hKmCJcEdapOYkQo3P5bNEduB5iHp50rshD1KHfDlppJZSRVfdeMjfU6fta 09s2bziHxHXfk6WophjSQaji9/G+/4XIsPeq+bSSpYrqzQU62Ia86BkWDtjmD7cSY4JsPdTV1T uTarp3oirRjBa2WCW0+punhL IronPort-SDR: ys1GDTncR0JUslY5A5g6yJhRUHtj0+XpIwuJZI9ypaIYkGFHOGdV8BVhBKKJS5nqiC3XHUcXy0 9ffQCn/ljgGeHdMjsuO+LhC2vZltnb/Y+gW8K4nyzWzVY5/rAXf9STHWb9qRDXqEL4DHLE03EJ TSCCTw0chhkIJM3xvKQoFO2k0w2pyZOvhWda8VJBbqewntvzEW/6UxSMG2uglW+Q29c3nnQNdu YPYKZOsEvgmo/iKSBsHabU3ofWbLbry4MiLKSqRl9BiahRum1oTHyHAAK03gX1iKQTkKKfIpBt ULg= WDCIronportException: Internal From: Alistair Francis To: qemu-devel@nongnu.org, qemu-riscv@nongnu.org Subject: [PATCH for 5.0 v1 2/2] riscv: AND stage-1 and stage-2 protection flags Date: Thu, 26 Mar 2020 15:44:09 -0700 Message-Id: <846f1e18f5922d818bc464ec32c144ef314ec724.1585262586.git.alistair.francis@wdc.com> X-Mailer: git-send-email 2.26.0 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: FreeBSD 9.x [fuzzy] X-Received-From: 68.232.143.124 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: alistair.francis@wdc.com, palmer@dabbelt.com, alistair23@gmail.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" X-ZohoMail-DKIM: fail (Header signature does not verify) Content-Type: text/plain; charset="utf-8" Take the result of stage-1 and stage-2 page table walks and AND the two protection flags together. This way we require both to set permissions instead of just stage-2. Signed-off-by: Alistair Francis Reviewed-by: Richard Henderson Tested-by: Anup Patel --- target/riscv/cpu_helper.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/target/riscv/cpu_helper.c b/target/riscv/cpu_helper.c index f36d184b7b..50e13a064f 100644 --- a/target/riscv/cpu_helper.c +++ b/target/riscv/cpu_helper.c @@ -707,7 +707,7 @@ bool riscv_cpu_tlb_fill(CPUState *cs, vaddr address, in= t size, #ifndef CONFIG_USER_ONLY vaddr im_address; hwaddr pa =3D 0; - int prot; + int prot, prot2; bool pmp_violation =3D false; bool m_mode_two_stage =3D false; bool hs_mode_two_stage =3D false; @@ -757,13 +757,15 @@ bool riscv_cpu_tlb_fill(CPUState *cs, vaddr address, = int size, /* Second stage lookup */ im_address =3D pa; =20 - ret =3D get_physical_address(env, &pa, &prot, im_address, + ret =3D get_physical_address(env, &pa, &prot2, im_address, access_type, mmu_idx, false, true); =20 qemu_log_mask(CPU_LOG_MMU, "%s 2nd-stage address=3D%" VADDR_PRIx " ret %d physica= l " TARGET_FMT_plx " prot %d\n", - __func__, im_address, ret, pa, prot); + __func__, im_address, ret, pa, prot2); + + prot &=3D prot2; =20 if (riscv_feature(env, RISCV_FEATURE_PMP) && (ret =3D=3D TRANSLATE_SUCCESS) && --=20 2.26.0