lkml.org 
[lkml]   [2020]   [Jun]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.4 235/261] powerpc/64s: Dont let DT CPU features set FSCR_DSCR
    Date
    From: Michael Ellerman <mpe@ellerman.id.au>

    commit 993e3d96fd08c3ebf7566e43be9b8cd622063e6d upstream.

    The device tree CPU features binding includes FSCR bit numbers which
    Linux is instructed to set by firmware.

    Whether that's a good idea or not, in the case of the DSCR the Linux
    implementation has a hard requirement that the FSCR_DSCR bit not be
    set by default. We use it to track when a process reads/writes to
    DSCR, so it must be clear to begin with.

    So if firmware tells us to set FSCR_DSCR we must ignore it.

    Currently this does not cause a bug in our DSCR handling because the
    value of FSCR that the device tree CPU features code establishes is
    only used by swapper. All other tasks use the value hard coded in
    init_task.thread.fscr.

    However we'd like to fix that in a future commit, at which point this
    will become necessary.

    Fixes: 5a61ef74f269 ("powerpc/64s: Support new device tree binding for discovering CPU features")
    Cc: stable@vger.kernel.org # v4.12+
    Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
    Link: https://lore.kernel.org/r/20200527145843.2761782-2-mpe@ellerman.id.au
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

    ---
    arch/powerpc/kernel/dt_cpu_ftrs.c | 8 ++++++++
    1 file changed, 8 insertions(+)

    --- a/arch/powerpc/kernel/dt_cpu_ftrs.c
    +++ b/arch/powerpc/kernel/dt_cpu_ftrs.c
    @@ -346,6 +346,14 @@ static int __init feat_enable_dscr(struc
    {
    u64 lpcr;

    + /*
    + * Linux relies on FSCR[DSCR] being clear, so that we can take the
    + * facility unavailable interrupt and track the task's usage of DSCR.
    + * See facility_unavailable_exception().
    + * Clear the bit here so that feat_enable() doesn't set it.
    + */
    + f->fscr_bit_nr = -1;
    +
    feat_enable(f);

    lpcr = mfspr(SPRN_LPCR);

    \
     
     \ /
      Last update: 2020-06-19 17:15    [W:4.100 / U:0.560 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site