irq_work: Remove return value from the irq_work_queue() function
authoranish kumar <anish198519851985@gmail.com>
Sun, 3 Feb 2013 21:08:23 +0000 (22:08 +0100)
committerIngo Molnar <mingo@kernel.org>
Mon, 4 Feb 2013 10:50:59 +0000 (11:50 +0100)
As no one is using the return value of irq_work_queue(),
so it is better to just make it void.

Signed-off-by: anish kumar <anish198519851985@gmail.com>
Acked-by: Steven Rostedt <rostedt@goodmis.org>
[ Fix stale comments, remove now unnecessary __irq_work_queue() intermediate function ]
Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
Link: http://lkml.kernel.org/r/1359925703-24304-1-git-send-email-fweisbec@gmail.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
include/linux/irq_work.h
kernel/irq_work.c

index 6a9e8f5399e2ab137b3c251eb49349fdc6b333ac..ce60c084635b5a9cef539ed79c8b34fe0b583a15 100644 (file)
@@ -16,7 +16,7 @@ void init_irq_work(struct irq_work *work, void (*func)(struct irq_work *))
        work->func = func;
 }
 
-bool irq_work_queue(struct irq_work *work);
+void irq_work_queue(struct irq_work *work);
 void irq_work_run(void);
 void irq_work_sync(struct irq_work *work);
 
index 64eddd59ed83852262b5655a56e8ebb284dcb119..c9d7478e4889937e053e283ede32a0ad321764e0 100644 (file)
@@ -63,12 +63,20 @@ void __weak arch_irq_work_raise(void)
 }
 
 /*
- * Queue the entry and raise the IPI if needed.
+ * Enqueue the irq_work @entry unless it's already pending
+ * somewhere.
+ *
+ * Can be re-enqueued while the callback is still in progress.
  */
-static void __irq_work_queue(struct irq_work *work)
+void irq_work_queue(struct irq_work *work)
 {
        bool empty;
 
+       /* Only queue if not already pending */
+       if (!irq_work_claim(work))
+               return;
+
+       /* Queue the entry and raise the IPI if needed. */
        preempt_disable();
 
        empty = llist_add(&work->llnode, &__get_cpu_var(irq_work_list));
@@ -78,25 +86,6 @@ static void __irq_work_queue(struct irq_work *work)
 
        preempt_enable();
 }
-
-/*
- * Enqueue the irq_work @entry, returns true on success, failure when the
- * @entry was already enqueued by someone else.
- *
- * Can be re-enqueued while the callback is still in progress.
- */
-bool irq_work_queue(struct irq_work *work)
-{
-       if (!irq_work_claim(work)) {
-               /*
-                * Already enqueued, can't do!
-                */
-               return false;
-       }
-
-       __irq_work_queue(work);
-       return true;
-}
 EXPORT_SYMBOL_GPL(irq_work_queue);
 
 /*