Changeset c11ee605 in mainline for uspace/srv/hid/display/display.c

Timestamp:

2020-05-11T15:36:46Z (5 years ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

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

Children:

83cb672

Parents:

e49b7997

Message:

Display server needs some locking

Sometimes destroying a window would race with event delivery. We add
one big lock for the display object that is taken before servicing
any client request or input event.

File:

• uspace/srv/hid/display/display.c (modified) (2 diffs)

uspace/srv/hid/display/display.c

@@ -65 +65 @@
         }
 
+        fibril_mutex_initialize(&disp->lock);
         list_initialize(&disp->clients);
         disp->next_wnd_id = 1;
@@ -84 +85 @@
         gfx_color_delete(disp->bg_color);
         free(disp);
+}
+
+/** Lock display.
+ *
+ * This should be called in any thread that wishes to access the display
+ * or its child objects (e.g. windows).
+ *
+ * @param disp Display
+ */
+void ds_display_lock(ds_display_t *disp)
+{
+        fibril_mutex_lock(&disp->lock);
+}
+
+/** Unlock display.
+ *
+ * @param disp Display
+ */
+void ds_display_unlock(ds_display_t *disp)
+{
+        fibril_mutex_unlock(&disp->lock);
 }