Commit 167e9c79 authored by Gerd Hoffmann's avatar Gerd Hoffmann
Browse files

framebuffer: make display updates thread safe 



Signed-off-by: default avatarGerd Hoffmann <kraxel@redhat.com>
Message-id: 20170421091632.30900-6-kraxel@redhat.com
Signed-off-by: default avatarGerd Hoffmann <kraxel@redhat.com>
parent fec5e8c9

hw/display/framebuffer.c

+5 −6
Original line number Diff line number Diff line
@@ -67,7 +67,7 @@ void framebuffer_update_display( 
    int *first_row, /* Input and output.  */

    int *last_row /* Output only */)

{

    hwaddr src_len;

    DirtyBitmapSnapshot *snap;

    uint8_t *dest;

    uint8_t *src;

    int first, last = 0;
@@ -78,7 +78,6 @@ void framebuffer_update_display( 


    i = *first_row;

    *first_row = -1;

    src_len = (hwaddr)src_width * rows;



    mem = mem_section->mr;

    if (!mem) {
@@ -102,9 +101,10 @@ void framebuffer_update_display( 
    src += i * src_width;

    dest += i * dest_row_pitch;



    for (; i < rows; i++) {

        dirty = memory_region_get_dirty(mem, addr, src_width,

    snap = memory_region_snapshot_and_clear_dirty(mem, addr, src_width * rows,

                                                  DIRTY_MEMORY_VGA);

    for (; i < rows; i++) {

        dirty = memory_region_snapshot_get_dirty(mem, snap, addr, src_width);

        if (dirty || invalidate) {

            fn(opaque, dest, src, cols, dest_col_pitch);

            if (first == -1)
@@ -115,11 +115,10 @@ void framebuffer_update_display( 
        src += src_width;

        dest += dest_row_pitch;

    }

    g_free(snap);

    if (first < 0) {

        return;

    }

    memory_region_reset_dirty(mem, mem_section->offset_within_region, src_len,

                              DIRTY_MEMORY_VGA);

    *first_row = first;

    *last_row = last;

}