diff options
author | Sam Bobroff <sbobroff@linux.ibm.com> | 2018-11-29 14:16:37 +1100 |
---|---|---|
committer | Michael Ellerman <mpe@ellerman.id.au> | 2019-02-05 11:55:41 +1100 |
commit | 3376cb91ed908eb0728900894a77d8206574dbcd (patch) | |
tree | a1535ed3c352f3b5931b2a76dbe61ad3b9f39eb2 /arch/powerpc/kernel/eeh_sysfs.c | |
parent | 26b523356f49a0117c8f9e32ca98aa6d6e496e1a (diff) | |
download | linux-rt-3376cb91ed908eb0728900894a77d8206574dbcd.tar.gz |
powerpc/eeh: Cleanup eeh_pe_clear_frozen_state()
The 'clear_sw_state' parameter for eeh_pe_clear_frozen_state() is
redundant because it has no effect (except in the rare case of a
hardware error part way through unfreezing a tree of PEs, where it
would dangerously allow partial de-isolation before returning
failure).
It is passed down to __eeh_pe_clear_frozen_state(), and from there to
eeh_unfreeze_pe(), where it causes EEH_PE_ISOLATED to be removed
from the state of each PE during the traversal. However, when the
traversal finishes, EEH_PE_ISOLATED is unconditionally removed by a
call to eeh_pe_state_clear() regardless of the parameter's value.
So remove the flag and pass false to eeh_unfreeze_pe() (to avoid the
rare case described above, as it was before the flag was introduced).
Also, perform the recursion directly in the function and eliminate a
bit of boilerplate.
There should be no change in functionality, except as mentioned above.
Signed-off-by: Sam Bobroff <sbobroff@linux.ibm.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Diffstat (limited to 'arch/powerpc/kernel/eeh_sysfs.c')
0 files changed, 0 insertions, 0 deletions