powerpc/rtas: avoid device tree lookups in rtas_os_term()
authorNathan Lynch <nathanl@linux.ibm.com>
Fri, 18 Nov 2022 15:07:41 +0000 (09:07 -0600)
committerMichael Ellerman <mpe@ellerman.id.au>
Wed, 7 Dec 2022 11:22:22 +0000 (22:22 +1100)
commited2213bfb192ab51f09f12e9b49b5d482c6493f3
tree2974cedc87276e9010227961eeb8ee9c7effb838
parentb10af504a2015d12c566b6b0a4c7e3b602949eeb
powerpc/rtas: avoid device tree lookups in rtas_os_term()

rtas_os_term() is called during panic. Its behavior depends on a couple
of conditions in the /rtas node of the device tree, the traversal of
which entails locking and local IRQ state changes. If the kernel panics
while devtree_lock is held, rtas_os_term() as currently written could
hang.

Instead of discovering the relevant characteristics at panic time,
cache them in file-static variables at boot. Note the lookup for
"ibm,extended-os-term" is converted to of_property_read_bool() since it
is a boolean property, not an RTAS function token.

Signed-off-by: Nathan Lynch <nathanl@linux.ibm.com>
Reviewed-by: Nicholas Piggin <npiggin@gmail.com>
Reviewed-by: Andrew Donnellan <ajd@linux.ibm.com>
[mpe: Incorporate suggested change from Nick]
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/20221118150751.469393-4-nathanl@linux.ibm.com
arch/powerpc/kernel/rtas.c