Changeset e0df6c2 in mainline for uspace/drv/uhci-hcd/transfer_list.c

Timestamp:

2011-02-12T03:04:40Z (13 years ago)

Author:

Jan Vesely <jano.vesely@…>

Branches:

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

Children:

67b6fc5

Parents:

d6115e5

Message:

Fix: properly check whether the removed batch is the first in the list
Add mutex to guard tranfser list

File:

• uspace/drv/uhci-hcd/transfer_list.c (modified) (7 diffs)

uspace/drv/uhci-hcd/transfer_list.c

@@ -52 +52 @@
         queue_head_init(instance->queue_head);
         list_initialize(&instance->batch_list);
+        fibril_mutex_initialize(&instance->guard);
         return EOK;
 }
@@ -76 +77 @@
         batch->qh->next_queue = instance->queue_head->next_queue;
 
+        fibril_mutex_lock(&instance->guard);
+
         if (instance->queue_head->element == instance->queue_head->next_queue) {
                 /* there is nothing scheduled */
@@ -82 +85 @@
                 usb_log_debug2("Added batch(%p) to queue %s first.\n",
                         batch, instance->name);
+                fibril_mutex_unlock(&instance->guard);
                 return;
         }
@@ -94 +98 @@
         usb_log_debug2("Added batch(%p) to queue %s last, first is %p.\n",
                 batch, instance->name, first );
+        fibril_mutex_unlock(&instance->guard);
 }
 /*----------------------------------------------------------------------------*/
@@ -105 +110 @@
 
         /* I'm the first one here */
-        if (batch->link.next == &instance->batch_list) {
+        if (batch->link.prev == &instance->batch_list) {
                 usb_log_debug("Removing tracer %p was first, next element %x.\n",
                         batch, batch->qh->next_queue);
@@ -121 +126 @@
 {
         assert(instance);
+        fibril_mutex_lock(&instance->guard);
         link_t *current = instance->batch_list.next;
         while (current != &instance->batch_list) {
@@ -132 +138 @@
                 current = next;
         }
+        fibril_mutex_unlock(&instance->guard);
 }
 /**