New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 830531 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Crash: raise-1d5ae944 / powerd_setuid_helper.cc

Project Member Reported by kbleicher@google.com, Apr 9 2018

Issue description

Top Chrome OS bug with latest Dev RC

67% / 52000+ Crashes

Adding derat@ to cc per latest CLs in powerd_setuid_helper.cc

https://crash.corp.google.com/browse?q=product.name%3D%27ChromeOS%27%20AND%20stable_signature%3D%27raise-1d5ae944%27%20AND%20product.Version%3D%2710539.0.0%27%20AND%20expanded_custom_data.ChromeCrashProto.magic_signature_1.name%3D%27%5BAssert%5D%20RunCommand%27&stbtiq=&reportid=&index=0

Stack Quality97%Show frame trust levels
0x00007a1570393dd2	(libc-2.23.so -raise.c:54 )	raise
0x00007a1570395bf5	(libc-2.23.so -abort.c:89 )	abort
0x00007a15708c7784	(libbase-core-395517.so -debugger_posix.cc:249 )	base::debug::BreakDebugger()
0x00007a15708e6406	(libbase-core-395517.so -logging.cc:755 )	logging::LogMessage::~LogMessage()
0x00007a15708e64da	(libbase-core-395517.so -logging.cc:877 )	logging::ErrnoLogMessage::~ErrnoLogMessage()
0x0000576d6d90f8c7	(powerd_setuid_helper -powerd_setuid_helper.cc:51 )	RunCommand(char const*, char const*, ...)
0x0000576d6d9104a6	(powerd_setuid_helper -powerd_setuid_helper.cc:109 )	main
0x00007a1570380735	(libc-2.23.so -libc-start.c:289 )	__libc_start_main
0x0000576d6d90f428	(powerd_setuid_helper + 0x00001428 )	_start
0x00007fff2903a737		




 
Summary: Crash: raise-1d5ae944 / powerd_setuid_helper.cc (was: Crash: raise-1d5ae944 / [Assert] RunCommand)

Comment 3 by derat@chromium.org, Apr 9 2018

Cc: vpalatin@chromium.org
Components: OS>Kernel>Power
Owner: derat@chromium.org
Status: Started (was: Untriaged)
[0101/170002:FATAL:powerd_setuid_helper.cc(51)] Check failed: execvp(command, argv) != -1. execv() failed: No such file or directory

Looks like ectool is missing on some devices (unsurprisingly):

109    RunCommand("ectool", "forcelidopen", state, NULL);

This is harmless (apart from the noise from the crashes). I'll make the helper check for ectool first.

Comment 4 by derat@chromium.org, Apr 9 2018

Cc: benchan@chromium.org
Actually, I should make powerd unconditionally depend on the ec-utils package.
>  I should make powerd unconditionally depend on the ec-utils package.

Yes, else ectool is missing on some base images even on machines with an actual Chrome EC implementation.
 Issue 829578  has been merged into this issue.
Labels: Restrict-View-Google
Project Member

Comment 8 by bugdroid1@chromium.org, Apr 10 2018

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

commit 3c8e62b6286e867f870d28a3edab4f88bd067d7e
Author: Daniel Erat <derat@chromium.org>
Date: Tue Apr 10 23:29:19 2018

power_manager: Depend on chromeos-base/ec-utils.

Make the power_manager package depend on ec-utils. powerd
unconditionally uses ectool's forcelidopen command to try to
prevent the system from suspending when it reboots while in
docked mode now.

BUG= chromium:339612 , chromium:830531 
TEST=built the package

Change-Id: I0bcf7802947295197f51e2fe4f14109f05d391cf
Reviewed-on: https://chromium-review.googlesource.com/1002879
Commit-Ready: Dan Erat <derat@chromium.org>
Tested-by: Dan Erat <derat@chromium.org>
Reviewed-by: Ben Chan <benchan@chromium.org>

[modify] https://crrev.com/3c8e62b6286e867f870d28a3edab4f88bd067d7e/chromeos-base/power_manager/power_manager-9999.ebuild

Comment 9 by derat@chromium.org, Apr 10 2018

Status: Fixed (was: Started)

Comment 10 by dgagnon@google.com, Apr 12 2018

Labels: XAct
Cc: kirtika@chromium.org briannorris@chromium.org diand...@chromium.org grundler@chromium.org
 Issue 832881  has been merged into this issue.
Labels: -Restrict-View-Google -Restrict-View-EditIssue

Sign in to add a comment