<div dir="ltr"><div class="gmail_quote"><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> What about declaring as 'tst_cgroup_test' ? i.e.<br>
><br>
>     const struct tst_cgroup_group *const tst_cgroup_test = &test_group;<br>
><br>
> As it is a reference to test's CGroup and the test_dir is the same<br>
> layer with drain_dir, so this will be easier to understand the relationship<br>
> with drain_group.<br>
><br>
>  +const struct tst_cgroup_group *const tst_cgroup_drain =<br>
>  &drain_group;<br>
<br>
I agree with your logic, but the variable name is too long even without<br>
'_test'. Perhaps we could shorten cgroup to cg? However I can submit a<br>
separate patch for that.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Ok, I agree with shortening that. Thanks!</div><br></div></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>