IPC event messages are too large

Bug #1367203 reported by Alexandros Frantzis
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mir
Triaged
Medium
Unassigned
mir (Ubuntu)
Triaged
Medium
Unassigned

Bug Description

For many events we send to the client (mir_event_type_surface, mir_event_type_resize etc) we pack the whole raw MirEvent structure, resulting in messages of size > 1KiB, from which only a few bytes are needed.

Tags: performance
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Yeah, this has been a theoretical issue for a while. Although having experimented with shrinking event structures I have not yet found anywhere that this is a noticeable performance issue. So we should improve it, but I'm not sure any improvement will provide measurable benefit.

Changed in mir:
status: New → Triaged
tags: added: clientapi performance
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

It's important to remember that the really large events (motion events) are not sent using the MirEvent structure. Input is sent in the raw event structure from Android, and then converted to MirEvent in the client. So this bug is indeed just about surface events (resize, attributes).

tags: removed: clientapi
Revision history for this message
Michał Sawicz (saviq) wrote :

Syncing task from Mir.

Changed in mir (Ubuntu):
importance: Undecided → Medium
status: New → Triaged
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.