drm/etnaviv: free events the usual way in recover worker
authorLucas Stach <l.stach@pengutronix.de>
Wed, 7 Jun 2023 13:02:17 +0000 (15:02 +0200)
committerLucas Stach <l.stach@pengutronix.de>
Mon, 17 Jul 2023 09:32:10 +0000 (11:32 +0200)
Clearing the whole bitmap at once is only a minor optimization in
a path that should be extremely cold. Free the events by calling
event_free() instead of directly manipulating the completion count
and event bitmap.

Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
Reviewed-by: Christian Gmeiner <cgmeiner@igalia.com>
drivers/gpu/drm/etnaviv/etnaviv_gpu.c

index 9279071..e067a27 100644 (file)
@@ -1450,8 +1450,7 @@ void etnaviv_gpu_recover_hang(struct etnaviv_gem_submit *submit)
        /* complete all events, the GPU won't do it after the reset */
        spin_lock(&gpu->event_spinlock);
        for_each_set_bit(i, gpu->event_bitmap, ETNA_NR_EVENTS)
-               complete(&gpu->event_free);
-       bitmap_zero(gpu->event_bitmap, ETNA_NR_EVENTS);
+               event_free(gpu, i);
        spin_unlock(&gpu->event_spinlock);
 
        etnaviv_gpu_hw_init(gpu);