qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 00/67] Make pixman an optional dependency
@ 2023-08-30  9:37 marcandre.lureau
  2023-08-30  9:37 ` [PATCH 01/67] ui: remove qemu_pixman_color() helper marcandre.lureau
                   ` (67 more replies)
  0 siblings, 68 replies; 157+ messages in thread
From: marcandre.lureau @ 2023-08-30  9:37 UTC (permalink / raw)
  To: qemu-devel; +Cc: Gerd Hoffmann, Marc-André Lureau

From: Marc-André Lureau <marcandre.lureau@redhat.com>

Hi,

QEMU system emulators can be made to compile and work without pixman.

Given how pervasively pixman types and API is used in all the code base, it was
a bit difficult to figure out how to cut the dependency.

I decided that it was important to keep VGA and graphics device working for
compatibility reasons, although some devices, such as xlnx Display Port, have
stronger dependency and have to be disabled. The ui/console code also has a lot
of pixman usage and a bit of a mess to deal with. I made large refactoring to
allow to compile out the VC code.

The series can be roughly divded as:
- a few related preliminary cleanups
- ui/console refactoring to allow ui/vc split
- add a 'pixman' option, and a minimal pixman-compat.h
- make some parts depend on 'pixman'

Graphic -display still work, although with some caveats. For ex, -display sdl or
cocoa don't have VCs, so starting QEMU will print the following warnings when
pixman is disabled:

qemu-system-x86_64: warning: compat_monitor0: this is a dummy VC driver. Use '-nographic' or a different chardev.
qemu-system-x86_64: warning: serial0: this is a dummy VC driver. Use '-nographic' or a different chardev.
qemu-system-x86_64: warning: parallel0: this is a dummy VC driver. Use '-nographic' or a different chardev.

But -display dbus is fully functionnal, for ex, as it operates at chardev/text
level.

Fixes:
https://gitlab.com/qemu-project/qemu/-/issues/1172

Marc-André Lureau (67):
  ui: remove qemu_pixman_color() helper
  ui: remove qemu_pixman_linebuf_copy()
  ui/qmp: move screendump to ui-qmp-cmds.c
  ui/vc: replace vc_chr_write() with generic qemu_chr_write()
  ui/vc: drop have_text
  ui/console: console_select() regardless of have_gfx
  ui/console: call dpy_gfx_update() regardless of have_gfx
  ui/console: drop have_gfx
  ui/console: get the DisplayState from new_console()
  ui/console: new_console() cannot fail
  ui/vc: VC always has a DisplayState now
  ui/vc: move VCChardev declaration at the top
  ui/vc: replace variable with static text attributes default
  ui/vc: fold text_update_xy()
  ui/vc: pass VCCharDev to VC-specific functions
  ui/vc: move VCCharDev specific fields out of QemuConsole
  ui/console: use OBJECT_DEFINE_TYPE for QemuConsole
  ui/console: change new_console() to use object initialization
  ui/console: introduce different console objects
  ui/console: instantiate a specific console type
  ui/console: register the console from qemu_console_init()
  ui/console: remove new_console()
  ui/console: specialize console_lookup_unused()
  ui/console: update the head from unused QemuConsole
  ui/console: allocate ui_timer in QemuConsole
  ui/vc: move cursor_timer initialization to QemuTextConsole class
  ui/console: free more QemuConsole resources
  ui/vc: move text fields to QemuTextConsole
  ui/console: move graphic fields to QemuGraphicConsole
  ui/vc: fold text_console_do_init() in vc_chr_open()
  ui/vc: move some text console initialization to qom handlers
  ui/console: simplify getting active_console size
  ui/console: remove need for g_width/g_height
  ui/vc: use common text console surface creation
  ui/console: declare console types in console.h
  ui/console: use QEMU_PIXMAN_COLOR helpers
  ui/console: rename vga_ functions → qemu_console_
  ui/console: assert(surface) where appropriate
  ui/console: fold text_console_update_cursor_timer
  ui/vc: skip text console resize when possible
  ui/console: minor stylistic changes
  ui/vc: move text console invalidate in helper
  ui/vc: do not parse VC-specific options in Spice and GTK
  ui/vc: change the argument for QemuTextConsole
  ui/vc: remove kby_put_keysym() and update function calls
  ui/vc: rename kbd_put → qemu_text_console functions
  ui/console: remove redundant format field
  ui/vc: preliminary QemuTextConsole changes before split
  ui/vc: split off the VC part from console.c
  ui/console: move DisplaySurface to its own header
  build-sys: add optional "pixman" feature
  ui: compile out some qemu-pixman functions when !PIXMAN
  ui: add pixman-compat.h
  ui/vc: console-vc requires PIXMAN
  qmp/hmp: disable screendump if PIXMAN is missing
  virtio-gpu: replace PIXMAN for region/rect test
  ui/console: when PIXMAN is unavailable, don't draw placeholder msg
  vhost-user-gpu: skip VHOST_USER_GPU_UPDATE when !PIXMAN
  ui/gl: opengl doesn't require PIXMAN
  ui/vnc: VNC requires PIXMAN
  ui/spice: SPICE/QXL requires PIXMAN
  ui/gtk: -display gtk requires PIXMAN
  ui/dbus: do not require PIXMAN
  arm/kconfig: XLNX_ZYNQMP_ARM depends on PIXMAN
  ppc/kconfig: make SAM460EX depend on PPC & PIXMAN
  sh4/kconfig: make R2D depend on SH4 & PIXMAN
  display/kconfig: make SM501 depend on PIXMAN

 configs/devices/ppc-softmmu/default.mak |    2 +-
 configs/devices/sh4-softmmu/default.mak |    2 +-
 meson.build                             |   11 +-
 qapi/char.json                          |    4 +
 qapi/ui.json                            |    3 +-
 include/chardev/char.h                  |    3 -
 include/ui/console.h                    |  118 +-
 include/ui/pixman-compat.h              |  190 +++
 include/ui/qemu-pixman.h                |   20 +-
 include/ui/rect.h                       |   55 +
 include/ui/surface.h                    |   91 ++
 ui/console-priv.h                       |   43 +
 hw/display/vhost-user-gpu.c             |    2 +
 hw/display/virtio-gpu.c                 |   30 +-
 ui/console-gl.c                         |    2 +-
 ui/console-vc-stubs.c                   |   59 +
 ui/console-vc.c                         | 1079 +++++++++++++++
 ui/console.c                            | 1603 +++--------------------
 ui/curses.c                             |    2 +-
 ui/dbus-listener.c                      |   62 +-
 ui/gtk.c                                |   11 +-
 ui/qemu-pixman.c                        |   25 +-
 ui/sdl2-input.c                         |    5 +-
 ui/sdl2.c                               |    5 +-
 ui/spice-app.c                          |    7 +-
 ui/spice-display.c                      |    2 +-
 ui/ui-hmp-cmds.c                        |    2 +
 ui/ui-qmp-cmds.c                        |  189 +++
 ui/vnc.c                                |   56 +-
 Kconfig.host                            |    3 +
 hmp-commands.hx                         |    2 +
 hw/arm/Kconfig                          |    2 +-
 hw/display/Kconfig                      |    3 +-
 hw/ppc/Kconfig                          |    2 +
 hw/sh4/Kconfig                          |    2 +
 meson_options.txt                       |    2 +
 ui/cocoa.m                              |    2 +-
 ui/meson.build                          |   25 +-
 38 files changed, 2093 insertions(+), 1633 deletions(-)
 create mode 100644 include/ui/pixman-compat.h
 create mode 100644 include/ui/rect.h
 create mode 100644 include/ui/surface.h
 create mode 100644 ui/console-priv.h
 create mode 100644 ui/console-vc-stubs.c
 create mode 100644 ui/console-vc.c

-- 
2.41.0



^ permalink raw reply	[flat|nested] 157+ messages in thread

end of thread, other threads:[~2023-09-06 11:01 UTC | newest]

Thread overview: 157+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-08-30  9:37 [PATCH 00/67] Make pixman an optional dependency marcandre.lureau
2023-08-30  9:37 ` [PATCH 01/67] ui: remove qemu_pixman_color() helper marcandre.lureau
2023-08-30 10:03   ` Philippe Mathieu-Daudé
2023-08-30 10:59   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 02/67] ui: remove qemu_pixman_linebuf_copy() marcandre.lureau
2023-08-30 10:59   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 03/67] ui/qmp: move screendump to ui-qmp-cmds.c marcandre.lureau
2023-08-30 11:00   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 04/67] ui/vc: replace vc_chr_write() with generic qemu_chr_write() marcandre.lureau
2023-08-30 10:04   ` Philippe Mathieu-Daudé
2023-08-30 11:01   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 05/67] ui/vc: drop have_text marcandre.lureau
2023-08-30 11:03   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 06/67] ui/console: console_select() regardless of have_gfx marcandre.lureau
2023-08-30 11:05   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 07/67] ui/console: call dpy_gfx_update() " marcandre.lureau
2023-08-30 11:06   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 08/67] ui/console: drop have_gfx marcandre.lureau
2023-08-30 11:06   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 09/67] ui/console: get the DisplayState from new_console() marcandre.lureau
2023-08-30 12:39   ` BALATON Zoltan
2023-09-01 14:10   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 10/67] ui/console: new_console() cannot fail marcandre.lureau
2023-08-30 11:07   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 11/67] ui/vc: VC always has a DisplayState now marcandre.lureau
2023-09-01 13:44   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 12/67] ui/vc: move VCChardev declaration at the top marcandre.lureau
2023-08-30 11:07   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 13/67] ui/vc: replace variable with static text attributes default marcandre.lureau
2023-09-01 13:45   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 14/67] ui/vc: fold text_update_xy() marcandre.lureau
2023-08-30 11:08   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 15/67] ui/vc: pass VCCharDev to VC-specific functions marcandre.lureau
2023-09-01 14:15   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 16/67] ui/vc: move VCCharDev specific fields out of QemuConsole marcandre.lureau
2023-09-01 14:16   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 17/67] ui/console: use OBJECT_DEFINE_TYPE for QemuConsole marcandre.lureau
2023-08-30 11:09   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 18/67] ui/console: change new_console() to use object initialization marcandre.lureau
2023-09-01 14:19   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 19/67] ui/console: introduce different console objects marcandre.lureau
2023-09-01 14:20   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 20/67] ui/console: instantiate a specific console type marcandre.lureau
2023-09-01 14:21   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 21/67] ui/console: register the console from qemu_console_init() marcandre.lureau
2023-09-01 14:21   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 22/67] ui/console: remove new_console() marcandre.lureau
2023-09-01 14:22   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 23/67] ui/console: specialize console_lookup_unused() marcandre.lureau
2023-09-01 14:23   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 24/67] ui/console: update the head from unused QemuConsole marcandre.lureau
2023-09-01 14:23   ` Daniel P. Berrangé
2023-08-30  9:37 ` [PATCH 25/67] ui/console: allocate ui_timer in QemuConsole marcandre.lureau
2023-09-01 14:24   ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 26/67] ui/vc: move cursor_timer initialization to QemuTextConsole class marcandre.lureau
2023-09-01 14:27   ` Daniel P. Berrangé
2023-09-04  9:55     ` Marc-André Lureau
2023-08-30  9:38 ` [PATCH 27/67] ui/console: free more QemuConsole resources marcandre.lureau
2023-09-01 14:29   ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 28/67] ui/vc: move text fields to QemuTextConsole marcandre.lureau
2023-09-01 14:30   ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 29/67] ui/console: move graphic fields to QemuGraphicConsole marcandre.lureau
2023-09-01 14:31   ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 30/67] ui/vc: fold text_console_do_init() in vc_chr_open() marcandre.lureau
2023-09-01 14:34   ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 31/67] ui/vc: move some text console initialization to qom handlers marcandre.lureau
2023-09-01 14:36   ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 32/67] ui/console: simplify getting active_console size marcandre.lureau
2023-09-01 14:37   ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 33/67] ui/console: remove need for g_width/g_height marcandre.lureau
2023-09-01 14:37   ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 34/67] ui/vc: use common text console surface creation marcandre.lureau
2023-09-01 14:38   ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 35/67] ui/console: declare console types in console.h marcandre.lureau
2023-09-01 14:39   ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 36/67] ui/console: use QEMU_PIXMAN_COLOR helpers marcandre.lureau
2023-08-30 15:13   ` Philippe Mathieu-Daudé
2023-08-30  9:38 ` [PATCH 37/67] ui/console: rename vga_ functions → qemu_console_ marcandre.lureau
2023-08-30 11:11   ` Daniel P. Berrangé
2023-08-30 12:44   ` BALATON Zoltan
2023-08-30 15:13   ` Philippe Mathieu-Daudé
2023-08-30  9:38 ` [PATCH 38/67] ui/console: assert(surface) where appropriate marcandre.lureau
2023-09-01 14:39   ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 39/67] ui/console: fold text_console_update_cursor_timer marcandre.lureau
2023-08-30 15:15   ` Philippe Mathieu-Daudé
2023-09-01 16:51   ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 40/67] ui/vc: skip text console resize when possible marcandre.lureau
2023-09-01 16:52   ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 41/67] ui/console: minor stylistic changes marcandre.lureau
2023-09-01 16:52   ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 42/67] ui/vc: move text console invalidate in helper marcandre.lureau
2023-09-01 16:54   ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 43/67] ui/vc: do not parse VC-specific options in Spice and GTK marcandre.lureau
2023-09-01 17:13   ` Daniel P. Berrangé
2023-09-04 10:56     ` Marc-André Lureau
2023-08-30  9:38 ` [PATCH 44/67] ui/vc: change the argument for QemuTextConsole marcandre.lureau
2023-09-01 17:15   ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 45/67] ui/vc: remove kby_put_keysym() and update function calls marcandre.lureau
2023-08-30 20:59   ` Akihiko Odaki
2023-09-04 12:42     ` Marc-André Lureau
2023-09-04 12:47       ` Akihiko Odaki
2023-08-30  9:38 ` [PATCH 46/67] ui/vc: rename kbd_put → qemu_text_console functions marcandre.lureau
2023-08-30 15:41   ` Philippe Mathieu-Daudé
2023-09-01 17:17     ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 47/67] ui/console: remove redundant format field marcandre.lureau
2023-09-01 17:18   ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 48/67] ui/vc: preliminary QemuTextConsole changes before split marcandre.lureau
2023-09-01 17:19   ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 49/67] ui/vc: split off the VC part from console.c marcandre.lureau
2023-09-01 17:23   ` Daniel P. Berrangé
2023-09-04 12:57     ` Marc-André Lureau
2023-09-04 13:05       ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 50/67] ui/console: move DisplaySurface to its own header marcandre.lureau
2023-09-01 17:24   ` Daniel P. Berrangé
2023-09-04 12:59     ` Marc-André Lureau
2023-09-04 13:05       ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 51/67] build-sys: add optional "pixman" feature marcandre.lureau
2023-08-30 15:48   ` Philippe Mathieu-Daudé
2023-08-30 15:55     ` Philippe Mathieu-Daudé
2023-09-01 17:25       ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 52/67] ui: compile out some qemu-pixman functions when !PIXMAN marcandre.lureau
2023-08-30  9:38 ` [PATCH 53/67] ui: add pixman-compat.h marcandre.lureau
2023-08-30 11:17   ` Daniel P. Berrangé
2023-08-30 11:23     ` Marc-André Lureau
2023-08-30  9:38 ` [PATCH 54/67] ui/vc: console-vc requires PIXMAN marcandre.lureau
2023-09-01 17:28   ` Daniel P. Berrangé
2023-09-04 13:41     ` Marc-André Lureau
2023-09-04 13:44       ` Daniel P. Berrangé
2023-08-30  9:38 ` [PATCH 55/67] qmp/hmp: disable screendump if PIXMAN is missing marcandre.lureau
2023-08-30 15:18   ` Philippe Mathieu-Daudé
2023-08-30  9:38 ` [PATCH 56/67] virtio-gpu: replace PIXMAN for region/rect test marcandre.lureau
2023-08-30  9:38 ` [PATCH 57/67] ui/console: when PIXMAN is unavailable, don't draw placeholder msg marcandre.lureau
2023-08-30  9:38 ` [PATCH 58/67] vhost-user-gpu: skip VHOST_USER_GPU_UPDATE when !PIXMAN marcandre.lureau
2023-08-30  9:38 ` [PATCH 59/67] ui/gl: opengl doesn't require PIXMAN marcandre.lureau
2023-08-30  9:38 ` [PATCH 60/67] ui/vnc: VNC requires PIXMAN marcandre.lureau
2023-09-04 14:01   ` Paolo Bonzini
2023-08-30  9:38 ` [PATCH 61/67] ui/spice: SPICE/QXL " marcandre.lureau
2023-08-30 15:21   ` Philippe Mathieu-Daudé
2023-09-04 14:42     ` Paolo Bonzini
2023-09-04 14:03   ` Paolo Bonzini
2023-08-30  9:38 ` [PATCH 62/67] ui/gtk: -display gtk " marcandre.lureau
2023-08-30  9:38 ` [PATCH 63/67] ui/dbus: do not require PIXMAN marcandre.lureau
2023-08-30  9:38 ` [PATCH 64/67] arm/kconfig: XLNX_ZYNQMP_ARM depends on PIXMAN marcandre.lureau
2023-08-30 15:25   ` Philippe Mathieu-Daudé
2023-08-30  9:38 ` [PATCH 65/67] ppc/kconfig: make SAM460EX depend on PPC & PIXMAN marcandre.lureau
2023-08-30 12:34   ` BALATON Zoltan
2023-09-04 16:24     ` Paolo Bonzini
2023-09-05 20:07     ` Marc-André Lureau
2023-09-06 11:00       ` Paolo Bonzini
2023-09-04 14:05   ` Paolo Bonzini
2023-08-30  9:38 ` [PATCH 66/67] sh4/kconfig: make R2D depend on SH4 " marcandre.lureau
2023-08-30  9:38 ` [PATCH 67/67] display/kconfig: make SM501 depend on PIXMAN marcandre.lureau
2023-08-30 15:27   ` Philippe Mathieu-Daudé
2023-08-30 10:52 ` [PATCH 00/67] Make pixman an optional dependency Thomas Huth
2023-08-30 11:01   ` Marc-André Lureau
2023-08-30 11:21     ` Daniel P. Berrangé
2023-08-30 11:32       ` Marc-André Lureau

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).