save fixed ms-queue
[model-checker-benchmarks.git] / ms-queue / my_queue.c
index 8fedd9c086379fd9ac7ca263f50837da528a08c3..6c0ccd4bea4c0f49b2a1fdd4f9b24d992dc0f749 100644 (file)
-#include "main.h"
+#include <threads.h>
+#include <stdlib.h>
+#include "librace.h"
+#include "model-assert.h"
 
-extern unsigned int iterations;
-extern private_t private;
-extern shared_mem_t *smp;
+#include "my_queue.h"
 
-void init_private(int pid)
-{
-       private.node = 2 + pid;
-       private.value = 1 + (pid * iterations);
+#define relaxed memory_order_relaxed
+#define release memory_order_release
+#define acquire memory_order_acquire
 
-}
+#define MAX_FREELIST 4 /* Each thread can own up to MAX_FREELIST free nodes */
+#define INITIAL_FREE 2 /* Each thread starts with INITIAL_FREE free nodes */
 
-void init_memory()
-{
-}
+#define POISON_IDX 0x666
 
+static unsigned int (*free_lists)[MAX_FREELIST];
+
+/* Search this thread's free list for a "new" node */
 static unsigned int new_node()
 {
-       return private.node;
+       int i;
+       int t = get_thread_num();
+       for (i = 0; i < MAX_FREELIST; i++) {
+               unsigned int node = load_32(&free_lists[t][i]);
+               if (node) {
+                       store_32(&free_lists[t][i], 0);
+                       return node;
+               }
+       }
+       /* free_list is empty? */
+       MODEL_ASSERT(0);
+       return 0;
 }
 
+/* Place this node index back on this thread's free list */
 static void reclaim(unsigned int node)
 {
-       private.node = node;
-}
+       int i;
+       int t = get_thread_num();
 
-void init_queue()
-{
-       unsigned int i;
-       pointer head;
-       pointer tail;
-       pointer next;
-
-       /* initialize queue */
-       head = MAKE_POINTER(1, 0);
-       tail = MAKE_POINTER(1, 0);
-       next = MAKE_POINTER(0, 0); // (NULL, 0)
+       /* Don't reclaim NULL node */
+       MODEL_ASSERT(node);
 
-       atomic_init(&smp->nodes[0].next, 0); // assumed inititalized in original example
+       for (i = 0; i < MAX_FREELIST; i++) {
+               /* Should never race with our own thread here */
+               unsigned int idx = load_32(&free_lists[t][i]);
 
-       atomic_store(&smp->head, head);
-       atomic_store(&smp->tail, tail);
-       atomic_store(&smp->nodes[1].next, next);
+               /* Found empty spot in free list */
+               if (idx == 0) {
+                       store_32(&free_lists[t][i], node);
+                       return;
+               }
+       }
+       /* free list is full? */
+       MODEL_ASSERT(0);
+}
 
-       /* initialize avail list */
-       for (i = 2; i < MAX_NODES; i++) {
-               next = MAKE_POINTER(i + 1, 0);
-               atomic_store(&smp->nodes[i].next, next);
+void init_queue(queue_t *q, int num_threads)
+{
+       int i, j;
+
+       /* Initialize each thread's free list with INITIAL_FREE pointers */
+       /* The actual nodes are initialized with poison indexes */
+       free_lists = malloc(num_threads * sizeof(*free_lists));
+       for (i = 0; i < num_threads; i++) {
+               for (j = 0; j < INITIAL_FREE; j++) {
+                       free_lists[i][j] = 2 + i * MAX_FREELIST + j;
+                       atomic_init(&q->nodes[free_lists[i][j]].next, MAKE_POINTER(POISON_IDX, 0));
+               }
        }
 
-       next = MAKE_POINTER(0, 0); // (NULL, 0)
-       atomic_store(&smp->nodes[MAX_NODES].next, next);
+       /* initialize queue */
+       atomic_init(&q->head, MAKE_POINTER(1, 0));
+       atomic_init(&q->tail, MAKE_POINTER(1, 0));
+       atomic_init(&q->nodes[1].next, MAKE_POINTER(0, 0));
 }
 
-void enqueue(unsigned int val)
+void enqueue(queue_t *q, unsigned int val)
 {
-       unsigned int success = 0;
+       int success = 0;
        unsigned int node;
        pointer tail;
        pointer next;
        pointer tmp;
 
        node = new_node();
-       smp->nodes[node].value = val;
-       tmp = atomic_load(&smp->nodes[node].next);
+       store_32(&q->nodes[node].value, val);
+       tmp = atomic_load_explicit(&q->nodes[node].next, relaxed);
        set_ptr(&tmp, 0); // NULL
-       atomic_store(&smp->nodes[node].next, tmp);
+       atomic_store_explicit(&q->nodes[node].next, tmp, relaxed);
 
        while (!success) {
-               tail = atomic_load(&smp->tail);
-               next = atomic_load(&smp->nodes[get_ptr(tail)].next);
-               if (tail == atomic_load(&smp->tail)) {
+               tail = atomic_load_explicit(&q->tail, acquire);
+               next = atomic_load_explicit(&q->nodes[get_ptr(tail)].next, acquire);
+               if (tail == atomic_load_explicit(&q->tail, relaxed)) {
+
+                       /* Check for uninitialized 'next' */
+                       MODEL_ASSERT(get_ptr(next) != POISON_IDX);
+
                        if (get_ptr(next) == 0) { // == NULL
-                               pointer val = MAKE_POINTER(node, get_count(next) + 1);
-                               success = atomic_compare_exchange_weak(&smp->nodes[get_ptr(tail)].next,
-                                               &next,
-                                               val);
+                               pointer value = MAKE_POINTER(node, get_count(next) + 1);
+                               success = atomic_compare_exchange_strong_explicit(&q->nodes[get_ptr(tail)].next,
+                                               &next, value, release, release);
                        }
                        if (!success) {
-                               unsigned int ptr = get_ptr(atomic_load(&smp->nodes[get_ptr(tail)].next));
-                               pointer val = MAKE_POINTER(ptr,
+                               unsigned int ptr = get_ptr(atomic_load_explicit(&q->nodes[get_ptr(tail)].next, acquire));
+                               pointer value = MAKE_POINTER(ptr,
                                                get_count(tail) + 1);
-                               atomic_compare_exchange_strong(&smp->tail,
-                                               &tail,
-                                               val);
+                               atomic_compare_exchange_strong_explicit(&q->tail,
+                                               &tail, value,
+                                               release, release);
                                thrd_yield();
                        }
                }
        }
-       atomic_compare_exchange_strong(&smp->tail,
+       atomic_compare_exchange_strong_explicit(&q->tail,
                        &tail,
-                       MAKE_POINTER(node, get_count(tail) + 1));
+                       MAKE_POINTER(node, get_count(tail) + 1),
+                       release, release);
 }
 
-unsigned int dequeue()
+bool dequeue(queue_t *q, unsigned int *retVal)
 {
-       unsigned int value;
-       unsigned int success;
+       int success = 0;
        pointer head;
        pointer tail;
        pointer next;
 
-       for (success = FALSE; success == FALSE; ) {
-               head = atomic_load(&smp->head);
-               tail = atomic_load(&smp->tail);
-               next = atomic_load(&smp->nodes[get_ptr(head)].next);
-               if (atomic_load(&smp->head) == head) {
+       while (!success) {
+               head = atomic_load_explicit(&q->head, acquire);
+               tail = atomic_load_explicit(&q->tail, relaxed);
+               next = atomic_load_explicit(&q->nodes[get_ptr(head)].next, acquire);
+               if (atomic_load_explicit(&q->head, relaxed) == head) {
                        if (get_ptr(head) == get_ptr(tail)) {
+
+                               /* Check for uninitialized 'next' */
+                               MODEL_ASSERT(get_ptr(next) != POISON_IDX);
+
                                if (get_ptr(next) == 0) { // NULL
-                                       return 0; // NULL
+                                       return false; // NULL
                                }
-                               atomic_compare_exchange_weak(&smp->tail,
+                               atomic_compare_exchange_strong_explicit(&q->tail,
                                                &tail,
-                                               MAKE_POINTER(get_ptr(next), get_count(tail) + 1));
+                                               MAKE_POINTER(get_ptr(next), get_count(tail) + 1),
+                                               release, release);
                                thrd_yield();
                        } else {
-                               value = smp->nodes[get_ptr(next)].value;
-                               success = atomic_compare_exchange_weak(&smp->head,
+                               *retVal = load_32(&q->nodes[get_ptr(next)].value);
+                               success = atomic_compare_exchange_strong_explicit(&q->head,
                                                &head,
-                                               MAKE_POINTER(get_ptr(next), get_count(head) + 1));
-                               if (success == FALSE) {
+                                               MAKE_POINTER(get_ptr(next), get_count(head) + 1),
+                                               release, release);
+                               if (!success)
                                        thrd_yield();
-                               }
                        }
                }
        }
        reclaim(get_ptr(head));
-       return value;
+       return true;
 }