On 4/4/25 16:23, Thomas Zimmermann wrote:
> +int drm_client_buffer_vmap(struct drm_client_buffer *buffer,
> + struct iosys_map *map_copy)
> {
> struct drm_gem_object *gem = buffer->gem;
> struct iosys_map *map = &buffer->map;
> int ret;
>
> drm_gem_lock(gem);
> -
> - ret = drm_gem_pin_locked(gem);
> - if (ret)
> - goto err_drm_gem_pin_locked;
> ret = drm_gem_vmap_locked(gem, map);
> - if (ret)
> - goto err_drm_gem_vmap;
> -
> drm_gem_unlock(gem);
The lock+unlock can be replaced with a single drm_gem_vmap_unlocked().
--
Best regards,
Dmitry
