<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jun 29, 2020 at 4:06 PM Yang Xu <<a href="mailto:xuyang2018.jy@cn.fujitsu.com">xuyang2018.jy@cn.fujitsu.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">When I test ksm04 on 3.10.0-1136.el7.x86_64, it failed as below:<br>
 ./ksm04<br>
tst_test.c:1247: INFO: Timeout per run is 0h 05m 00s<br>
tst_device.c:417: INFO: No device is mounted at /tmp/cgroup_mem<br>
tst_cgroup.c:96: INFO: Cgroup(memory) v1 mount at /tmp/cgroup_mem success<br>
tst_test.c:1292: BROK: Test killed by SIGSEGV<br>
<br>
dmesg as below:<br>
[10096.547468] traps: ksm04[25785] general protection ip:410673 sp:7ffc1b1dc870 error:0 in ksm04[400000+22000]<br>
<br>
addr2line -e ksm04 -f 410673<br>
tst_cgroup_set_path<br>
/root/ltp/lib/tst_cgroup.c:199<br>
<br>
It detects uninitialized mem when using a->next, so it was killed.<br>
<br>
Also ksm04_1 and oom5 have the same problem.<br>
<br>
Signed-off-by: Yang Xu <<a href="mailto:xuyang2018.jy@cn.fujitsu.com" target="_blank">xuyang2018.jy@cn.fujitsu.com</a>><br>
---<br>
 lib/tst_cgroup.c | 1 +<br>
 1 file changed, 1 insertion(+)<br>
<br>
diff --git a/lib/tst_cgroup.c b/lib/tst_cgroup.c<br>
index 341ecf599..ba420d5f8 100644<br>
--- a/lib/tst_cgroup.c<br>
+++ b/lib/tst_cgroup.c<br>
@@ -190,6 +190,7 @@ static void tst_cgroup_set_path(const char *cgroup_dir)<br>
        tst_cgroup_path = SAFE_MALLOC(sizeof(struct tst_cgroup_path));<br>
        tst_cgroup_path->mnt_path = SAFE_MALLOC(strlen(cgroup_dir) + 1);<br>
        tst_cgroup_path->new_path = SAFE_MALLOC(strlen(cgroup_new_dir) + 1);<br>
+       tst_cgroup_path->next = NULL;<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">I remember I have fixed this in my local branch(probably v4), but it gets lost in patch merging.</div></div><div class="gmail_default" style="font-size:small">Anyway, thanks for fixing, pushed.</div><div> </div></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>