Change virtio-gpu Venus link, pointing it at the Mesa Venus
documentation instead of the protocol. The Mesa doc provides more
information and also has a link to the protocol.
Suggested-by: Akihiko Odaki
Reviewed-by: Akihiko Odaki
Signed-off-by: Dmitry Osipenko
---
docs/system/devices/virtio
sults
in a lightweight context implementations that yield better performance.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
docs/system/devices/virtio-gpu.rst | 11 +++
hw/display/virtio-gpu-gl.c | 2 ++
hw/display/
ontexts in virglrenderer.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Reviewed-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
hw/display/virtio-gpu-gl.c | 5 ++
hw/display/virtio-gpu-virgl.c | 130 +
include/hw/virtio/vi
vblank_mode=0 for GTK). Refreshment of display's
content shouldn't disable the active display. Fix it by keeping the
scanout's state unchanged when display is redrawn.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
--
Extend virtio-gpu documentation with a link to the Mesa VirGL
documentation.
Suggested-by: Akihiko Odaki
Reviewed-by: Akihiko Odaki
Signed-off-by: Dmitry Osipenko
---
docs/system/devices/virtio-gpu.rst | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/docs/system/devices
vblank_mode=0 for GTK). Refreshment of display's
content shouldn't disable the active display. Fix it by keeping the
scanout's state unchanged when display is redrawn.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
--
Reviewed-by: Akihiko Odaki
Signed-off-by: Dmitry Osipenko
[dmitry.osipe...@collabora.com: Extended and corrected doc]
---
docs/system/devices/virtio-gpu.rst | 101 -
1 file changed, 98 insertions(+), 3 deletions(-)
diff --git a/docs/system/devices/virtio-gpu.rst
b/docs
Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
ui/sdl2-gl.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/ui/sdl2-gl.c b/ui/sdl2-gl.c
index 3be17d1079af..ea2301305d3e 100644
--- a/ui/sdl2-gl.c
+++ b/ui/sdl2-gl.c
@@ -168,6 +168,9 @@ QEMUGLContext sdl2_gl_create_
ror message
if DRM context can't be initialized instead of giving a warning
message, as was suggested by Akihiko Odaki.
- Added patchew's dependecy tag to the cover letter as was suggested by
Akihiko Odaki.
Alex Bennée (1):
docs/system: virtio-gpu: Document host/g
Print out error messages when virgl fence creation fails to aid debugging
of the fence-related bugs.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
hw/display/virtio-gpu-virgl.c | 27 ++-
1 file
ael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Pierre-Eric Pelloux-Prayer
Signed-off-by: Dmitry Osipenko
---
include/ui/sdl2.h | 7 +
meson.build | 6 ++---
ui/sdl2-gl.c | 66 +++
ui/sdl2.c | 42
On 5/24/25 02:33, Dmitry Osipenko wrote:
> + * - Asahi DRM native context
> +- `Downstream version`_ of Asahi Linux kernel
> +- `mr1274`_
> +
> +.. _mr1384:
> https://gitlab.freedesktop.org/virgl/virglrenderer/-/merge_requests/1384
> +.. _mr1274:
> https://gitl
On 5/24/25 02:32, Dmitry Osipenko wrote:
> @@ -872,6 +872,8 @@ void virtio_gpu_virgl_process_cmd(VirtIOGPU *g,
>struct virtio_gpu_ctrl_command *cmd)
> {
> bool cmd_suspended = false;
> +uint32_t flags;
Realized I re-introduced th
> + }
mem_obj should be destroyed on error and when created texture is released
--
Best regards,
Dmitry
Reviewed-by: Akihiko Odaki
Signed-off-by: Dmitry Osipenko
[dmitry.osipe...@collabora.com: Extended and corrected doc]
---
docs/system/devices/virtio-gpu.rst | 101 -
1 file changed, 98 insertions(+), 3 deletions(-)
diff --git a/docs/system/devices/virtio-gpu.rst
b/docs
the cover letter as was suggested by
Akihiko Odaki.
Alex Bennée (1):
docs/system: virtio-gpu: Document host/guest requirements
Dmitry Osipenko (8):
ui/sdl2: Restore original context after new context creation
virtio-gpu: Handle virgl fence creation errors
virtio-gpu: Support asynchronous
Extend virtio-gpu documentation with a link to the Mesa VirGL
documentation.
Suggested-by: Akihiko Odaki
Reviewed-by: Akihiko Odaki
Signed-off-by: Dmitry Osipenko
---
docs/system/devices/virtio-gpu.rst | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/docs/system/devices
ael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Pierre-Eric Pelloux-Prayer
Signed-off-by: Dmitry Osipenko
---
include/ui/sdl2.h | 7 +
meson.build | 6 ++---
ui/sdl2-gl.c | 66 +++
ui/sdl2.c | 42
Change virtio-gpu Venus link, pointing it at the Mesa Venus
documentation instead of the protocol. The Mesa doc provides more
information and also has a link to the protocol.
Suggested-by: Akihiko Odaki
Reviewed-by: Akihiko Odaki
Signed-off-by: Dmitry Osipenko
---
docs/system/devices/virtio
vblank_mode=0 for GTK). Refreshment of display's
content shouldn't disable the active display. Fix it by keeping the
scanout's state unchanged when display is redrawn.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
--
ontexts in virglrenderer.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Reviewed-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
hw/display/virtio-gpu-gl.c | 5 ++
hw/display/virtio-gpu-virgl.c | 130 +
include/hw/virtio/vi
sults
in a lightweight context implementations that yield better performance.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
docs/system/devices/virtio-gpu.rst | 11 +++
hw/display/virtio-gpu-gl.c | 2 ++
hw/display/
Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
ui/sdl2-gl.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/ui/sdl2-gl.c b/ui/sdl2-gl.c
index e01d9ab0c7bf..b1fe96d6af22 100644
--- a/ui/sdl2-gl.c
+++ b/ui/sdl2-gl.c
@@ -168,6 +168,9 @@ QEMUGLContext sdl2_gl_create_
vblank_mode=0 for GTK). Refreshment of display's
content shouldn't disable the active display. Fix it by keeping the
scanout's state unchanged when display is redrawn.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
--
Print out error messages when virgl fence creation fails to aid debugging
of the fence-related bugs.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
hw/display/virtio-gpu-virgl.c | 28 +++-
1 file
On 5/20/25 11:26, Yiwei Zhang wrote:
> On Mon, May 19, 2025 at 7:29 PM Dmitry Osipenko
> wrote:
>>
>> On 5/18/25 18:26, Yiwei Zhang wrote:
>>> +#if VIRGL_VERSION_MAJOR >= 1
>>> +static void virgl_write_cont
it msg updates based on feedbacks from BALATON
>>
>> Cc: qemu-sta...@nongnu.org
>> Fixes: 94d0ea1c1928 ("virtio-gpu: Support Venus context")
>> Signed-off-by: Yiwei Zhang
>
> Queued to virtio-gpu/next (in maintainer/may-2025), thanks.
>
>
Wanted to no
cb be used by vrend for signalling
fences without ring_idx info?
--
Best regards,
Dmitry
ne uses separate fence list.
--
Best regards,
Dmitry
hdr.flags & VIRTIO_GPU_FLAG_INFO_RING_IDX) {
> +continue;
> +}
> +#endif
Is it possible that virglrenderer will ever write a context fence in
virgl_renderer_create_fence()? Do we really need this check?
--
Best regards,
Dmitry
t
never showed a problem during venus testing in the past.
Reviewed-by: Dmitry Osipenko
There could be multiple sources of a problem RE frames going backwards.
I'm always testing venus in conjunction with async-cb patch that adds
per-context fencing support and it doesn't help the X11
to me on IRC a year ago and the root of the problem
was identified only couple months ago.
[1]
https://lore.kernel.org/qemu-devel/5aedf1ad-d9b0-4edb-a050-f3d9bee9b...@collabora.com/
As of today, the GTK problem isn't understood.
--
Best regards,
Dmitry
On 5/4/25 10:20, Akihiko Odaki wrote:
> On 2025/04/30 3:56, Dmitry Osipenko wrote:
>> On 4/28/25 15:59, Alex Bennée wrote:
>>> Seeing as I've taken a few patches to here now I might as well put
>>> myself forward to maintain virtio-gpu. I've marked it as Od
the
patches and QEMU doesn't crash anymore. Now the code changes look sane
to me.
Will be great if Akihiko Odaki could comment too.
--
Best regards,
Dmitry
cu_thread (opaque=opaque@entry=0x0) at
../util/rcu.c:301
#13 0x55c1cc68 in qemu_thread_start (args=0x57993d30) at
../util/qemu-thread-posix.c:541
#14 0x7540fba8 in start_thread () at /lib64/libc.so.6
#15 0x75493b8c in __clone3 () at /lib64/libc.so.6
--
Best regards,
Dmitry
that I wanted to propose myself as reviewer for the
virtio-gpu patches. Will do it soon.
Akihiko Odaki is also good at reviewing virtio-gpu patches. Wondering if
Akihiko would want to be added as reviewer or co-maintainer of virtio-gpu?
--
Best regards,
Dmitry
), "blob", size, data);
> +memory_region_init_ram_ptr(mr, OBJECT(g), "blob", size, data);
> memory_region_add_subregion(&b->hostmem, offset, mr);
> memory_region_set_enabled(mr, true);
>
This change makes QEMU to crash.
AFAICT, it effectively reverts code to old bugged version [1] that was
rejected in the past.
+Akihiko Odaki
[1]
https://lore.kernel.org/qemu-devel/2023091530.24064-10-ray.hu...@amd.com/
--
Best regards,
Dmitry
ial stale GL errors, otherwise your code will handle unrelated GL
errors and report them as memfd import failures.
--
Best regards,
Dmitry
On 4/27/25 14:53, 刘聪 wrote:
> Hi Dmitry,
>
> The virglrender patch can fix the virgl issue, but the native context still
> fails to run on my machine.
> I'm not sure if anyone has successfully run it on an ARM64 machine before.
Thanks for the testing!
> When running
On 4/14/25 17:47, Dmitry Osipenko wrote:
> On 4/11/25 04:42, 刘聪 wrote:
>>
>>
>>
>>> -Original Messages-
>>> From: "Dmitry Osipenko"
>>> Send time:Friday, 04/11/2025 05:59:11
>>> To: "Cong Liu"
On 4/11/25 04:42, 刘聪 wrote:
>
>
>
>> -Original Messages-
>> From: "Dmitry Osipenko"
>> Send time:Friday, 04/11/2025 05:59:11
>> To: "Cong Liu"
>> Cc: jiqian.c...@amd.com, akihiko.od...@daynix.com, alex.ben...@linaro.o
y building QEMU against your
virglrenderer version, but also setting LD_LIBRARY_PATH properly at
runtime. Best to remove system version of virglrenderer if unsure,
2. Can you reproduce this problem using tcg instead of kvm?
--
Best regards,
Dmitry
glImportMemoryFdEXT(mem_obj, size, GL_HANDLE_TYPE_OPAQUE_FD_EXT, fd);
> +if (!glIsMemoryObjectEXT(mem_obj)) {
This check should always succeed. glGetError() should be used for
glImportMemoryFdEXT() failure checking.
Won't hurt to print error messages instead of failing silently.
--
Best regards,
Dmitry
On 3/10/25 15:25, Akihiko Odaki wrote:
> On 2025/03/10 21:05, Dmitry Osipenko wrote:
>> From: Alex Bennée
>>
>> This attempts to tidy up the VirtIO GPU documentation to make the list
>> of requirements clearer. There are still a lot of moving parts and the
>> di
gested by Akihiko Odaki.
- Added patchew's dependecy tag to the cover letter as was suggested by
Akihiko Odaki.
Alex Bennée (1):
docs/system: virtio-gpu: Document host/guest requirements
Dmitry Osipenko (8):
ui/sdl2: Restore original context after new context creation
virtio-gpu
ontexts in virglrenderer.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Reviewed-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
hw/display/virtio-gpu-gl.c | 3 +
hw/display/virtio-gpu-virgl.c | 147 +++--
include/hw/virtio/vi
ontexts in virglrenderer.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Reviewed-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
hw/display/virtio-gpu-gl.c | 3 +
hw/display/virtio-gpu-virgl.c | 147 +++--
include/hw/virtio/vi
Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
ui/sdl2-gl.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/ui/sdl2-gl.c b/ui/sdl2-gl.c
index e01d9ab0c7bf..b1fe96d6af22 100644
--- a/ui/sdl2-gl.c
+++ b/ui/sdl2-gl.c
@@ -168,6 +168,9 @@ QEMUGLContext sdl2_gl_create_
ontexts in virglrenderer.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Reviewed-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
hw/display/virtio-gpu-gl.c | 3 +
hw/display/virtio-gpu-virgl.c | 147 +++--
include/hw/virtio/vi
ael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Pierre-Eric Pelloux-Prayer
Signed-off-by: Dmitry Osipenko
---
include/ui/sdl2.h | 7 ++
meson.build | 6 ++---
ui/sdl2-gl.c | 64 +++
ui/sdl2.c | 42 +
Signed-off-by: Dmitry Osipenko
[dmitry.osipe...@collabora.com: Extended and corrected doc]
---
docs/system/devices/virtio-gpu.rst | 90 +-
1 file changed, 87 insertions(+), 3 deletions(-)
diff --git a/docs/system/devices/virtio-gpu.rst
b/docs/system/devices/virtio
vblank_mode=0 for GTK). Refreshment of display's
content shouldn't disable the active display. Fix it by keeping the
scanout's state unchanged when display is redrawn.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
--
Extend virtio-gpu documentation with a link to the Mesa VirGL
documentation.
Suggested-by: Akihiko Odaki
Reviewed-by: Akihiko Odaki
Signed-off-by: Dmitry Osipenko
---
docs/system/devices/virtio-gpu.rst | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/docs/system/devices
sults
in a lightweight context implementations that yield better performance.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
docs/system/devices/virtio-gpu.rst | 11 +++
hw/display/virtio-gpu-gl.c | 2 ++
hw/display/
Signed-off-by: Dmitry Osipenko
[dmitry.osipe...@collabora.com: Extended and corrected doc]
---
docs/system/devices/virtio-gpu.rst | 101 -
1 file changed, 98 insertions(+), 3 deletions(-)
diff --git a/docs/system/devices/virtio-gpu.rst
b/docs/system/devices/virtio
ael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Pierre-Eric Pelloux-Prayer
Signed-off-by: Dmitry Osipenko
---
include/ui/sdl2.h | 7 ++
meson.build | 6 ++---
ui/sdl2-gl.c | 64 +++
ui/sdl2.c | 42 +
Extend virtio-gpu documentation with a link to the Mesa VirGL
documentation.
Suggested-by: Akihiko Odaki
Reviewed-by: Akihiko Odaki
Signed-off-by: Dmitry Osipenko
---
docs/system/devices/virtio-gpu.rst | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/docs/system/devices
vblank_mode=0 for GTK). Refreshment of display's
content shouldn't disable the active display. Fix it by keeping the
scanout's state unchanged when display is redrawn.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
--
vblank_mode=0 for GTK). Refreshment of display's
content shouldn't disable the active display. Fix it by keeping the
scanout's state unchanged when display is redrawn.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
--
Print out error messages when virgl fence creation fails to aid debugging
of the fence-related bugs.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
hw/display/virtio-gpu-virgl.c | 13 -
1 file changed, 12
Change virtio-gpu Venus link, pointing it at the Mesa Venus
documentation instead of the protocol. The Mesa doc provides more
information and also has a link to the protocol.
Suggested-by: Akihiko Odaki
Reviewed-by: Akihiko Odaki
Signed-off-by: Dmitry Osipenko
---
docs/system/devices/virtio
Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
ui/sdl2-gl.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/ui/sdl2-gl.c b/ui/sdl2-gl.c
index e01d9ab0c7bf..b1fe96d6af22 100644
--- a/ui/sdl2-gl.c
+++ b/ui/sdl2-gl.c
@@ -168,6 +168,9 @@ QEMUGLContext sdl2_gl_create_
irgl_init() to fail with a error message
if DRM context can't be initialized instead of giving a warning
message, as was suggested by Akihiko Odaki.
- Added patchew's dependecy tag to the cover letter as was suggested by
Akihiko Odaki.
Alex Bennée (1):
docs/system:
On 3/10/25 14:35, Akihiko Odaki wrote:
> On 2025/03/10 20:12, Dmitry Osipenko wrote:
>> From: Alex Bennée
>>
>> This attempts to tidy up the VirtIO GPU documentation to make the list
>> of requirements clearer. There are still a lot of moving parts and the
>> di
Signed-off-by: Dmitry Osipenko
[dmitry.osipe...@collabora.com: Extended and corrected doc]
---
docs/system/devices/virtio-gpu.rst | 94 +-
1 file changed, 91 insertions(+), 3 deletions(-)
diff --git a/docs/system/devices/virtio-gpu.rst
b/docs/system/devices/virtio
Change virtio-gpu Venus link, pointing it at the Mesa Venus
documentation instead of the protocol. The Mesa doc provides more
information and also has a link to the protocol.
Suggested-by: Akihiko Odaki
Reviewed-by: Akihiko Odaki
Signed-off-by: Dmitry Osipenko
---
docs/system/devices/virtio
sults
in a lightweight context implementations that yield better performance.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
docs/system/devices/virtio-gpu.rst | 11 +++
hw/display/virtio-gpu-gl.c | 2 ++
hw/display/
vblank_mode=0 for GTK). Refreshment of display's
content shouldn't disable the active display. Fix it by keeping the
scanout's state unchanged when display is redrawn.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
--
Print out error messages when virgl fence creation fails to aid debugging
of the fence-related bugs.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
hw/display/virtio-gpu-virgl.c | 13 -
1 file changed, 12
Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
ui/sdl2-gl.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/ui/sdl2-gl.c b/ui/sdl2-gl.c
index e01d9ab0c7bf..b1fe96d6af22 100644
--- a/ui/sdl2-gl.c
+++ b/ui/sdl2-gl.c
@@ -168,6 +168,9 @@ QEMUGLContext sdl2_gl_create_
ael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Pierre-Eric Pelloux-Prayer
Signed-off-by: Dmitry Osipenko
---
include/ui/sdl2.h | 7 ++
meson.build | 6 ++---
ui/sdl2-gl.c | 64 +++
ui/sdl2.c | 42 +
vblank_mode=0 for GTK). Refreshment of display's
content shouldn't disable the active display. Fix it by keeping the
scanout's state unchanged when display is redrawn.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
--
Change virtio-gpu Venus link, pointing it at the Mesa Venus
documentation instead of the protocol. The Mesa doc provides more
information and also has a link to the protocol.
Suggested-by: Akihiko Odaki
Reviewed-by: Akihiko Odaki
Signed-off-by: Dmitry Osipenko
---
docs/system/devices/virtio
sults
in a lightweight context implementations that yield better performance.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
docs/system/devices/virtio-gpu.rst | 11 +++
hw/display/virtio-gpu-gl.c | 2 ++
hw/display/
Extend virtio-gpu documentation with a link to the Mesa VirGL
documentation.
Suggested-by: Akihiko Odaki
Reviewed-by: Akihiko Odaki
Signed-off-by: Dmitry Osipenko
---
docs/system/devices/virtio-gpu.rst | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/docs/system/devices
vblank_mode=0 for GTK). Refreshment of display's
content shouldn't disable the active display. Fix it by keeping the
scanout's state unchanged when display is redrawn.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
--
Print out error messages when virgl fence creation fails to aid debugging
of the fence-related bugs.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
hw/display/virtio-gpu-virgl.c | 13 -
1 file changed, 12
of giving a warning
message, as was suggested by Akihiko Odaki.
- Added patchew's dependecy tag to the cover letter as was suggested by
Akihiko Odaki.
Alex Bennée (1):
docs/system: virtio-gpu: Document host/guest requirements
Dmitry Osipenko (8):
ui/sdl2: Restore
n, like host-vk-gpu.c for example.
--
Best regards,
Dmitry
On 2/18/25 09:27, Dmitry Osipenko wrote:
>> To be honest, I'm concerned that you may be using QEMU as a staging tree
>> for Mesa/virglrenderer. Submitting a documentation to QEMU as a
>> preparation to submit one to Mesa is not OK.
>>
>> You shouldn't su
On 2/13/25 07:32, Akihiko Odaki wrote:
> On 2025/02/10 6:03, Dmitry Osipenko wrote:
>> On 2/6/25 08:41, Akihiko Odaki wrote:
>>> On 2025/02/06 2:40, Dmitry Osipenko wrote:
>>>> On 2/3/25 08:31, Akihiko Odaki wrote:
>>>> ...
>>>>>> Req
; Why immediately? Surely if we are unmapping we can defer it. Or is this
> a case of having stale mappings making the life of new allocations
> harder?
Unmapping currently works synchronously for virtio-gpu in QEMU, hence
deferring it blocks whole virtio-gpu up to 100+ ms. And if multiple
unmappings are done in a row, then it's 100ms multiplied by the number
of unmappings.
--
Best regards,
Dmitry
On 2/14/25 18:26, Dmitry Osipenko wrote:
> On 2/14/25 17:06, Alex Bennée wrote:
>> dongwon@intel.com writes:
>>
>>> From: Dongwon Kim
>>>
>>> This partially reverts commit 77bf310084dad38b3a2badf01766c659056f1cf2
>>> which causes some guest
On 2/13/25 07:47, Akihiko Odaki wrote:
> On 2025/02/10 1:56, Dmitry Osipenko wrote:
>> From: Alex Bennée
>>
>> This attempts to tidy up the VirtIO GPU documentation to make the list
>> of requirements clearer. There are still a lot of moving parts and the
>> di
On 2/14/25 17:33, Alex Bennée wrote:
> Dmitry Osipenko writes:
>
>> This patchset adds DRM native context support to VirtIO-GPU on Qemu.
>>
>> Contarary to Virgl and Venus contexts that mediates high level GFX APIs,
>> DRM native context [1] mediates lower l
g (e.g. Wayland Compositor) possibly due to
>> simulataneous accesses on the guest frame buffer by host compositor
>> and the guest.
>>
>> Reported-by: Dmitry Osipenko
>> Reported-by: Alex Bennée
>> Cc: Marc-André Lureau
>> Cc: Vivek Kasireddy
>&g
On 2/6/25 01:13, Dmitry Osipenko wrote:
> On 2/5/25 23:08, Dmitry Osipenko wrote:
>>> Thanks for showing me the video. I will take a look and check what would go
>>> wrong here. I kinda understand corruption may happen
>>> in some scenario but I don't know wha
On 2/6/25 08:41, Akihiko Odaki wrote:
> On 2025/02/06 2:40, Dmitry Osipenko wrote:
>> On 2/3/25 08:31, Akihiko Odaki wrote:
>> ...
>>>> Requirements don't vary much. For example virglrenderer minigbm support
>>>> is mandatory for crosvm, while for QEMU i
Signed-off-by: Dmitry Osipenko
[dmitry.osipe...@collabora.com: Extended and corrected doc]
---
docs/system/devices/virtio-gpu.rst | 106 -
1 file changed, 104 insertions(+), 2 deletions(-)
diff --git a/docs/system/devices/virtio-gpu.rst
b/docs/system/devices/virtio
Extend virtio-gpu documentation with a link to the Mesa VirGL
documentation.
Suggested-by: Akihiko Odaki
Reviewed-by: Akihiko Odaki
Signed-off-by: Dmitry Osipenko
---
docs/system/devices/virtio-gpu.rst | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/docs/system/devices
Change virtio-gpu Venus link, pointing it at the Mesa Venus
documentation instead of the protocol. The Mesa doc provides more
information and also has a link to the protocol.
Suggested-by: Akihiko Odaki
Reviewed-by: Akihiko Odaki
Signed-off-by: Dmitry Osipenko
---
docs/system/devices/virtio
ontexts in virglrenderer.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Reviewed-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
hw/display/virtio-gpu-gl.c | 3 +
hw/display/virtio-gpu-virgl.c | 147 +++--
include/hw/virtio/vi
vblank_mode=0 for GTK). Refreshment of display's
content shouldn't disable the active display. Fix it by keeping the
scanout's state unchanged when display is redrawn.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
--
sults
in a lightweight context implementations that yield better performance.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
docs/system/devices/virtio-gpu.rst | 11 +++
hw/display/virtio-gpu-gl.c | 2 ++
hw/display/
vblank_mode=0 for GTK). Refreshment of display's
content shouldn't disable the active display. Fix it by keeping the
scanout's state unchanged when display is redrawn.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
--
ael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Pierre-Eric Pelloux-Prayer
Signed-off-by: Dmitry Osipenko
---
include/ui/sdl2.h | 7 ++
meson.build | 6 ++---
ui/sdl2-gl.c | 64 +++
ui/sdl2.c | 42 +
Print out error messages when virgl fence creation fails to aid debugging
of the fence-related bugs.
Reviewed-by: Akihiko Odaki
Acked-by: Michael S. Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
hw/display/virtio-gpu-virgl.c | 13 -
1 file changed, 12
example and made virtio_gpu_virgl_init() to fail with a error message
if DRM context can't be initialized instead of giving a warning
message, as was suggested by Akihiko Odaki.
- Added patchew's dependecy tag to the cover letter as was suggested by
Akihiko Odaki.
Tsirkin
Tested-by: Alex Bennée
Signed-off-by: Dmitry Osipenko
---
ui/sdl2-gl.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/ui/sdl2-gl.c b/ui/sdl2-gl.c
index e01d9ab0c7bf..b1fe96d6af22 100644
--- a/ui/sdl2-gl.c
+++ b/ui/sdl2-gl.c
@@ -168,6 +168,9 @@ QEMUGLContext sdl2_gl_create_
On 2/3/25 09:04, Akihiko Odaki wrote:
> On 2025/02/03 8:21, Dmitry Osipenko wrote:
>> From: Alex Bennée
>>
>> This attempts to tidy up the VirtIO GPU documentation to make the list
>> of requirements clearer. There are still a lot of moving parts and the
>> di
1 - 100 of 1233 matches
Mail list logo
