<div dir="ltr"><div>Hi Li, <br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jan 24, 2022 at 1:44 AM Li Wang <<a href="mailto:liwang@redhat.com">liwang@redhat.com</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">Luke Nowakowski-Krijger <<a href="mailto:luke.nowakowskikrijger@canonical.com" target="_blank">luke.nowakowskikrijger@canonical.com</a>> wrote:<br>
<br>
As we already built the controller files mapping from V2 to V1<br>
in C library and you actually add many new (in patch 5/16).<br>
<br>
I'm thinking maybe we could make use of it in tst_cgctl.c to<br>
avoid handling these (in shell) separately.<br>
<br>
Something like:<br>
<br>
    # ./tst_cgctl  set  "$pid"  "cgroup.procs"  "$target_pid"<br>
    # ./tst_cgctl  set  "$pid"  "memory.max"  "$ACTIVE_MEM_LIMIT"<br>
<br>
Otherwise, it seems to make no sense to add so many new<br>
files mapping (like that patch 5/16) at this moment.<br>
<br>
What do you think?<br>
<br>
<br></blockquote><div>I think this would be nice except that we would need to keep track of the tst_cg_cgroup structs if we wanted to use safe_cg_* functions in the C lib. This would be fine if we only wanted to use control files in the test_dir but it gets more complicated if there are other directories below it that we would want to set. At least as far as I understand it.<br></div><div><br></div><div>And as Richard mentioned its probably a better idea to just only add the control files for controllers as we absolutely need them so this wouldn't be too useful. Plus I think it's easy enough from shell to do a version check and write to the right control file/directory directly.</div><div><br></div><div>So I personally don't think its as important, but I could see in the future implementing something like this so it mimics the C api. What do you think? <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">
-- <br>
Regards,<br>
Li Wang<br>
<br></blockquote><div><br></div><div>Apologies for the hiatus,  I know it's not easy to get back into review mode for patches you haven't thought about in a while. <br></div><div><br></div><div>Thanks, <br></div><div>- Luke<br></div></div></div>