From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932445AbbIUOAS (ORCPT ); Mon, 21 Sep 2015 10:00:18 -0400 Received: from mga14.intel.com ([192.55.52.115]:37228 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932166AbbIUOAN (ORCPT ); Mon, 21 Sep 2015 10:00:13 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.17,568,1437462000"; d="scan'208";a="809814229" From: Jani Nikula To: Andrzej Hajda , linux-kernel@vger.kernel.org Cc: Andrzej Hajda , Bartlomiej Zolnierkiewicz , Marek Szyprowski , Daniel Vetter , David Airlie , intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org Subject: Re: [PATCH 03/38] drm/i915: fix handling gen8_emit_flush_coherentl3_wa result In-Reply-To: <1442842450-29769-4-git-send-email-a.hajda@samsung.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <1442842450-29769-1-git-send-email-a.hajda@samsung.com> <1442842450-29769-4-git-send-email-a.hajda@samsung.com> User-Agent: Notmuch/0.20.2+57~gff3a03d (http://notmuchmail.org) Emacs/24.4.1 (x86_64-pc-linux-gnu) Date: Mon, 21 Sep 2015 16:59:58 +0300 Message-ID: <87vbb398kh.fsf@intel.com> 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 Mon, 21 Sep 2015, Andrzej Hajda wrote: > The function can return negative value. > > The problem has been detected using proposed semantic patch > scripts/coccinelle/tests/unsigned_lesser_than_zero.cocci [1]. > > [1]: http://permalink.gmane.org/gmane.linux.kernel/2038576 > > Signed-off-by: Andrzej Hajda Reviewed-by: Jani Nikula > --- > drivers/gpu/drm/i915/intel_lrc.c | 7 ++++--- > 1 file changed, 4 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_lrc.c b/drivers/gpu/drm/i915/intel_lrc.c > index fe06accb0..ff9a481 100644 > --- a/drivers/gpu/drm/i915/intel_lrc.c > +++ b/drivers/gpu/drm/i915/intel_lrc.c > @@ -1254,9 +1254,10 @@ static int gen8_init_indirectctx_bb(struct intel_engine_cs *ring, > > /* WaFlushCoherentL3CacheLinesAtContextSwitch:bdw */ > if (IS_BROADWELL(ring->dev)) { > - index = gen8_emit_flush_coherentl3_wa(ring, batch, index); > - if (index < 0) > - return index; > + int rc = gen8_emit_flush_coherentl3_wa(ring, batch, index); > + if (rc < 0) > + return rc; > + index = rc; > } > > /* WaClearSlmSpaceAtContextSwitch:bdw,chv */ > -- > 1.9.1 > -- Jani Nikula, Intel Open Source Technology Center