Skip to content
Snippets Groups Projects
Commit 698816ca authored by Stefan Schmidt's avatar Stefan Schmidt
Browse files

Merge branch 'preinit-fixup' into 'kirkstone'

Smaller fixups to openharmony-preinit script/procedure

See merge request eclipse/oniro-core/meta-openharmony!92
parents 1f853071 32b200f9
No related branches found
No related tags found
1 merge request!92Smaller fixups to openharmony-preinit script/procedure
Pipeline #14817 passed
......@@ -73,6 +73,9 @@ mkdir -p /data/vendor_ce
mkdir -p /data/vendor_de
mkdir -p /data/vendor/hardware
mkdir -p /data/weston
mkdir -p /data/log/hilog
mkdir -p /data/log/faultlog/faultlogger
mkdir -p /data/log/faultlog/temp
mkdir -p /usr/lib/dri
chmod -R 777 /data
......@@ -100,6 +103,9 @@ if systemctl -q is-enabled param.service; then
sleep "$STARTUP_CMD_SLEEP"
fi
# Explicitly set default value to silence error message about parameter not being set
setparam persist.ace.trace.enabled 0
# - pre-init stage
# - init stage
......@@ -304,12 +310,14 @@ fi
# "disabled" : 1
if systemctl -q is-enabled hdcd.service; then
setparam persist.hdc.port 35000
setparam persist.hdc.root 1 # We are running hdcd as root
setparam ro.hdc.secure 0 # Do not enable secure mode
sleep "$STARTUP_CMD_SLEEP"
/system/bin/hdcd -t &
fi
if systemctl -q is-enabled faultloggerd.service; then
/usr/bin/install -m 775 -d /data/log/faultlog /data/log/faultlog/faultlogger /data/log/faultlog/temp/
setparam ro.logsystem.usertype 6 # UserType::OVERSEAS_COMMERCIAL
/system/bin/faultloggerd &
fi
# SPDX-FileCopyrightText: Huawei Inc.
#
# SPDX-License-Identifier: Apache-2.0
This patch is for //base/startup/init_lite repository of OpenHarmony 3.0
When initializing sockets, no less than 2 info and 2 error messages are logged for each socket. It is caused by a codepath that allows for specifying a socket fd using an environment variable. When the variable is not defined, all 4 messages are logged. As this is really the normal case, I have changed all 4 messages to debug messages.
Signed-off-by: Esben Haabendal <esben@geanix.com>
Upstream-Status: Pending
diff --git a/interfaces/innerkits/socket/init_socket.c b/interfaces/innerkits/socket/init_socket.c
index be64939e993e..247ba5bfbbfe 100755
--- a/interfaces/innerkits/socket/init_socket.c
+++ b/interfaces/innerkits/socket/init_socket.c
@@ -39,10 +39,10 @@ static int GetControlFromEnv(const char *path, int length)
if (path == NULL || length <= 0) {
return -1;
}
- INIT_LOGI("GetControlFromEnv path is %s ", path);
+ INIT_LOGD("GetControlFromEnv path is %s ", path);
const char *val = getenv(path);
if (val == NULL) {
- INIT_LOGE("GetControlFromEnv val is null %d", errno);
+ INIT_LOGD("GetControlFromEnv %s val is null", path);
return -1;
}
errno = 0;
@@ -50,9 +50,9 @@ static int GetControlFromEnv(const char *path, int length)
if (errno) {
return -1;
}
- INIT_LOGI("GetControlFromEnv fd is %d ", fd);
+ INIT_LOGD("GetControlFromEnv fd is %d ", fd);
if (fcntl(fd, F_GETFD) < 0) {
- INIT_LOGE("GetControlFromEnv errno %d ", errno);
+ INIT_LOGE("GetControlFromEnv %s fnctl errno %d ", path, errno);
return -1;
}
return fd;
@@ -67,10 +67,10 @@ int GetControlSocket(const char *name)
if (snprintf_s(path, sizeof(path), sizeof(path) - 1, OHOS_SOCKET_ENV_PREFIX"%s", name) == -1) {
return -1;
}
- INIT_LOGI("GetControlSocket path is %s ", path);
+ INIT_LOGD("GetControlSocket path is %s ", path);
int fd = GetControlFromEnv(path, MAX_SOCKET_ENV_PREFIX_LEN);
if (fd < 0) {
- INIT_LOGE("GetControlFromEnv fail ");
+ INIT_LOGD("GetControlFromEnv fail ");
return -1;
}
struct sockaddr_un addr;
......@@ -62,6 +62,7 @@ SRC_URI += "file://appspawn-procps.patch;patchdir=${S}/base/startup/appspawn_sta
SRC_URI += "file://base_startup_appspawn_standard-disable-longProcName-resetting.patch;patchdir=${S}/base/startup/appspawn_standard"
SRC_URI += "file://test_xts_acts-Align-tests-list-with-mandatory-set.patch;patchdir=${S}/test/xts/acts"
SRC_URI += "file://init_lite-silence-GetControlFromEnv-spam.patch;patchdir=${S}/base/startup/init_lite"
SRC_URI += "file://param_service_standalone.patch;patchdir=${S}/base/startup/init_lite"
SRC_URI += "file://param_service-Add-to-startup-l2-part.patch;patchdir=${S}/base/startup/appspawn_standard"
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment