Changeset 79ae36dd in mainline for uspace/app/tester/devs/devman2.c

Timestamp:

2011-06-08T19:01:55Z (13 years ago)

Author:

Martin Decky <martin@…>

Branches:

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

Children:

0eff68e

Parents:

764d71e

Message:

new async framework with integrated exchange tracking

• strict isolation between low-level IPC and high-level async framework with integrated exchange tracking
  • each IPC connection is represented by an async_sess_t structure
  • each IPC exchange is represented by an async_exch_t structure
  • exchange management is either based on atomic messages (EXCHANGE_ATOMIC), locking (EXCHANGE_SERIALIZE) or connection cloning (EXCHANGE_CLONE)
• async_obsolete: temporary compatibility layer to keep old async clients working (several pieces of code are currently broken, but only non-essential functionality)
• IPC_M_PHONE_HANGUP is now method no. 0 (for elegant boolean evaluation)
• IPC_M_DEBUG_ALL has been renamed to IPC_M_DEBUG
• IPC_M_PING has been removed (VFS protocol now has VFS_IN_PING)
• console routines in libc have been rewritten for better abstraction
• additional use for libc-private header files (FILE structure opaque to the client)
• various cstyle changes (typos, indentation, missing externs in header files, improved comments, etc.)

File:

• uspace/app/tester/devs/devman2.c (modified) (3 diffs)

uspace/app/tester/devs/devman2.c

@@ -42 +42 @@
 #include <devman.h>
 #include <str.h>
+#include <async.h>
 #include <vfs/vfs.h>
+#include <vfs/vfs_sess.h>
 #include <sys/stat.h>
 #include <fcntl.h>
@@ -68 +70 @@
                         continue;
                 }
-                int phone = fd_phone(fd);
+                async_sess_t *sess = fd_session(EXCHANGE_SERIALIZE, fd);
                 close(fd);
-                if (phone < 0) {
-                        TPRINTF("Failed opening phone: %s.\n", str_error(phone));
-                        rc = phone;
+                if (sess == NULL) {
+                        TPRINTF("Failed opening phone: %s.\n", str_error(errno));
+                        rc = errno;
                         err_msg = "Failed opening file descriptor phone";
                         continue;
                 }
-                async_hangup(phone);
+                async_hangup(sess);
                 TPRINTF("Path `%s' okay.\n", path);
                 free(path);
@@ -83 +85 @@
         }
         
-        if (path != NULL) {
+        if (path != NULL)
                 free(path);
-        }
-
+        
         return err_msg;
 }