Changeset 498ced1 in mainline for uspace/srv/net/tcp/conn.c

Timestamp:

2018-08-11T02:43:32Z (6 years ago)

Author:

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

Branches:

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

Children:

05882233

Parents:

b13d80b

git-author:

Jiří Zárevúcky <jiri.zarevucky@…> (2018-08-11 02:29:02)

git-committer:

Jiří Zárevúcky <jiri.zarevucky@…> (2018-08-11 02:43:32)

Message:

Unify reference counting and remove some unnecessary instances of <atomic.h>

File:

• uspace/srv/net/tcp/conn.c (modified) (3 diffs)

uspace/srv/net/tcp/conn.c

@@ -128 +128 @@
 
         /* One for the user, one for not being in closed state */
-        atomic_set(&conn->refcnt, 2);
+        refcount_init(&conn->refcnt);
+        refcount_up(&conn->refcnt);
 
         /* Allocate receive buffer */
@@ -238 +239 @@
 void tcp_conn_addref(tcp_conn_t *conn)
 {
-        log_msg(LOG_DEFAULT, LVL_DEBUG2, "%s: tcp_conn_addref(%p) before=%zu",
-            conn->name, conn, atomic_get(&conn->refcnt));
-        atomic_inc(&conn->refcnt);
+        log_msg(LOG_DEFAULT, LVL_DEBUG2, "%s: tcp_conn_addref(%p)",
+            conn->name, conn);
+
+        refcount_up(&conn->refcnt);
 }
 
@@ -251 +253 @@
 void tcp_conn_delref(tcp_conn_t *conn)
 {
-        log_msg(LOG_DEFAULT, LVL_DEBUG2, "%s: tcp_conn_delref(%p) before=%zu",
-            conn->name, conn, atomic_get(&conn->refcnt));
-
-        if (atomic_predec(&conn->refcnt) == 0)
+        log_msg(LOG_DEFAULT, LVL_DEBUG2, "%s: tcp_conn_delref(%p)",
+            conn->name, conn);
+
+        if (refcount_down(&conn->refcnt))
                 tcp_conn_free(conn);
 }