Merge "first_stage_init: add support to skip module load failures" into rvc-d1-dev am: 427e663b96

Change-Id: I661d19db45513f1c5c3a943b8bbbc2b482fdea5a
This commit is contained in:
TreeHugger Robot 2020-05-01 23:43:01 +00:00 committed by Automerger Merge Worker
commit 9f42e0d770
3 changed files with 24 additions and 6 deletions

View file

@ -16,6 +16,7 @@
#include "first_stage_console.h"
#include <stdio.h>
#include <sys/stat.h>
#include <sys/sysmacros.h>
#include <sys/types.h>
@ -87,8 +88,18 @@ void StartConsole() {
_exit(127);
}
bool FirstStageConsole(const std::string& cmdline) {
return cmdline.find("androidboot.first_stage_console=1") != std::string::npos;
int FirstStageConsole(const std::string& cmdline) {
auto pos = cmdline.find("androidboot.first_stage_console=");
if (pos != std::string::npos) {
int val = 0;
if (sscanf(cmdline.c_str() + pos, "androidboot.first_stage_console=%d", &val) != 1) {
return FirstStageConsoleParam::DISABLED;
}
if (val <= FirstStageConsoleParam::MAX_PARAM_VALUE && val >= 0) {
return val;
}
}
return FirstStageConsoleParam::DISABLED;
}
} // namespace init

View file

@ -21,8 +21,15 @@
namespace android {
namespace init {
enum FirstStageConsoleParam {
DISABLED = 0,
CONSOLE_ON_FAILURE = 1,
IGNORE_FAILURE = 2,
MAX_PARAM_VALUE = IGNORE_FAILURE,
};
void StartConsole();
bool FirstStageConsole(const std::string& cmdline);
int FirstStageConsole(const std::string& cmdline);
} // namespace init
} // namespace android

View file

@ -200,16 +200,16 @@ int FirstStageMain(int argc, char** argv) {
}
Modprobe m({"/lib/modules"}, module_load_file);
auto want_console = ALLOW_FIRST_STAGE_CONSOLE && FirstStageConsole(cmdline);
auto want_console = ALLOW_FIRST_STAGE_CONSOLE ? FirstStageConsole(cmdline) : 0;
if (!m.LoadListedModules(!want_console)) {
if (want_console) {
if (want_console != FirstStageConsoleParam::DISABLED) {
LOG(ERROR) << "Failed to load kernel modules, starting console";
} else {
LOG(FATAL) << "Failed to load kernel modules";
}
}
if (want_console) {
if (want_console == FirstStageConsoleParam::CONSOLE_ON_FAILURE) {
StartConsole();
}