Changeset 854151c in mainline for uspace/srv

Timestamp:

2011-01-07T12:32:23Z (14 years ago)

Author:

Martin Decky <martin@…>

Branches:

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

Children:

b404a98

Parents:

3d5e190

Message:

store addresses as uint8_t values (char can be both signed or unsigned depending on platform)
improve MAC address printouts

Location:

uspace/srv

Files:

• hw/netif/dp8390/dp8390.c (modified) (1 diff)
• net/il/arp/arp.c (modified) (2 diffs)
• net/il/arp/arp.h (modified) (3 diffs)
• net/nil/eth/eth.c (modified) (1 diff)
• net/nil/eth/eth.h (modified) (1 diff)
• net/nil/nildummy/nildummy.h (modified) (1 diff)

uspace/srv/hw/netif/dp8390/dp8390.c

@@ -263 +263 @@
         printf("Ethernet address ");
         for (i = 0; i < 6; i++)
-                printf("%x%c", dep->de_address.ea_addr[i], i < 5 ? ':' : '\n');
+                printf("%02x%c", dep->de_address.ea_addr[i], i < 5 ? ':' : '\n');
         
         /*

uspace/srv/net/il/arp/arp.c

@@ -215 +215 @@
         (*proto)->service = service;
         (*proto)->addr = address;
-        (*proto)->addr_data = address->value;
+        (*proto)->addr_data = (uint8_t *) address->value;
         
         rc = arp_addr_initialize(&(*proto)->addresses);
@@ -267 +267 @@
                         free(proto->addr_data);
                         proto->addr = address;
-                        proto->addr_data = address->value;
+                        proto->addr_data = (uint8_t *) address->value;
                 } else {
                         rc = arp_proto_create(&proto, protocol, address);

uspace/srv/net/il/arp/arp.h

@@ -96 +96 @@
         measured_string_t *addr;
         /** Actual device hardware address data. */
-        char *addr_data;
+        uint8_t *addr_data;
         /** Broadcast device hardware address. */
         measured_string_t *broadcast_addr;
         /** Broadcast device hardware address data. */
-        char *broadcast_data;
+        uint8_t *broadcast_data;
         /** Device identifier. */
         device_id_t device_id;
@@ -142 +142 @@
         measured_string_t *addr;
         /** Actual device protocol address data. */
-        char *addr_data;
+        uint8_t *addr_data;
         /** Address map. */
         arp_addr_t addresses;
@@ -154 +154 @@
          * Hardware address for the translation. NULL denotes an incomplete
          * record with possible waiters.
-         */ 
+         */
         measured_string_t *hw_addr;
         /** Condition variable used for waiting for completion of the record. */

uspace/srv/net/nil/eth/eth.c

@@ -407 +407 @@
         
         printf("%s: Device registered (id: %d, service: %d: mtu: %zu, "
-            "mac: %x:%x:%x:%x:%x:%x, flags: 0x%x)\n",
+            "mac: %02x:%02x:%02x:%02x:%02x:%02x, flags: 0x%x)\n",
             NAME, device->device_id, device->service, device->mtu,
             device->addr_data[0], device->addr_data[1],

uspace/srv/net/nil/eth/eth.h

@@ -91 +91 @@
         /** Actual device hardware address. */
         measured_string_t *addr;
+        
         /** Actual device hardware address data. */
-        char *addr_data;
+        uint8_t *addr_data;
 };
 

uspace/srv/net/nil/nildummy/nildummy.h

@@ -78 +78 @@
         measured_string_t *addr;
         /** Actual device hardware address data. */
-        char *addr_data;
+        uint8_t *addr_data;
 };