lkml.org 
[lkml]   [2023]   [Mar]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v3 11/60] arm64: kaslr: drop special case for ThunderX in kaslr_requires_kpti()
    Date
    ThunderX is an obsolete platform that shipped without support for the
    EFI_RNG_PROTOCOL in its firmware. Now that we no longer misidentify
    small KASLR offsets as randomization being enabled, we can drop the
    explicit check for ThunderX as well, given that KASLR is known to be
    unavailable.

    Note that we never enable KPTI on these systems, in spite of what this
    function returns. However, using non-global mappings for code that is
    executable at EL1 is what tickles the erratum on these cores, regardless
    of whether KPTI is enabled or not, so non-global mappings should simply
    never be used here.

    Signed-off-by: Ard Biesheuvel <ardb@kernel.org>
    ---
    arch/arm64/kernel/cpu_errata.c | 2 +-
    arch/arm64/kernel/cpufeature.c | 12 ------------
    2 files changed, 1 insertion(+), 13 deletions(-)

    diff --git a/arch/arm64/kernel/cpu_errata.c b/arch/arm64/kernel/cpu_errata.c
    index 307faa2b4395ed9f..530bbd6a2f6331fd 100644
    --- a/arch/arm64/kernel/cpu_errata.c
    +++ b/arch/arm64/kernel/cpu_errata.c
    @@ -261,7 +261,7 @@ static const struct midr_range cavium_erratum_23154_cpus[] = {
    #endif

    #ifdef CONFIG_CAVIUM_ERRATUM_27456
    -const struct midr_range cavium_erratum_27456_cpus[] = {
    +static const struct midr_range cavium_erratum_27456_cpus[] = {
    /* Cavium ThunderX, T88 pass 1.x - 2.1 */
    MIDR_RANGE(MIDR_THUNDERX, 0, 0, 1, 1),
    /* Cavium ThunderX, T81 pass 1.0 */
    diff --git a/arch/arm64/kernel/cpufeature.c b/arch/arm64/kernel/cpufeature.c
    index 2e3e5513977733b7..e9a138b7e3b22cc7 100644
    --- a/arch/arm64/kernel/cpufeature.c
    +++ b/arch/arm64/kernel/cpufeature.c
    @@ -1621,18 +1621,6 @@ bool kaslr_requires_kpti(void)
    return false;
    }

    - /*
    - * Systems affected by Cavium erratum 24756 are incompatible
    - * with KPTI.
    - */
    - if (IS_ENABLED(CONFIG_CAVIUM_ERRATUM_27456)) {
    - extern const struct midr_range cavium_erratum_27456_cpus[];
    -
    - if (is_midr_in_range_list(read_cpuid_id(),
    - cavium_erratum_27456_cpus))
    - return false;
    - }
    -
    return kaslr_enabled();
    }

    --
    2.39.2
    \
     
     \ /
      Last update: 2023-03-27 00:47    [W:7.046 / U:0.196 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site