From mboxrd@z Thu Jan 1 00:00:00 1970 From: Imre Deak Subject: Re: [PATCH 3/9] drm/i915: Wrap the preallocated BIOS framebuffer and preserve for KMS fbcon v2 Date: Wed, 27 Mar 2013 16:07:59 +0200 Message-ID: <1364393279.11915.9.camel@intelbox> References: <1364340792-7278-1-git-send-email-jbarnes@virtuousgeek.org> <1364340792-7278-4-git-send-email-jbarnes@virtuousgeek.org> <20130326235356.GA14007@cantiga.alporthouse.com> Reply-To: imre.deak@intel.com Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by gabe.freedesktop.org (Postfix) with ESMTP id 6B5FAE5CE0 for ; Wed, 27 Mar 2013 07:08:04 -0700 (PDT) In-Reply-To: <20130326235356.GA14007@cantiga.alporthouse.com> 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 List-Id: intel-gfx@lists.freedesktop.org On Tue, 2013-03-26 at 23:53 +0000, Chris Wilson wrote: > On Tue, Mar 26, 2013 at 04:33:06PM -0700, Jesse Barnes wrote: > > v2: check for non-native modes and adjust (Jesse) > > fixup aperture and cmap frees (Imre) > The aperture is already freed via framebuffer_release(). And wrong > patch? Ah, haven't checked in there.. So aperture shouldn't be freed by us. --Imre