diff options
author | Mahesh Salgaonkar <mahesh@linux.vnet.ibm.com> | 2015-09-14 16:39:44 +0530 |
---|---|---|
committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2015-09-15 11:15:16 +1000 |
commit | 1764f2452565bc7203f6e4523a3ea59bfffc35ee (patch) | |
tree | ff96c2b73dad358be5cdd2a0678d83faf96fb800 /include | |
parent | 1311068c72f82a0ccfd8cb539d7f7f062d077a23 (diff) | |
download | skiboot-1764f2452565bc7203f6e4523a3ea59bfffc35ee.zip skiboot-1764f2452565bc7203f6e4523a3ea59bfffc35ee.tar.gz skiboot-1764f2452565bc7203f6e4523a3ea59bfffc35ee.tar.bz2 |
opal: Fix hang in time_wait* calls on HMI for TB errors.
On TOD/TB errors timebase register stops/freezes until HMI error recovery
gets TOD/TB back into running state. However, while HMI recovery is in
progress there are chances where some code path may invoke time_wait*()
calls which depends on running TB value. In an event of TB not moving,
time_wait* calls would keep looping resulting into a hang on that CPU.
On OpenPower systems we are seeing system hang on TOD/TB errors. The hang
is seen inside OPAL HMI handler while invoking prlog/perror(). The reason
is, on OpenPower systems prlog/perror() depends on LPC UART console
driver to flush log messages to the console. UART read/write calls invoke
time_wait_nopoll() inside opb_[read|write]() functions. When TB is in
stopped state this causes a hang in prlog/perror() calls.
This patch fixes this issue by modifying time_wait_[no]poll() to check
for TB validity and return immediately.
Signed-off-by: Mahesh Salgaonkar <mahesh@linux.vnet.ibm.com>
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'include')
-rw-r--r-- | include/cpu.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/include/cpu.h b/include/cpu.h index d2c1825..03a51f9 100644 --- a/include/cpu.h +++ b/include/cpu.h @@ -85,6 +85,7 @@ struct cpu_thread { uint32_t *core_hmi_state_ptr; /* Mask to indicate thread id in core. */ uint8_t thread_mask; + bool tb_invalid; }; /* This global is set to 1 to allow secondaries to callin, |