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:10:21 +1000 |
commit | aafb5ced8cb6404a1dc91b93c92bc0b013b7e733 (patch) | |
tree | 2b387e3843d0ba83b42ba88a13294c6586e6bc30 | |
parent | c70c825dc8ff1fb309aa678eebe10300fb3c832a (diff) | |
download | skiboot-aafb5ced8cb6404a1dc91b93c92bc0b013b7e733.zip skiboot-aafb5ced8cb6404a1dc91b93c92bc0b013b7e733.tar.gz skiboot-aafb5ced8cb6404a1dc91b93c92bc0b013b7e733.tar.bz2 |
centaur: print message on disabling xscoms to centaur due to many errors
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
(cherry picked from commit aa341a3fbf23dc7c74a93ecff8662688a063cb8b)
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
-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); |