Code

bindings/perl/Collectd.pm: Improved the way access to @plugins is synchronized.
authorSebastian Harl <sh@tokkee.org>
Fri, 1 Aug 2008 22:50:18 +0000 (00:50 +0200)
committerFlorian Forster <octo@huhu.verplant.org>
Mon, 11 Aug 2008 14:54:32 +0000 (16:54 +0200)
commitdffb71cd6668e98f93c12da2ee2bd7a728d7292a
tree8b6d0b5cf5c9c6b8a258eebc5f8b90e352b6e069
parentf95e32a1284d9686593591ab80fa0fb7a0524d30
bindings/perl/Collectd.pm: Improved the way access to @plugins is synchronized.

So far, a lock has been placed on @plugins, so that no two threads could
access the list of plugins simultaneously. This could cause problems which in
certain situations could even lead to deadlocks. E.g. when using the perl
plugin in combination with the rrdtool plugin with debugging enabled one would
get hit by the following situation: the perl plugin holds the lock on @plugins
and then dispatches values to the rrdtool plugin from some Perl plugin's read
function. The rrdtool plugin then tries to acquire its cache lock. At the same
time some other plugin dispatches values to the rrdtool plugin as well and
this thread now holds the lock on the rrdtool cache. While holding that lock,
the rrdtool plugin might dispatch a debug logging message and thus calls the
perl plugin's log-callback which tries to get the lock on @plugins thus
causing a deadlock.

This has been resolved by the following two changes:

 * Restrict the lock to the list of plugins of one type. This allows to access
   e.g. read and log plugins in parallel.

 * Unlock the variable before calling the Perl callback function. This
   further prevents nested locks.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>
bindings/perl/Collectd.pm