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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id 610CFC19F2D for ; Tue, 9 Aug 2022 14:02:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=/5PqYGoosrGtUq00ZXyyzicCDDlZJQcitiOwx77GFXk=; b=c4LNhOl6kNDPXfJxLbT2li0wHV ip4PNVWJV6stkNDP5QdDe7/fKcTHXxfRWkDUE4VZhqhjmHNXXnidZJ8S1Xo9kGhELOnP+/cn6QDQ/ AHIJbRFkciiSTJqnL83B2Rel9q8i5FMHBw7xOleDIGdyriG7lS6qdPZmGSrmaxB+2NxPTXom1l8tb 2DOlStE1KBjJ4jzfoMtuHILciqdXO0Ds01lA5EGdPeFA97DGkDbVBK7uF924MF4/8kMd6QlTWqmGx 8pUowuz0FYuMn7RmBeydSzB9xh27atnwlDcIU3AH0C0Tqt8pEX4NOt52Sfv/dAMmnRdx3b2bTz01N m793bFrg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oLPnQ-004GQX-C5; Tue, 09 Aug 2022 14:01:24 +0000 Received: from sin.source.kernel.org ([2604:1380:40e1:4800::1]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oLPnN-004GNm-NW for linux-arm-kernel@lists.infradead.org; Tue, 09 Aug 2022 14:01:23 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sin.source.kernel.org (Postfix) with ESMTPS id BA3D1CE1718; Tue, 9 Aug 2022 14:01:13 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 0D94DC433C1; Tue, 9 Aug 2022 14:01:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1660053672; bh=u+a4xxI8QuroONub9HjlQCWDta7sG4zKmZv7pm4e4YQ=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=BqCEXUMORsYoPmP0TX81BzFZY+JxVRuIDNIJDwJyMmP1ljzVK0zj3ajutrpW6JxP3 v9AcpNw3Jd+XSv3AcYY5JxJ3WjdLf8tylmnTHCMzBAJmuoWWNQ1wSz4uh8G+BcR7H7 qzp31MJLYX9n2DH5YKN3sMJDHK9WTvgZSceadpRZwuCsxLYct7IQX4R9GIMTt4h0iz 2Cmm0vkMz+q23/7NHrWRw9dtTkAY+pHpPQr/5h78SGZFNOdRetRdf5bA8m0gL5czd6 layPBfn/2nP/+JtU/3GNMs7icW9SY8QkXZO3IbCrXv5UdOOGZF0uXBSw38UVPIjojv Kzn2k3i4UllqA== Date: Tue, 9 Aug 2022 10:01:10 -0400 From: Sasha Levin To: Ard Biesheuvel Cc: Greg Kroah-Hartman , linux-kernel@vger.kernel.org, stable@vger.kernel.org, Anshuman Khandual , Will Deacon , catalin.marinas@arm.com, bp@suse.de, Jason@zx2c4.com, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH AUTOSEL 5.19 05/58] arm64: kernel: drop unnecessary PoC cache clean+invalidate Message-ID: References: <20220808013118.313965-1-sashal@kernel.org> <20220808013118.313965-5-sashal@kernel.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220809_070121_976853_6FA8717C X-CRM114-Status: GOOD ( 18.59 ) 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-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, Aug 08, 2022 at 11:05:29AM +0200, Ard Biesheuvel wrote: >On Mon, 8 Aug 2022 at 03:31, Sasha Levin wrote: >> >> From: Ard Biesheuvel >> >> [ Upstream commit 2e945851e26836c0f2d34be3763ddf55870e49fe ] >> >> Some early boot code runs before the virtual placement of the kernel is >> finalized, and we used to go back to the very start and recreate the ID >> map along with the page tables describing the virtual kernel mapping, >> and this involved setting some global variables with the caches off. >> >> In order to ensure that global state created by the KASLR code is not >> corrupted by the cache invalidation that occurs in that case, we needed >> to clean those global variables to the PoC explicitly. >> >> This is no longer needed now that the ID map is created only once (and >> the associated global variable updates are no longer repeated). So drop >> the cache maintenance that is no longer necessary. >> >> Signed-off-by: Ard Biesheuvel >> Reviewed-by: Anshuman Khandual >> Link: https://lore.kernel.org/r/20220624150651.1358849-9-ardb@kernel.org >> Signed-off-by: Will Deacon >> Signed-off-by: Sasha Levin > >NAK > >This patch *must* *not* be backported. It will break the boot. Appologies for this one, this was a technical issue on my end and I owe a beer for yourself and few other folks that should have been filtered out. I'll drop all your patches from the AUTOSEL queue. -- Thanks, Sasha _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel