[<prev] [next>] [day] [month] [year] [list]
Message-Id: <20221016200324.17930-1-someguy@effective-light.com>
Date: Sun, 16 Oct 2022 16:03:23 -0400
From: Hamza Mahfooz <someguy@...ective-light.com>
To: linux-kernel@...r.kernel.org
Cc: Hamza Mahfooz <someguy@...ective-light.com>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>,
Thomas Zimmermann <tzimmermann@...e.de>,
David Airlie <airlied@...il.com>,
Daniel Vetter <daniel@...ll.ch>,
dri-devel@...ts.freedesktop.org
Subject: [PATCH] drm/framebuffer: break out of loop, if the requested element is found
We can stop searching through the linked-list after we have found the
element we are looking for. So, break out of list_for_each_entry()
after we have found the element we are looking for in drm_mode_rmfb().
Signed-off-by: Hamza Mahfooz <someguy@...ective-light.com>
---
drivers/gpu/drm/drm_framebuffer.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/drm_framebuffer.c b/drivers/gpu/drm/drm_framebuffer.c
index 2dd97473ca10..557177a4c5be 100644
--- a/drivers/gpu/drm/drm_framebuffer.c
+++ b/drivers/gpu/drm/drm_framebuffer.c
@@ -441,8 +441,11 @@ int drm_mode_rmfb(struct drm_device *dev, u32 fb_id,
mutex_lock(&file_priv->fbs_lock);
list_for_each_entry(fbl, &file_priv->fbs, filp_head)
- if (fb == fbl)
+ if (fb == fbl) {
found = 1;
+ break;
+ }
+
if (!found) {
mutex_unlock(&file_priv->fbs_lock);
goto fail_unref;
--
2.37.3
Powered by blists - more mailing lists