[PATCH xf86-video-amdgpu 1/3] Always delete entry from list in drm_queue_handler

Michel Dänzer michel at daenzer.net
Wed Aug 29 16:56:46 UTC 2018


From: Michel Dänzer <michel.daenzer at amd.com>

We left entries without a handler hook in the list, so the list could
keep taking longer to process and use up more memory.

Signed-off-by: Michel Dänzer <michel.daenzer at amd.com>
---
 src/amdgpu_drm_queue.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/amdgpu_drm_queue.c b/src/amdgpu_drm_queue.c
index b13d28014..f9db81c08 100644
--- a/src/amdgpu_drm_queue.c
+++ b/src/amdgpu_drm_queue.c
@@ -82,7 +82,7 @@ amdgpu_drm_queue_handler(struct xorg_list *signalled, unsigned int frame,
 	xorg_list_for_each_entry_safe(e, tmp, &amdgpu_drm_queue, list) {
 		if (e->seq == seq) {
 			if (!e->handler) {
-				e->abort(e->crtc, e->data);
+				amdgpu_drm_queue_handle_one(e);
 				break;
 			}
 
-- 
2.18.0



More information about the amd-gfx mailing list