pmqos: Add cleanup function on normal exit

This patch adds a cleanup function, that frees all the dynamically
allocated data and assigns 0 to the static variables. This cleanup
function is invoked when the pmqos module is normally terminated.

Change-Id: I0471f3ca13a4ba8a385b7829afab16dbf68f4988
Signed-off-by: Wook Song <wook16.song@samsung.com>

diff --git a/src/pmqos/pmqos.c b/src/pmqos/pmqos.c
index 395a513..ea40cb7 100644 (file)
--- a/src/pmqos/pmqos.c
+++ b/src/pmqos/pmqos.c
@@ -49,7 +49,7 @@ static dd_list *pmqos_head;
 static Ecore_Timer *unlock_timer;
 static struct timespec unlock_timer_start_st;
 static struct timespec unlock_timer_end_st;
-static struct pmqos_cpu unlock_timer_owner = {"NULL", 0};
+static struct pmqos_cpu unlock_timer_owner = {"", 0};
 
 int set_pmqos(const char *name, int val)
 {
@@ -455,9 +455,30 @@ static void pmqos_init(void *data)
        register_notifier(DEVICE_NOTIFIER_BOOTING_DONE, booting_done, NULL);
 }
 
+static void pmqos_free(void)
+{
+       /* Assign 0 to static variables */
+       memset(&unlock_timer_start_st, 0, sizeof(struct timespec));
+       memset(&unlock_timer_end_st, 0, sizeof(struct timespec));
+       memset(&unlock_timer_owner, 0, sizeof(struct pmqos_cpu));
+
+       /* Clean up pmqos_head */
+       if (pmqos_head) {
+               DD_LIST_FREE_LIST_FULL(pmqos_head, free);
+               pmqos_head = NULL;
+       }
+
+       /* Clean up unlock_timer */
+       if (unlock_timer) {
+               ecore_timer_del(unlock_timer);
+               unlock_timer = NULL;
+       }
+}
+
 static void pmqos_exit(void *data)
 {
        unregister_notifier(DEVICE_NOTIFIER_BOOTING_DONE, booting_done);
+       pmqos_free();
 }
 
 static const struct device_ops pmqos_device_ops = {