[v2] Support mapping virtio-gpu virgl hostmem blobs using MAP_FIXED API

[RFC PATCH v2 0/2] Support mapping virtio-gpu virgl hostmem blobs using MAP_FIXED API

Posted by Dmitry Osipenko 3 weeks, 6 days ago

Virglrender got a new unstable API that allows mapping host blobs at a given
memory address using MAP_FIXED mmap flag [1]. Usage of this new API brings
major performance and stability improvement for venus and drm native contexts,
see commit message of the RFC patch for details.

Sending early to collect review feeback and have patch prepared by the
time new version of libvirglrenderer will be released with the stabilized
API.

[1] https://gitlab.freedesktop.org/virgl/virglrenderer/-/merge_requests/1374

Based-on: 20251116125641.2255794-1-dmitry.osipenko@collabora.com

Changelog:

v2: - Addressed v1 review comments from Akihiko Odaki

    - Added patch that removes unnecessary memory_region_set_enabled(),
      suggested by Akihiko Odaki

Dmitry Osipenko (2):
  virtio-gpu: Remove superfluous memory_region_set_enabled()
  virtio-gpu: Support mapping hostmem blobs with map_fixed

 hw/display/virtio-gpu-gl.c     | 45 +++++++++++++++++++-
 hw/display/virtio-gpu-virgl.c  | 76 ++++++++++++++++++++++++++++++++--
 include/hw/virtio/virtio-gpu.h |  3 ++
 3 files changed, 119 insertions(+), 5 deletions(-)

-- 
2.51.1

Re: [RFC PATCH v2 0/2] Support mapping virtio-gpu virgl hostmem blobs using MAP_FIXED API

Posted by Yiwei Zhang 3 weeks, 5 days ago

On Sun, Nov 16, 2025 at 6:15 AM Dmitry Osipenko
<dmitry.osipenko@collabora.com> wrote:
>
> Virglrender got a new unstable API that allows mapping host blobs at a given
> memory address using MAP_FIXED mmap flag [1]. Usage of this new API brings
> major performance and stability improvement for venus and drm native contexts,
> see commit message of the RFC patch for details.
>
> Sending early to collect review feeback and have patch prepared by the
> time new version of libvirglrenderer will be released with the stabilized
> API.
>
> [1] https://gitlab.freedesktop.org/virgl/virglrenderer/-/merge_requests/1374
>
> Based-on: 20251116125641.2255794-1-dmitry.osipenko@collabora.com
>
> Changelog:
>
> v2: - Addressed v1 review comments from Akihiko Odaki
>
>     - Added patch that removes unnecessary memory_region_set_enabled(),
>       suggested by Akihiko Odaki
>
> Dmitry Osipenko (2):
>   virtio-gpu: Remove superfluous memory_region_set_enabled()
>   virtio-gpu: Support mapping hostmem blobs with map_fixed
>
>  hw/display/virtio-gpu-gl.c     | 45 +++++++++++++++++++-
>  hw/display/virtio-gpu-virgl.c  | 76 ++++++++++++++++++++++++++++++++--
>  include/hw/virtio/virtio-gpu.h |  3 ++
>  3 files changed, 119 insertions(+), 5 deletions(-)
>
> --
> 2.51.1
>

Nice work! I'd say Venus loves it soooo much ; )

Tested-by: Yiwei Zhang <zzyiwei@gmail.com>

Re: [RFC PATCH v2 0/2] Support mapping virtio-gpu virgl hostmem blobs using MAP_FIXED API

Posted by Dmitry Osipenko 3 weeks, 5 days ago

On 11/17/25 08:56, Yiwei Zhang wrote:
> On Sun, Nov 16, 2025 at 6:15 AM Dmitry Osipenko
> <dmitry.osipenko@collabora.com> wrote:
>>
>> Virglrender got a new unstable API that allows mapping host blobs at a given
>> memory address using MAP_FIXED mmap flag [1]. Usage of this new API brings
>> major performance and stability improvement for venus and drm native contexts,
>> see commit message of the RFC patch for details.
>>
>> Sending early to collect review feeback and have patch prepared by the
>> time new version of libvirglrenderer will be released with the stabilized
>> API.
>>
>> [1] https://gitlab.freedesktop.org/virgl/virglrenderer/-/merge_requests/1374
>>
>> Based-on: 20251116125641.2255794-1-dmitry.osipenko@collabora.com
>>
>> Changelog:
>>
>> v2: - Addressed v1 review comments from Akihiko Odaki
>>
>>     - Added patch that removes unnecessary memory_region_set_enabled(),
>>       suggested by Akihiko Odaki
>>
>> Dmitry Osipenko (2):
>>   virtio-gpu: Remove superfluous memory_region_set_enabled()
>>   virtio-gpu: Support mapping hostmem blobs with map_fixed
>>
>>  hw/display/virtio-gpu-gl.c     | 45 +++++++++++++++++++-
>>  hw/display/virtio-gpu-virgl.c  | 76 ++++++++++++++++++++++++++++++++--
>>  include/hw/virtio/virtio-gpu.h |  3 ++
>>  3 files changed, 119 insertions(+), 5 deletions(-)
>>
>> --
>> 2.51.1
>>
> 
> Nice work! I'd say Venus loves it soooo much ; )
> 
> Tested-by: Yiwei Zhang <zzyiwei@gmail.com>

Thanks!

-- 
Best regards,
Dmitry