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=-4.7 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED 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 3903BC4320A for ; Mon, 2 Aug 2021 13:22:42 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (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 0B8BF61108 for ; Mon, 2 Aug 2021 13:22:42 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 0B8BF61108 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Subject:Cc:To:From:Message-ID:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=sly/s2Oj/Z1ZbAHy13XlAC9r1TVwiuSHffRRR5xodbs=; b=oRYjIQkcBgyill kutZD+zh48vmBM2ckzFOAzkPFdfIOxL00uhJXlCi4rInMqidW6uhqLPSG5WUt5Mf3+kS7+P7YFUrB pJeI3dL4NtQxWVK+ZmbkW+RjpK+253aIrJeY3aJsmTm+HgXmaLyqPMj54x+SKYaTMOGos8iMj6UHP a96W36iCSUJm+/2XZ/l4PkAils0NTBmwlkj7TFrvlvMmWR9Jt/F8JyiQiphlAQ68qDt2iPl428vni jeDZnRZYjjgYsCy5hcDhEnCZC+rF3hvjSE3MHo5smr6jfBzd/H8QU4Y4dlyX8JxA5ZBHN18nL34NA M+n8NE8DPznwr6tM5AOw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mAXsC-00GMFm-43; Mon, 02 Aug 2021 13:20:52 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mAXs5-00GMEm-6P for linux-arm-kernel@lists.infradead.org; Mon, 02 Aug 2021 13:20:49 +0000 Received: from disco-boy.misterjones.org (disco-boy.misterjones.org [51.254.78.96]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id F08DE61108; Mon, 2 Aug 2021 13:20:44 +0000 (UTC) Received: from sofa.misterjones.org ([185.219.108.64] helo=why.misterjones.org) by disco-boy.misterjones.org with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1mAXs2-002SNp-Tz; Mon, 02 Aug 2021 14:20:43 +0100 Date: Mon, 02 Aug 2021 14:20:42 +0100 Message-ID: <87mtq00yqd.wl-maz@kernel.org> From: Marc Zyngier To: Quentin Perret Cc: kvm@vger.kernel.org, kvmarm@lists.cs.columbia.edu, linux-arm-kernel@lists.infradead.org, James Morse , Suzuki K Poulose , Alexandru Elisei , Will Deacon , Catalin Marinas , kernel-team@android.com, stable@vger.kernel.org Subject: Re: [PATCH v2 1/2] arm64: Move .hyp.rodata outside of the _sdata.._edata range In-Reply-To: References: <20210802123830.2195174-1-maz@kernel.org> <20210802123830.2195174-2-maz@kernel.org> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI-EPG/1.14.7 (Harue) FLIM-LB/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL-LB/10.8 EasyPG/1.0.0 Emacs/27.1 (x86_64-pc-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") X-SA-Exim-Connect-IP: 185.219.108.64 X-SA-Exim-Rcpt-To: qperret@google.com, kvm@vger.kernel.org, kvmarm@lists.cs.columbia.edu, linux-arm-kernel@lists.infradead.org, james.morse@arm.com, suzuki.poulose@arm.com, alexandru.elisei@arm.com, will@kernel.org, catalin.marinas@arm.com, kernel-team@android.com, stable@vger.kernel.org X-SA-Exim-Mail-From: maz@kernel.org X-SA-Exim-Scanned: No (on disco-boy.misterjones.org); SAEximRunCond expanded to false X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210802_062045_290575_F66A7D7A X-CRM114-Status: GOOD ( 24.52 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Quentin, On Mon, 02 Aug 2021 14:11:07 +0100, Quentin Perret wrote: > > Hi Marc, > > On Monday 02 Aug 2021 at 13:38:29 (+0100), Marc Zyngier wrote: > > The HYP rodata section is currently lumped together with the BSS, > > which isn't exactly what is expected (it gets registered with > > kmemleak, for example). > > > > Move it away so that it is actually marked RO. As an added > > benefit, it isn't registered with kmemleak anymore. > > 2d7bf218ca73 ("KVM: arm64: Add .hyp.data..ro_after_init ELF section") > states explicitly that the hyp ro_after_init section should remain RW in > the host as it is expected to modify it before initializing EL2. But I > can't seem to trigger anything with this patch applied, so I'll look > into this a bit more. The switch to RO happens quite late. And if the host was to actually try and change things there, it would be screwed anyway (we will have already removed the pages from its S2). I wouldn't be surprised if this was a consequence of the way we now build the HYP object, and the comment in the original commit may not be valid anymore. > > > Fixes: 380e18ade4a5 ("KVM: arm64: Introduce a BSS section for use at Hyp") > > Not sure this is the patch to blame? My bad, this is plain wrong. I'm not sure it can be applied earlier though if my rambling above is correct. Thanks, M. -- Without deviation from the norm, progress is not possible. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel