Changeset 04552324 in mainline for kernel/generic/src/synch/workqueue.c

Timestamp:

2012-12-03T14:54:05Z (13 years ago)

Author:

Adam Hraska <adam.hraska+hos@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

a440a0f

Parents:

69146b93

Message:

cht, workq: Got rid of warnings in release builds.

File:

• kernel/generic/src/synch/workqueue.c (modified) (5 diffs)

kernel/generic/src/synch/workqueue.c

@@ -139 +139 @@
 static void cv_wait(struct work_queue *workq);
 static void nonblock_init(void);
+
+#ifdef CONFIG_DEBUG
 static bool workq_corrupted(struct work_queue *workq);
 static bool work_item_corrupted(work_t *work_item);
-
+#endif
 
 /** Creates worker thread for the system-wide worker queue. */
@@ -204 +206 @@
         irq_spinlock_lock(&workq->lock, true);
         bool stopped = workq->stopping;
+#ifdef CONFIG_DEBUG
         size_t running_workers = workq->cur_worker_cnt;
+#endif
         irq_spinlock_unlock(&workq->lock, true);
         
@@ -940 +944 @@
 }
 
+#ifdef CONFIG_DEBUG
 /** Returns true if the workq is definitely corrupted; false if not sure. 
  * 
@@ -946 +951 @@
 static bool workq_corrupted(struct work_queue *workq)
 {
-#ifdef CONFIG_DEBUG
         /* 
          * Needed to make the most current cookie value set by workq_preinit()
@@ -955 +959 @@
         memory_barrier();
         return NULL == workq || workq->cookie != WORKQ_MAGIC;
-#else
-        return false;
+}
+
+/** Returns true if the work_item is definitely corrupted; false if not sure. 
+ * 
+ * Must be used with the work queue protecting spinlock locked.
+ */
+static bool work_item_corrupted(work_t *work_item)
+{
+        return NULL == work_item || work_item->cookie != WORK_ITEM_MAGIC;
+}
 #endif
-}
-
-/** Returns true if the work_item is definitely corrupted; false if not sure. 
- * 
- * Must be used with the work queue protecting spinlock locked.
- */
-static bool work_item_corrupted(work_t *work_item)
-{
-#ifdef CONFIG_DEBUG
-        return NULL == work_item || work_item->cookie != WORK_ITEM_MAGIC;
-#else
-        return false;
-#endif
-}
 
 /** @}