Message ID | 20170613044545.25834-1-hegdevasant@linux.vnet.ibm.com |
---|---|
State | Accepted |
Headers | show |
On Tue, Jun 13, 2017 at 10:15:45AM +0530, Vasant Hegde wrote: > Presently we print word0 and word1 in error log. word0 contains > sequence number and command class. One has to understand word0 > format to identify command class. > > Lets explicitly print command class, sub command etc. > > CC: Ananth N Mavinakayanahalli <ananth@linux.vnet.ibm.com> > Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com> > --- > Changes in v2: > Updated description. Note that we still have two messages in OPAL console > and we log event to FSP after R/R completes. Acked-by: Ananth N Mavinakayanahalli <ananth@linux.vnet.ibm.com>
Vasant Hegde <hegdevasant@linux.vnet.ibm.com> writes: > Presently we print word0 and word1 in error log. word0 contains > sequence number and command class. One has to understand word0 > format to identify command class. > > Lets explicitly print command class, sub command etc. > > CC: Ananth N Mavinakayanahalli <ananth@linux.vnet.ibm.com> > Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com> > --- > Changes in v2: > Updated description. Note that we still have two messages in OPAL console > and we log event to FSP after R/R completes. Merged to master as of 807a3acc8fd66af1e1c6e7154aa5029c9b91bb3b and 5.4.x as of b0e07980fb3454c68f196e943c65f4f1d8954450
diff --git a/hw/fsp/fsp.c b/hw/fsp/fsp.c index 162d9b4..4a8eaee 100644 --- a/hw/fsp/fsp.c +++ b/hw/fsp/fsp.c @@ -2168,8 +2168,8 @@ static void fsp_timeout_poll(void *data __unused) w1 = req->word1; mstate = req->state; prlog(PR_WARNING, "FSP: Response from FSP timed out," - " word0 = %x, word1 = %x state: %d\n", - w0, w1, mstate); + " cmd = %x subcmd = %x mod = %x state: %d\n", + w0 & 0xff, w1 & 0xff, (w1 >> 8) & 0xff, mstate); fsp_reg_dump(); fsp_cmdclass_resp_bitmask &= ~(1ull << index); cmdclass->timesent = 0; @@ -2180,8 +2180,9 @@ static void fsp_timeout_poll(void *data __unused) unlock(&fsp_lock); fsp_hir_reason_plid = log_simple_error( &e_info(OPAL_RC_FSP_POLL_TIMEOUT), - "FSP: Response from FSP timed out, word0 = %x," - "word1 = %x state: %d\n", w0, w1, mstate); + "FSP: Response from FSP timed out," + " cmd = %x subcmd = %x mod = %x state: %d\n", + w0 & 0xff, w1 & 0xff, (w1 >> 8) & 0xff, mstate); } next_bit: cmdclass_resp_bitmask = cmdclass_resp_bitmask >> 1;
Presently we print word0 and word1 in error log. word0 contains sequence number and command class. One has to understand word0 format to identify command class. Lets explicitly print command class, sub command etc. CC: Ananth N Mavinakayanahalli <ananth@linux.vnet.ibm.com> Signed-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com> --- Changes in v2: Updated description. Note that we still have two messages in OPAL console and we log event to FSP after R/R completes. -Vasant hw/fsp/fsp.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-)