From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751654AbdIKMDD (ORCPT ); Mon, 11 Sep 2017 08:03:03 -0400 Received: from mga03.intel.com ([134.134.136.65]:64850 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751574AbdIKMDC (ORCPT ); Mon, 11 Sep 2017 08:03:02 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.42,377,1500966000"; d="scan'208";a="899089311" From: Jani Nikula To: Christophe JAILLET , joonas.lahtinen@linux.intel.com, rodrigo.vivi@intel.com, airlied@linux.ie Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org, Christophe JAILLET Subject: Re: [PATCH] drm/i915: Fix an error handling in 'intel_framebuffer_init()' In-Reply-To: <20170910085642.13673-1-christophe.jaillet@wanadoo.fr> References: <20170910085642.13673-1-christophe.jaillet@wanadoo.fr> Date: Mon, 11 Sep 2017 15:02:35 +0300 Message-ID: <87zia130kk.fsf@nikula.org> MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, 10 Sep 2017, Christophe JAILLET wrote: > We should go through the error handling path to decrease the > 'framebuffer_references' as done everywhere else in this function. > > Fixes: 2e2adb05736c ("drm/i915: Add render decompression support") > Signed-off-by: Christophe JAILLET Pushed to drm-intel-next-queued, thanks for the patch. For future reference, the intel-gfx list will suffice. BR, Jani. > --- > drivers/gpu/drm/i915/intel_display.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c > index 7cd392f2cd94..478fa449003f 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -14077,7 +14077,7 @@ static int intel_framebuffer_init(struct intel_framebuffer *intel_fb, > > if (mode_cmd->handles[i] != mode_cmd->handles[0]) { > DRM_DEBUG_KMS("bad plane %d handle\n", i); > - return -EINVAL; > + goto err; > } > > stride_alignment = intel_fb_stride_alignment(fb, i); -- Jani Nikula, Intel Open Source Technology Center