<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">Richard Palethorpe <<a href="mailto:rpalethorpe@suse.de">rpalethorpe@suse.de</a>> wrote:<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">
>> I have done a direct conversion of the test cases to the new API, but<br>
>> I am not sure that it makes sense to call tst_cgroup_move_current<br>
>> within the run method of a test because after the first iteration it<br>
>><br>
><br>
> Hmm, I feel that is a rare scenario in our real test. Mostly we<br>
> just need to set it once in a process.<br>
<br>
I suppose we can just move it into setup then?<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">For tests that only need a one-time CGroup setting we can, but for</div><div class="gmail_default" style="font-size:small">multiple requirements of the same controller in many processes it</div><div class="gmail_default" style="font-size:small">seems not working.</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">
> That's true, but it is mainly caused by different versions of<br>
> CGroup. We could NOT unify the unsupported behavior, so<br>
> maybe the wiser choice is to let _CPUSET test skipping(TCONF)<br>
> directly on CGroup_V2?<br>
<br>
Yes. I wonder if tst_cgroup_move_current should return a value to<br>
indicate it was a NOP? Or maybe it should throw an error when called on<br>
CGroup_V2 and it would have been a NOP?<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">That sound like we have to handle the returned value/error additionally?</div><div class="gmail_default" style="font-size:small">If so, I'd prefer to encapsulate that in CGroup-lib.</div></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>