<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Hi Richard,</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Sep 24, 2020 at 7:11 PM Richard Palethorpe <<a href="mailto:rpalethorpe@suse.com" target="_blank">rpalethorpe@suse.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">It is not possible to use a controller in V2 cgroups if it has been<br>
mounted as a V1 controller. So if V1 is mounted we use it regardless<br>
of if V2 is available.<br>
<br>
We have to include a space in tst_is_mounted so that we do not match<br>
cgroup2.<br>
<br>
Signed-off-by: Richard Palethorpe <<a href="mailto:rpalethorpe@suse.com" target="_blank">rpalethorpe@suse.com</a>><br>
---<br>
 lib/tst_cgroup.c | 2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/lib/tst_cgroup.c b/lib/tst_cgroup.c<br>
index ba413d874..73ddd4b82 100644<br>
--- a/lib/tst_cgroup.c<br>
+++ b/lib/tst_cgroup.c<br>
@@ -44,7 +44,7 @@ enum tst_cgroup_ver tst_cgroup_version(void)<br>
         enum tst_cgroup_ver cg_ver;<br>
<br>
         if (tst_cgroup_check("cgroup2")) {<br>
-                if (!tst_is_mounted("cgroup2") && tst_is_mounted("cgroup"))<br>
+                if (tst_is_mounted("cgroup "))<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Add a space in the suffix still not work as expected.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">The reason is that tst_is_mounted("cgroup ") also get non-zero return if system only mount cgroup_v2, which lead to choose cgroup_v1 in LTP test.</div></div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small"># cat /proc/mounts |grep cgroup<br>cgroup2 /sys/fs/cgroup cgroup2 rw,seclabel,nosuid,nodev,noexec,relatime 0 0<br></div><div><br></div></div>-- <br><div dir="ltr"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>