> On Feb. 20, 2024, 4:36 a.m., Benjamin Mahler wrote: > > src/linux/cgroups2.hpp > > Lines 27-41 (patched) > > <https://reviews.apache.org/r/74873/diff/1/?file=2285174#file2285174line27> > > > > hm.. it doesn't seem like the caller needs to know about these, can you > > put them in the .cpp file?
Updated. - Devin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74873/#review226224 ----------------------------------------------------------- On Feb. 26, 2024, 8:58 p.m., Devin Leamy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/74873/ > ----------------------------------------------------------- > > (Updated Feb. 26, 2024, 8:58 p.m.) > > > Review request for mesos. > > > Repository: mesos > > > Description > ------- > > Make all cgroup2 controls names constants that are available from: > ``` > cgroups2::control::<control> > ``` > > > Diffs > ----- > > src/linux/cgroups2.cpp 6beb8dd2c43f548dd86be16e828b4abfe6fa3fd3 > > > Diff: https://reviews.apache.org/r/74873/diff/2/ > > > Testing > ------- > > > Thanks, > > Devin Leamy > >
