On Mon, Mar 13, 2023 at 10:22:24AM +0800, Dmitry Osipenko wrote:
> On 3/12/23 20:51, Dmitry Osipenko wrote:
> > On 3/12/23 12:22, Huang Rui wrote:
> >> From: Antonio Caggiano <antonio.caggi...@collabora.com>
> >>
> >> Request Venus when initializing VirGL.
> >>
> >> Signed-off-by: Antonio Caggiano <antonio.caggi...@collabora.com>
> >> ---
> >> hw/display/virtio-gpu-virgl.c | 4 ++++
> >> 1 file changed, 4 insertions(+)
> >>
> >> diff --git a/hw/display/virtio-gpu-virgl.c b/hw/display/virtio-gpu-virgl.c
> >> index fe03dc916f..f5ce206b93 100644
> >> --- a/hw/display/virtio-gpu-virgl.c
> >> +++ b/hw/display/virtio-gpu-virgl.c
> >> @@ -803,7 +803,11 @@ int virtio_gpu_virgl_init(VirtIOGPU *g)
> >> {
> >> int ret;
> >>
> >> +#ifdef VIRGL_RENDERER_VENUS
> >> + ret = virgl_renderer_init(g, VIRGL_RENDERER_VENUS,
> >> &virtio_gpu_3d_cbs);
> >> +#else
> >> ret = virgl_renderer_init(g, 0, &virtio_gpu_3d_cbs);
> >> +#endif
> >
> > Note that Venus now requires VIRGL_RENDERER_RENDER_SERVER flag to be
> > set. Please test the patches with the latest virglrenderer and etc.
> >
> > The #ifdef also doesn't allow adding new flags, it should look like:
> >
> > #ifdef VIRGL_RENDERER_VENUS
> > flags |= VIRGL_RENDERER_RENDER_SERVER;
> > #endif
> >
> > ret = virgl_renderer_init(g, flags, &virtio_gpu_3d_cbs);
> >
>
> BTW, Alex reviewed the Venus v3 patches a month ago [1] and the review
> comments need to be addressed. AFAICS, you're actually using the very
> old Venus patches here that stopped working about a year ago, so again
> you're using a very outdated virglrenderer version.
>
> Please take it all into account if you'll beat me to posting the next
> version of Venus patches ;)
>
> [1]
> https://lore.kernel.org/qemu-devel/20220926142422.22325-1-antonio.caggi...@collabora.com/
>
Thanks Dmitry point it out, I will use the latest v3 patches, and try to
address the comments in next version. :-)
Thanks,
Ray
