lkml.org 
[lkml]   [2019]   [Nov]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.14 025/211] powerpc/eeh: Fix use of EEH_PE_KEEP on wrong field
    Date
    From: Sam Bobroff <sbobroff@linux.ibm.com>

    [ Upstream commit 473af09b56dc4be68e4af33220ceca6be67aa60d ]

    eeh_add_to_parent_pe() sometimes removes the EEH_PE_KEEP flag, but it
    incorrectly removes it from pe->type, instead of pe->state.

    However, rather than clearing it from the correct field, remove it.
    Inspection of the code shows that it can't ever have had any effect
    (even if it had been cleared from the correct field), because the
    field is never tested after it is cleared by the statement in
    question.

    The clear statement was added by commit 807a827d4e74 ("powerpc/eeh:
    Keep PE during hotplug"), but it didn't explain why it was necessary.

    Signed-off-by: Sam Bobroff <sbobroff@linux.ibm.com>
    Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    arch/powerpc/kernel/eeh_pe.c | 2 +-
    1 file changed, 1 insertion(+), 1 deletion(-)

    diff --git a/arch/powerpc/kernel/eeh_pe.c b/arch/powerpc/kernel/eeh_pe.c
    index 8545a9523b9bc..7339ca4fdc19e 100644
    --- a/arch/powerpc/kernel/eeh_pe.c
    +++ b/arch/powerpc/kernel/eeh_pe.c
    @@ -381,7 +381,7 @@ int eeh_add_to_parent_pe(struct eeh_dev *edev)
    while (parent) {
    if (!(parent->type & EEH_PE_INVALID))
    break;
    - parent->type &= ~(EEH_PE_INVALID | EEH_PE_KEEP);
    + parent->type &= ~EEH_PE_INVALID;
    parent = parent->parent;
    }

    --
    2.20.1


    \
     
     \ /
      Last update: 2019-11-27 21:47    [W:2.379 / U:0.256 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site