Changes in uspace/lib/packet/generic/packet_server.c [774e6d1a:dd5046dd] in mainline

File:

• uspace/lib/packet/generic/packet_server.c (modified) (7 diffs)

uspace/lib/packet/generic/packet_server.c

@@ -135 +135 @@
 /** Creates a new packet of dimensions at least as given.
  *
+ * Should be used only when the global data are locked.
+ *
  * @param[in] length    The total length of the packet, including the header,
  *                      the addresses and the data of the packet.
@@ -151 +153 @@
         packet_t *packet;
         int rc;
-
-        assert(fibril_mutex_is_locked(&ps_globals.lock));
 
         // already locked
@@ -233 +233 @@
 /** Release the packet and returns it to the appropriate free packet queue.
  *
+ * Should be used only when the global data are locked.
+ *
  * @param[in] packet    The packet to be released.
  *
@@ -240 +242 @@
         int index;
         int result;
-
-        assert(fibril_mutex_is_locked(&ps_globals.lock));
 
         for (index = 0; (index < FREE_QUEUES_COUNT - 1) &&
@@ -322 +322 @@
 int
 packet_server_message(ipc_callid_t callid, ipc_call_t *call, ipc_call_t *answer,
-    size_t *answer_count)
+    int *answer_count)
 {
         packet_t *packet;
 
         *answer_count = 0;
-        switch (IPC_GET_IMETHOD(*call)) {
+        switch (IPC_GET_METHOD(*call)) {
         case IPC_M_PHONE_HUNGUP:
                 return EOK;
@@ -333 +333 @@
         case NET_PACKET_CREATE_1:
                 packet = packet_get_local(DEFAULT_ADDR_LEN, DEFAULT_PREFIX,
-                    IPC_GET_CONTENT(*call), DEFAULT_SUFFIX);
+                    IPC_GET_CONTENT(call), DEFAULT_SUFFIX);
                 if (!packet)
                         return ENOMEM;
                 *answer_count = 2;
-                IPC_SET_ARG1(*answer, (sysarg_t) packet->packet_id);
-                IPC_SET_ARG2(*answer, (sysarg_t) packet->length);
+                IPC_SET_ARG1(*answer, (ipcarg_t) packet->packet_id);
+                IPC_SET_ARG2(*answer, (ipcarg_t) packet->length);
                 return EOK;
         
         case NET_PACKET_CREATE_4:
                 packet = packet_get_local(
-                    ((DEFAULT_ADDR_LEN < IPC_GET_ADDR_LEN(*call)) ?
-                    IPC_GET_ADDR_LEN(*call) : DEFAULT_ADDR_LEN),
-                    DEFAULT_PREFIX + IPC_GET_PREFIX(*call),
-                    IPC_GET_CONTENT(*call),
-                    DEFAULT_SUFFIX + IPC_GET_SUFFIX(*call));
+                    ((DEFAULT_ADDR_LEN < IPC_GET_ADDR_LEN(call)) ?
+                    IPC_GET_ADDR_LEN(call) : DEFAULT_ADDR_LEN),
+                    DEFAULT_PREFIX + IPC_GET_PREFIX(call),
+                    IPC_GET_CONTENT(call),
+                    DEFAULT_SUFFIX + IPC_GET_SUFFIX(call));
                 if (!packet)
                         return ENOMEM;
                 *answer_count = 2;
-                IPC_SET_ARG1(*answer, (sysarg_t) packet->packet_id);
-                IPC_SET_ARG2(*answer, (sysarg_t) packet->length);
+                IPC_SET_ARG1(*answer, (ipcarg_t) packet->packet_id);
+                IPC_SET_ARG2(*answer, (ipcarg_t) packet->length);
                 return EOK;
         
         case NET_PACKET_GET:
-                packet = pm_find(IPC_GET_ID(*call));
+                packet = pm_find(IPC_GET_ID(call));
                 if (!packet_is_valid(packet))
                         return ENOENT;
@@ -362 +362 @@
         
         case NET_PACKET_GET_SIZE:
-                packet = pm_find(IPC_GET_ID(*call));
+                packet = pm_find(IPC_GET_ID(call));
                 if (!packet_is_valid(packet))
                         return ENOENT;
-                IPC_SET_ARG1(*answer, (sysarg_t) packet->length);
+                IPC_SET_ARG1(*answer, (ipcarg_t) packet->length);
                 *answer_count = 1;
                 return EOK;
         
         case NET_PACKET_RELEASE:
-                return packet_release_wrapper(IPC_GET_ID(*call));
+                return packet_release_wrapper(IPC_GET_ID(call));
         }