diff options
author | Alexey Kardashevskiy <aik@ozlabs.ru> | 2019-06-25 14:29:04 +1000 |
---|---|---|
committer | Stewart Smith <stewart@linux.ibm.com> | 2019-06-27 11:35:22 +1000 |
commit | d2005818bea35e74b8991a615ac5bee389263126 (patch) | |
tree | 31879c389fa27ad2b4eb6ed8f8acafa5fd961c26 /hw | |
parent | 6e049e3c4fc5a046fc50bc38a2ea0826b18a173c (diff) | |
download | skiboot-d2005818bea35e74b8991a615ac5bee389263126.zip skiboot-d2005818bea35e74b8991a615ac5bee389263126.tar.gz skiboot-d2005818bea35e74b8991a615ac5bee389263126.tar.bz2 |
npu2: Increase timeout for L2/L3 cache purging
On NVLink2 bridge reset, we purge all L2/L3 caches in the system.
This is an asynchronous operation, we have a 2ms timeout here. There are
reports that this is not enough and "PURGE L3 on core xxx timed out"
messages appear (for the reference: on the test setup this takes
280us..780us).
This defines the timeout as a macro and changes this from 2ms to 20ms.
This adds a tracepoint to tell how long it took to purge all the caches.
Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
Signed-off-by: Stewart Smith <stewart@linux.ibm.com>
Diffstat (limited to 'hw')
-rw-r--r-- | hw/npu2.c | 20 |
1 files changed, 13 insertions, 7 deletions
@@ -336,7 +336,7 @@ static int wait_l2_purge(uint32_t chip_id, uint32_t core_id) uint64_t val; uint64_t addr = XSCOM_ADDR_P9_EX(core_id, L2_PRD_PURGE_CMD_REG); unsigned long now = mftb(); - unsigned long end = now + msecs_to_tb(2); + unsigned long end = now + msecs_to_tb(L2_L3_PRD_PURGE_TIMEOUT_MS); int rc; while (1) { @@ -386,7 +386,7 @@ static int wait_l3_purge(uint32_t chip_id, uint32_t core_id) uint64_t val; uint64_t addr = XSCOM_ADDR_P9_EX(core_id, L3_PRD_PURGE_REG); unsigned long now = mftb(); - unsigned long end = now + msecs_to_tb(2); + unsigned long end = now + msecs_to_tb(L2_L3_PRD_PURGE_TIMEOUT_MS); int rc; /* Trigger bit is automatically set to zero when flushing is done */ @@ -414,6 +414,7 @@ static int64_t purge_l2_l3_caches(void) struct cpu_thread *t; uint64_t core_id, prev_core_id = (uint64_t)-1; int rc; + unsigned long now = mftb(); for_each_ungarded_cpu(t) { /* Only need to do it once per core chiplet */ @@ -423,10 +424,10 @@ static int64_t purge_l2_l3_caches(void) prev_core_id = core_id; rc = start_l2_purge(t->chip_id, core_id); if (rc) - return rc; + goto trace_exit; rc = start_l3_purge(t->chip_id, core_id); if (rc) - return rc; + goto trace_exit; } prev_core_id = (uint64_t)-1; @@ -439,12 +440,17 @@ static int64_t purge_l2_l3_caches(void) rc = wait_l2_purge(t->chip_id, core_id); if (rc) - return rc; + goto trace_exit; rc = wait_l3_purge(t->chip_id, core_id); if (rc) - return rc; + goto trace_exit; } - return OPAL_SUCCESS; + +trace_exit: + prlog(PR_TRACE, "L2/L3 purging took %ldus\n", + tb_to_usecs(mftb() - now)); + + return rc; } static int64_t npu2_dev_cfg_exp_devcap(void *dev, |