From mboxrd@z Thu Jan 1 00:00:00 1970 From: Adam Jackson Subject: Re: [PATCH] drm/i915: Initialise GTT MTRR to -1 Date: Fri, 23 Mar 2012 13:43:14 -0400 Message-ID: <4F6CB632.8020705@redhat.com> References: <1332524329-6359-1-git-send-email-chris@chris-wilson.co.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by gabe.freedesktop.org (Postfix) with ESMTP id 4F1489E86C for ; Fri, 23 Mar 2012 10:43:23 -0700 (PDT) In-Reply-To: <1332524329-6359-1-git-send-email-chris@chris-wilson.co.uk> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: Chris Wilson Cc: intel-gfx@lists.freedesktop.org, Ben Widawsky List-Id: intel-gfx@lists.freedesktop.org On 3/23/12 1:38 PM, Chris Wilson wrote: > Fixes a regression from 9e984bc1 (drm/i915: Don't do MTRR setup if PAT > is enabled) where we left the MTRR as 0 and so tried to free a MTRR we > did not own during unload. > > Reported-by: Ben Widawsky > Cc: Ben Widawsky > Cc: Adam Jackson > Signed-off-by: Chris Wilson Brown-paper-bag-by: Adam Jackson Also reviewed-by. Thanks! - ajax