* Refactored the kernel daemon code into a class.

* Added a second kernel daemon service, resource resizer, which is
  mainly supposed to be used for resizing allocations asynchrounously.


git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@27122 a95241bf-73f2-0310-859d-f6bbb57e9c96

headers/private/kernel/kernel_daemon.h

@@ -6,12 +6,21 @@
 #define _KERNEL_DAEMON_H
 
 
-#include <OS.h>
+#include <KernelExport.h>
 
 
 #ifdef __cplusplus
-extern "C"
+extern "C" {
 #endif
+
+status_t register_resource_resizer(daemon_hook function, void* arg,
+			int frequency);
+status_t unregister_resource_resizer(daemon_hook function, void* arg);
+
 status_t kernel_daemon_init(void);
 
+#ifdef __cplusplus
+}
+#endif
+
 #endif	/* _KERNEL_DAEMON_H */

src/system/kernel/kernel_daemon.cpp

@@ -33,46 +33,90 @@ struct daemon : DoublyLinkedListLinkImpl<struct daemon> {
 
 typedef DoublyLinkedList<struct daemon> DaemonList;
 
-static mutex sDaemonMutex = MUTEX_INITIALIZER("kernel daemon");
+
-static DaemonList sDaemons;
+class KernelDaemon {
+public:
+			status_t			Init(const char* name);
+
+			status_t			Register(daemon_hook function, void* arg,
+									int frequency);
+			status_t			Unregister(daemon_hook function, void* arg);
+
+private:
+	static	status_t			_DaemonThreadEntry(void* data);
+			status_t			_DaemonThread();
+
+private:
+			mutex				fLock;
+			DaemonList			fDaemons;
+			thread_id			fThread;
+};
 
 
-static status_t
+static KernelDaemon sKernelDaemon;
-kernel_daemon(void* data)
+static KernelDaemon sResourceResizer;
+
+
+status_t
+KernelDaemon::Init(const char* name)
 {
-	int32 iteration = 0;
+	new(&fDaemons) DaemonList;
+	mutex_init(&fLock, name);
 
-	while (true) {
+	fThread = spawn_kernel_thread(&_DaemonThreadEntry, name, B_LOW_PRIORITY,
-		mutex_lock(&sDaemonMutex);
+		this);
+	if (fThread < 0)
+		return fThread;
 
-		DaemonList::Iterator iterator = sDaemons.GetIterator();
+	send_signal_etc(fThread, SIGCONT, B_DO_NOT_RESCHEDULE);
-
-		// iterate through the list and execute each daemon if needed
-		while (iterator.HasNext()) {
-			struct daemon* daemon = iterator.Next();
-
-			if (((iteration + daemon->offset) % daemon->frequency) == 0)
-				daemon->function(daemon->arg, iteration);
-		}
-		mutex_unlock(&sDaemonMutex);
-
-		iteration++;
-		snooze(100000);	// 0.1 seconds
-	}
 
 	return B_OK;
 }
 
 
-//	#pragma mark -
+status_t
-
+KernelDaemon::Register(daemon_hook function, void* arg, int frequency)
-
-extern "C" status_t
-unregister_kernel_daemon(daemon_hook function, void* arg)
 {
-	MutexLocker _(sDaemonMutex);
+	if (function == NULL || frequency < 1)
+		return B_BAD_VALUE;
 
-	DaemonList::Iterator iterator = sDaemons.GetIterator();
+	struct daemon* daemon = new(std::nothrow) struct ::daemon;
+	if (daemon == NULL)
+		return B_NO_MEMORY;
+
+	daemon->function = function;
+	daemon->arg = arg;
+	daemon->frequency = frequency;
+
+	MutexLocker _(fLock);
+
+	if (frequency > 1) {
+		// we try to balance the work-load for each daemon run
+		// (beware, it's a very simple algorithm, yet effective)
+
+		DaemonList::Iterator iterator = fDaemons.GetIterator();
+		int32 num = 0;
+
+		while (iterator.HasNext()) {
+			if (iterator.Next()->frequency == frequency)
+				num++;
+		}
+
+		daemon->offset = num % frequency;
+	} else
+		daemon->offset = 0;
+
+	fDaemons.Add(daemon);
+	return B_OK;
+}
+
+
+status_t
+KernelDaemon::Unregister(daemon_hook function, void* arg)
+{
+	MutexLocker _(fLock);
+
+	DaemonList::Iterator iterator = fDaemons.GetIterator();
 
 	// search for the daemon and remove it from the list
 	while (iterator.HasNext()) {
@@ -90,51 +134,81 @@ unregister_kernel_daemon(daemon_hook function, void* arg)
 }
 
 
+/*static*/ status_t
+KernelDaemon::_DaemonThreadEntry(void* data)
+{
+	return ((KernelDaemon*)data)->_DaemonThread();
+}
+
+
+status_t
+KernelDaemon::_DaemonThread()
+{
+	int32 iteration = 0;
+
+	while (true) {
+		mutex_lock(&fLock);
+
+		DaemonList::Iterator iterator = fDaemons.GetIterator();
+
+		// iterate through the list and execute each daemon if needed
+		while (iterator.HasNext()) {
+			struct daemon* daemon = iterator.Next();
+
+			if (((iteration + daemon->offset) % daemon->frequency) == 0)
+				daemon->function(daemon->arg, iteration);
+		}
+		mutex_unlock(&fLock);
+
+		iteration++;
+		snooze(100000);	// 0.1 seconds
+	}
+
+	return B_OK;
+}
+
+
+//	#pragma mark -
+
+
 extern "C" status_t
 register_kernel_daemon(daemon_hook function, void* arg, int frequency)
 {
-	if (function == NULL || frequency < 1)
+	return sKernelDaemon.Register(function, arg, frequency);
-		return B_BAD_VALUE;
+}
 
-	struct daemon* daemon = new(std::nothrow) struct daemon();
-	if (daemon == NULL)
-		return B_NO_MEMORY;
 
-	daemon->function = function;
+extern "C" status_t
-	daemon->arg = arg;
+unregister_kernel_daemon(daemon_hook function, void* arg)
-	daemon->frequency = frequency;
+{
+	return sKernelDaemon.Unregister(function, arg);
+}
 
-	MutexLocker _(sDaemonMutex);
-	
-	if (frequency > 1) {
-		// we try to balance the work-load for each daemon run
-		// (beware, it's a very simple algorithm, yet effective)
 
-		DaemonList::Iterator iterator = sDaemons.GetIterator();
+extern "C" status_t
-		int32 num = 0;
+register_resource_resizer(daemon_hook function, void* arg, int frequency)
+{
+	return sResourceResizer.Register(function, arg, frequency);
+}
 
-		while (iterator.HasNext()) {
-			if (iterator.Next()->frequency == frequency)
-				num++;
-		}
 
-		daemon->offset = num % frequency;
+extern "C" status_t
-	} else
+unregister_resource_resizer(daemon_hook function, void* arg)
-		daemon->offset = 0;
+{
-
+	return sResourceResizer.Unregister(function, arg);
-	sDaemons.Add(daemon);
-	return B_OK;
 }
 
 
 extern "C" status_t
 kernel_daemon_init(void)
 {
-	new(&sDaemons) DaemonList;
+	new(&sKernelDaemon) KernelDaemon;
+	if (sKernelDaemon.Init("kernel daemon") != B_OK)
+		panic("kernel_daemon_init(): failed to init kernel daemon");
 
-	thread_id thread = spawn_kernel_thread(&kernel_daemon, "kernel daemon",
+	new(&sResourceResizer) KernelDaemon;
-		B_LOW_PRIORITY, NULL);
+	if (sResourceResizer.Init("resource resizer") != B_OK)
-	send_signal_etc(thread, SIGCONT, B_DO_NOT_RESCHEDULE);
+		panic("kernel_daemon_init(): failed to init resource resizer");
 
 	return B_OK;
 }