From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Wilson Subject: Re: [PATCH 00/11] aliasing ppgtt support v2 Date: Tue, 29 Nov 2011 09:42:25 +0000 Message-ID: References: <1322512539-3923-1-git-send-email-daniel.vetter@ffwll.ch> 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 59F4E9E74E for ; Tue, 29 Nov 2011 01:42:45 -0800 (PST) In-Reply-To: <1322512539-3923-1-git-send-email-daniel.vetter@ffwll.ch> 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: intel-gfx@lists.freedesktop.org Cc: Daniel Vetter List-Id: intel-gfx@lists.freedesktop.org On Mon, 28 Nov 2011 21:35:27 +0100, Daniel Vetter wrote: > > Hi all, > > Changes since the last submission: > - fixed issues pointed out by Chris Wilson on irc. > - fixed an oops on pre-snb, shame on me for that one. > - added two new patches to only bind objects to the global gtt when required. > - added a new patch so that userspace can find out whether ppgtt is on. This is > required to use MI_STORE/LOAD commands correctly from userspace batchbuffers. > Luckily no currently released userspace code depends on this, it's just prep > work. > > Comments, test reports, reviews and flames highly welcome. The lazy-gtt is just missing a guard to ensure the buffer is bound in the global gtt before reading through those PTE (impacts other code to avoid allocating mappable GTT space). The beauty is that it forced me to grok the rest of the lazy-gtt, it's deceptively simple. Aside from the lazy-gtt sufficiently speeding up command submission and reopening an old wound that prevents me from usefully analysing performance, the series is tested-by myself. I've so far reviewed everything but the actual mechanics of the PDE, which is not saying much ;-) -Chris -- Chris Wilson, Intel Open Source Technology Centre