<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Hi Richard,</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Richard Palethorpe <<a href="mailto:rpalethorpe@suse.com">rpalethorpe@suse.com</a>> wrote:<br></div></div><div class="gmail_quote"><div> </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">
--- a/include/tst_cgroup.h<br>
+++ b/include/tst_cgroup.h<br>
<span class="gmail_default" style="font-size:small">...</span><br>
<br>
 /* A Control Group in LTP's aggregated hierarchy */<br>
 struct tst_cgroup_group;<br>
<br>
+/* Populated with a reference to this tests's CGroup */<br>
+extern const struct tst_cgroup_group *const tst_cgroup;<br>
+extern const struct tst_cgroup_group *const tst_cgroup_drain;<br></blockquote><div> </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">
--- a/lib/tst_cgroup.c<br>
+++ b/lib/tst_cgroup.c<br>
@@ -18,7 +18,6 @@<br>
 #include "lapi/mount.h"<br>
 #include "lapi/mkdirat.h"<br>
 #include "tst_safe_file_at.h"<br>
-#include "tst_cgroup.h"<br>
<br>
 struct cgroup_root;<br>
<br>
@@ -138,6 +137,14 @@ struct tst_cgroup_group {<br>
        struct cgroup_dir *dirs[ROOTS_MAX + 1];<br>
 };<br>
<br>
+/* If controllers are required via the tst_test struct then this is<br>
+ * populated with the test's CGroup.<br>
+ */<br>
+static struct tst_cgroup_group test_group;<br>
+static struct tst_cgroup_group drain_group;<br>
+<span class="gmail_default" style="font-size:small"></span>const struct tst_cgroup_group *const tst_cgroup = &test_group;<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">What about declaring as 'tst_cgroup_test' ? i.e.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small"><div class="gmail_default"><span class="gmail_default">    </span>const struct tst_cgroup_group *const tst_cgroup_test = &test_group;</div><div class="gmail_default"><br></div></div><div class="gmail_default" style="font-size:small">As it is a reference to test's CGroup and the test_dir is the same</div><div class="gmail_default" style="font-size:small">layer with drain_dir, so this will be easier to understand the relationship</div><div class="gmail_default" style="font-size:small">with drain_group.</div><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">
+const struct tst_cgroup_group *const tst_cgroup_drain = &drain_group;</blockquote><div><br></div><div class="gmail_default" style="font-size:small">The rest part looks quite good.</div></div><div class="gmail_default" style="font-size:small">Reviewed-by: Li Wang <<a href="mailto:liwang@redhat.com">liwang@redhat.com</a>></div><div><br></div><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>