Changes in / [93ebe4e:5fe7692] in mainline

Files:

• defaults/arm32/gta02/Makefile.config (modified) (1 diff)
• uspace/lib/c/include/adt/int_map.h (modified) (1 diff)
• uspace/srv/devman/devman.c (modified) (3 diffs)
• uspace/srv/devman/main.c (modified) (1 diff)
• uspace/srv/devman/util.c (modified) (1 diff)
• uspace/srv/devman/util.h (modified) (1 diff)

defaults/arm32/gta02/Makefile.config

@@ -4 +4 @@
 # RAM disk format
 RDFMT = tmpfs
+
+# Support for userspace debuggers
+CONFIG_UDEBUG = n
+
+# Compile kernel tests
+CONFIG_TEST = n
+
+# Barebone build with essential binaries only
+CONFIG_BAREBONE = y

uspace/lib/c/include/adt/int_map.h

@@ -246 +246 @@
                         if (item->value) { \
                                 if (dtor) \
-                                        dtor(item->value); \
+                                        free(item->value); \
                                 item->value = NULL; \
                         } \

uspace/srv/devman/devman.c

@@ -266 +266 @@
         }
         
-        ssize_t read_bytes = safe_read(fd, buf, len);
-        if (read_bytes <= 0) {
+        if (read(fd, buf, len) <= 0) {
                 printf(NAME ": unable to read file '%s'.\n", conf_path);
                 goto cleanup;
         }
-        buf[read_bytes] = 0;
+        buf[len] = 0;
         
         suc = parse_match_ids(buf, ids);
@@ -1124 +1123 @@
 fun_node_t *find_fun_node_by_path(dev_tree_t *tree, char *path)
 {
-        assert(path != NULL);
-
-        bool is_absolute = path[0] == '/';
-        if (!is_absolute) {
-                return NULL;
-        }
-
         fibril_rwlock_read_lock(&tree->rwlock);
         
@@ -1140 +1132 @@
         char *rel_path = path;
         char *next_path_elem = NULL;
-        bool cont = true;
+        bool cont = (rel_path[0] == '/');
         
         while (cont && fun != NULL) {

uspace/srv/devman/main.c

@@ -477 +477 @@
                 dev = fun->dev;
 
-        /*
-         * For a valid function to connect to we need a device. The root
-         * function, for example, has no device and cannot be connected to.
-         * This means @c dev needs to be valid regardless whether we are
-         * connecting to a device or to a function.
-         */
-        if (dev == NULL) {
+        if (fun == NULL && dev == NULL) {
                 printf(NAME ": devman_forward error - no device or function with "
                     "handle %" PRIun " was found.\n", handle);

uspace/srv/devman/util.c

@@ -111 +111 @@
 }
 
-ssize_t safe_read(int fd, void *buffer, size_t size)
-{
-        if (size == 0) {
-                return 0;
-        }
-
-        uint8_t *buf_ptr = (uint8_t *) buffer;
-
-        size_t total_read = 0;
-        while (total_read < size) {
-                ssize_t bytes_read = read(fd, buf_ptr, size - total_read);
-                if (bytes_read < 0) {
-                        /* Error. */
-                        return bytes_read;
-                } else if (bytes_read == 0) {
-                        /* Possibly end of file. */
-                        break;
-                } else {
-                        /* Read at least something. */
-                        buf_ptr += bytes_read;
-                        total_read += bytes_read;
-                }
-        }
-
-        return (ssize_t) total_read;
-}
-
 /** @}
  */

uspace/srv/devman/util.h

@@ -47 +47 @@
 extern void replace_char(char *, char, char);
 
-extern ssize_t safe_read(int, void *, size_t);
-
 #endif