diff options
| author | Stewart Smith <stewart@linux.vnet.ibm.com> | 2016-09-14 15:09:02 +1000 |
|---|---|---|
| committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2016-09-14 15:09:02 +1000 |
| commit | aa341a3fbf23dc7c74a93ecff8662688a063cb8b (patch) | |
| tree | c783ad603a86baa439e6885e24d17ce292ca17c8 /hw | |
| parent | ebbd74db20521ebb9e7ed2a697f963492ddc3cec (diff) | |
| download | skiboot-aa341a3fbf23dc7c74a93ecff8662688a063cb8b.zip skiboot-aa341a3fbf23dc7c74a93ecff8662688a063cb8b.tar.gz skiboot-aa341a3fbf23dc7c74a93ecff8662688a063cb8b.tar.bz2 | |
centaur: print message on disabling xscoms to centaur due to many errors
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'hw')
| -rw-r--r-- | hw/centaur.c | 14 |
1 files changed, 13 insertions, 1 deletions
diff --git a/hw/centaur.c b/hw/centaur.c index 8b7311f..d577da3 100644 --- a/hw/centaur.c +++ b/hw/centaur.c @@ -336,8 +336,20 @@ int64_t centaur_xscom_read(uint32_t id, uint64_t pcb_addr, uint64_t *val) */ if (rc) { centaur->error_count++; - if (centaur->error_count > CENTAUR_ERR_OFFLINE_THRESHOLD) + if (centaur->error_count > CENTAUR_ERR_OFFLINE_THRESHOLD) { centaur->online = false; + /** + * @fwts-label CentaurOfflinedTooManyErrors + * @fwts-advice OPAL marked a Centaur (memory buffer) + * as offline due to CENTAUR_ERR_OFFLINE_THRESHOLD (10) + * consecutive errors on XSCOMs to this centaur. + * OPAL will now return OPAL_XSCOM_CTR_OFFLINED and not + * try any further XSCOMs. This is likely caused by + * some hardware issue or PRD recovery issue. + */ + prlog(PR_ERR, "CENTAUR: Offlined %x due to > %d consecutive XSCOM errors. No more XSCOMs to this centaur.\n", + id, CENTAUR_ERR_OFFLINE_THRESHOLD); + } } else centaur->error_count = 0; unlock(¢aur->lock); |