Changeset 1e2e0c1e in mainline for uspace/srv/net/structures

Timestamp:

2010-02-17T21:15:03Z (16 years ago)

Author:

Lukas Mejdrech <lukasmejdrech@…>

Branches:

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

Children:

e6b7b198

Parents:

bfd7aac

Message:

• change in the pq_add interface

Location:

uspace/srv/net/structures/packet

Files:

• packet.c (modified) (3 diffs)
• packet.h (modified) (1 diff)
• packet_server.c (modified) (1 diff)

uspace/srv/net/structures/packet/packet.c

@@ -184 +184 @@
 }
 
-packet_t pq_add( packet_t first, packet_t packet, size_t order, size_t metric ){
+int pq_add( packet_t * first, packet_t packet, size_t order, size_t metric ){
         packet_t        item;
 
-        if( ! packet_is_valid( packet )) return NULL;
+        if(( ! first ) || ( ! packet_is_valid( packet ))) return EINVAL;
         pq_set_order( packet, order, metric );
-        if( packet_is_valid( first )){
-                item = first;
+        if( packet_is_valid( * first )){
+                item = * first;
                 do{
                         if( item->order < order ){
@@ -198 +198 @@
                                         item->next = packet->packet_id;
                                         packet->previous = item->packet_id;
-                                        return first;
+                                        return EOK;
                                 }
                         }else{
@@ -205 +205 @@
                                 item->previous = packet->packet_id;
                                 item = pm_find( packet->previous );
-                                if( item ) item->next = packet->packet_id;
-                                return item ? first : packet;
+                                if( item ){
+                                        item->next = packet->packet_id;
+                                }else{
+                                        * first = packet;
+                                }
+                                return EOK;
                         }
                 }while( packet_is_valid( item ));
         }
-        return packet;
+        * first = packet;
+        return EOK;
 }
 

uspace/srv/net/structures/packet/packet.h

@@ -86 +86 @@
  *  The queue is sorted in the ascending order.
  *  The packet is inserted right before the packets of the same order value.
- *  @param[in] first The first packet of the queue. May be NULL.
+ *  @param[in,out] first The first packet of the queue. Sets the first packet of the queue. The original first packet may be shifted by the new packet.
  *  @param[in] packet The packet to be added.
  *  @param[in] order The packet order value.
  *  @param[in] metric The metric value of the packet.
- *  @returns The first packet of the queue. The original first packet may be shifted by the new packet.
- *  @returns NULL if the packet is not valid.
+ *  @returns EOK on success.
+ *  @returns EINVAL if the first parameter is NULL.
+ *  @returns EINVAL if the packet is not valid.
  */
-packet_t        pq_add( packet_t first, packet_t packet, size_t order, size_t metric );
+int     pq_add( packet_t * first, packet_t packet, size_t order, size_t metric );
 
 /** Finds the packet with the given order.

uspace/srv/net/structures/packet/packet_server.c

@@ -229 +229 @@
 void packet_release( packet_t packet ){
         int index;
+        int result;
 
         // remove debug dump
 //      printf( "packet %d released\n", packet->packet_id );
         for( index = 0; ( index < FREE_QUEUES_COUNT - 1 ) && ( packet->length > ps_globals.sizes[ index ] ); ++ index );
-        ps_globals.free[ index ] = pq_add( ps_globals.free[ index ], packet, packet->length, packet->length );
-        assert( ps_globals.free[ index ] );
+        result = pq_add( & ps_globals.free[ index ], packet, packet->length, packet->length );
+        assert( result == EOK );
 }