Issue 643062 link

Starred by 1 user

Issue metadata

Status:	Archived
Owner:	drinkcat@chromium.org
Closed:	Sep 2016
Cc:	█ rspangler@chromium.org gwendal@chromium.org █ djkurtz@chromium.org █ ejcaruso@chromium.org
EstimatedDays:	----
NextAction:	----
OS:	Chrome
Pri:	2
Type:	Bug
VerifyIn-55 VerifyIn-56 VerifyIn-57 VerifyIn-58 VerifyIn-59 VerifyIn-60 VerifyIn-61

ec: Write a tool to parse panicinfo from feedback reports

Project Member Reported by drinkcat@chromium.org, Sep 1 2016

Issue description

One item that was leftover from  Issue 527904  is a tool to parse EC panicinfo from feedback reports.

For example: a feedback may include a report such as: https://feedback.corp.google.com/product/208/neutron?lView=rd&lReport=19371262196

cros_ec_panicinfo=<base64>: AQINAHBOACALAAAAeAQAIGFmrd4JAAAACQAAALBXACAAAAAAAAAAAAAAAAAAAAAAkQkBCAEAAAAA
AAAAn5dGhQEAAACbl0aFCQAAAPrRAAAAAAAAIQ5kiiMcdt0AAAAAAAAAAAAAAAAAAAAAdAAAAFBu
YyE=

`ectool panicinfo` was able to parse it as:

Saved panic data:
=== HANDLER EXCEPTION: 0b ====== xPSR: 00000000 ===
r0 :00000001 r1 :00000000 r2 :8546979f r3 :00000001
r4 :dead6661 r5 :00000009 r6 :00000009 r7 :200057b0
r8 :00000000 r9 :00000000 r10:00000000 r11:00000000
r12:8546979b sp :20000478 lr :00000009 pc :0000d1fa

An offline tool could go further and resolve the PC address to the source code line.

Comment 1 by drinkcat@chromium.org, Sep 1 2016

CLs:
 - https://chromium-review.googlesource.com/379639
 - https://chromium-review.googlesource.com/#/c/379697/

Project Member

Comment 2 by bugdroid1@chromium.org, Sep 7 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/ec/+/107cb0df63efb527a470553b0a4e6f1f1ab17eb6

commit 107cb0df63efb527a470553b0a4e6f1f1ab17eb6
Author: Nicolas Boichat <drinkcat@google.com>
Date: Thu Sep 01 06:35:48 2016

util: Add ec_parse_panicinfo tool to parse binary panicinfo

To be able to parse binary panicinfo from feedback reports, we need
a host tool:
 - Move panicinfo generic parsing functions to a separate C file
 - Create a new host utility to parse panicinfo

BRANCH=none
BUG= chromium:643062 
TEST=base64 -d | ec_parse_panicinfo

Change-Id: Idd8560a2894f270d0ab3a9f654c333135759e57f
Reviewed-on: https://chromium-review.googlesource.com/379639
Commit-Ready: Nicolas Boichat <drinkcat@chromium.org>
Tested-by: Nicolas Boichat <drinkcat@chromium.org>
Reviewed-by: Randall Spangler <rspangler@chromium.org>

[add] https://crrev.com/107cb0df63efb527a470553b0a4e6f1f1ab17eb6/util/ec_panicinfo.c
[add] https://crrev.com/107cb0df63efb527a470553b0a4e6f1f1ab17eb6/util/ec_panicinfo.h
[modify] https://crrev.com/107cb0df63efb527a470553b0a4e6f1f1ab17eb6/util/build.mk
[modify] https://crrev.com/107cb0df63efb527a470553b0a4e6f1f1ab17eb6/util/ectool.c
[add] https://crrev.com/107cb0df63efb527a470553b0a4e6f1f1ab17eb6/util/ec_parse_panicinfo.c

Project Member

Comment 3 by bugdroid1@chromium.org, Sep 7 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/bf90295c4113b0f3a818c6797b62d112ab7b77da

commit bf90295c4113b0f3a818c6797b62d112ab7b77da
Author: Nicolas Boichat <drinkcat@google.com>
Date: Thu Sep 01 06:39:28 2016

ec-devutils: Build and install ec_parse_panicinfo in chroot

CQ-DEPEND=CL:379639
BUG= chromium:643062 
TEST=base64 -d | ec_parse_panicinfo

Change-Id: Ieffd6bc4730348a315b717f43ed132f76f2b182d
Reviewed-on: https://chromium-review.googlesource.com/379697
Commit-Ready: Nicolas Boichat <drinkcat@chromium.org>
Tested-by: Nicolas Boichat <drinkcat@chromium.org>
Reviewed-by: Randall Spangler <rspangler@chromium.org>

[modify] https://crrev.com/bf90295c4113b0f3a818c6797b62d112ab7b77da/chromeos-base/ec-devutils/ec-devutils-9999.ebuild