Changeset 3ca3430 in mainline for uspace/srv/devman/devman.c

Timestamp:

2010-12-17T23:32:02Z (14 years ago)

Author:

Vojtech Horky <vojtechhorky@…>

Branches:

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

Children:

6e50466

Parents:

7551706b

Message:

Bugfix in hash table usage in devman

The hash table with device class must use different comparing due to
different backend structures.

File:

• uspace/srv/devman/devman.c (modified) (5 diffs)

uspace/srv/devman/devman.c

@@ -62 +62 @@
 }
 
+static int devmap_devices_class_compare(unsigned long key[], hash_count_t keys,
+    link_t *item)
+{
+        dev_class_info_t *class_info
+            = hash_table_get_instance(item, dev_class_info_t, devmap_link);
+        assert(class_info != NULL);
+
+        return (class_info->devmap_handle == (devmap_handle_t) key[0]);
+}
+
 static void devices_remove_callback(link_t *item)
 {
@@ -75 +85 @@
         .hash = devices_hash,
         .compare = devmap_devices_compare,
+        .remove_callback = devices_remove_callback
+};
+
+static hash_table_operations_t devmap_devices_class_ops = {
+        .hash = devices_hash,
+        .compare = devmap_devices_class_compare,
         .remove_callback = devices_remove_callback
 };
@@ -1043 +1059 @@
         
         info = (dev_class_info_t *) malloc(sizeof(dev_class_info_t));
-        if (info != NULL)
+        if (info != NULL) {
                 memset(info, 0, sizeof(dev_class_info_t));
+                list_initialize(&info->dev_classes);
+                list_initialize(&info->devmap_link);
+                list_initialize(&info->link);
+        }
         
         return info;
@@ -1168 +1188 @@
         fibril_rwlock_initialize(&class_list->rwlock);
         hash_table_create(&class_list->devmap_devices, DEVICE_BUCKETS, 1,
-            &devmap_devices_ops);
+            &devmap_devices_class_ops);
 }
 
@@ -1216 +1236 @@
         hash_table_insert(&class_list->devmap_devices, &key, &cli->devmap_link);
         fibril_rwlock_write_unlock(&class_list->rwlock);
+
+        assert(find_devmap_class_device(class_list, cli->devmap_handle) != NULL);
 }