debugd: cros_ec_panicinfo missing from feedback reports |
|||||||||||
Issue descriptionTurns out "/bin/cat /sys/kernel/debug/cros_ec/panicinfo 2> /dev/null" cannot be run as non-priviledged user, so we always miss the panic information. This is because /sys/kernel/debug is only accessible by user in debugfs-access group.
,
Sep 5 2016
Wrongly referenced the old bug in https://chromium-review.googlesource.com/#/c/379678/, requesting merge to 54.
,
Sep 6 2016
Your change meets the bar and is auto-approved for M54 (branch: 2840)
,
Sep 6 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/platform2/+/0b522b85877d3b42ff11e2df57eb271bd3aa9c7d commit 0b522b85877d3b42ff11e2df57eb271bd3aa9c7d Author: Nicolas Boichat <drinkcat@google.com> Date: Thu Sep 01 02:36:46 2016 debugd: log_tool: Change group to kDebugfsGroup to obtain EC panicinfo /sys/kernel/debug is only accessible by user in debugfs-access group, even though the panicinfo file itself is world-readable. BUG= chromium:527904 BUG= chromium:643031 TEST=File feedback, panicinfo is present Change-Id: I98fff3a511b944639bab10f78db8581923c4a32e Previous-Reviewed-on: https://chromium-review.googlesource.com/379678 (cherry picked from commit f59e261a0cfd99b36a6772dcc266b915ad3baa26) Reviewed-on: https://chromium-review.googlesource.com/381755 Reviewed-by: Nicolas Boichat <drinkcat@chromium.org> Commit-Queue: Nicolas Boichat <drinkcat@chromium.org> Tested-by: Nicolas Boichat <drinkcat@chromium.org> [modify] https://crrev.com/0b522b85877d3b42ff11e2df57eb271bd3aa9c7d/debugd/src/log_tool.cc
,
Sep 6 2016
,
Sep 9 2016
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Sep 9 2016
,
Mar 4 2017
,
Apr 17 2017
,
May 30 2017
,
Aug 1 2017
,
Oct 14 2017
|
|||||||||||
►
Sign in to add a comment |
|||||||||||
Comment 1 by drinkcat@chromium.org
, Sep 1 2016