<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">On Thu, Sep 24, 2020 at 10:06 PM Richard Palethorpe <<a href="mailto:rpalethorpe@suse.de">rpalethorpe@suse.de</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">Hello Li,<br>
<br>
Li Wang <<a href="mailto:liwang@redhat.com" target="_blank">liwang@redhat.com</a>> writes:<br>
<br>
> Hi Richard,<br>
><br>
> On Thu, Sep 24, 2020 at 7:11 PM Richard Palethorpe <<a href="mailto:rpalethorpe@suse.com" target="_blank">rpalethorpe@suse.com</a>><br>
> wrote:<br>
><br>
>> 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") &&<br>
>> tst_is_mounted("cgroup"))<br>
>> +                if (tst_is_mounted("cgroup "))<br>
>><br>
><br>
> Add a space in the suffix still not work as expected.<br>
><br>
> The reason is that tst_is_mounted("cgroup ") also get non-zero return if<br>
> system only mount cgroup_v2, which lead to choose cgroup_v1 in LTP test.<br>
><br>
> # cat /proc/mounts |grep cgroup<br>
> cgroup2 /sys/fs/cgroup cgroup2 rw,seclabel,nosuid,nodev,noexec,relatime 0 0<br>
<br>
I wonder if it would be better to simply try mounting/using V2 and if<br>
that fails try V1?<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">That will be work but not good, because if cgroup mount fails, how do we know it is a bug or not support?</div></div><div> </div></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>