[v1] use trace events and fix garbled output

[PATCH 0/2] use trace events and fix garbled output

Carwyn Ellis posted 2 patches 2 years, 3 months ago

Diff against v3
Download series mbox

Test checkpatch failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20220104180659.91619-1-carwynellis@gmail.com

There is a newer version of this series

hw/display/trace-events |  4 +++
hw/display/vmware_vga.c | 63 +++++++++++++++++++++--------------------
2 files changed, 37 insertions(+), 30 deletions(-)

Expand all Fold all

[PATCH 0/2] use trace events and fix garbled output

Posted by Carwyn Ellis 2 years, 3 months ago

This patchset supersedes the earlier submission and incorporates
feedback from Laurent Vivier and Gerd Hoffmann.

There are two patches addressing the following in the vmware vga display
code

 - use of fprintf to log debug output to STDERR

   This has been replaced with trace events.

 - garbled display due to lost display updates

   This prevents an issue that can cause garbled display output when
   a high number of screen updates are being requested.

   The queue is now flushed when it reaches capacity.

   The code traversing the queue when updates are being applied to the
   display has also been simplified, since we always start the traversal
   at the beginning of the queue to ensure that all updates are applied.

Carwyn Ellis (2):
  hw/display/vmware_vga: replace fprintf calls with trace events
  hw/display/vmware_vga: do not discard screen updates

 hw/display/trace-events |  4 +++
 hw/display/vmware_vga.c | 63 +++++++++++++++++++++--------------------
 2 files changed, 37 insertions(+), 30 deletions(-)

-- 
2.34.1