On 21/3/24 08:01, Kim, Dongwon wrote:
Hi Phlippe,
-----Original Message-----
From: Philippe Mathieu-Daudé <phi...@linaro.org>
Sent: Wednesday, March 20, 2024 11:57 PM
To: Kim, Dongwon <dongwon....@intel.com>; qemu-devel@nongnu.org
Cc: marcandre.lur...@redhat.com
Subject: Re: [PATCH 0/3] ui/console: initialize QemuDmaBuf in ui/console
Hi Dongwon,
On 20/3/24 21:50, dongwon....@intel.com wrote:
From: Dongwon Kim <dongwon....@intel.com>
QemuDmaBuf struct is defined and primarily used by ui/console/gl so it
is better to handle its creation, initialization and access within
ui/console rather than within hw modules such as hw/display/virtio-gpu
and hw/vfio/display.
To achieve this, new methods for allocating, initializing the struct,
and accessing certain fields necessary for hardware modules have been
introduced in ui/console.c.
(3rd patch)
Furthermore, modifications have been made to hw/display/virtio-gpu and
hw/vfio/display to utilize these new methods instead of setting up the
struct independently.
(1st and 2nd patches)
Thanks for splitting, unfortunately the series isn't buildable / bisectable
since the
methods use in patches 1&2 are only introduced in patch 3 :/
[Kim, Dongwon] Maybe changing order of patches like 3-1-2 would be acceptable?
No, because of the g_free() call in dpy_gl_release_dmabuf().
Maybe Marc-André is OK with the previous version...
Dongwon Kim (3):
hw/virtio: intialize QemuDmaBuf using the function from ui/console
hw/vfio: intialize QemuDmaBuf using the function from ui/console
ui/console: add methods for allocating, intializing and accessing
QemuDmaBuf