> On Aug. 3, 2016, 7:07 p.m., Chug Rolke wrote: > > src/qpid/sys/epoll/EpollPoller.cpp, line 552 > > <https://reviews.apache.org/r/50759/diff/1/?file=1461569#file1461569line552> > > > > Before these changes the code was very friendly to low power > > configurations by not waking up unnecessarily. Now it wakes up once a > > minute regardless which might be enough to never let the host get to sleep. > > > > How about waking up once an hour or once every three hours?
the test case causes memory growth of 50MB per hour. One minute is probably a decent balance. - Cliff ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50759/#review144647 ----------------------------------------------------------- On Aug. 4, 2016, 3:32 a.m., Cliff Jansen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50759/ > ----------------------------------------------------------- > > (Updated Aug. 4, 2016, 3:32 a.m.) > > > Review request for qpid and Andrew Stitcher. > > > Bugs: qpid-7373 > https://issues.apache.org/jira/browse/qpid-7373 > > > Repository: qpid-cpp > > > Description > ------- > > If a C++ broker is lightly loaded with many short lived connections such that > at least one worker thread remains unwoken from the epoll_wait, its > DeletionManager::ThreadStatus::handles grows without bound and the associated > handles retain a shared_ptr ref and never go away. > > This patch allows IO threads a maximum of one minute to accumulate > DeletionManager shared pointer references before resuming the wait loop which > involves calling > > PollerHandleDeletionManager.markAllUnusedInThisThread(); > > The overhead is small on brokers light thread load and non-existent on busy > brokers. > > > Diffs > ----- > > src/qpid/sys/epoll/EpollPoller.cpp 6fdf996 > > Diff: https://reviews.apache.org/r/50759/diff/ > > > Testing > ------- > > Jira test case, PollerTest.cpp > > > Thanks, > > Cliff Jansen > >
