<div dir="ltr"><div>Hi Richard and Li, <br></div><div><br></div><div>Sorry for the hiatus and the delay on getting this done. There was some other stuff keeping me from finishing things up here.</div><div><br></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jan 24, 2022 at 4:22 AM 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 Luke,<br>
<br>
Luke Nowakowski-Krijger <<a href="mailto:luke.nowakowskikrijger@canonical.com" target="_blank">luke.nowakowskikrijger@canonical.com</a>> writes:<br>
<br>
> Implement tst_cgroup_load_config which consumes the state given by<br>
> tst_cgroup_print_config() to update the internal test state to reflect<br>
> the given config.<br>
><br>
> This allows for programs using the cgroup C API to load and reload<br>
> state, allowing functionality such as calling tst_cgroup_require and<br>
> tst_cgroup_cleanup to function properly between programs or between<br>
> invocations of a binary using the C API.<br>
<br>
I'm afraid I have to say this looks way more complicated than it needs<br>
to be. We control the input format after all.<br>
<br>
If you can make each line the same format then it may be possible to<br>
just use a single scanf on each line. Note that it's only possible to<br>
have ~14 controllers, so we can even afford to repeat the root info on<br>
each line.<br>
<br></blockquote><div><br></div><div>I have read through all the feedback and I agree with all of it. I implemented something along the lines of what you described here and I agree that it looks a lot better and much much more simple. Hopefully get that out to you soon to review. <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>
Thank you,<br>
Richard.<br></blockquote><div><br></div><div>Thanks, <br></div><div>Luke  <br></div></div></div>