<div dir="ltr"><div>Hi Li, <br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jan 11, 2022 at 4:43 AM Li Wang <<a href="mailto:liwang@redhat.com">liwang@redhat.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">Luke Nowakowski-Krijger <<a href="mailto:luke.nowakowskikrijger@canonical.com" target="_blank">luke.nowakowskikrijger@canonical.com</a>> wrote:<br>
<br>
>  tools/cgroup/Makefile    |  7 ++++<br>
>  tools/cgroup/tst_cgctl.c | 69 ++++++++++++++++++++++++++++++++++++++++<br>
<br>
Looks like putting this tst_cgctl.c in testcase/lib/ will be better,<br>
we have no necessary to create it under a bit far directory, and<br>
that tool/ is more generic for LTP, but this process is only for<br>
shell tests.<br>
<br></blockquote><div><br></div><div>Yeah I will move it over here. I was not sure exactly where it should go.  <br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> +static int cgctl_cleanup(const char *config)<br>
> +{<br>
> +    tst_cgroup_scan();<br>
> +    tst_cgroup_load_config(config);<br>
<br>
This tst_cgroup_load_config() does not work as expected.<br>
>From my manual check, the ltp and drain dir have been created<br>
but it prints " Created_Ltp_Dir=no Created_Drain_Dir=no" strings.<br>
<br>
...<br>
Detected Roots:<br>
/sys/fs/cgroup/memory Mounted_Root=no Created_Ltp_Dir=no<br>
Created_Drain_Dir=no Test_Id=test-1801<br>
...<br>
<br>
<br></blockquote><div><br></div><div>Hm, I'm not sure what this could be. Are you sure when you were reading the printed info it was in the same invocation as when it was being created? Because the tst_cgroup_print_config is just pretty directly printing out the state of the cgroup framework.</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> +    tst_cgroup_cleanup();<br>
<br>
This does not work as expected too, but the problem should exist<br>
in previous patches. Anyway, I will look into the details tomorrow.<br>
<br>
# ./tst_cgctl cleanup "$_cgroup_state"<br>
tst_cgroup.c:414: TBROK: Could not find root from config. Roots<br>
changing between calls?<br>
<br></blockquote><div><br></div><div>Was "/sys/fs/cgroup/cpu,cpuacct" one of the roots that was printed out? Because if so the way I have it now it would not be able to parse that. I will look into fixing this.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
> +<br>
> +    return 0;<br>
> +}<br>
<br>
<br>
--<br>
Regards,<br>
Li Wang<br>
<br></blockquote><div><br></div><div>Thanks again for review,</div><div><br></div><div>- Luke<br></div></div></div>