[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [PATCH 0/3] ui/console: initialize QemuDmaBuf in ui/console
From: |
Kim, Dongwon |
Subject: |
RE: [PATCH 0/3] ui/console: initialize QemuDmaBuf in ui/console |
Date: |
Thu, 21 Mar 2024 07:01:31 +0000 |
Hi Phlippe,
> -----Original Message-----
> From: Philippe Mathieu-Daudé <philmd@linaro.org>
> Sent: Wednesday, March 20, 2024 11:57 PM
> To: Kim, Dongwon <dongwon.kim@intel.com>; qemu-devel@nongnu.org
> Cc: marcandre.lureau@redhat.com
> Subject: Re: [PATCH 0/3] ui/console: initialize QemuDmaBuf in ui/console
>
> Hi Dongwon,
>
> On 20/3/24 21:50, dongwon.kim@intel.com wrote:
> > From: Dongwon Kim <dongwon.kim@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?
>
> > 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
- [PATCH 0/3] ui/console: initialize QemuDmaBuf in ui/console, dongwon . kim, 2024/03/20
- [PATCH 1/3] hw/virtio: initialize QemuDmaBuf using the function from ui/console, dongwon . kim, 2024/03/20
- [PATCH 2/3] hw/vfio: intialize QemuDmaBuf using the function from ui/console, dongwon . kim, 2024/03/20
- [PATCH 3/3] ui/console: add methods for allocating, intializing and accessing QemuDmaBuf, dongwon . kim, 2024/03/20
- Re: [PATCH 0/3] ui/console: initialize QemuDmaBuf in ui/console, Philippe Mathieu-Daudé, 2024/03/21
- RE: [PATCH 0/3] ui/console: initialize QemuDmaBuf in ui/console,
Kim, Dongwon <=
- Re: [PATCH 0/3] ui/console: initialize QemuDmaBuf in ui/console, Philippe Mathieu-Daudé, 2024/03/21