----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67923/#review206128 -----------------------------------------------------------
Ship it! LGTM! Thanks Ben. I think longer term, we will modify caller to call `verify`, making read/write/create pure helper for writing to cgroup filesystem. - Jie Yu On July 16, 2018, 2:03 a.m., Benjamin Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67923/ > ----------------------------------------------------------- > > (Updated July 16, 2018, 2:03 a.m.) > > > Review request for mesos, Gilbert Song and Jie Yu. > > > Bugs: MESOS-8418 > https://issues.apache.org/jira/browse/MESOS-8418 > > > Repository: mesos > > > Description > ------- > > It turns out that cgroups::verify is expensive and leads to a severe > performance issue on the agent during container metrics collection > if there are a lot of containers on the agent. See MESOS-8418. > > Since cgroups::verify serves to provide a helpful error message, > this patch preserves the error message, but only if the read fails. > > Longer term, there probably needs to be some re-structuring of the > code to make verification caller-controlled, or perhaps the verify > code can occur consistently post-operation (as done in this patch), > or perhaps verify can be optimized substantially. > > > Diffs > ----- > > src/linux/cgroups.cpp b12e63c112a7aa7a6f7150359ff5409f8214067e > > > Diff: https://reviews.apache.org/r/67923/diff/1/ > > > Testing > ------- > > Ran through internal CI. > > > Thanks, > > Benjamin Mahler > >