Changeset 25a179e in mainline for uspace/lib/c/generic/iplink_srv.c

Timestamp:

2017-12-20T19:51:45Z (7 years ago)

Author:

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

Branches:

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

Children:

f96b6c8

Parents:

d4a829e

Message:

IPC return values are always errno constants. Adjust types to reflect that.

In principle, IPC server is not allowed to return non-errno values via
the "main" return value, because kernel interprets it (e.g. EHANGUP).
It's still possible to return arbitrary additional return values alongside EOK,
which are not interpreted in normal communication.

File:

• uspace/lib/c/generic/iplink_srv.c (modified) (8 diffs)

uspace/lib/c/generic/iplink_srv.c

@@ -78 +78 @@
                 async_answer_0(callid, rc);
         
-        async_answer_0(iid, (sysarg_t) rc);
+        async_answer_0(iid, rc);
 }
 
@@ -90 +90 @@
 
         if (!async_data_write_receive(&callid, &size)) {
-                async_answer_0(callid, (sysarg_t) EREFUSED);
-                async_answer_0(iid, (sysarg_t) EREFUSED);
+                async_answer_0(callid, EREFUSED);
+                async_answer_0(iid, EREFUSED);
         }
 
@@ -104 +104 @@
                 async_answer_0(callid, rc);
         
-        async_answer_0(iid, (sysarg_t) rc);
+        async_answer_0(iid, rc);
 }
 
@@ -127 +127 @@
         int rc = async_data_write_finalize(callid, &addr, size);
         if (rc != EOK) {
-                async_answer_0(callid, (sysarg_t) rc);
-                async_answer_0(iid, (sysarg_t) rc);
+                async_answer_0(callid, rc);
+                async_answer_0(iid, rc);
         }
         
         rc = srv->ops->addr_add(srv, &addr);
-        async_answer_0(iid, (sysarg_t) rc);
+        async_answer_0(iid, rc);
 }
 
@@ -155 +155 @@
         int rc = async_data_write_finalize(callid, &addr, size);
         if (rc != EOK) {
-                async_answer_0(callid, (sysarg_t) rc);
-                async_answer_0(iid, (sysarg_t) rc);
+                async_answer_0(callid, rc);
+                async_answer_0(iid, rc);
         }
         
         rc = srv->ops->addr_remove(srv, &addr);
-        async_answer_0(iid, (sysarg_t) rc);
+        async_answer_0(iid, rc);
 }
 
@@ -204 +204 @@
         int rc = async_data_write_finalize(callid, &sdu.dest, size);
         if (rc != EOK) {
-                async_answer_0(callid, (sysarg_t) rc);
-                async_answer_0(iid, (sysarg_t) rc);
+                async_answer_0(callid, rc);
+                async_answer_0(iid, rc);
         }
         
@@ -321 +321 @@
         }
         
-        sysarg_t retval;
+        int retval;
         async_wait_for(req, &retval);
         if (retval != EOK)
@@ -347 +347 @@
         }
         
-        sysarg_t retval;
+        int retval;
         async_wait_for(req, &retval);
         if (retval != EOK)