[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <3c436b32da8d76df7eace4dbbde79e808058ad8b@intel.com>
Date: Tue, 05 Aug 2025 10:52:09 +0300
From: Jani Nikula <jani.nikula@...ux.intel.com>
To: Liao Yuanhong <liaoyuanhong@...o.com>, Joonas Lahtinen
<joonas.lahtinen@...ux.intel.com>, Rodrigo Vivi <rodrigo.vivi@...el.com>,
Tvrtko Ursulin <tursulin@...ulin.net>, David Airlie <airlied@...il.com>,
Simona Vetter <simona@...ll.ch>, Liao Yuanhong <liaoyuanhong@...o.com>,
Nitin Gote <nitin.r.gote@...el.com>, Krzysztof Niemiec
<krzysztof.niemiec@...el.com>, "open list:DRM DRIVERS"
<dri-devel@...ts.freedesktop.org>, open list
<linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/4] drm/i915/gem: Remove goto label that are only used
once
On Tue, 05 Aug 2025, Liao Yuanhong <liaoyuanhong@...o.com> wrote:
> Remove unnecessary goto labels to improve code readability.
Please don't. It's fine as it is.
> Signed-off-by: Liao Yuanhong <liaoyuanhong@...o.com>
> ---
> drivers/gpu/drm/i915/gem/i915_gem_region.c | 10 ++++------
> 1 file changed, 4 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/gem/i915_gem_region.c b/drivers/gpu/drm/i915/gem/i915_gem_region.c
> index 636768d0f57e..539d1716d454 100644
> --- a/drivers/gpu/drm/i915/gem/i915_gem_region.c
> +++ b/drivers/gpu/drm/i915/gem/i915_gem_region.c
> @@ -92,15 +92,13 @@ __i915_gem_object_create_region(struct intel_memory_region *mem,
> flags |= I915_BO_ALLOC_PM_EARLY;
>
> err = mem->ops->init_object(mem, obj, offset, size, page_size, flags);
> - if (err)
> - goto err_object_free;
> + if (err) {
> + i915_gem_object_free(obj);
> + return ERR_PTR(err);
> + }
>
> trace_i915_gem_object_create(obj);
> return obj;
> -
> -err_object_free:
> - i915_gem_object_free(obj);
> - return ERR_PTR(err);
> }
>
> struct drm_i915_gem_object *
--
Jani Nikula, Intel
Powered by blists - more mailing lists