<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">Li Wang <<a href="mailto:liwang@redhat.com">liwang@redhat.com</a>> wrote:<br></div><div dir="ltr" class="gmail_attr"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail_default" style="font-size:small">...</span><br><br>
+#define PATH_SYS_CGROUP                "/sys/fs/cgroup"<br>
+#define PATH_TMP_CG1_MEM       "/tmp/cg1_mem"<br>
+#define PATH_TMP_CG1_CST       "/tmp/cg1_cst"<br>
+#define PATH_TMP_CGROUP2       "/tmp/cgroup2" /* cgroup v2 has only single hierarchy */<br>
+<br></blockquote><div><br></div></div><div class="gmail_default" style="font-size:small">Seems there is no need to distinguish CG1 and CG2 mount directory</div><div class="gmail_default" style="font-size:small">in tst_cgroup.h, because we have introduced the 'tst_cgroup_mnt_path'</div><div class="gmail_default" style="font-size:small">to mount cgroup unified, it will choose cgroup versions automatically.</div><div><br></div><div class="gmail_default" style="font-size:small">So maybe we can change these above definitions to:</div><div class="gmail_default" style="font-size:small"><br></div>-#define PATH_SYS_CGROUP                "/sys/fs/cgroup"<br>-#define PATH_TMP_CG1_MEM       "/tmp/cg1_mem"<br>-#define PATH_TMP_CG1_CST       "/tmp/cg1_cst"<br>-#define PATH_TMP_CGROUP2       "/tmp/cgroup2" /* cgroup v2 has only single hierarchy */<br>+#define PATH_TMP_CGROUP_MEM    "/tmp/cgroup_mem"<br>+#define PATH_TMP_CGROUP_CST    "/tmp/cgroup_cst"<br><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>