[REFACTOR] redesign and move remove_probe_from_list()

from src/modules/driver/storage.c
to src/modules/driver/probes_manager.c

diff --git a/driver/probes_manager.c b/driver/probes_manager.c
index 25ec140..a80ccc2 100644 (file)
--- a/driver/probes_manager.c
+++ b/driver/probes_manager.c
@@ -145,6 +145,24 @@ static int add_probe_to_list(unsigned long addr, kernel_probe_t **pprobe)
        return 0;
 }
 
+/* Removes non-predefined kernel probe from the list. */
+static int remove_probe_from_list(unsigned long addr)
+{
+       kernel_probe_t *p;
+
+       /* check if such probe does exist */
+       p = find_probe(addr);
+       if (!p) {
+               /* We do not care about it. Nothing bad. */
+               return -EINVAL;
+       }
+
+       hlist_del_rcu(&p->hlist);
+       kfree(p);
+
+       return 0;
+}
+
 int
 add_probe (unsigned long addr)
 {

diff --git a/driver/storage.c b/driver/storage.c
index 42a26a3..208eb5e 100644 (file)
--- a/driver/storage.c
+++ b/driver/storage.c
@@ -1259,25 +1259,6 @@ kernel_probe_t* find_probe (unsigned long addr)
        return node ? p : NULL;
 }
 
-int remove_probe_from_list (unsigned long addr)
-{
-       kernel_probe_t *p;
-
-       //check if such probe does exist
-       p = find_probe (addr);
-       if (!p) {
-               /* We do not care about it. Nothing bad. */
-               return 0;
-       }
-
-       hlist_del_rcu (&p->hlist);
-
-       kfree (p);
-
-       return 0;
-}
-
-
 int put_us_event (char *data, unsigned long len)
 {
        unsigned long spinlock_flags = 0L;

diff --git a/driver/storage.h b/driver/storage.h
index 13f9daf..9c12aa9 100644 (file)
--- a/driver/storage.h
+++ b/driver/storage.h
@@ -69,11 +69,6 @@ void unlink_bundle(void);
 void dbi_find_and_set_handler_for_probe(kernel_probe_t *p);
 
 /*
-    Removes non-predefined kernel probe from the list.
-*/
-extern int remove_probe_from_list (unsigned long addr);
-
-/*
     Searches non-predefined kernel probe in the list.
 */
 extern kernel_probe_t *find_probe (unsigned long addr);