diff mbox

PSI: Report a PEL for PSI timeout

Message ID 20150205121836.32018.47540.stgit@thinktux.in.ibm.com
State Superseded
Headers show

Commit Message

Ananth N Mavinakayanahalli Feb. 5, 2015, 12:18 p.m. UTC
We currently just log an error when we don't find an active PSI link
15 minutes after it went down. Add a PEL log, with sufficient severity
so it gets pushed to the administrator.

Signed-off-by: Ananth N Mavinakayanahalli <ananth@in.ibm.com>
---
 hw/psi.c           |   10 +++++++---
 include/errorlog.h |    1 +
 2 files changed, 8 insertions(+), 3 deletions(-)

Comments

Vasant Hegde Feb. 5, 2015, 12:58 p.m. UTC | #1
On 02/05/2015 05:48 PM, Ananth N Mavinakayanahalli wrote:
> We currently just log an error when we don't find an active PSI link
> 15 minutes after it went down. Add a PEL log, with sufficient severity
> so it gets pushed to the administrator.
> 
> Signed-off-by: Ananth N Mavinakayanahalli <ananth@in.ibm.com>

Tested-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com>

-Vasant
Ananth N Mavinakayanahalli Feb. 5, 2015, 2:41 p.m. UTC | #2
On Thu, Feb 05, 2015 at 06:28:43PM +0530, Vasant Hegde wrote:
> On 02/05/2015 05:48 PM, Ananth N Mavinakayanahalli wrote:
> > We currently just log an error when we don't find an active PSI link
> > 15 minutes after it went down. Add a PEL log, with sufficient severity
> > so it gets pushed to the administrator.
> > 
> > Signed-off-by: Ananth N Mavinakayanahalli <ananth@in.ibm.com>
> 
> Tested-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com>

Thank you Vasant!
diff mbox

Patch

diff --git a/hw/psi.c b/hw/psi.c
index 70403fd..0d68e2c 100644
--- a/hw/psi.c
+++ b/hw/psi.c
@@ -31,6 +31,7 @@ 
 #include <i2c.h>
 #include <timebase.h>
 #include <platform.h>
+#include <errorlog.h>
 
 static LIST_HEAD(psis);
 static u64 psi_link_timer;
@@ -43,6 +44,10 @@  static void psi_activate_phb(struct psi *psi);
 
 static struct lock psi_lock = LOCK_UNLOCKED;
 
+DEFINE_LOG_ENTRY(OPAL_RC_PSI_TIMEOUT, OPAL_PLATFORM_ERR_EVT, OPAL_PSI,
+		OPAL_PLATFORM_FIRMWARE,
+		OPAL_UNRECOVERABLE_ERR_LOSS_OF_FUNCTION, OPAL_NA, NULL);
+
 void psi_set_link_polling(bool active)
 {
 	printf("PSI: %sing link polling\n",
@@ -200,9 +205,8 @@  static void psi_link_poll(void *data __unused)
 				now + secs_to_tb(PSI_LINK_RECOVERY_TIMEOUT);
 
 		if (tb_compare(now, psi_link_timeout) == TB_AAFTERB) {
-			prerror("PSI: Timed out looking for a PSI link\n");
-
-			/* Log error to the host from here */
+			log_simple_error(&e_info(OPAL_RC_PSI_TIMEOUT),
+				"PSI: Link timeout -- loss of FSP\n");
 		}
 
 		/* Poll every 10 seconds */
diff --git a/include/errorlog.h b/include/errorlog.h
index 7879b7b..1bcc03e 100644
--- a/include/errorlog.h
+++ b/include/errorlog.h
@@ -268,6 +268,7 @@  enum opal_reasoncode {
 /* PSI */
 	OPAL_RC_PSI_INIT        = OPAL_PS | 0x10,
 	OPAL_RC_PSI_IRQ_RESET   = OPAL_PS | 0x11,
+	OPAL_RC_PSI_TIMEOUT	= OPAL_PS | 0X12,
 /* XSCOM */
 	OPAL_RC_XSCOM_RW		= OPAL_XS | 0x10,
 	OPAL_RC_XSCOM_INDIRECT_RW	= OPAL_XS | 0x11,