Changeset 76b5a95c in mainline for uspace/srv

Timestamp:

2011-03-23T20:53:30Z (15 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

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

Children:

71af5a4

Parents:

5716e9a (diff), ab10b842 (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Merge mainline changes.

Location:

uspace/srv

Files:

• bd/ata_bd/ata_bd.c (modified) (23 diffs)
• bd/ata_bd/ata_bd.h (modified) (4 diffs)
• bd/ata_bd/ata_hw.h (modified) (4 diffs)
• devman/devman.c (modified) (34 diffs)
• devman/devman.h (modified) (13 diffs)
• devman/main.c (modified) (14 diffs)
• devman/match.c (modified) (1 diff)
• devmap/devmap.c (modified) (2 diffs)
• hid/console/gcons.c (modified) (1 diff)
• hid/kbd/Makefile (modified) (1 diff)
• hid/kbd/ctl/pl050.c (deleted)
• loader/arch/abs32le/_link.ld.in (modified) (2 diffs)
• loader/arch/amd64/_link.ld.in (modified) (7 diffs)
• loader/arch/arm32/_link.ld.in (modified) (5 diffs)
• loader/arch/ia32/_link.ld.in (modified) (3 diffs)
• loader/arch/ia64/_link.ld.in (modified) (5 diffs)
• loader/arch/mips32/_link.ld.in (modified) (5 diffs)
• loader/arch/ppc32/_link.ld.in (modified) (6 diffs)
• loader/arch/sparc64/_link.ld.in (modified) (4 diffs)
• loader/elf_load.c (modified) (7 diffs)
• net/tl/udp/udp.c (modified) (1 diff)

uspace/srv/bd/ata_bd/ata_bd.c

@@ -55 +55 @@
 #include <as.h>
 #include <fibril_synch.h>
+#include <stdint.h>
 #include <str.h>
 #include <devmap.h>
@@ -61 +62 @@
 #include <errno.h>
 #include <bool.h>
+#include <byteorder.h>
 #include <task.h>
 #include <macros.h>
@@ -73 +75 @@
 #define LEGACY_CTLS 4
 
-/** Physical block size. Should be always 512. */
-static const size_t block_size = 512;
+/**
+ * Size of data returned from Identify Device or Identify Packet Device
+ * command.
+ */
+static const size_t identify_data_size = 512;
 
 /** Size of the communication area. */
@@ -105 +110 @@
 static int ata_bd_write_blocks(int disk_id, uint64_t ba, size_t cnt,
     const void *buf);
-static int ata_bd_read_block(int disk_id, uint64_t ba, size_t cnt,
+static int ata_rcmd_read(int disk_id, uint64_t ba, size_t cnt,
     void *buf);
-static int ata_bd_write_block(int disk_id, uint64_t ba, size_t cnt,
+static int ata_rcmd_write(int disk_id, uint64_t ba, size_t cnt,
     const void *buf);
 static int disk_init(disk_t *d, int disk_id);
 static int drive_identify(int drive_id, void *buf);
+static int identify_pkt_dev(int dev_idx, void *buf);
+static int ata_cmd_packet(int dev_idx, const void *cpkt, size_t cpkt_size,
+    void *obuf, size_t obuf_size);
+static int ata_pcmd_inquiry(int dev_idx, void *obuf, size_t obuf_size);
+static int ata_pcmd_read_12(int dev_idx, uint64_t ba, size_t cnt,
+    void *obuf, size_t obuf_size);
 static void disk_print_summary(disk_t *d);
 static int coord_calc(disk_t *d, uint64_t ba, block_coord_t *bc);
@@ -203 +214 @@
         printf("%s: ", d->model);
 
-        switch (d->amode) {
-        case am_chs:
-                printf("CHS %u cylinders, %u heads, %u sectors",
-                    disk->geom.cylinders, disk->geom.heads, disk->geom.sectors);
-                break;
-        case am_lba28:
-                printf("LBA-28");
-                break;
-        case am_lba48:
-                printf("LBA-48");
-                break;
+        if (d->dev_type == ata_reg_dev) {
+                switch (d->amode) {
+                case am_chs:
+                        printf("CHS %u cylinders, %u heads, %u sectors",
+                            disk->geom.cylinders, disk->geom.heads,
+                            disk->geom.sectors);
+                        break;
+                case am_lba28:
+                        printf("LBA-28");
+                        break;
+                case am_lba48:
+                        printf("LBA-48");
+                        break;
+                }
+        } else {
+                printf("PACKET");
         }
 
@@ -313 +329 @@
                             IPC_GET_ARG2(call));
                         cnt = IPC_GET_ARG3(call);
-                        if (cnt * block_size > comm_size) {
+                        if (cnt * disk[disk_id].block_size > comm_size) {
                                 retval = ELIMIT;
                                 break;
@@ -323 +339 @@
                             IPC_GET_ARG2(call));
                         cnt = IPC_GET_ARG3(call);
-                        if (cnt * block_size > comm_size) {
+                        if (cnt * disk[disk_id].block_size > comm_size) {
                                 retval = ELIMIT;
                                 break;
@@ -330 +346 @@
                         break;
                 case BD_GET_BLOCK_SIZE:
-                        async_answer_1(callid, EOK, block_size);
+                        async_answer_1(callid, EOK, disk[disk_id].block_size);
                         continue;
                 case BD_GET_NUM_BLOCKS:
@@ -353 +369 @@
         identify_data_t idata;
         uint8_t model[40];
+        ata_inquiry_data_t inq_data;
         uint16_t w;
         uint8_t c;
@@ -359 +376 @@
         unsigned i;
 
+        d->present = false;
+        fibril_mutex_initialize(&d->lock);
+
+        /* Try identify command. */
         rc = drive_identify(disk_id, &idata);
-        if (rc != EOK) {
-                d->present = false;
-                return rc;
-        }
-
-        if ((idata.caps & cap_lba) == 0) {
+        if (rc == EOK) {
+                /* Success. It's a register (non-packet) device. */
+                printf("ATA register-only device found.\n");
+                d->dev_type = ata_reg_dev;
+        } else if (rc == EIO) {
+                /*
+                 * There is something, but not a register device.
+                 * It could be a packet device.
+                 */
+                rc = identify_pkt_dev(disk_id, &idata);
+                if (rc == EOK) {
+                        /* We have a packet device. */
+                        d->dev_type = ata_pkt_dev;
+                } else {
+                        /* Nope. Something's there, but not recognized. */
+                        return EIO;
+                }
+        } else {
+                /* Operation timed out. That means there is no device there. */
+                return EIO;
+        }
+
+        printf("device caps: 0x%04x\n", idata.caps);
+        if (d->dev_type == ata_pkt_dev) {
+                /* Packet device */
+                d->amode = 0;
+
+                d->geom.cylinders = 0;
+                d->geom.heads = 0;
+                d->geom.sectors = 0;
+
+                d->blocks = 0;
+        } else if ((idata.caps & rd_cap_lba) == 0) {
                 /* Device only supports CHS addressing. */
                 d->amode = am_chs;
@@ -422 +470 @@
         d->model[pos] = '\0';
 
+        if (d->dev_type == ata_pkt_dev) {
+                /* Send inquiry. */
+                rc = ata_pcmd_inquiry(0, &inq_data, sizeof(inq_data));
+                if (rc != EOK) {
+                        printf("Device inquiry failed.\n");
+                        d->present = false;
+                        return EIO;
+                }
+
+                /* Check device type. */
+                if (INQUIRY_PDEV_TYPE(inq_data.pdev_type) != PDEV_TYPE_CDROM)
+                        printf("Warning: Peripheral device type is not CD-ROM.\n");
+
+                /* Assume 2k block size for now. */
+                d->block_size = 2048;
+        } else {
+                /* Assume register Read always uses 512-byte blocks. */
+                d->block_size = 512;
+        }
+
         d->present = true;
-        fibril_mutex_initialize(&d->lock);
-
         return EOK;
 }
@@ -435 +501 @@
 
         while (cnt > 0) {
-                rc = ata_bd_read_block(disk_id, ba, 1, buf);
+                if (disk[disk_id].dev_type == ata_reg_dev)
+                        rc = ata_rcmd_read(disk_id, ba, 1, buf);
+                else
+                        rc = ata_pcmd_read_12(disk_id, ba, 1, buf,
+                            disk[disk_id].block_size);
+
                 if (rc != EOK)
                         return rc;
@@ -441 +512 @@
                 ++ba;
                 --cnt;
-                buf += block_size;
+                buf += disk[disk_id].block_size;
         }
 
@@ -453 +524 @@
         int rc;
 
+        if (disk[disk_id].dev_type != ata_reg_dev)
+                return ENOTSUP;
+
         while (cnt > 0) {
-                rc = ata_bd_write_block(disk_id, ba, 1, buf);
+                rc = ata_rcmd_write(disk_id, ba, 1, buf);
                 if (rc != EOK)
                         return rc;
@@ -460 +534 @@
                 ++ba;
                 --cnt;
-                buf += block_size;
+                buf += disk[disk_id].block_size;
         }
 
@@ -473 +547 @@
  * @param disk_id       Device ID, 0 or 1.
  * @param buf           Pointer to a 512-byte buffer.
+ *
+ * @return              ETIMEOUT on timeout (this can mean the device is
+ *                      not present). EIO if device responds with error.
  */
 static int drive_identify(int disk_id, void *buf)
@@ -484 +561 @@
 
         if (wait_status(0, ~SR_BSY, NULL, TIMEOUT_PROBE) != EOK)
-                return EIO;
+                return ETIMEOUT;
 
         pio_write_8(&cmd->drive_head, drv_head);
@@ -493 +570 @@
          */
         if (wait_status(SR_DRDY, ~SR_BSY, NULL, TIMEOUT_PROBE) != EOK)
-                return EIO;
+                return ETIMEOUT;
 
         pio_write_8(&cmd->command, CMD_IDENTIFY_DRIVE);
 
         if (wait_status(0, ~SR_BSY, &status, TIMEOUT_PROBE) != EOK)
-                return EIO;
+                return ETIMEOUT;
 
         /* Read data from the disk buffer. */
 
         if ((status & SR_DRQ) != 0) {
-                for (i = 0; i < block_size / 2; i++) {
+                for (i = 0; i < identify_data_size / 2; i++) {
                         data = pio_read_16(&cmd->data_port);
                         ((uint16_t *) buf)[i] = data;
@@ -509 +586 @@
         }
 
+        if ((status & SR_ERR) != 0) {
+                return EIO;
+        }
+
+        return EOK;
+}
+
+/** Issue Identify Packet Device command.
+ *
+ * Reads @c identify data into the provided buffer. This is used to detect
+ * whether an ATAPI device is present and if so, to determine its parameters.
+ *
+ * @param dev_idx       Device index, 0 or 1.
+ * @param buf           Pointer to a 512-byte buffer.
+ */
+static int identify_pkt_dev(int dev_idx, void *buf)
+{
+        uint16_t data;
+        uint8_t status;
+        uint8_t drv_head;
+        size_t i;
+
+        drv_head = ((dev_idx != 0) ? DHR_DRV : 0);
+
+        if (wait_status(0, ~SR_BSY, NULL, TIMEOUT_PROBE) != EOK)
+                return EIO;
+
+        pio_write_8(&cmd->drive_head, drv_head);
+
+        /* For ATAPI commands we do not need to wait for DRDY. */
+        if (wait_status(0, ~SR_BSY, NULL, TIMEOUT_PROBE) != EOK)
+                return EIO;
+
+        pio_write_8(&cmd->command, CMD_IDENTIFY_PKT_DEV);
+
+        if (wait_status(0, ~SR_BSY, &status, TIMEOUT_BSY) != EOK)
+                return EIO;
+
+        /* Read data from the device buffer. */
+
+        if ((status & SR_DRQ) != 0) {
+                for (i = 0; i < identify_data_size / 2; i++) {
+                        data = pio_read_16(&cmd->data_port);
+                        ((uint16_t *) buf)[i] = data;
+                }
+        }
+
         if ((status & SR_ERR) != 0)
                 return EIO;
+
+        return EOK;
+}
+
+/** Issue packet command (i. e. write a command packet to the device).
+ *
+ * Only data-in commands are supported (e.g. inquiry, read).
+ *
+ * @param dev_idx       Device index (0 or 1)
+ * @param obuf          Buffer for storing data read from device
+ * @param obuf_size     Size of obuf in bytes
+ *
+ * @return EOK on success, EIO on error.
+ */
+static int ata_cmd_packet(int dev_idx, const void *cpkt, size_t cpkt_size,
+    void *obuf, size_t obuf_size)
+{
+        size_t i;
+        uint8_t status;
+        uint8_t drv_head;
+        disk_t *d;
+        size_t data_size;
+        uint16_t val;
+
+        d = &disk[dev_idx];
+        fibril_mutex_lock(&d->lock);
+
+        /* New value for Drive/Head register */
+        drv_head =
+            ((dev_idx != 0) ? DHR_DRV : 0);
+
+        if (wait_status(0, ~SR_BSY, NULL, TIMEOUT_PROBE) != EOK) {
+                fibril_mutex_unlock(&d->lock);
+                return EIO;
+        }
+
+        pio_write_8(&cmd->drive_head, drv_head);
+
+        if (wait_status(0, ~(SR_BSY|SR_DRQ), NULL, TIMEOUT_BSY) != EOK) {
+                fibril_mutex_unlock(&d->lock);
+                return EIO;
+        }
+
+        /* Byte count <- max. number of bytes we can read in one transfer. */
+        pio_write_8(&cmd->cylinder_low, 0xfe);
+        pio_write_8(&cmd->cylinder_high, 0xff);
+
+        pio_write_8(&cmd->command, CMD_PACKET);
+
+        if (wait_status(SR_DRQ, ~SR_BSY, &status, TIMEOUT_BSY) != EOK) {
+                fibril_mutex_unlock(&d->lock);
+                return EIO;
+        }
+
+        /* Write command packet. */
+        for (i = 0; i < (cpkt_size + 1) / 2; i++)
+                pio_write_16(&cmd->data_port, ((uint16_t *) cpkt)[i]);
+
+        if (wait_status(0, ~SR_BSY, &status, TIMEOUT_BSY) != EOK) {
+                fibril_mutex_unlock(&d->lock);
+                return EIO;
+        }
+
+        if ((status & SR_DRQ) == 0) {
+                fibril_mutex_unlock(&d->lock);
+                return EIO;
+        }
+
+        /* Read byte count. */
+        data_size = (uint16_t) pio_read_8(&cmd->cylinder_low) +
+            ((uint16_t) pio_read_8(&cmd->cylinder_high) << 8);
+
+        /* Check whether data fits into output buffer. */
+        if (data_size > obuf_size) {
+                /* Output buffer is too small to store data. */
+                fibril_mutex_unlock(&d->lock);
+                return EIO;
+        }
+
+        /* Read data from the device buffer. */
+        for (i = 0; i < (data_size + 1) / 2; i++) {
+                val = pio_read_16(&cmd->data_port);
+                ((uint16_t *) obuf)[i] = val;
+        }
+
+        if (status & SR_ERR) {
+                fibril_mutex_unlock(&d->lock);
+                return EIO;
+        }
+
+        fibril_mutex_unlock(&d->lock);
+
+        return EOK;
+}
+
+/** Issue ATAPI Inquiry.
+ *
+ * @param dev_idx       Device index (0 or 1)
+ * @param obuf          Buffer for storing inquiry data read from device
+ * @param obuf_size     Size of obuf in bytes
+ *
+ * @return EOK on success, EIO on error.
+ */
+static int ata_pcmd_inquiry(int dev_idx, void *obuf, size_t obuf_size)
+{
+        ata_pcmd_inquiry_t cp;
+        int rc;
+
+        memset(&cp, 0, sizeof(cp));
+
+        cp.opcode = PCMD_INQUIRY;
+        cp.alloc_len = min(obuf_size, 0xff); /* Allocation length */
+
+        rc = ata_cmd_packet(0, &cp, sizeof(cp), obuf, obuf_size);
+        if (rc != EOK)
+                return rc;
+
+        return EOK;
+}
+
+/** Issue ATAPI read(12) command.
+ *
+ * Output buffer must be large enough to hold the data, otherwise the
+ * function will fail.
+ *
+ * @param dev_idx       Device index (0 or 1)
+ * @param ba            Starting block address
+ * @param cnt           Number of blocks to read
+ * @param obuf          Buffer for storing inquiry data read from device
+ * @param obuf_size     Size of obuf in bytes
+ *
+ * @return EOK on success, EIO on error.
+ */
+static int ata_pcmd_read_12(int dev_idx, uint64_t ba, size_t cnt,
+    void *obuf, size_t obuf_size)
+{
+        ata_pcmd_read_12_t cp;
+        int rc;
+
+        if (ba > UINT32_MAX)
+                return EINVAL;
+
+        memset(&cp, 0, sizeof(cp));
+
+        cp.opcode = PCMD_READ_12;
+        cp.ba = host2uint32_t_be(ba);
+        cp.nblocks = host2uint32_t_be(cnt);
+
+        rc = ata_cmd_packet(0, &cp, sizeof(cp), obuf, obuf_size);
+        if (rc != EOK)
+                return rc;
 
         return EOK;
@@ -524 +799 @@
  * @return EOK on success, EIO on error.
  */
-static int ata_bd_read_block(int disk_id, uint64_t ba, size_t blk_cnt,
+static int ata_rcmd_read(int disk_id, uint64_t ba, size_t blk_cnt,
     void *buf)
 {
@@ -579 +854 @@
                 /* Read data from the device buffer. */
 
-                for (i = 0; i < block_size / 2; i++) {
+                for (i = 0; i < disk[disk_id].block_size / 2; i++) {
                         data = pio_read_16(&cmd->data_port);
                         ((uint16_t *) buf)[i] = data;
@@ -601 +876 @@
  * @return EOK on success, EIO on error.
  */
-static int ata_bd_write_block(int disk_id, uint64_t ba, size_t cnt,
+static int ata_rcmd_write(int disk_id, uint64_t ba, size_t cnt,
     const void *buf)
 {
@@ -655 +930 @@
                 /* Write data to the device buffer. */
 
-                for (i = 0; i < block_size / 2; i++) {
+                for (i = 0; i < disk[disk_id].block_size / 2; i++) {
                         pio_write_16(&cmd->data_port, ((uint16_t *) buf)[i]);
                 }

uspace/srv/bd/ata_bd/ata_bd.h

@@ -53 +53 @@
 };
 
-/** Block addressing mode. */
-enum addr_mode {
+enum ata_dev_type {
+        ata_reg_dev,    /* Register device (no packet feature set support) */
+        ata_pkt_dev     /* Packet device (supports packet feature set). */
+};
+
+/** Register device block addressing mode. */
+enum rd_addr_mode {
         am_chs,         /**< CHS block addressing */
         am_lba28,       /**< LBA-28 block addressing */
@@ -62 +67 @@
 /** Block coordinates */
 typedef struct {
-        /** Addressing mode used */
-        enum addr_mode amode;
+        enum rd_addr_mode amode;
 
         union {
@@ -90 +94 @@
 typedef struct {
         bool present;
-        enum addr_mode amode;
+
+        /** Device type */
+        enum ata_dev_type dev_type;
+
+        /** Addressing mode to use (if register device) */
+        enum rd_addr_mode amode;
 
         /*
@@ -102 +111 @@
 
         uint64_t blocks;
+        size_t block_size;
 
         char model[STR_BOUNDS(40) + 1];

uspace/srv/bd/ata_bd/ata_hw.h

@@ -134 +134 @@
         CMD_WRITE_SECTORS       = 0x30,
         CMD_WRITE_SECTORS_EXT   = 0x34,
+        CMD_PACKET              = 0xA0,
+        CMD_IDENTIFY_PKT_DEV    = 0xA1,
         CMD_IDENTIFY_DRIVE      = 0xEC
 };
 
-/** Data returned from @c identify command. */
+/** Data returned from identify device and identify packet device command. */
 typedef struct {
         uint16_t gen_conf;
@@ -159 +161 @@
         uint16_t max_rw_multiple;
         uint16_t _res48;
-        uint16_t caps;
+        uint16_t caps;          /* Different meaning for packet device */
         uint16_t _res50;
         uint16_t pio_timing;
@@ -214 +216 @@
 } identify_data_t;
 
-enum ata_caps {
-        cap_iordy       = 0x0800,
-        cap_iordy_cbd   = 0x0400,
-        cap_lba         = 0x0200,
-        cap_dma         = 0x0100
+/** Capability bits for register device. */
+enum ata_regdev_caps {
+        rd_cap_iordy            = 0x0800,
+        rd_cap_iordy_cbd        = 0x0400,
+        rd_cap_lba              = 0x0200,
+        rd_cap_dma              = 0x0100
+};
+
+/** Capability bits for packet device. */
+enum ata_pktdev_caps {
+        pd_cap_ildma            = 0x8000,
+        pd_cap_cmdqueue         = 0x4000,
+        pd_cap_overlap          = 0x2000,
+        pd_cap_need_softreset   = 0x1000,       /* Obsolete (ATAPI-6) */
+        pd_cap_iordy            = 0x0800,
+        pd_cap_iordy_dis        = 0x0400,
+        pd_cap_lba              = 0x0200,       /* Must be on */
+        pd_cap_dma              = 0x0100
 };
 
@@ -226 +241 @@
 };
 
+/** ATA packet command codes. */
+enum ata_pkt_command {
+        PCMD_INQUIRY            = 0x12,
+        PCMD_READ_12            = 0xa8
+};
+
+/** ATAPI Inquiry command */
+typedef struct {
+        uint8_t opcode;         /**< Operation code (PCMD_INQUIRY) */
+        uint8_t _res0;
+        uint8_t _res1;
+        uint8_t _res2;
+        uint8_t alloc_len;      /**< Allocation length */
+        uint8_t _res3;
+        uint8_t _res4;
+        uint8_t _res5;
+        uint32_t _res6;
+} __attribute__ ((packed)) ata_pcmd_inquiry_t;
+
+/** ATAPI Read(12) command */
+typedef struct {
+        uint8_t opcode;         /**< Operation code (PCMD_READ_12) */
+        uint8_t _res0;
+        uint32_t ba;            /**< Starting block address */
+        uint32_t nblocks;       /**< Number of blocks to transfer */
+        uint8_t _res1;
+        uint8_t _res2;
+} __attribute__ ((packed)) ata_pcmd_read_12_t;
+
+/** Data returned from Inquiry command (mandatory part) */
+typedef struct {
+        uint8_t pdev_type;      /** Reserved, Peripheral device type */
+        uint8_t rmb;            /** RMB, Reserved */
+        uint8_t std_version;    /** ISO version, ECMA version, ANSI version */
+        uint8_t atapi_ver_rdf;  /** ATAPI version, Response data format */
+        uint8_t additional_len; /** Additional length */
+        uint8_t _res0;
+        uint8_t _res1;
+        uint8_t _res2;
+        uint8_t vendor_id[8];   /** Vendor ID */
+        uint8_t product_id[8];  /** Product ID */
+        uint8_t product_rev[4]; /** Product revision level */
+} ata_inquiry_data_t;
+
+/** Extract value of ata_inquiry_data_t.pdev_type */
+#define INQUIRY_PDEV_TYPE(val) ((val) & 0x1f)
+
+/** Values for ata_inquiry_data_t.pdev_type */
+enum ata_pdev_type {
+        PDEV_TYPE_CDROM         = 0x05
+};
+
 #endif
 

uspace/srv/devman/devman.c

@@ -41 +41 @@
 #include "devman.h"
 
+fun_node_t *find_node_child(fun_node_t *parent, const char *name);
+
 /* hash table operations */
 
@@ -51 +53 @@
     link_t *item)
 {
-        node_t *dev = hash_table_get_instance(item, node_t, devman_link);
+        dev_node_t *dev = hash_table_get_instance(item, dev_node_t, devman_dev);
         return (dev->handle == (devman_handle_t) key[0]);
 }
 
-static int devmap_devices_compare(unsigned long key[], hash_count_t keys,
+static int devman_functions_compare(unsigned long key[], hash_count_t keys,
     link_t *item)
 {
-        node_t *dev = hash_table_get_instance(item, node_t, devmap_link);
-        return (dev->devmap_handle == (devmap_handle_t) key[0]);
+        fun_node_t *fun = hash_table_get_instance(item, fun_node_t, devman_fun);
+        return (fun->handle == (devman_handle_t) key[0]);
+}
+
+static int devmap_functions_compare(unsigned long key[], hash_count_t keys,
+    link_t *item)
+{
+        fun_node_t *fun = hash_table_get_instance(item, fun_node_t, devmap_fun);
+        return (fun->devmap_handle == (devmap_handle_t) key[0]);
 }
 
@@ -82 +91 @@
 };
 
+static hash_table_operations_t devman_functions_ops = {
+        .hash = devices_hash,
+        .compare = devman_functions_compare,
+        .remove_callback = devices_remove_callback
+};
+
 static hash_table_operations_t devmap_devices_ops = {
         .hash = devices_hash,
-        .compare = devmap_devices_compare,
+        .compare = devmap_functions_compare,
         .remove_callback = devices_remove_callback
 };
@@ -373 +388 @@
 }
 
-/** Create root device node in the device tree.
+/** Create root device and function node in the device tree.
  *
  * @param tree  The device tree.
  * @return      True on success, false otherwise.
  */
-bool create_root_node(dev_tree_t *tree)
-{
-        node_t *node;
-
-        printf(NAME ": create_root_node\n");
-
+bool create_root_nodes(dev_tree_t *tree)
+{
+        fun_node_t *fun;
+        dev_node_t *dev;
+        
+        printf(NAME ": create_root_nodes\n");
+        
         fibril_rwlock_write_lock(&tree->rwlock);
-        node = create_dev_node();
-        if (node != NULL) {
-                insert_dev_node(tree, node, clone_string(""), NULL);
-                match_id_t *id = create_match_id();
-                id->id = clone_string("root");
-                id->score = 100;
-                add_match_id(&node->match_ids, id);
-                tree->root_node = node;
-        }
+        
+        /*
+         * Create root function. This is a pseudo function to which
+         * the root device node is attached. It allows us to match
+         * the root device driver in a standard manner, i.e. against
+         * the parent function.
+         */
+        
+        fun = create_fun_node();
+        if (fun == NULL) {
+                fibril_rwlock_write_unlock(&tree->rwlock);
+                return false;
+        }
+        
+        insert_fun_node(tree, fun, clone_string(""), NULL);
+        match_id_t *id = create_match_id();
+        id->id = clone_string("root");
+        id->score = 100;
+        add_match_id(&fun->match_ids, id);
+        tree->root_node = fun;
+        
+        /*
+         * Create root device node.
+         */
+        dev = create_dev_node();
+        if (dev == NULL) {
+                fibril_rwlock_write_unlock(&tree->rwlock);
+                return false;
+        }
+        
+        insert_dev_node(tree, dev, fun);
+        
         fibril_rwlock_write_unlock(&tree->rwlock);
-
-        return node != NULL;
+        
+        return dev != NULL;
 }
 
@@ -414 +453 @@
  *                      is found.
  */
-driver_t *find_best_match_driver(driver_list_t *drivers_list, node_t *node)
+driver_t *find_best_match_driver(driver_list_t *drivers_list, dev_node_t *node)
 {
         driver_t *best_drv = NULL, *drv = NULL;
@@ -442 +481 @@
  * @param drv           The driver.
  */
-void attach_driver(node_t *node, driver_t *drv)
+void attach_driver(dev_node_t *dev, driver_t *drv)
 {
         printf(NAME ": attach_driver %s to device %s\n",
-            drv->name, node->pathname);
+            drv->name, dev->pfun->pathname);
         
         fibril_mutex_lock(&drv->driver_mutex);
         
-        node->drv = drv;
-        list_append(&node->driver_devices, &drv->devices);
+        dev->drv = drv;
+        list_append(&dev->driver_devices, &drv->devices);
         
         fibril_mutex_unlock(&drv->driver_mutex);
@@ -530 +569 @@
 static void pass_devices_to_driver(driver_t *driver, dev_tree_t *tree)
 {
-        node_t *dev;
+        dev_node_t *dev;
         link_t *link;
         int phone;
@@ -551 +590 @@
         link = driver->devices.next;
         while (link != &driver->devices) {
-                dev = list_get_instance(link, node_t, driver_devices);
+                dev = list_get_instance(link, dev_node_t, driver_devices);
                 if (dev->passed_to_driver) {
                         link = link->next;
@@ -669 +708 @@
 }
 
-/** Create devmap path and name for the device. */
-static void devmap_register_tree_device(node_t *node, dev_tree_t *tree)
+/** Create devmap path and name for the function. */
+void devmap_register_tree_function(fun_node_t *fun, dev_tree_t *tree)
 {
         char *devmap_pathname = NULL;
         char *devmap_name = NULL;
         
-        asprintf(&devmap_name, "%s", node->pathname);
+        asprintf(&devmap_name, "%s", fun->pathname);
         if (devmap_name == NULL)
                 return;
@@ -689 +728 @@
         
         devmap_device_register_with_iface(devmap_pathname,
-            &node->devmap_handle, DEVMAN_CONNECT_FROM_DEVMAP);
-        
-        tree_add_devmap_device(tree, node);
+            &fun->devmap_handle, DEVMAN_CONNECT_FROM_DEVMAP);
+        
+        tree_add_devmap_function(tree, fun);
         
         free(devmap_name);
@@ -702 +741 @@
  * @param node          The device's node in the device tree.
  */
-void add_device(int phone, driver_t *drv, node_t *node, dev_tree_t *tree)
+void add_device(int phone, driver_t *drv, dev_node_t *dev, dev_tree_t *tree)
 {
         /*
@@ -709 +748 @@
          */
         printf(NAME ": add_device (driver `%s', device `%s')\n", drv->name,
-            node->name);
+            dev->pfun->name);
         
         sysarg_t rc;
@@ -716 +755 @@
         /* Send the device to the driver. */
         devman_handle_t parent_handle;
-        if (node->parent) {
-                parent_handle = node->parent->handle;
+        if (dev->pfun) {
+                parent_handle = dev->pfun->handle;
         } else {
                 parent_handle = 0;
         }
 
-        aid_t req = async_send_2(phone, DRIVER_ADD_DEVICE, node->handle,
+        aid_t req = async_send_2(phone, DRIVER_ADD_DEVICE, dev->handle,
             parent_handle, &answer);
         
         /* Send the device's name to the driver. */
-        rc = async_data_write_start(phone, node->name,
-            str_size(node->name) + 1);
+        rc = async_data_write_start(phone, dev->pfun->name,
+            str_size(dev->pfun->name) + 1);
         if (rc != EOK) {
                 /* TODO handle error */
@@ -737 +776 @@
         switch(rc) {
         case EOK:
-                node->state = DEVICE_USABLE;
-                devmap_register_tree_device(node, tree);
+                dev->state = DEVICE_USABLE;
                 break;
         case ENOENT:
-                node->state = DEVICE_NOT_PRESENT;
+                dev->state = DEVICE_NOT_PRESENT;
                 break;
         default:
-                node->state = DEVICE_INVALID;
-        }
-        
-        node->passed_to_driver = true;
+                dev->state = DEVICE_INVALID;
+        }
+        
+        dev->passed_to_driver = true;
 
         return;
@@ -759 +797 @@
  *                      successfully assigned to the device, false otherwise.
  */
-bool assign_driver(node_t *node, driver_list_t *drivers_list, dev_tree_t *tree)
-{
+bool assign_driver(dev_node_t *dev, driver_list_t *drivers_list,
+    dev_tree_t *tree)
+{
+        assert(dev != NULL);
+        assert(drivers_list != NULL);
+        assert(tree != NULL);
+        
         /*
          * Find the driver which is the most suitable for handling this device.
          */
-        driver_t *drv = find_best_match_driver(drivers_list, node);
+        driver_t *drv = find_best_match_driver(drivers_list, dev);
         if (drv == NULL) {
                 printf(NAME ": no driver found for device '%s'.\n",
-                    node->pathname);
+                    dev->pfun->pathname);
                 return false;
         }
         
         /* Attach the driver to the device. */
-        attach_driver(node, drv);
+        attach_driver(dev, drv);
         
         fibril_mutex_lock(&drv->driver_mutex);
@@ -786 +829 @@
                 int phone = async_connect_me_to(drv->phone, DRIVER_DEVMAN, 0, 0);
                 if (phone >= 0) {
-                        add_device(phone, drv, node, tree);
+                        add_device(phone, drv, dev, tree);
                         async_hangup(phone);
                 }
@@ -810 +853 @@
         hash_table_create(&tree->devman_devices, DEVICE_BUCKETS, 1,
             &devman_devices_ops);
-        hash_table_create(&tree->devmap_devices, DEVICE_BUCKETS, 1,
+        hash_table_create(&tree->devman_functions, DEVICE_BUCKETS, 1,
+            &devman_functions_ops);
+        hash_table_create(&tree->devmap_functions, DEVICE_BUCKETS, 1,
             &devmap_devices_ops);
         
         fibril_rwlock_initialize(&tree->rwlock);
         
-        /* Create root node and add it to the device tree. */
-        if (!create_root_node(tree))
+        /* Create root function and root device and add them to the device tree. */
+        if (!create_root_nodes(tree))
                 return false;
 
         /* Find suitable driver and start it. */
-        return assign_driver(tree->root_node, drivers_list, tree);
+        return assign_driver(tree->root_node->child, drivers_list, tree);
 }
 
@@ -829 +874 @@
  * @return              A device node structure.
  */
-node_t *create_dev_node(void)
-{
-        node_t *res = malloc(sizeof(node_t));
+dev_node_t *create_dev_node(void)
+{
+        dev_node_t *res = malloc(sizeof(dev_node_t));
         
         if (res != NULL) {
-                memset(res, 0, sizeof(node_t));
-                list_initialize(&res->children);
-                list_initialize(&res->match_ids.ids);
-                list_initialize(&res->classes);
+                memset(res, 0, sizeof(dev_node_t));
+                list_initialize(&res->functions);
+                link_initialize(&res->driver_devices);
+                link_initialize(&res->devman_dev);
         }
         
@@ -847 +892 @@
  * @param node          The device node structure.
  */
-void delete_dev_node(node_t *node)
-{
-        assert(list_empty(&node->children));
-        assert(node->parent == NULL);
-        assert(node->drv == NULL);
-        
-        clean_match_ids(&node->match_ids);
-        free_not_null(node->name);
-        free_not_null(node->pathname);
-        free(node);
+void delete_dev_node(dev_node_t *dev)
+{
+        assert(list_empty(&dev->functions));
+        assert(dev->pfun == NULL);
+        assert(dev->drv == NULL);
+        
+        free(dev);
 }
 
@@ -865 +907 @@
  * @return              The device node.
  */
-node_t *find_dev_node_no_lock(dev_tree_t *tree, devman_handle_t handle)
+dev_node_t *find_dev_node_no_lock(dev_tree_t *tree, devman_handle_t handle)
 {
         unsigned long key = handle;
@@ -873 +915 @@
         
         link = hash_table_find(&tree->devman_devices, &key);
-        return hash_table_get_instance(link, node_t, devman_link);
+        return hash_table_get_instance(link, dev_node_t, devman_dev);
 }
 
@@ -882 +924 @@
  * @return              The device node.
  */
-node_t *find_dev_node(dev_tree_t *tree, devman_handle_t handle)
-{
-        node_t *node = NULL;
+dev_node_t *find_dev_node(dev_tree_t *tree, devman_handle_t handle)
+{
+        dev_node_t *dev = NULL;
         
         fibril_rwlock_read_lock(&tree->rwlock);
-        node = find_dev_node_no_lock(tree, handle);
+        dev = find_dev_node_no_lock(tree, handle);
         fibril_rwlock_read_unlock(&tree->rwlock);
         
-        return node;
-}
-
+        return dev;
+}
+
+/* Function nodes */
+
+/** Create a new function node.
+ *
+ * @return              A function node structure.
+ */
+fun_node_t *create_fun_node(void)
+{
+        fun_node_t *res = malloc(sizeof(fun_node_t));
+        
+        if (res != NULL) {
+                memset(res, 0, sizeof(fun_node_t));
+                link_initialize(&res->dev_functions);
+                list_initialize(&res->match_ids.ids);
+                list_initialize(&res->classes);
+                link_initialize(&res->devman_fun);
+                link_initialize(&res->devmap_fun);
+        }
+        
+        return res;
+}
+
+/** Delete a function node.
+ *
+ * @param fun           The device node structure.
+ */
+void delete_fun_node(fun_node_t *fun)
+{
+        assert(fun->dev == NULL);
+        assert(fun->child == NULL);
+        
+        clean_match_ids(&fun->match_ids);
+        free_not_null(fun->name);
+        free_not_null(fun->pathname);
+        free(fun);
+}
+
+/** Find the function node with the specified handle.
+ *
+ * @param tree          The device tree where we look for the device node.
+ * @param handle        The handle of the function.
+ * @return              The function node.
+ */
+fun_node_t *find_fun_node_no_lock(dev_tree_t *tree, devman_handle_t handle)
+{
+        unsigned long key = handle;
+        link_t *link;
+        
+        assert(fibril_rwlock_is_locked(&tree->rwlock));
+        
+        link = hash_table_find(&tree->devman_functions, &key);
+        if (link == NULL)
+                return NULL;
+        
+        return hash_table_get_instance(link, fun_node_t, devman_fun);
+}
+
+/** Find the function node with the specified handle.
+ *
+ * @param tree          The device tree where we look for the device node.
+ * @param handle        The handle of the function.
+ * @return              The function node.
+ */
+fun_node_t *find_fun_node(dev_tree_t *tree, devman_handle_t handle)
+{
+        fun_node_t *fun = NULL;
+        
+        fibril_rwlock_read_lock(&tree->rwlock);
+        fun = find_fun_node_no_lock(tree, handle);
+        fibril_rwlock_read_unlock(&tree->rwlock);
+        
+        return fun;
+}
 
 /** Create and set device's full path in device tree.
@@ -901 +1016 @@
  *                      resources etc.).
  */
-static bool set_dev_path(node_t *node, node_t *parent)
-{
-        assert(node->name != NULL);
-        
-        size_t pathsize = (str_size(node->name) + 1);
+static bool set_fun_path(fun_node_t *fun, fun_node_t *parent)
+{
+        assert(fun->name != NULL);
+        
+        size_t pathsize = (str_size(fun->name) + 1);
         if (parent != NULL)
                 pathsize += str_size(parent->pathname) + 1;
         
-        node->pathname = (char *) malloc(pathsize);
-        if (node->pathname == NULL) {
+        fun->pathname = (char *) malloc(pathsize);
+        if (fun->pathname == NULL) {
                 printf(NAME ": failed to allocate device path.\n");
                 return false;
@@ -916 +1031 @@
         
         if (parent != NULL) {
-                str_cpy(node->pathname, pathsize, parent->pathname);
-                str_append(node->pathname, pathsize, "/");
-                str_append(node->pathname, pathsize, node->name);
+                str_cpy(fun->pathname, pathsize, parent->pathname);
+                str_append(fun->pathname, pathsize, "/");
+                str_append(fun->pathname, pathsize, fun->name);
         } else {
-                str_cpy(node->pathname, pathsize, node->name);
+                str_cpy(fun->pathname, pathsize, fun->name);
         }
         
@@ -936 +1051 @@
  *                      etc.).
  */
-bool insert_dev_node(dev_tree_t *tree, node_t *node, char *dev_name,
-    node_t *parent)
-{
-        assert(node != NULL);
+bool insert_dev_node(dev_tree_t *tree, dev_node_t *dev, fun_node_t *pfun)
+{
+        assert(dev != NULL);
         assert(tree != NULL);
-        assert(dev_name != NULL);
         assert(fibril_rwlock_is_write_locked(&tree->rwlock));
         
-        node->name = dev_name;
-        if (!set_dev_path(node, parent)) {
+        /* Add the node to the handle-to-node map. */
+        dev->handle = ++tree->current_handle;
+        unsigned long key = dev->handle;
+        hash_table_insert(&tree->devman_devices, &key, &dev->devman_dev);
+
+        /* Add the node to the list of its parent's children. */
+        printf("insert_dev_node: dev=%p, dev->pfun := %p\n", dev, pfun);
+        dev->pfun = pfun;
+        pfun->child = dev;
+        
+        return true;
+}
+
+/** Insert new function into device tree.
+ *
+ * @param tree          The device tree.
+ * @param node          The newly added function node. 
+ * @param dev_name      The name of the newly added function.
+ * @param parent        Owning device node.
+ *
+ * @return              True on success, false otherwise (insufficient resources
+ *                      etc.).
+ */
+bool insert_fun_node(dev_tree_t *tree, fun_node_t *fun, char *fun_name,
+    dev_node_t *dev)
+{
+        fun_node_t *pfun;
+        
+        assert(fun != NULL);
+        assert(tree != NULL);
+        assert(fun_name != NULL);
+        assert(fibril_rwlock_is_write_locked(&tree->rwlock));
+        
+        /*
+         * The root function is a special case, it does not belong to any
+         * device so for the root function dev == NULL.
+         */
+        pfun = (dev != NULL) ? dev->pfun : NULL;
+        
+        fun->name = fun_name;
+        if (!set_fun_path(fun, pfun)) {
                 return false;
         }
         
         /* Add the node to the handle-to-node map. */
-        node->handle = ++tree->current_handle;
-        unsigned long key = node->handle;
-        hash_table_insert(&tree->devman_devices, &key, &node->devman_link);
+        fun->handle = ++tree->current_handle;
+        unsigned long key = fun->handle;
+        hash_table_insert(&tree->devman_functions, &key, &fun->devman_fun);
 
         /* Add the node to the list of its parent's children. */
-        node->parent = parent;
-        if (parent != NULL)
-                list_append(&node->sibling, &parent->children);
+        fun->dev = dev;
+        if (dev != NULL)
+                list_append(&fun->dev_functions, &dev->functions);
         
         return true;
 }
 
-/** Find device node with a specified path in the device tree.
+/** Find function node with a specified path in the device tree.
  * 
- * @param path          The path of the device node in the device tree.
+ * @param path          The path of the function node in the device tree.
  * @param tree          The device tree.
- * @return              The device node if it is present in the tree, NULL
+ * @return              The function node if it is present in the tree, NULL
  *                      otherwise.
  */
-node_t *find_dev_node_by_path(dev_tree_t *tree, char *path)
+fun_node_t *find_fun_node_by_path(dev_tree_t *tree, char *path)
 {
         fibril_rwlock_read_lock(&tree->rwlock);
         
-        node_t *dev = tree->root_node;
+        fun_node_t *fun = tree->root_node;
         /*
-         * Relative path to the device from its parent (but with '/' at the
+         * Relative path to the function from its parent (but with '/' at the
          * beginning)
          */
@@ -982 +1134 @@
         bool cont = (rel_path[0] == '/');
         
-        while (cont && dev != NULL) {
+        while (cont && fun != NULL) {
                 next_path_elem  = get_path_elem_end(rel_path + 1);
                 if (next_path_elem[0] == '/') {
@@ -991 +1143 @@
                 }
                 
-                dev = find_node_child(dev, rel_path + 1);
+                fun = find_node_child(fun, rel_path + 1);
                 
                 if (cont) {
@@ -1002 +1154 @@
         fibril_rwlock_read_unlock(&tree->rwlock);
         
-        return dev;
-}
-
-/** Find child device node with a specified name.
+        return fun;
+}
+
+/** Find child function node with a specified name.
  *
  * Device tree rwlock should be held at least for reading.
  *
- * @param parent        The parent device node.
- * @param name          The name of the child device node.
- * @return              The child device node.
- */
-node_t *find_node_child(node_t *parent, const char *name)
-{
-        node_t *dev;
+ * @param parent        The parent function node.
+ * @param name          The name of the child function.
+ * @return              The child function node.
+ */
+fun_node_t *find_node_child(fun_node_t *pfun, const char *name)
+{
+        fun_node_t *fun;
         link_t *link;
         
-        link = parent->children.next;
-        
-        while (link != &parent->children) {
-                dev = list_get_instance(link, node_t, sibling);
+        link = pfun->child->functions.next;
+        
+        while (link != &pfun->child->functions) {
+                fun = list_get_instance(link, fun_node_t, dev_functions);
                 
-                if (str_cmp(name, dev->name) == 0)
-                        return dev;
+                if (str_cmp(name, fun->name) == 0)
+                        return fun;
                 
                 link = link->next;
@@ -1107 +1259 @@
 }
 
-/** Add the device to the class.
+/** Add the device function to the class.
  *
  * The device may be added to multiple classes and a class may contain multiple
@@ -1120 +1272 @@
  *                      with the class.
  */
-dev_class_info_t *add_device_to_class(node_t *dev, dev_class_t *cl,
+dev_class_info_t *add_function_to_class(fun_node_t *fun, dev_class_t *cl,
     const char *base_dev_name)
 {
-        dev_class_info_t *info = create_dev_class_info();
+        dev_class_info_t *info;
+
+        assert(fun != NULL);
+        assert(cl != NULL);
+
+        info = create_dev_class_info();
+
         
         if (info != NULL) {
                 info->dev_class = cl;
-                info->dev = dev;
+                info->fun = fun;
                 
                 /* Add the device to the class. */
@@ -1135 +1293 @@
                 
                 /* Add the class to the device. */
-                list_append(&info->dev_classes, &dev->classes);
+                list_append(&info->dev_classes, &fun->classes);
                 
                 /* Create unique name for the device within the class. */
@@ -1189 +1347 @@
         list_initialize(&class_list->classes);
         fibril_rwlock_initialize(&class_list->rwlock);
-        hash_table_create(&class_list->devmap_devices, DEVICE_BUCKETS, 1,
+        hash_table_create(&class_list->devmap_functions, DEVICE_BUCKETS, 1,
             &devmap_devices_class_ops);
 }
@@ -1196 +1354 @@
 /* Devmap devices */
 
-node_t *find_devmap_tree_device(dev_tree_t *tree, devmap_handle_t devmap_handle)
-{
-        node_t *dev = NULL;
+fun_node_t *find_devmap_tree_function(dev_tree_t *tree, devmap_handle_t devmap_handle)
+{
+        fun_node_t *fun = NULL;
         link_t *link;
         unsigned long key = (unsigned long) devmap_handle;
         
         fibril_rwlock_read_lock(&tree->rwlock);
-        link = hash_table_find(&tree->devmap_devices, &key);
+        link = hash_table_find(&tree->devmap_functions, &key);
         if (link != NULL)
-                dev = hash_table_get_instance(link, node_t, devmap_link);
+                fun = hash_table_get_instance(link, fun_node_t, devmap_fun);
         fibril_rwlock_read_unlock(&tree->rwlock);
         
-        return dev;
-}
-
-node_t *find_devmap_class_device(class_list_t *classes,
+        return fun;
+}
+
+fun_node_t *find_devmap_class_function(class_list_t *classes,
     devmap_handle_t devmap_handle)
 {
-        node_t *dev = NULL;
+        fun_node_t *fun = NULL;
         dev_class_info_t *cli;
         link_t *link;
@@ -1220 +1378 @@
         
         fibril_rwlock_read_lock(&classes->rwlock);
-        link = hash_table_find(&classes->devmap_devices, &key);
+        link = hash_table_find(&classes->devmap_functions, &key);
         if (link != NULL) {
                 cli = hash_table_get_instance(link, dev_class_info_t,
                     devmap_link);
-                dev = cli->dev;
+                fun = cli->fun;
         }
         fibril_rwlock_read_unlock(&classes->rwlock);
         
-        return dev;
-}
-
-void class_add_devmap_device(class_list_t *class_list, dev_class_info_t *cli)
+        return fun;
+}
+
+void class_add_devmap_function(class_list_t *class_list, dev_class_info_t *cli)
 {
         unsigned long key = (unsigned long) cli->devmap_handle;
         
         fibril_rwlock_write_lock(&class_list->rwlock);
-        hash_table_insert(&class_list->devmap_devices, &key, &cli->devmap_link);
+        hash_table_insert(&class_list->devmap_functions, &key, &cli->devmap_link);
         fibril_rwlock_write_unlock(&class_list->rwlock);
 
-        assert(find_devmap_class_device(class_list, cli->devmap_handle) != NULL);
-}
-
-void tree_add_devmap_device(dev_tree_t *tree, node_t *node)
-{
-        unsigned long key = (unsigned long) node->devmap_handle;
+        assert(find_devmap_class_function(class_list, cli->devmap_handle) != NULL);
+}
+
+void tree_add_devmap_function(dev_tree_t *tree, fun_node_t *fun)
+{
+        unsigned long key = (unsigned long) fun->devmap_handle;
         fibril_rwlock_write_lock(&tree->rwlock);
-        hash_table_insert(&tree->devmap_devices, &key, &node->devmap_link);
+        hash_table_insert(&tree->devmap_functions, &key, &fun->devmap_fun);
         fibril_rwlock_write_unlock(&tree->rwlock);
 }

uspace/srv/devman/devman.h

@@ -56 +56 @@
 #define DEVMAP_SEPARATOR '\\'
 
-struct node;
-typedef struct node node_t;
+struct dev_node;
+typedef struct dev_node dev_node_t;
+
+struct fun_node;
+typedef struct fun_node fun_node_t;
 
 typedef enum {
@@ -117 +120 @@
 } device_state_t;
 
-/** Representation of a node in the device tree. */
-struct node {
+/** Device node in the device tree. */
+struct dev_node {
         /** The global unique identifier of the device. */
         devman_handle_t handle;
-        /** The name of the device specified by its parent. */
-        char *name;
-        
-        /**
-         * Full path and name of the device in device hierarchi (i. e. in full
-         * path in device tree).
-         */
-        char *pathname;
-        
-        /** The node of the parent device. */
-        node_t *parent;
-        
-        /**
-         * Pointers to previous and next child devices in the linked list of
-         * parent device's node.
-         */
-        link_t sibling;
-        
-        /** List of child device nodes. */
-        link_t children;
-        /** List of device ids for device-to-driver matching. */
-        match_id_list_t match_ids;
+        
+        /** (Parent) function the device is attached to. */
+        fun_node_t *pfun;
+        
+        /** List of device functions. */
+        link_t functions;
         /** Driver of this device. */
         driver_t *drv;
         /** The state of the device. */
         device_state_t state;
-        /**
-         * Pointer to the previous and next device in the list of devices
-         * owned by one driver.
-         */
+        /** Link to list of devices owned by driver (driver_t.devices) */
         link_t driver_devices;
         
-        /** The list of device classes to which this device belongs. */
-        link_t classes;
-        /** Devmap handle if the device is registered by devmapper. */
-        devmap_handle_t devmap_handle;
-        
         /**
          * Used by the hash table of devices indexed by devman device handles.
          */
-        link_t devman_link;
-        
-        /**
-         * Used by the hash table of devices indexed by devmap device handles.
-         */
-        link_t devmap_link;
-
+        link_t devman_dev;
+        
         /**
          * Whether this device was already passed to the driver.
@@ -173 +147 @@
         bool passed_to_driver;
 };
+
+/** Function node in the device tree. */
+struct fun_node {
+        /** The global unique identifier of the function */
+        devman_handle_t handle;
+        /** Name of the function, assigned by the device driver */
+        char *name;
+        
+        /** Full path and name of the device in device hierarchy */
+        char *pathname;
+        
+        /** Device which this function belongs to */
+        dev_node_t *dev;
+        
+        /** Link to list of functions in the device (ddf_dev_t.functions) */
+        link_t dev_functions;
+        
+        /** Child device node (if any attached). */
+        dev_node_t *child;
+        /** List of device ids for device-to-driver matching. */
+        match_id_list_t match_ids;
+        
+        /** The list of device classes to which this device function belongs. */
+        link_t classes;
+        /** Devmap handle if the device function is registered by devmap. */
+        devmap_handle_t devmap_handle;
+        
+        /**
+         * Used by the hash table of functions indexed by devman device handles.
+         */
+        link_t devman_fun;
+        
+        /**
+         * Used by the hash table of functions indexed by devmap device handles.
+         */
+        link_t devmap_fun;
+};
+
 
 /** Represents device tree. */
 typedef struct dev_tree {
         /** Root device node. */
-        node_t *root_node;
+        fun_node_t *root_node;
         
         /**
@@ -191 +203 @@
         hash_table_t devman_devices;
         
+        /** Hash table of all devices indexed by devman handles. */
+        hash_table_t devman_functions;
+        
         /**
          * Hash table of devices registered by devmapper, indexed by devmap
          * handles.
          */
-        hash_table_t devmap_devices;
+        hash_table_t devmap_functions;
 } dev_tree_t;
 
@@ -227 +242 @@
 
 /**
- * Provides n-to-m mapping between device nodes and classes - each device may
- * be register to the arbitrary number of classes and each class may contain
- * the arbitrary number of devices.
+ * Provides n-to-m mapping between function nodes and classes - each function
+ * can register in an arbitrary number of classes and each class can contain
+ * an arbitrary number of device functions.
  */
 typedef struct dev_class_info {
@@ -235 +250 @@
         dev_class_t *dev_class;
         /** The device. */
-        node_t *dev;
+        fun_node_t *fun;
         
         /**
@@ -249 +264 @@
         link_t dev_classes;
         
-        /** The name of the device within the class. */
+        /** The name of the device function within the class. */
         char *dev_name;
         /** The handle of the device by device mapper in the class namespace. */
@@ -270 +285 @@
          * indexed by devmap handles.
          */
-        hash_table_t devmap_devices;
+        hash_table_t devmap_functions;
         
         /** Fibril mutex for list of classes. */
@@ -278 +293 @@
 /* Match ids and scores */
 
-extern int get_match_score(driver_t *, node_t *);
+extern int get_match_score(driver_t *, dev_node_t *);
 
 extern bool parse_match_ids(char *, match_id_list_t *);
@@ -292 +307 @@
 extern int lookup_available_drivers(driver_list_t *, const char *);
 
-extern driver_t *find_best_match_driver(driver_list_t *, node_t *);
-extern bool assign_driver(node_t *, driver_list_t *, dev_tree_t *);
+extern driver_t *find_best_match_driver(driver_list_t *, dev_node_t *);
+extern bool assign_driver(dev_node_t *, driver_list_t *, dev_tree_t *);
 
 extern void add_driver(driver_list_t *, driver_t *);
-extern void attach_driver(node_t *, driver_t *);
-extern void add_device(int, driver_t *, node_t *, dev_tree_t *);
+extern void attach_driver(dev_node_t *, driver_t *);
+extern void add_device(int, driver_t *, dev_node_t *, dev_tree_t *);
 extern bool start_driver(driver_t *);
 
@@ -310 +325 @@
 /* Device nodes */
 
-extern node_t *create_dev_node(void);
-extern void delete_dev_node(node_t *node);
-extern node_t *find_dev_node_no_lock(dev_tree_t *tree,
+extern dev_node_t *create_dev_node(void);
+extern void delete_dev_node(dev_node_t *node);
+extern dev_node_t *find_dev_node_no_lock(dev_tree_t *tree,
     devman_handle_t handle);
-extern node_t *find_dev_node(dev_tree_t *tree, devman_handle_t handle);
-extern node_t *find_dev_node_by_path(dev_tree_t *, char *);
-extern node_t *find_node_child(node_t *, const char *);
+extern dev_node_t *find_dev_node(dev_tree_t *tree, devman_handle_t handle);
+extern dev_node_t *find_dev_function(dev_node_t *, const char *);
+
+extern fun_node_t *create_fun_node(void);
+extern void delete_fun_node(fun_node_t *);
+extern fun_node_t *find_fun_node_no_lock(dev_tree_t *tree,
+    devman_handle_t handle);
+extern fun_node_t *find_fun_node(dev_tree_t *tree, devman_handle_t handle);
+extern fun_node_t *find_fun_node_by_path(dev_tree_t *, char *);
 
 /* Device tree */
 
 extern bool init_device_tree(dev_tree_t *, driver_list_t *);
-extern bool create_root_node(dev_tree_t *);
-extern bool insert_dev_node(dev_tree_t *, node_t *, char *, node_t *);
+extern bool create_root_nodes(dev_tree_t *);
+extern bool insert_dev_node(dev_tree_t *, dev_node_t *, fun_node_t *);
+extern bool insert_fun_node(dev_tree_t *, fun_node_t *, char *, dev_node_t *);
 
 /* Device classes */
@@ -330 +352 @@
 extern size_t get_new_class_dev_idx(dev_class_t *);
 extern char *create_dev_name_for_class(dev_class_t *, const char *);
-extern dev_class_info_t *add_device_to_class(node_t *, dev_class_t *,
+extern dev_class_info_t *add_function_to_class(fun_node_t *, dev_class_t *,
     const char *);
 
@@ -341 +363 @@
 /* Devmap devices */
 
-extern node_t *find_devmap_tree_device(dev_tree_t *, devmap_handle_t);
-extern node_t *find_devmap_class_device(class_list_t *, devmap_handle_t);
-
-extern void class_add_devmap_device(class_list_t *, dev_class_info_t *);
-extern void tree_add_devmap_device(dev_tree_t *, node_t *);
+extern void devmap_register_tree_function(fun_node_t *, dev_tree_t *);
+
+extern fun_node_t *find_devmap_tree_function(dev_tree_t *, devmap_handle_t);
+extern fun_node_t *find_devmap_class_function(class_list_t *, devmap_handle_t);
+
+extern void class_add_devmap_function(class_list_t *, dev_class_info_t *);
+extern void tree_add_devmap_function(dev_tree_t *, fun_node_t *);
 
 #endif

uspace/srv/devman/main.c

@@ -199 +199 @@
 static int assign_driver_fibril(void *arg)
 {
-        node_t *node = (node_t *) arg;
-        assign_driver(node, &drivers_list, &device_tree);
+        dev_node_t *dev_node = (dev_node_t *) arg;
+        assign_driver(dev_node, &drivers_list, &device_tree);
         return EOK;
 }
 
-/** Handle child device registration.
+/** Handle function registration.
  *
  * Child devices are registered by their parent's device driver.
  */
-static void devman_add_child(ipc_callid_t callid, ipc_call_t *call)
-{
-        devman_handle_t parent_handle = IPC_GET_ARG1(*call);
-        sysarg_t match_count = IPC_GET_ARG2(*call);
+static void devman_add_function(ipc_callid_t callid, ipc_call_t *call)
+{
+        fun_type_t ftype = (fun_type_t) IPC_GET_ARG1(*call);
+        devman_handle_t dev_handle = IPC_GET_ARG2(*call);
+        sysarg_t match_count = IPC_GET_ARG3(*call);
         dev_tree_t *tree = &device_tree;
         
         fibril_rwlock_write_lock(&tree->rwlock);
-        node_t *parent = find_dev_node_no_lock(&device_tree, parent_handle);
-        
-        if (parent == NULL) {
+
+        dev_node_t *dev = NULL;
+        dev_node_t *pdev = find_dev_node_no_lock(&device_tree, dev_handle);
+        
+        if (pdev == NULL) {
                 fibril_rwlock_write_unlock(&tree->rwlock);
                 async_answer_0(callid, ENOENT);
@@ -223 +226 @@
         }
         
-        char *dev_name = NULL;
-        int rc = async_data_write_accept((void **)&dev_name, true, 0, 0, 0, 0);
+        if (ftype != fun_inner && ftype != fun_exposed) {
+                /* Unknown function type */
+                printf(NAME ": Error, unknown function type provided by driver!\n");
+
+                fibril_rwlock_write_unlock(&tree->rwlock);
+                async_answer_0(callid, EINVAL);
+                return;
+        }
+        
+        char *fun_name = NULL;
+        int rc = async_data_write_accept((void **)&fun_name, true, 0, 0, 0, 0);
         if (rc != EOK) {
                 fibril_rwlock_write_unlock(&tree->rwlock);
@@ -231 +243 @@
         }
         
-        node_t *node = create_dev_node();
-        if (!insert_dev_node(&device_tree, node, dev_name, parent)) {
+        fun_node_t *fun = create_fun_node();
+        if (!insert_fun_node(&device_tree, fun, fun_name, pdev)) {
                 fibril_rwlock_write_unlock(&tree->rwlock);
-                delete_dev_node(node);
+                delete_fun_node(fun);
                 async_answer_0(callid, ENOMEM);
                 return;
         }
 
+        if (ftype == fun_inner) {
+                dev = create_dev_node();
+                if (dev == NULL) {
+                        fibril_rwlock_write_unlock(&tree->rwlock);
+                        delete_fun_node(fun);
+                        async_answer_0(callid, ENOMEM);
+                        return;
+                }
+
+                insert_dev_node(tree, dev, fun);
+        }
+
         fibril_rwlock_write_unlock(&tree->rwlock);
         
-        printf(NAME ": devman_add_child %s\n", node->pathname);
-        
-        devman_receive_match_ids(match_count, &node->match_ids);
-
-        /*
-         * Try to find a suitable driver and assign it to the device.  We do
-         * not want to block the current fibril that is used for processing
-         * incoming calls: we will launch a separate fibril to handle the
-         * driver assigning. That is because assign_driver can actually include
-         * task spawning which could take some time.
-         */
-        fid_t assign_fibril = fibril_create(assign_driver_fibril, node);
-        if (assign_fibril == 0) {
+        printf(NAME ": devman_add_function %s\n", fun->pathname);
+        
+        devman_receive_match_ids(match_count, &fun->match_ids);
+
+        if (ftype == fun_inner) {
+                assert(dev != NULL);
                 /*
-                 * Fallback in case we are out of memory.
-                 * Probably not needed as we will die soon anyway ;-).
+                 * Try to find a suitable driver and assign it to the device.  We do
+                 * not want to block the current fibril that is used for processing
+                 * incoming calls: we will launch a separate fibril to handle the
+                 * driver assigning. That is because assign_driver can actually include
+                 * task spawning which could take some time.
                  */
-                (void) assign_driver_fibril(node);
+                fid_t assign_fibril = fibril_create(assign_driver_fibril, dev);
+                if (assign_fibril == 0) {
+                        /*
+                         * Fallback in case we are out of memory.
+                         * Probably not needed as we will die soon anyway ;-).
+                         */
+                        (void) assign_driver_fibril(fun);
+                } else {
+                        fibril_add_ready(assign_fibril);
+                }
         } else {
-                fibril_add_ready(assign_fibril);
-        }
-
+                devmap_register_tree_function(fun, tree);
+        }
+        
         /* Return device handle to parent's driver. */
-        async_answer_1(callid, EOK, node->handle);
+        async_answer_1(callid, EOK, fun->handle);
 }
 
@@ -288 +317 @@
          * mapper.
          */
-        class_add_devmap_device(&class_list, cli);
+        class_add_devmap_function(&class_list, cli);
         
         free(devmap_pathname);
 }
 
-static void devman_add_device_to_class(ipc_callid_t callid, ipc_call_t *call)
+static void devman_add_function_to_class(ipc_callid_t callid, ipc_call_t *call)
 {
         devman_handle_t handle = IPC_GET_ARG1(*call);
@@ -306 +335 @@
         }       
         
-        node_t *dev = find_dev_node(&device_tree, handle);
-        if (dev == NULL) {
+        fun_node_t *fun = find_fun_node(&device_tree, handle);
+        if (fun == NULL) {
                 async_answer_0(callid, ENOENT);
                 return;
@@ -313 +342 @@
         
         dev_class_t *cl = get_dev_class(&class_list, class_name);
-        dev_class_info_t *class_info = add_device_to_class(dev, cl, NULL);
+        dev_class_info_t *class_info = add_function_to_class(fun, cl, NULL);
         
         /* Register the device's class alias by devmapper. */
         devmap_register_class_dev(class_info);
         
-        printf(NAME ": device '%s' added to class '%s', class name '%s' was "
-            "asigned to it\n", dev->pathname, class_name, class_info->dev_name);
+        printf(NAME ": function'%s' added to class '%s', class name '%s' was "
+            "asigned to it\n", fun->pathname, class_name, class_info->dev_name);
 
         async_answer_0(callid, EOK);
@@ -372 +401 @@
                         cont = false;
                         continue;
-                case DEVMAN_ADD_CHILD_DEVICE:
-                        devman_add_child(callid, &call);
+                case DEVMAN_ADD_FUNCTION:
+                        devman_add_function(callid, &call);
                         break;
                 case DEVMAN_ADD_DEVICE_TO_CLASS:
-                        devman_add_device_to_class(callid, &call);
+                        devman_add_function_to_class(callid, &call);
                         break;
                 default:
@@ -387 +416 @@
 /** Find handle for the device instance identified by the device's path in the
  * device tree. */
-static void devman_device_get_handle(ipc_callid_t iid, ipc_call_t *icall)
+static void devman_function_get_handle(ipc_callid_t iid, ipc_call_t *icall)
 {
         char *pathname;
@@ -397 +426 @@
         }
         
-        node_t * dev = find_dev_node_by_path(&device_tree, pathname);
+        fun_node_t *fun = find_fun_node_by_path(&device_tree, pathname);
         
         free(pathname);
 
-        if (dev == NULL) {
+        if (fun == NULL) {
                 async_answer_0(iid, ENOENT);
                 return;
         }
-        
-        async_answer_1(iid, EOK, dev->handle);
+
+        async_answer_1(iid, EOK, fun->handle);
 }
 
@@ -426 +455 @@
                         continue;
                 case DEVMAN_DEVICE_GET_HANDLE:
-                        devman_device_get_handle(callid, &call);
+                        devman_function_get_handle(callid, &call);
                         break;
                 default:
@@ -438 +467 @@
 {
         devman_handle_t handle = IPC_GET_ARG2(*icall);
-        
-        node_t *dev = find_dev_node(&device_tree, handle);
-        if (dev == NULL) {
-                printf(NAME ": devman_forward error - no device with handle %" PRIun
-                    " was found.\n", handle);
+        devman_handle_t fwd_h;
+        fun_node_t *fun = NULL;
+        dev_node_t *dev = NULL;
+        
+        fun = find_fun_node(&device_tree, handle);
+        if (fun == NULL)
+                dev = find_dev_node(&device_tree, handle);
+        else
+                dev = fun->dev;
+
+        if (fun == NULL && dev == NULL) {
+                printf(NAME ": devman_forward error - no device or function with "
+                    "handle %" PRIun " was found.\n", handle);
                 async_answer_0(iid, ENOENT);
                 return;
         }
+
+        if (fun == NULL && !drv_to_parent) {
+                printf(NAME ": devman_forward error - cannot connect to "
+                    "handle %" PRIun ", refers to a device.\n", handle);
+                async_answer_0(iid, ENOENT);
+                return;
+        }
         
         driver_t *driver = NULL;
         
         if (drv_to_parent) {
-                if (dev->parent != NULL)
-                        driver = dev->parent->drv;
+                /* Connect to parent function of a device (or device function). */
+                if (dev->pfun->dev != NULL)
+                        driver = dev->pfun->dev->drv;
+                fwd_h = dev->pfun->handle;
         } else if (dev->state == DEVICE_USABLE) {
+                /* Connect to the specified function */
                 driver = dev->drv;
                 assert(driver != NULL);
+
+                fwd_h = handle;
         }
         
@@ -478 +527 @@
         }
 
-        printf(NAME ": devman_forward: forward connection to device %s to "
-            "driver %s.\n", dev->pathname, driver->name);
-        async_forward_fast(iid, driver->phone, method, dev->handle, 0, IPC_FF_NONE);
+        if (fun != NULL) {
+                printf(NAME ": devman_forward: forward connection to function %s to "
+                    "driver %s.\n", fun->pathname, driver->name);
+        } else {
+                printf(NAME ": devman_forward: forward connection to device %s to "
+                    "driver %s.\n", dev->pfun->pathname, driver->name);
+        }
+
+        async_forward_fast(iid, driver->phone, method, fwd_h, 0, IPC_FF_NONE);
 }
 
@@ -488 +543 @@
 {
         devmap_handle_t devmap_handle = IPC_GET_ARG2(*icall);
-        node_t *dev;
-
-        dev = find_devmap_tree_device(&device_tree, devmap_handle);
-        if (dev == NULL)
-                dev = find_devmap_class_device(&class_list, devmap_handle);
-        
-        if (dev == NULL || dev->drv == NULL) {
+        fun_node_t *fun;
+        dev_node_t *dev;
+
+        fun = find_devmap_tree_function(&device_tree, devmap_handle);
+        if (fun == NULL)
+                fun = find_devmap_class_function(&class_list, devmap_handle);
+        
+        if (fun == NULL || fun->dev->drv == NULL) {
                 async_answer_0(iid, ENOENT);
                 return;
         }
+        
+        dev = fun->dev;
         
         if (dev->state != DEVICE_USABLE || dev->drv->phone <= 0) {
@@ -504 +562 @@
         }
         
-        async_forward_fast(iid, dev->drv->phone, DRIVER_CLIENT, dev->handle, 0,
+        async_forward_fast(iid, dev->drv->phone, DRIVER_CLIENT, fun->handle, 0,
             IPC_FF_NONE);
         printf(NAME ": devman_connection_devmapper: forwarded connection to "
-            "device %s to driver %s.\n", dev->pathname, dev->drv->name);
+            "device %s to driver %s.\n", fun->pathname, dev->drv->name);
 }
 

uspace/srv/devman/match.c

@@ -57 +57 @@
 }
 
-int get_match_score(driver_t *drv, node_t *dev)
+int get_match_score(driver_t *drv, dev_node_t *dev)
 {
         link_t *drv_head = &drv->match_ids.ids;
-        link_t *dev_head = &dev->match_ids.ids;
+        link_t *dev_head = &dev->pfun->match_ids.ids;
         
         if (list_empty(drv_head) || list_empty(dev_head))

uspace/srv/devmap/devmap.c

@@ -123 +123 @@
 static devmap_handle_t last_handle = 0;
 static devmap_device_t *null_devices[NULL_DEVICES];
+
+/*
+ * Dummy list for null devices. This is necessary so that null devices can
+ * be used just as any other devices, e.g. in devmap_device_unregister_core().
+ */
+static LIST_INITIALIZE(dummy_null_driver_devices);
 
 static devmap_handle_t devmap_create_handle(void)
@@ -953 +959 @@
         device->name = dev_name;
         
-        /* Insert device into list of all devices
-           and into null devices array */
+        /*
+         * Insert device into list of all devices and into null devices array.
+         * Insert device into a dummy list of null driver's devices so that it
+         * can be safely removed later.
+         */
         list_append(&device->devices, &devices_list);
+        list_append(&device->driver_devices, &dummy_null_driver_devices);
         null_devices[i] = device;
         

uspace/srv/hid/console/gcons.c

@@ -285 +285 @@
         ssize_t nx = (ssize_t) mouse_x + dx;
         ssize_t ny = (ssize_t) mouse_y + dy;
-
-        if (!use_gcons)
+        
+        /* Until gcons is initalized we don't have the screen resolution */
+        if (xres == 0 || yres == 0)
                 return;
         

uspace/srv/hid/kbd/Makefile

@@ -78 +78 @@
                 SOURCES += \
                         port/pl050.c \
-                        ctl/pl050.c
+                        ctl/pc.c
         endif
 endif

uspace/srv/loader/arch/abs32le/_link.ld.in

@@ -3 +3 @@
  * is the base address and the special interp section.
  */
+
 STARTUP(LIBC_PREFIX/arch/UARCH/src/entry.o)
 ENTRY(__entry)
@@ -54 +55 @@
         } :data
         
-        . = ALIGN(0x1000);
-        
-        _heap = .;
-        
         /DISCARD/ : {
                 *(*);

uspace/srv/loader/arch/amd64/_link.ld.in

@@ +1 @@
+/*
+ * The difference from _link.ld.in for regular statically-linked apps
+ * is the base address and the special interp section.
+ */
+
 STARTUP(LIBC_PREFIX/arch/UARCH/src/entry.o)
 ENTRY(__entry)
@@ -6 +11 @@
         text PT_LOAD FLAGS(5);
         data PT_LOAD FLAGS(6);
+        debug PT_NOTE;
 }
 
@@ -11 +17 @@
         .interp : {
                 *(.interp);
-        } : interp
-
-        /* . = 0x0000700000001000;*/
+        } :interp
+        
+        /* . = 0x0000700000001000; */
         . = 0x70001000;
         
@@ -19 +25 @@
                 *(.init);
         } :text
+        
         .text : {
                 *(.text);
@@ -27 +34 @@
                 *(.data);
         } :data
+        
         .tdata : {
                 _tdata_start = .;
@@ -32 +40 @@
                 _tdata_end = .;
         } :data
+        
         .tbss : {
                 _tbss_start = .;
@@ -37 +46 @@
                 _tbss_end = .;
         } :data
+        
         _tls_alignment = MAX(ALIGNOF(.tdata), ALIGNOF(.tbss));
+        
         .bss : {
                 *(COMMON);
                 *(.bss);
         } :data
-
-        . = ALIGN(0x1000);
-        _heap = .;
+        
+#ifdef CONFIG_LINE_DEBUG
+        .comment 0 : { *(.comment); } :debug
+        .debug_abbrev 0 : { *(.debug_abbrev); } :debug
+        .debug_aranges 0 : { *(.debug_aranges); } :debug
+        .debug_info 0 : { *(.debug_info); } :debug
+        .debug_line 0 : { *(.debug_line); } :debug
+        .debug_loc 0 : { *(.debug_loc); } :debug
+        .debug_pubnames 0 : { *(.debug_pubnames); } :debug
+        .debug_pubtypes 0 : { *(.debug_pubtypes); } :debug
+        .debug_ranges 0 : { *(.debug_ranges); } :debug
+        .debug_str 0 : { *(.debug_str); } :debug
+#endif
         
         /DISCARD/ : {
                 *(*);
         }
-
 }

uspace/srv/loader/arch/arm32/_link.ld.in

@@ -3 +3 @@
  * is the base address.
  */
+
 STARTUP(LIBC_PREFIX/arch/UARCH/src/entry.o)
 ENTRY(__entry)
@@ -16 +17 @@
                 *(.interp);
         } : interp
-
+        
         . = 0x70001000;
-
+        
         .init ALIGN(0x1000): SUBALIGN(0x1000) {
                 *(.init);
-        } : text
+        } :text
+        
         .text : {
                 *(.text);
-        *(.rodata*);
+                *(.rodata*);
         } :text
         
@@ -32 +34 @@
                 *(.sdata);
         } :data
+        
         .tdata : {
                 _tdata_start = .;
@@ -37 +40 @@
                 _tdata_end = .;
         } :data
+        
         .tbss : {
                 _tbss_start = .;
@@ -42 +46 @@
                 _tbss_end = .;
         } :data
+        
         _tls_alignment = MAX(ALIGNOF(.tdata), ALIGNOF(.tbss));
+        
         .bss : {
                 *(.sbss);
                 *(.scommon);
-        *(COMMON);
-        *(.bss);
+                *(COMMON);
+                *(.bss);
         } :data
-        
-        . = ALIGN(0x1000);
-        _heap = .;
         
         /DISCARD/ : {
                 *(*);
         }
-
 }

uspace/srv/loader/arch/ia32/_link.ld.in

@@ -3 +3 @@
  * is the base address and the special interp section.
  */
+
 STARTUP(LIBC_PREFIX/arch/UARCH/src/entry.o)
 ENTRY(__entry)
@@ -10 +11 @@
         text PT_LOAD FILEHDR PHDRS FLAGS(5);
         data PT_LOAD FLAGS(6);
+        debug PT_NOTE;
 }
 
@@ -52 +54 @@
         } :data
         
-        . = ALIGN(0x1000);
-        
-        _heap = .;
+#ifdef CONFIG_LINE_DEBUG
+        .comment 0 : { *(.comment); } :debug
+        .debug_abbrev 0 : { *(.debug_abbrev); } :debug
+        .debug_aranges 0 : { *(.debug_aranges); } :debug
+        .debug_info 0 : { *(.debug_info); } :debug
+        .debug_line 0 : { *(.debug_line); } :debug
+        .debug_loc 0 : { *(.debug_loc); } :debug
+        .debug_pubnames 0 : { *(.debug_pubnames); } :debug
+        .debug_pubtypes 0 : { *(.debug_pubtypes); } :debug
+        .debug_ranges 0 : { *(.debug_ranges); } :debug
+        .debug_str 0 : { *(.debug_str); } :debug
+#endif
         
         /DISCARD/ : {

uspace/srv/loader/arch/ia64/_link.ld.in

@@ -12 +12 @@
                 *(.interp);
         } :interp
-
+        
         /* On Itanium code sections must be aligned to 16 bytes. */
         . = ALIGN(0x800000000 + SIZEOF_HEADERS, 16);
-
+        
         .init : {
                 *(.init);
-        } : text
+        } :text
+        
         .text : {
                 *(.text);
                 *(.rodata*);
         } :text
-
+        
         . = . + 0x4000;
-
+        
         .got : {
                 _gp = .;
                 *(.got*);
-        } :data 
+        } :data
+        
         .data : {
                 *(.opd);
@@ -35 +37 @@
                 *(.sdata);
         } :data
+        
         .tdata : {
                 _tdata_start = .;
@@ -40 +43 @@
                 _tdata_end = .;
         } :data
+        
         .tbss : {
                 _tbss_start = .;
@@ -45 +49 @@
                 _tbss_end = .;
         } :data
+        
         _tls_alignment = MAX(ALIGNOF(.tdata), ALIGNOF(.tbss));
+        
         .bss : {
                 *(.sbss);
@@ -52 +58 @@
                 *(.bss);
         } :data
-
-        . = ALIGN(0x4000);
-        _heap = .;
- 
+        
         /DISCARD/ : {
                 *(*);
-        }
+        }
 }

uspace/srv/loader/arch/mips32/_link.ld.in

@@ -3 +3 @@
  * is the base address.
  */
+
 STARTUP(LIBC_PREFIX/arch/UARCH/src/entry.o)
 ENTRY(__entry)
@@ -16 +17 @@
                 *(.interp);
         } :interp
-
+        
         . = 0x70004000;
         
@@ -22 +23 @@
                 *(.init);
         } :text
+        
         .text : {
-                *(.text);
+                *(.text);
                 *(.rodata*);
         } :text
-
+        
+        . = . + 0x4000;
+        
         .data : {
                 *(.data);
                 *(.data.rel*);
         } :data
-
+        
         .got : {
                 _gp = .;
                 *(.got);
         } :data
-
+        
         .tdata : {
                 _tdata_start = .;
@@ -42 +46 @@
                 _tdata_end = .;
         } :data
+        
         .tbss : {
                 _tbss_start = .;
@@ -47 +52 @@
                 _tbss_end = .;
         } :data
+        
         _tls_alignment = MAX(ALIGNOF(.tdata), ALIGNOF(.tbss));
-
+        
         .sbss : {
                 *(.scommon);
                 *(.sbss);
-        }       
+        }
+        
         .bss : {
                 *(.bss);
                 *(COMMON);
         } :data
-
-        . = ALIGN(0x4000);
-        _heap = .;
-
+        
         /DISCARD/ : {
                 *(*);

uspace/srv/loader/arch/ppc32/_link.ld.in

@@ -3 +3 @@
  * is the base address.
  */
+
 STARTUP(LIBC_PREFIX/arch/UARCH/src/entry.o)
 ENTRY(__entry)
@@ -16 +17 @@
                 *(.interp);
         } :interp
-
+        
         . = 0x70001000;
-
+        
         .init ALIGN(0x1000) : SUBALIGN(0x1000) {
                 *(.init);
         } :text
+        
         .text : {
                 *(.text);
@@ -31 +33 @@
                 *(.sdata);
         } :data
+        
         .tdata : {
                 _tdata_start = .;
@@ -36 +39 @@
                 _tdata_end = .;
         } :data
+        
         .tbss : {
                 _tbss_start = .;
@@ -41 +45 @@
                 _tbss_end = .;
         } :data
+        
         _tls_alignment = MAX(ALIGNOF(.tdata), ALIGNOF(.tbss));
+        
         .bss : {
                 *(.sbss);
@@ -47 +53 @@
                 *(.bss);
         } :data
-
-        . = ALIGN(0x1000);
-        _heap = .;
         
         /DISCARD/ : {
                 *(*);
         }
-
 }

uspace/srv/loader/arch/sparc64/_link.ld.in

@@ -12 +12 @@
                 *(.interp);
         } :interp
-
+        
         . = 0x70004000 + SIZEOF_HEADERS;
-
+        
         .init : {
                 *(.init);
         } :text
+        
         .text : {
                 *(.text);
                 *(.rodata*);
         } :text
-
+        
         . = . + 0x4000;
-
+        
         .got : {
                  _gp = .;
                  *(.got*);
         } :data
+        
         .data : {
                 *(.data);
                 *(.sdata);
         } :data
+        
         .tdata : {
                 _tdata_start = .;
@@ -38 +41 @@
                 _tdata_end = .;
         } :data
+        
         .tbss : {
                 _tbss_start = .;
@@ -43 +47 @@
                 _tbss_end = .;
         } :data
+        
         _tls_alignment = MAX(ALIGNOF(.tdata), ALIGNOF(.tbss));
+        
         .bss : {
                 *(.sbss);
@@ -49 +55 @@
                 *(.bss);
         } :data
-
-        . = ALIGN(0x4000);
-        _heap = .;
         
         /DISCARD/ : {
                 *(*);
         }
-
 }

uspace/srv/loader/elf_load.c

@@ -109 +109 @@
         int fd;
         int rc;
-
+        
         fd = open(file_name, O_RDONLY);
         if (fd < 0) {
@@ -300 +300 @@
         case PT_NULL:
         case PT_PHDR:
+        case PT_NOTE:
                 break;
         case PT_LOAD:
@@ -310 +311 @@
         case PT_DYNAMIC:
         case PT_SHLIB:
-        case PT_NOTE:
         case PT_LOPROC:
         case PT_HIPROC:
@@ -344 +344 @@
         seg_ptr = (void *) seg_addr;
 
-        DPRINTF("Load segment at addr %p, size 0x%x\n", seg_addr,
+        DPRINTF("Load segment at addr %p, size 0x%x\n", (void *) seg_addr,
                 entry->p_memsz);
 
@@ -372 +372 @@
         mem_sz = entry->p_memsz + (entry->p_vaddr - base);
 
-        DPRINTF("Map to seg_addr=%p-%p.\n", seg_addr,
-        entry->p_vaddr + bias + ALIGN_UP(entry->p_memsz, PAGE_SIZE));
+        DPRINTF("Map to seg_addr=%p-%p.\n", (void *) seg_addr,
+            (void *) (entry->p_vaddr + bias +
+            ALIGN_UP(entry->p_memsz, PAGE_SIZE)));
 
         /*
@@ -386 +387 @@
         }
 
-        DPRINTF("as_area_create(%p, 0x%x, %d) -> 0x%lx\n",
-                base + bias, mem_sz, flags, (uintptr_t)a);
+        DPRINTF("as_area_create(%p, %#zx, %d) -> %p\n",
+            (void *) (base + bias), mem_sz, flags, (void *) a);
 
         /*
@@ -464 +465 @@
                     (void *)((uint8_t *)entry->sh_addr + elf->bias);
                 DPRINTF("Dynamic section found at %p.\n",
-                        (uintptr_t)elf->info->dynamic);
+                    (void *) elf->info->dynamic);
                 break;
         default:

uspace/srv/net/tl/udp/udp.c

@@ -740 +740 @@
         int socket_id;
         size_t addrlen;
-        size_t size;
+        size_t size = 0;
         ipc_call_t answer;
         size_t answer_count;