0

我被要求使用比较和交换在 c 中实现一个无锁队列,但是我对指针的了解相当有限。

我一直在使用以下代码来测试我的(尚未完成的)出队实现,但我相信它会无限循环,因为我不太确定如何正确使用运算符的指针/地址。

因为我对汇编程序一无所知,所以我已经得到了这个 CAS 函数。

long __cdecl compare_exchange(long *flag, long oldvalue, long newvalue)
{
    __asm
    {
        mov ecx, flag
        mov eax, oldvalue
        mov ebx, newvalue
        lock cmpxchg [ecx], ebx
        jz iftrue
    }
    return 0;
    iftrue: return 1;
}

我当前的(相关)代码如下......

typedef struct QueueItem
{
    int data;
    struct QueueItem* next;
}item;

struct Queue
{
    item *head;
    item *tail;
}*queue;

int Dequeue()
{
    item *head;

    do
    {
        head = queue->head;
        if(head == NULL)
            return NULL_ITEM;
        printf("%d, %d, %d\n", (long *)queue->head, (long)&head, (long)&head->next);
    }
    while(!compare_exchange((long *)queue->head, (long)&head, (long)&head->next)); // Infinite loop.

    return head->data;
}

int main(int argc, char *argv[])
{
    item i, j;

    queue = (struct Queue *) malloc(sizeof(struct Queue));

    // Manually enqueue some data for testing dequeue.
    i.data = 5;
    j.data = 10;
    i.next = &j;
    j.next = NULL;

    queue->head = &i;

    printf("Dequeued: %d\n", Dequeue());
    printf("Dequeued: %d\n", Dequeue());
}

我应该在 do while 循环中使用 not 运算符吗?如果我不使用该运算符,我会得到“Dequeued 5”x2 的输出,这表明交换没有发生,我应该使用 not。如果是这样,我哪里错了?我会把钱放在指针/地址运算符问题上。

4

1 回答 1

0

指针和值存在混淆。这是更正的代码:

 do
 {
    head = queue->head;
    if(head == NULL)
        return 0;
    printf("%d, %d, %d %d\n", (long *)queue->head, (long)head, (long)head->next, head->data);
  }  while (!compare_exchange((long *)&queue->head,   (long)head, (long)head->next));

您试图编写 queue->head 指向的内容,而不是 queue->head 本身的值。

另外,为了让它在多核上正常工作,我相信你需要将 head 定义为 volatile。

struct Queue
{
   volatile item *head;
   item *tail;
}*queue;
于 2012-08-24T03:51:01.690 回答