tequilaOS/platform_system_core
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Merge "ueventd: don't double fork firmware handlers"

Browse source 
am: 0f4fb5497a

Change-Id: I2a40b76e937853fd2f1eed61cc0fe4f6fa465ba2
This commit is contained in:
Tom Cherry 2017-07-06 02:10:57 +00:00 committed by android-build-merger
commit e944b7e0a4
2 changed files with 11 additions and 19 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
init/firmware_handler.cpp
View file

@ -110,31 +110,16 @@ void HandleFirmwareEvent(const Uevent& uevent) {
if (uevent.subsystem != "firmware" || uevent.action != "add") return;
// Loading the firmware in a child means we can do that in parallel...
// We double fork instead of waiting for these processes.
pid_t pid = fork();
auto pid = fork();
if (pid == -1) {
PLOG(ERROR) << "could not fork to process firmware event for " << uevent.firmware;
return;
}
if (pid == 0) {
pid = fork();
if (pid == -1) {
PLOG(ERROR) << "could not fork a sceond time to process firmware event for "
<< uevent.firmware;
_exit(EXIT_FAILURE);
}
if (pid == 0) {
Timer t;
ProcessFirmwareEvent(uevent);
LOG(INFO) << "loading " << uevent.path << " took " << t;
_exit(EXIT_SUCCESS);
}
Timer t;
ProcessFirmwareEvent(uevent);
LOG(INFO) << "loading " << uevent.path << " took " << t;
_exit(EXIT_SUCCESS);
}
waitpid(pid, nullptr, 0);
}
} // namespace init

7
init/ueventd.cpp
View file

@ -268,6 +268,13 @@ int ueventd_main(int argc, char** argv) {
cold_boot.Run();
}
// We use waitpid() in ColdBoot, so we can't ignore SIGCHLD until now.
signal(SIGCHLD, SIG_IGN);
// Reap and pending children that exited between the last call to waitpid() and setting SIG_IGN
// for SIGCHLD above.
while (waitpid(-1, nullptr, WNOHANG) > 0) {
}
uevent_listener.Poll([&device_handler](const Uevent& uevent) {
HandleFirmwareEvent(uevent);
device_handler.HandleDeviceEvent(uevent);