wenchao-hao
/
lmkd
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

lmkd/tests: handle kill reports being confused with reap reports am: 13b5b0ce06 

Original change: https://android-review.googlesource.com/c/platform/system/memory/lmkd/+/3134320

Change-Id: I5dc709c99cf789242aad16b7902f714bdb5cec51
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Suren Baghdasaryan 2024-08-12 21:01:14 +00:00 committed by Automerger Merge Worker
parent ff8b8464d1 13b5b0ce06
commit 72c26015dd
1 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
tests/lmkd_tests.cpp
View File

@ -42,6 +42,7 @@ using namespace android::base;
#define LMKD_REAP_FAIL_TEMPLATE "process_mrelease %d failed" #define LMKD_REAP_FAIL_TEMPLATE "process_mrelease %d failed"
#define LMKD_KILL_LINE_START LMKD_LOGCAT_MARKER ": Kill" #define LMKD_KILL_LINE_START LMKD_LOGCAT_MARKER ": Kill"
#define LMKD_KILLED_LINE_START LMKD_LOGCAT_MARKER ": Process got killed"
#define LMKD_REAP_LINE_START LMKD_LOGCAT_MARKER ": Process" #define LMKD_REAP_LINE_START LMKD_LOGCAT_MARKER ": Process"
#define LMKD_REAP_TIME_TEMPLATE LMKD_LOGCAT_MARKER ": Process %d was reaped in %ldms" #define LMKD_REAP_TIME_TEMPLATE LMKD_LOGCAT_MARKER ": Process %d was reaped in %ldms"
#define LMKD_REAP_MRELESE_ERR_MARKER ": process_mrelease" #define LMKD_REAP_MRELESE_ERR_MARKER ": process_mrelease"
@ -223,8 +224,10 @@ TEST_F(LmkdTest, TargetReaping) {
long rss, swap; long rss, swap;
ASSERT_TRUE(ParseProcSize(line, rss, swap)) << "Kill report format is invalid"; ASSERT_TRUE(ParseProcSize(line, rss, swap)) << "Kill report format is invalid";
line_start = 0;
retry:
// find reap duration report // find reap duration report
line_start = logcat_out.find(LMKD_REAP_LINE_START); line_start = logcat_out.find(LMKD_REAP_LINE_START, line_start);
if (line_start == std::string::npos) { if (line_start == std::string::npos) {
// Target might have exited before reaping started // Target might have exited before reaping started
line_start = logcat_out.find(LMKD_REAP_MRELESE_ERR_MARKER); line_start = logcat_out.find(LMKD_REAP_MRELESE_ERR_MARKER);
@ -240,6 +243,11 @@ TEST_F(LmkdTest, TargetReaping) {
line_end = logcat_out.find('\n', line_start); line_end = logcat_out.find('\n', line_start);
line = logcat_out.substr( line = logcat_out.substr(
line_start, line_end == std::string::npos ? std::string::npos : line_end - line_start); line_start, line_end == std::string::npos ? std::string::npos : line_end - line_start);
if (line.find(LMKD_KILLED_LINE_START) != std::string::npos) {
// we found process kill report, keep looking for reaping report
line_start = line_end;
goto retry;
}
long reap_time; long reap_time;
ASSERT_TRUE(ParseReapTime(line, pid, reap_time) && reap_time >= 0) ASSERT_TRUE(ParseReapTime(line, pid, reap_time) && reap_time >= 0)
<< "Reaping time report format is invalid"; << "Reaping time report format is invalid";