Changeset c632c96 in mainline for uspace/lib/ui

Timestamp:

2021-10-25T00:32:45Z (4 years ago)

Author:

jxsvoboda <5887334+jxsvoboda@…>

Branches:

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

Children:

39ab17c

Parents:

f59212cc

git-author:

Jiri Svoboda <jiri@…> (2021-10-20 22:22:04)

git-committer:

jxsvoboda <5887334+jxsvoboda@…> (2021-10-25 00:32:45)

Message:

Unmap and clear console before executing a binary

Fixes running edit, nav, improves fdisk, etc.

Location:

uspace/lib/ui

Files:

• include/ui/ui.h (modified) (1 diff)
• src/ui.c (modified) (2 diffs)
• test/ui.c (modified) (1 diff)

uspace/lib/ui/include/ui/ui.h

@@ -52 +52 @@
 extern bool ui_is_textmode(ui_t *);
 extern bool ui_is_fullscreen(ui_t *);
+extern errno_t ui_suspend(ui_t *);
+extern errno_t ui_resume(ui_t *);
 
 #endif

uspace/lib/ui/src/ui.c

@@ -40 +40 @@
 #include <fibril.h>
 #include <gfx/color.h>
+#include <gfx/cursor.h>
 #include <gfx/render.h>
 #include <io/console.h>
@@ -355 +356 @@
 }
 
+/** Free up console for other users.
+ *
+ * Release console resources for another application (that the current
+ * task is starting). After the other application finishes, resume
+ * operation with ui_resume(). No calls to UI must happen inbetween
+ * and no events must be processed (i.e. the calling function must not
+ * return control to UI.
+ *
+ * @param ui UI
+ * @return EOK on success or an error code
+ */
+errno_t ui_suspend(ui_t *ui)
+{
+        if (ui->cgc == NULL)
+                return EOK;
+
+        return console_gc_suspend(ui->cgc);
+}
+
+/** Resume suspended UI.
+ *
+ * Reclaim console resources (after child application has finished running)
+ * and restore UI operation previously suspended by calling ui_suspend().
+ *
+ * @param ui UI
+ * @return EOK on success or an error code
+ */
+errno_t ui_resume(ui_t *ui)
+{
+        errno_t rc;
+
+        if (ui->cgc == NULL)
+                return EOK;
+
+        rc = console_gc_resume(ui->cgc);
+        if (rc != EOK)
+                return rc;
+
+        return gfx_cursor_set_visible(console_gc_get_ctx(ui->cgc), false);
+}
+
 /** Terminate user interface.
  *

uspace/lib/ui/test/ui.c

@@ -67 +67 @@
 {
         ui_destroy(NULL);
+}
+
+/** ui_suspend() / ui_resume() do nothing if we don't have a console */
+PCUT_TEST(suspend_resume)
+{
+        ui_t *ui = NULL;
+        errno_t rc;
+
+        rc = ui_create_disp(NULL, &ui);
+        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
+        PCUT_ASSERT_NOT_NULL(ui);
+
+        rc = ui_suspend(ui);
+        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
+        rc = ui_resume(ui);
+        PCUT_ASSERT_ERRNO_VAL(EOK, rc);
+
+        ui_destroy(ui);
 }