[collectd] [PATCH] Linux ACPI thermal zone plugin

Michal Miroslaw mirq-linux at rere.qmqm.pl
Mon Jul 28 21:52:15 CEST 2008


On Mon, Jul 28, 2008 at 11:30:20AM +0200, Sebastian Harl wrote:
> On Mon, Jul 28, 2008 at 11:21:11AM +0200, Sebastian Harl wrote:
> [libacpi]
> > The library supports sysfs and procfs as well.
> Sorry, I was wrong - currently, the library supports procfs only. The
> sysfs support is not finished yet and only available in some VCS. :-/
> Anyway, it might still make sense to use it for the non-thermal stuff.

As I see it, it is just introducing abstraction that's not adding any
value. What might be useful, is a function (maybe a plugin)
that parses files like /proc/<pid>/status or /proc/acpi/battery/*/state
('name: value' lines) and uses selected values - then the Linux-ACPI
plugins (battery, thermal) could be reduced to couple of lines each.

 -- Michał Mirosław




More information about the collectd mailing list