workqueue: Clean up enum work_bits and related constants

[linux-2.6-microblaze.git] / include / linux / workqueue.h

diff --git a/include/linux/workqueue.h b/include/linux/workqueue.h
index 1565bab..0ad534f 100644 (file)
--- a/include/linux/workqueue.h
+++ b/include/linux/workqueue.h
@@ -24,41 +24,49 @@
 
 enum work_bits {
        WORK_STRUCT_PENDING_BIT = 0,    /* work item is pending execution */
-       WORK_STRUCT_INACTIVE_BIT= 1,    /* work item is inactive */
-       WORK_STRUCT_PWQ_BIT     = 2,    /* data points to pwq */
-       WORK_STRUCT_LINKED_BIT  = 3,    /* next work is linked to this one */
+       WORK_STRUCT_INACTIVE_BIT,       /* work item is inactive */
+       WORK_STRUCT_PWQ_BIT,            /* data points to pwq */
+       WORK_STRUCT_LINKED_BIT,         /* next work is linked to this one */
 #ifdef CONFIG_DEBUG_OBJECTS_WORK
-       WORK_STRUCT_STATIC_BIT  = 4,    /* static initializer (debugobjects) */
-       WORK_STRUCT_COLOR_SHIFT = 5,    /* color for workqueue flushing */
-#else
-       WORK_STRUCT_COLOR_SHIFT = 4,    /* color for workqueue flushing */
+       WORK_STRUCT_STATIC_BIT,         /* static initializer (debugobjects) */
 #endif
+       WORK_STRUCT_FLAG_BITS,
 
+       /* color for workqueue flushing */
+       WORK_STRUCT_COLOR_SHIFT = WORK_STRUCT_FLAG_BITS,
        WORK_STRUCT_COLOR_BITS  = 4,
 
        /*
-        * Reserve 8 bits off of pwq pointer w/ debugobjects turned off.
-        * This makes pwqs aligned to 256 bytes and allows 16 workqueue
-        * flush colors.
+        * When WORK_STRUCT_PWQ is set, reserve 8 bits off of pwq pointer w/
+        * debugobjects turned off. This makes pwqs aligned to 256 bytes (512
+        * bytes w/ DEBUG_OBJECTS_WORK) and allows 16 workqueue flush colors.
+        *
+        * MSB
+        * [ pwq pointer ] [ flush color ] [ STRUCT flags ]
+        *                     4 bits        4 or 5 bits
         */
-       WORK_STRUCT_FLAG_BITS   = WORK_STRUCT_COLOR_SHIFT +
-                                 WORK_STRUCT_COLOR_BITS,
+       WORK_STRUCT_PWQ_SHIFT   = WORK_STRUCT_COLOR_SHIFT + WORK_STRUCT_COLOR_BITS,
 
-       /* data contains off-queue information when !WORK_STRUCT_PWQ */
-       WORK_OFFQ_FLAG_BASE     = WORK_STRUCT_COLOR_SHIFT,
-
-       __WORK_OFFQ_CANCELING   = WORK_OFFQ_FLAG_BASE,
+       /*
+        * data contains off-queue information when !WORK_STRUCT_PWQ.
+        *
+        * MSB
+        * [ pool ID ] [ OFFQ flags ] [ STRUCT flags ]
+        *                 1 bit        4 or 5 bits
+        */
+       WORK_OFFQ_FLAG_SHIFT    = WORK_STRUCT_FLAG_BITS,
+       WORK_OFFQ_CANCELING_BIT = WORK_OFFQ_FLAG_SHIFT,
+       WORK_OFFQ_FLAG_END,
+       WORK_OFFQ_FLAG_BITS     = WORK_OFFQ_FLAG_END - WORK_OFFQ_FLAG_SHIFT,
 
        /*
-        * When a work item is off queue, its high bits point to the last
-        * pool it was on.  Cap at 31 bits and use the highest number to
-        * indicate that no pool is associated.
+        * When a work item is off queue, the high bits encode off-queue flags
+        * and the last pool it was on. Cap pool ID to 31 bits and use the
+        * highest number to indicate that no pool is associated.
         */
-       WORK_OFFQ_FLAG_BITS     = 1,
-       WORK_OFFQ_POOL_SHIFT    = WORK_OFFQ_FLAG_BASE + WORK_OFFQ_FLAG_BITS,
+       WORK_OFFQ_POOL_SHIFT    = WORK_OFFQ_FLAG_SHIFT + WORK_OFFQ_FLAG_BITS,
        WORK_OFFQ_LEFT          = BITS_PER_LONG - WORK_OFFQ_POOL_SHIFT,
        WORK_OFFQ_POOL_BITS     = WORK_OFFQ_LEFT <= 31 ? WORK_OFFQ_LEFT : 31,
-
 };
 
 enum work_flags {
@@ -88,12 +96,10 @@ enum wq_misc_consts {
 };
 
 /* Convenience constants - of type 'unsigned long', not 'enum'! */
-#define WORK_OFFQ_CANCELING    (1ul << __WORK_OFFQ_CANCELING)
+#define WORK_OFFQ_CANCELING    (1ul << WORK_OFFQ_CANCELING_BIT)
 #define WORK_OFFQ_POOL_NONE    ((1ul << WORK_OFFQ_POOL_BITS) - 1)
 #define WORK_STRUCT_NO_POOL    (WORK_OFFQ_POOL_NONE << WORK_OFFQ_POOL_SHIFT)
-
-#define WORK_STRUCT_FLAG_MASK    ((1ul << WORK_STRUCT_FLAG_BITS) - 1)
-#define WORK_STRUCT_WQ_DATA_MASK (~WORK_STRUCT_FLAG_MASK)
+#define WORK_STRUCT_PWQ_MASK   (~((1ul << WORK_STRUCT_PWQ_SHIFT) - 1))
 
 #define WORK_DATA_INIT()       ATOMIC_LONG_INIT((unsigned long)WORK_STRUCT_NO_POOL)
 #define WORK_DATA_STATIC_INIT()        \