Re: [PATCH v3 01/23] arm64: alternative: Allow alternative_insn to always issue the first instruction

Catalin Marinas
2020-05-04T14:47:53Z

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 <catalin.marinas@arm.com>
> 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 <catalin.marinas@arm.com>
> Cc: Will Deacon <will@kernel.org>

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