Changeset bb9ec2d in mainline for uspace/lib/c/generic/loader.c

Timestamp:

2017-03-07T20:47:35Z (7 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

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

Children:

a737667e

Parents:

e796dc8

git-author:

Jiri Zarevucky <zarevucky.jiri@…> (2017-03-07 20:47:35)

git-committer:

Jakub Jermar <jakub@…> (2017-03-07 20:47:35)

Message:

Merge from lp:~zarevucky-jiri/helenos/vfs-2.5/ revision 1941-1944

Original commit messages:

1944: Jiri Zarevucky 2013-08-06 Replace legacy file descriptor presetting with inbox.
1943: Jiri Zarevucky 2013-08-06 Do not preserve open state when passing file descriptor to another task. Allow receiver to specify, whether the descriptor is low or high.
1942: Jiri Zarevucky 2013-08-06 C style.
1941: Jiri Zarevucky 2013-08-06 Make loader accept file reference instead of a pathname.

Modifications:

• Keep version of elf_load_file() that accepts file name
• Changes required for loading dynamically linked executables
• Update to newer list_foreach

File:

• uspace/lib/c/generic/loader.c (modified) (2 diffs)

uspace/lib/c/generic/loader.c

@@ -147 +147 @@
 }
 
-/** Set pathname of the program to load.
- *
- * Sets the name of the program file to load. The name can be relative
- * to the current working directory (it will be absolutized before
- * sending to the loader).
+/** Set the program to load.
  *
  * @param ldr  Loader connection structure.
- * @param path Pathname of the program file.
- *
- * @return Zero on success or negative error code.
- *
- */
-int loader_set_pathname(loader_t *ldr, const char *path)
-{
-        size_t pa_len;
-        char *pa = vfs_absolutize(path, &pa_len);
-        if (!pa)
-                return ENOMEM;
-        
-        /* Send program pathname */
-        async_exch_t *exch = async_exchange_begin(ldr->sess);
-        
+ * @param name Name to set for the spawned program.
+ * @param file Program file.
+ *
+ * @return Zero on success or negative error code.
+ *
+ */
+int loader_set_program(loader_t *ldr, const char *name, int file)
+{
+        async_exch_t *exch = async_exchange_begin(ldr->sess);
+
         ipc_call_t answer;
-        aid_t req = async_send_0(exch, LOADER_SET_PATHNAME, &answer);
-        sysarg_t rc = async_data_write_start(exch, (void *) pa, pa_len);
-        
-        async_exchange_end(exch);
-        free(pa);
-        
+        aid_t req = async_send_0(exch, LOADER_SET_PROGRAM, &answer);
+
+        sysarg_t rc = async_data_write_start(exch, name, str_size(name) + 1);
+        if (rc == EOK) {
+                async_exch_t *vfs_exch = vfs_exchange_begin();
+                rc = vfs_pass_handle(vfs_exch, file, exch);
+                vfs_exchange_end(vfs_exch);
+        }
+
+        async_exchange_end(exch);
+
         if (rc != EOK) {
                 async_forget(req);
                 return (int) rc;
         }
-        
+
         async_wait_for(req, &rc);
         return (int) rc;
 }
+
+/** Set the program to load by path.
+ *
+ * @param ldr  Loader connection structure.
+ * @param path Program path.
+ *
+ * @return Zero on success or negative error code.
+ *
+ */
+int loader_set_program_path(loader_t *ldr, const char *path)
+{
+        const char *name = str_rchr(path, '/');
+        if (name == NULL) {
+                name = path;
+        } else {
+                name++;
+        }
+        
+        int fd = vfs_lookup(path);
+        if (fd < 0) {
+                return fd;
+        }
+        
+        int rc = loader_set_program(ldr, name, fd);
+        close(fd);
+        return rc;
+}
+
 
 /** Set command-line arguments for the program.
@@ -244 +268 @@
 }
 
-/** Set preset files for the program.
- *
- * Sets the vector of preset files to be passed to the loaded
- * program. By convention, the first three files represent stdin,
- * stdout and stderr respectively.
- *
- * @param ldr   Loader connection structure.
- * @param files NULL-terminated array of pointers to files.
- *
- * @return Zero on success or negative error code.
- *
- */
-int loader_set_files(loader_t *ldr, int * const files[])
-{
-        /* Send serialized files to the loader */
+/** Add a file to the task's inbox.
+ *
+ * @param ldr        Loader connection structure.
+ * @param name       Identification of the file.
+ * @param file       The file's descriptor.
+ *
+ * @return Zero on success or negative error code.
+ *
+ */
+int loader_add_inbox(loader_t *ldr, const char *name, int file)
+{
         async_exch_t *exch = async_exchange_begin(ldr->sess);
         async_exch_t *vfs_exch = vfs_exchange_begin();
         
-        int i;
-        for (i = 0; files[i]; i++);
-
-        ipc_call_t answer;
-        aid_t req = async_send_1(exch, LOADER_SET_FILES, i, &answer);
-
-        sysarg_t rc = EOK;
-        
-        for (i = 0; files[i]; i++) {
-                rc = vfs_pass_handle(vfs_exch, *files[i], exch);
-                if (rc != EOK)
-                        break;
-        }
-        
-        vfs_exchange_end(vfs_exch);
-        async_exchange_end(exch);
-
-        if (rc != EOK) {
-                async_forget(req);
-                return (int) rc;
-        }
-        
-        async_wait_for(req, &rc);
+        aid_t req = async_send_0(exch, LOADER_ADD_INBOX, NULL);
+        
+        sysarg_t rc = async_data_write_start(exch, name, str_size(name) + 1);
+        if (rc == EOK) {
+                rc = vfs_pass_handle(vfs_exch, file, exch);
+        }
+        
+        async_exchange_end(vfs_exch);
+        async_exchange_end(exch);
+        
+        if (rc == EOK) {
+                async_wait_for(req, &rc);
+        } else {
+                async_forget(req);
+        }
+        
         return (int) rc;
 }