Changeset 96368f56 in mainline

Timestamp:

2025-01-16T13:33:58Z (32 hours ago)

Author:

Jiří Zárevúcky <zarevucky.jiri@…>

Children:

fa3ed5b

Parents:

e0e2264

Message:

Use <lib/refcount.h> for kobject reference counting

Location:

kernel/generic

Files:

• include/cap/cap.h (modified) (2 diffs)
• src/cap/cap.c (modified) (4 diffs)

kernel/generic/include/cap/cap.h

@@ -37 +37 @@
 
 #include <abi/cap.h>
+#include <adt/hash_table.h>
+#include <adt/hash.h>
+#include <adt/list.h>
+#include <atomic.h>
+#include <lib/ra.h>
+#include <lib/refcount.h>
+#include <synch/mutex.h>
 #include <typedefs.h>
-#include <adt/list.h>
-#include <adt/hash.h>
-#include <adt/hash_table.h>
-#include <lib/ra.h>
-#include <synch/mutex.h>
-#include <atomic.h>
 
 typedef enum {
@@ -80 +81 @@
 typedef struct kobject {
         kobject_type_t type;
-        atomic_size_t refcnt;
+        atomic_refcount_t refcnt;
 
         /** Mutex protecting caps_list */

kernel/generic/src/cap/cap.c

@@ -426 +426 @@
 void kobject_initialize(kobject_t *kobj, kobject_type_t type, void *raw)
 {
-        atomic_store(&kobj->refcnt, 1);
+        refcount_init(&kobj->refcnt);
 
         mutex_initialize(&kobj->caps_list_lock, MUTEX_PASSIVE);
@@ -455 +455 @@
                 if (cap->kobject->type == type) {
                         kobj = cap->kobject;
-                        atomic_inc(&kobj->refcnt);
+                        refcount_up(&kobj->refcnt);
                 }
         }
@@ -469 +469 @@
 void kobject_add_ref(kobject_t *kobj)
 {
-        atomic_inc(&kobj->refcnt);
+        refcount_up(&kobj->refcnt);
 }
 
@@ -481 +481 @@
 void kobject_put(kobject_t *kobj)
 {
-        if (atomic_postdec(&kobj->refcnt) == 1) {
+        if (refcount_down(&kobj->refcnt)) {
                 KOBJECT_OP(kobj)->destroy(kobj->raw);
                 kobject_free(kobj);