[PATCH 1/3] percpu_ida: introduce kobject for percpu_ida pool

[Ming Lei]

So that we can export some allocation/free information
for monitoring percpu_ida performance.

Signed-off-by: Ming Lei <tom.leim...@gmail.com>
---
 include/linux/percpu_ida.h |   16 ++++++++++++++++
 lib/percpu_ida.c           |   21 ++++++++++++++++++---
 2 files changed, 34 insertions(+), 3 deletions(-)

diff --git a/include/linux/percpu_ida.h b/include/linux/percpu_ida.h
index f5cfdd6..463e3b3 100644
--- a/include/linux/percpu_ida.h
+++ b/include/linux/percpu_ida.h
@@ -8,6 +8,7 @@
 #include <linux/spinlock_types.h>
 #include <linux/wait.h>
 #include <linux/cpumask.h>
+#include <linux/kobject.h>
 
 struct percpu_ida_cpu;
 
@@ -52,6 +53,8 @@ struct percpu_ida {
                unsigned                nr_free;
                unsigned                *freelist;
        } ____cacheline_aligned_in_smp;
+
+       struct kobject kobj;
 };
 
 /*
@@ -79,4 +82,17 @@ int percpu_ida_for_each_free(struct percpu_ida *pool, 
percpu_ida_cb fn,
        void *data);
 
 unsigned percpu_ida_free_tags(struct percpu_ida *pool, int cpu);
+
+static inline int percpu_ida_kobject_add(struct percpu_ida *pool,
+               struct kobject *parent, const char *name)
+{
+       if (pool->kobj.state_initialized)
+               return kobject_add(&pool->kobj, parent, name);
+       return 0;
+}
+static inline void percpu_ida_kobject_del(struct percpu_ida *pool)
+{
+       if (pool->kobj.state_in_sysfs)
+               kobject_del(&pool->kobj);
+}
 #endif /* __PERCPU_IDA_H__ */
diff --git a/lib/percpu_ida.c b/lib/percpu_ida.c
index 93d145e..56ae350 100644
--- a/lib/percpu_ida.c
+++ b/lib/percpu_ida.c
@@ -260,6 +260,20 @@ void percpu_ida_free(struct percpu_ida *pool, unsigned tag)
 }
 EXPORT_SYMBOL_GPL(percpu_ida_free);
 
+static void percpu_ida_release(struct kobject *kobj)
+{
+       struct percpu_ida *pool = container_of(kobj,
+                       struct percpu_ida, kobj);
+
+       free_percpu(pool->tag_cpu);
+       free_pages((unsigned long) pool->freelist,
+                  get_order(pool->nr_tags * sizeof(unsigned)));
+}
+
+static struct kobj_type percpu_ida_ktype = {
+       .release        = percpu_ida_release,
+};
+
 /**
  * percpu_ida_destroy - release a tag pool's resources
  * @pool: pool to free
@@ -268,9 +282,8 @@ EXPORT_SYMBOL_GPL(percpu_ida_free);
  */
 void percpu_ida_destroy(struct percpu_ida *pool)
 {
-       free_percpu(pool->tag_cpu);
-       free_pages((unsigned long) pool->freelist,
-                  get_order(pool->nr_tags * sizeof(unsigned)));
+       if (pool->kobj.state_initialized)
+               kobject_put(&pool->kobj);
 }
 EXPORT_SYMBOL_GPL(percpu_ida_destroy);
 
@@ -324,6 +337,8 @@ int __percpu_ida_init(struct percpu_ida *pool, unsigned 
long nr_tags,
        for_each_possible_cpu(cpu)
                spin_lock_init(&per_cpu_ptr(pool->tag_cpu, cpu)->lock);
 
+       kobject_init(&pool->kobj, &percpu_ida_ktype);
+
        return 0;
 err:
        percpu_ida_destroy(pool);
-- 
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/