[Qemu-commits] [qemu/qemu] 4d6316: console: add graphic_hw_update_done()

[Peter Maydell]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 4d6316218bf7bf3b8c7c7165b072cc314511a7a7
      
https://github.com/qemu/qemu/commit/4d6316218bf7bf3b8c7c7165b072cc314511a7a7
  Author: Marc-André Lureau <address@hidden>
  Date:   2020-01-02 (Thu, 02 Jan 2020)

  Changed paths:
    M hw/display/qxl-render.c
    M hw/display/qxl.c
    M include/ui/console.h
    M ui/console.c

  Log Message:
  -----------
  console: add graphic_hw_update_done()

Add a function to be called when a graphic update is done.

Declare the QXL renderer as async: render_update_cookie_num counts the
number of outstanding updates, and graphic_hw_update_done() is called
when it reaches none.

(note: this is preliminary work for asynchronous screendump support)

Signed-off-by: Marc-André Lureau <address@hidden>
Reviewed-by: Gerd Hoffmann <address@hidden>
Reviewed-by: Daniel P. Berrangé <address@hidden>


  Commit: 46e5841cd2d8ffbaaa95ee7a1358e9650f9a1b93
      
https://github.com/qemu/qemu/commit/46e5841cd2d8ffbaaa95ee7a1358e9650f9a1b93
  Author: Marc-André Lureau <address@hidden>
  Date:   2020-01-02 (Thu, 02 Jan 2020)

  Changed paths:
    M ui/console.c
    M ui/trace-events

  Log Message:
  -----------
  ppm-save: pass opened fd

This will allow to pre-open the file before running the async finish
handler and avoid potential monitor fdset races.

(note: this is preliminary work for asynchronous screendump support)

Signed-off-by: Marc-André Lureau <address@hidden>
Reviewed-by: Daniel P. Berrangé <address@hidden>


  Commit: f908475f2f327ffe3508ba5ddfa5c53ad656958d
      
https://github.com/qemu/qemu/commit/f908475f2f327ffe3508ba5ddfa5c53ad656958d
  Author: Marc-André Lureau <address@hidden>
  Date:   2020-01-02 (Thu, 02 Jan 2020)

  Changed paths:
    M include/ui/qemu-pixman.h

  Log Message:
  -----------
  ui: add pixman image g_autoptr support

Signed-off-by: Marc-André Lureau <address@hidden>
Reviewed-by: Daniel P. Berrangé <address@hidden>


  Commit: f60a1cdca52dd9cf49aad768471d9e0ea6751f76
      
https://github.com/qemu/qemu/commit/f60a1cdca52dd9cf49aad768471d9e0ea6751f76
  Author: Marc-André Lureau <address@hidden>
  Date:   2020-01-02 (Thu, 02 Jan 2020)

  Changed paths:
    M include/qom/object.h

  Log Message:
  -----------
  object: add g_autoptr support

Signed-off-by: Marc-André Lureau <address@hidden>
Reviewed-by: Daniel P. Berrangé <address@hidden>


  Commit: c5f2bce5ee75ec3267b025f2053bac728da9a6c3
      
https://github.com/qemu/qemu/commit/c5f2bce5ee75ec3267b025f2053bac728da9a6c3
  Author: Marc-André Lureau <address@hidden>
  Date:   2020-01-02 (Thu, 02 Jan 2020)

  Changed paths:
    M ui/console.c

  Log Message:
  -----------
  screendump: replace FILE with QIOChannel and fix close()/qemu_close()

The file opened for ppm_save() may be a /dev/fdset, in which case a
dup fd is added to the fdset. It should be removed by calling
qemu_close(), instead of the implicit close() on fclose().

I don't see a convenient way to solve that with stdio streams, so I
switched the code to QIOChannel which uses qemu_close().

Signed-off-by: Marc-André Lureau <address@hidden>
Reviewed-by: Daniel P. Berrangé <address@hidden>


  Commit: ee13240e60a02836426140d384e363be5d8f8bbe
      
https://github.com/qemu/qemu/commit/ee13240e60a02836426140d384e363be5d8f8bbe
  Author: Marc-André Lureau <address@hidden>
  Date:   2020-01-02 (Thu, 02 Jan 2020)

  Changed paths:
    M include/qemu/osdep.h
    M util/osdep.c

  Log Message:
  -----------
  osdep: add qemu_unlink()

Add a helper function to match qemu_open() which may return files
under the /dev/fdset prefix. Those shouldn't be removed, since it's
only a qemu namespace.

Signed-off-by: Marc-André Lureau <address@hidden>
Reviewed-by: Daniel P. Berrangé <address@hidden>


  Commit: 53a61ecbb16b28c707f0b126193d226a25c28cb2
      
https://github.com/qemu/qemu/commit/53a61ecbb16b28c707f0b126193d226a25c28cb2
  Author: Marc-André Lureau <address@hidden>
  Date:   2020-01-02 (Thu, 02 Jan 2020)

  Changed paths:
    M ui/console.c

  Log Message:
  -----------
  screendump: use qemu_unlink()

Don't attempt to remove /dev/fdset files.

Signed-off-by: Marc-André Lureau <address@hidden>
Reviewed-by: Daniel P. Berrangé <address@hidden>


  Commit: 81e6a7345a1299c8d0ce879e6abfe928a24df62b
      
https://github.com/qemu/qemu/commit/81e6a7345a1299c8d0ce879e6abfe928a24df62b
  Author: Peter Maydell <address@hidden>
  Date:   2020-01-06 (Mon, 06 Jan 2020)

  Changed paths:
    M hw/display/qxl-render.c
    M hw/display/qxl.c
    M include/qemu/osdep.h
    M include/qom/object.h
    M include/ui/console.h
    M include/ui/qemu-pixman.h
    M ui/console.c
    M ui/trace-events
    M util/osdep.c

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/elmarco/tags/screendump-pull-request' 
into staging

console: screendump improvements

Hi,

The following patches have been extracted from the "[PATCH v6 00/25]
monitor: add asynchronous command type", as they are
reviewable/mergeable independantly.

They introduce some internal API changes, and fix
qemu_open()/qemu_close()/unlink() misusages which should be quite
harmless.

# gpg: Signature made Fri 03 Jan 2020 09:07:51 GMT
# gpg:                using RSA key 87A9BD933F87C606D276F62DDAE8E10975969CE5
# gpg:                issuer "address@hidden"
# gpg: Good signature from "Marc-André Lureau <address@hidden>" [full]
# gpg:                 aka "Marc-André Lureau <address@hidden>" [full]
# Primary key fingerprint: 87A9 BD93 3F87 C606 D276  F62D DAE8 E109 7596 9CE5

* remotes/elmarco/tags/screendump-pull-request:
  screendump: use qemu_unlink()
  osdep: add qemu_unlink()
  screendump: replace FILE with QIOChannel and fix close()/qemu_close()
  object: add g_autoptr support
  ui: add pixman image g_autoptr support
  ppm-save: pass opened fd
  console: add graphic_hw_update_done()

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/6fb0dae9efa9...81e6a7345a12