Changeset 4224ef7 in mainline for uspace/lib

Timestamp:

2019-08-06T18:18:37Z (6 years ago)

Author:

Matthieu Riolo <matthieu.riolo@…>

Children:

e55741e

Parents:

dd5c623

git-author:

Michal Koutný <xm.koutny+hos@…> (2015-05-11 16:50:40)

git-committer:

Matthieu Riolo <matthieu.riolo@…> (2019-08-06 18:18:37)

Message:

sysman: Add VFS FS server autostart

• VFS autostart instrumentation removes explicit dependency on FS servers.
• Compositor service properly named, it's now resolved as implicit dependency.

Conflicts:

boot/Makefile.common
uspace/lib/gui/window.c
uspace/srv/locsrv/locsrv.c
uspace/srv/vfs/vfs.c
uspace/srv/vfs/vfs_ops.c

Location:

uspace/lib

Files:

• gui/window.c (modified) (4 diffs)
• sysman/src/ctl.c (modified) (1 diff)

uspace/lib/gui/window.c

@@ -616 +616 @@
         win->surface = NULL;
 
+        unsigned int ipc_flags = IPC_FLAG_AUTOSTART;
         service_id_t reg_dsid;
-        errno_t rc = loc_service_get_id(winreg, &reg_dsid, 0);
+        errno_t rc = loc_service_get_id(winreg, &reg_dsid, ipc_flags);
         if (rc != EOK) {
                 free(win);
@@ -624 +625 @@
 
         async_sess_t *reg_sess =
-            loc_service_connect(reg_dsid, INTERFACE_COMPOSITOR, 0);
+            loc_service_connect(reg_dsid, INTERFACE_COMPOSITOR, ipc_flags);
+
         if (reg_sess == NULL) {
                 free(win);
@@ -639 +641 @@
         }
 
-        win->osess = loc_service_connect(out_dsid, INTERFACE_COMPOSITOR, 0);
+
+        win->osess = loc_service_connect(out_dsid, INTERFACE_COMPOSITOR, ipc_flags);
+
         if (win->osess == NULL) {
                 free(win);
@@ -645 +649 @@
         }
 
-        win->isess = loc_service_connect(in_dsid, INTERFACE_COMPOSITOR, 0);
+        win->isess = loc_service_connect(in_dsid, INTERFACE_COMPOSITOR, ipc_flags);
+
         if (win->isess == NULL) {
                 async_hangup(win->osess);

uspace/lib/sysman/src/ctl.c

@@ -33 +33 @@
 #include <sysman/sysman.h>
 
+/*
+ * TODO
+ * Non-blocking favor of this API is effectively incomplete as it doesn't
+ * provide means how to obtain result of the start operation.
+ * Probably devise individual API for brokers that could exploit the fact that
+ * broker knows when appropriate exposee is created and the request succeeded.
+ * Still though, it's necessary to centralize timeout into sysman.
+ */
 int sysman_unit_start(const char *unit_name, int flags)
 {