From mboxrd@z Thu Jan 1 00:00:00 1970 From: Catalin Marinas Subject: Re: [PATCH v3 01/23] arm64: alternative: Allow alternative_insn to always issue the first instruction Date: Mon, 4 May 2020 15:47:44 +0100 Message-ID: <20200504144743.GB2884@gaia> References: <20200421142603.3894-1-catalin.marinas@arm.com> <20200421142603.3894-2-catalin.marinas@arm.com> <20200427165737.GD15808@arm.com> <20200428114354.GE3868@gaia> <20200429102600.GA30377@arm.com> <20200429140425.GC10651@gaia> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from foss.arm.com ([217.140.110.172]:46512 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728399AbgEDOrx (ORCPT ); Mon, 4 May 2020 10:47:53 -0400 Content-Disposition: inline In-Reply-To: <20200429140425.GC10651@gaia> Sender: linux-arch-owner@vger.kernel.org List-ID: To: Dave Martin Cc: linux-arch@vger.kernel.org, Richard Earnshaw , Szabolcs Nagy , Andrey Konovalov , Kevin Brodsky , Peter Collingbourne , linux-mm@kvack.org, Vincenzo Frascino , Will Deacon , linux-arm-kernel@lists.infradead.org On Wed, Apr 29, 2020 at 03:04:25PM +0100, Catalin Marinas wrote: > From 73f3869cb68fab1505d7b400ae8a39a19c5fc9e9 Mon Sep 17 00:00:00 2001 > From: Catalin Marinas > Date: Wed, 27 Nov 2019 09:07:30 +0000 > Subject: [PATCH] arm64: alternative: Always emit the first instruction in > ALTERNATIVE blocks > > Currently with the ALTERNATIVE macro or alternative_insn, the cfg (or > enable) arguments disable the entire asm block. Change the macros to > only omit the alternative block on !IS_ENABLED(cfg). In addition, remove > the cfg arguments to to ALTERNATIVE in those few calls where it is still > passed. There is no change to the resulting kernel image with defconfig. > > alternative_insn's enable argument will be used in a subsequent patch > and we are keeping the ALTERNATIVE C macro arguments in line with the > asm version. > > Signed-off-by: Catalin Marinas > Cc: Will Deacon I no longer need this patch as I moved the page tag zeroing from clear_page to the actual mte.S file (called via set_pte_at()). So don't bother reviewing this. -- Catalin