Fix: remove break in epoll loop of apps. thread
authorDavid Goulet <dgoulet@efficios.com>
Wed, 18 Dec 2013 23:34:44 +0000 (18:34 -0500)
committerDavid Goulet <dgoulet@efficios.com>
Thu, 19 Dec 2013 17:30:01 +0000 (12:30 -0500)
commit226e8f15071226f3d7911fd213000f8959c4bc76
tree586dcaef51db15ddeea66f722f95ced831360f65
parent9e52b0d84210aaed7b6bb70709a6dafbffbe922c
Fix: remove break in epoll loop of apps. thread

In *heavy* stress test with a large number of applications (> 7000 a
second), the manage application thread could starve the delete process
by breaking just after adding an application to the poll set.

Also, we've observed that somehow the application unregister process is
not done on most of the application by breaking the loop at each delete
from the poll set. We are still uncertain why but one theory is that
epoll detects that an I/O operation is ready (here a shutdown) and an
other subsystem of the session daemon uses that socket for I/O which
flags the poll event as "has been taken care of" thus the loop never
sees it because of that break.

The notify socket thread does not use a break between poll operation
which leads us to that conclusion with the manage apps thread.

We don't use epoll with edge-trigger thus a POLLERR/POLLHUP should
always be return as long as it's not taken care of.

Acked-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Signed-off-by: David Goulet <dgoulet@efficios.com>
src/bin/lttng-sessiond/main.c
This page took 0.025454 seconds and 4 git commands to generate.