Re: [PATCH v4 09/21] drm/etnaviv: Prepare to dynamic dma-buf locking specification

[cco]

To whoom it may belong: Deepest sorrow to inform you that my husband Helmut passed by on July 21. This access will be cleared in a view days. Irene
-------- Ursprüngliche Nachricht --------Von: Christian König <christian.koenig at amd.com> Datum: 01.09.22  08:50  (GMT+01:00) An: Dmitry Osipenko <dmitry.osipenko at collabora.com>, David Airlie <airlied at linux.ie>, Gerd Hoffmann <kraxel at redhat.com>, Gurchetan Singh <gurchetansingh at chromium.org>, Chia-I Wu <olvaffe at gmail.com>, Daniel Vetter <daniel at ffwll.ch>, Daniel Almeida <daniel.almeida at collabora.com>, Gert Wollny <gert.wollny at collabora.com>, Gustavo Padovan <gustavo.padovan at collabora.com>, Daniel Stone <daniel at fooishbar.org>, Tomeu Vizoso <tomeu.vizoso at collabora.com>, Maarten Lankhorst <maarten.lankhorst at linux.intel.com>, Maxime Ripard <mripard at kernel.org>, Thomas Zimmermann <tzimmermann at suse.de>, Rob Clark <robdclark at gmail.com>, Sumit Semwal <sumit.semwal at linaro.org>, "Pan, Xinhui" <Xinhui.Pan at amd.com>, Thierry Reding <thierry.reding at gmail.com>, Tomasz Figa <tfiga at chromium.org>, Marek Szyprowski <m.szyprowski at samsung.com>, Mauro Carvalho Chehab <mchehab at kernel.org>, Alex Deucher <alexander.deucher at amd.com>, Jani Nikula <jani.nikula at linux.intel.com>, Joonas Lahtinen <joonas.lahtinen at linux.intel.com>, Rodrigo Vivi <rodrigo.vivi at intel.com>, Tvrtko Ursulin <tvrtko.ursulin at linux.intel.com>, Thomas Hellström <thomas_os at shipmail.org>, Qiang Yu <yuq825 at gmail.com>, Srinivas Kandagatla <srinivas.kandagatla at linaro.org>, Amol Maheshwari <amahesh at qti.qualcomm.com>, Jason Gunthorpe <jgg at ziepe.ca>, Leon Romanovsky <leon at kernel.org>, Juergen Gross <jgross at suse.com>, Stefano Stabellini <sstabellini at kernel.org>, Oleksandr Tyshchenko <oleksandr_tyshchenko at epam.com>, Tomi Valkeinen <tomba at kernel.org>, Russell King <linux at armlinux.org.uk>, Lucas Stach <l.stach at pengutronix.de>, Christian Gmeiner <christian.gmeiner at gmail.com> Cc: dri-devel at lists.freedesktop.org, linux-kernel at vger.kernel.org, Dmitry Osipenko <digetx at gmail.com>, linux-media at vger.kernel.org, linaro-mm-sig at lists.linaro.org, amd-gfx at lists.freedesktop.org, intel-gfx at lists.freedesktop.org, kernel at collabora.com, virtualization at lists.linux-foundation.org, linux-rdma at vger.kernel.org, linux-arm-msm at vger.kernel.org Betreff: Re: [PATCH v4 09/21] drm/etnaviv: Prepare to dynamic dma-buf locking
  specification Am 31.08.22 um 17:37 schrieb Dmitry Osipenko:> Prepare Etnaviv driver to the common dynamic dma-buf locking convention> by starting to use the unlocked versions of dma-buf API functions.>> Signed-off-by: Dmitry Osipenko <dmitry.osipenko at collabora.com>Interesting, where is the matching vmap()?Anyway, this patch is Acked-by: Christian König <christian.koenig at amd.com>> --->   drivers/gpu/drm/etnaviv/etnaviv_gem_prime.c | 2 +->   1 file changed, 1 insertion(+), 1 deletion(-)>> diff --git a/drivers/gpu/drm/etnaviv/etnaviv_gem_prime.c b/drivers/gpu/drm/etnaviv/etnaviv_gem_prime.c> index 3fa2da149639..7031db145a77 100644> --- a/drivers/gpu/drm/etnaviv/etnaviv_gem_prime.c> +++ b/drivers/gpu/drm/etnaviv/etnaviv_gem_prime.c> @@ -65,7 +65,7 @@ static void etnaviv_gem_prime_release(struct etnaviv_gem_object *etnaviv_obj)>   	struct iosys_map map = IOSYS_MAP_INIT_VADDR(etnaviv_obj->vaddr);>   >   	if (etnaviv_obj->vaddr)> -		dma_buf_vunmap(etnaviv_obj->base.import_attach->dmabuf, &map);> +		dma_buf_vunmap_unlocked(etnaviv_obj->base.import_attach->dmabuf, &map);>   >   	/* Don't drop the pages for imported dmabuf, as they are not>   	 * ours, just free the array we allocated:
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20220925/2a29b03f/attachment.htm>