From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 0/7] updated ppgtt patches Date: Thu, 9 Feb 2012 22:06:32 +0100 Message-ID: <20120209210632.GA26354@phenom.ffwll.local> References: <1328804150-19522-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 mail-ww0-f43.google.com (mail-ww0-f43.google.com [74.125.82.43]) by gabe.freedesktop.org (Postfix) with ESMTP id 816269E7BF for ; Thu, 9 Feb 2012 13:06:52 -0800 (PST) Received: by wgbdr13 with SMTP id dr13so1742387wgb.12 for ; Thu, 09 Feb 2012 13:06:51 -0800 (PST) Content-Disposition: inline In-Reply-To: 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: Eugeni Dodonov Cc: Daniel Vetter , Intel Graphics Development List-Id: intel-gfx@lists.freedesktop.org On Thu, Feb 09, 2012 at 04:21:31PM -0200, Eugeni Dodonov wrote: > On Thu, Feb 9, 2012 at 14:15, Daniel Vetter wrote: > > > Hi all, > > > > I've rebased and updated my ppgtt patches. Only changes are somewhat > > improved > > commit messages (especially for the final patch that actually enables > > ppgtt) and > > an added module option in the last patch to please Chris. > > > > Assuming nothing pops up I plan to merge this in 1-2 days. > > > > Cheers, Daniel > > > > Daniel Vetter (7): > > agp/intel-gtt: export the scratch page dma address > > agp/intel-gtt: export the gtt pagetable iomapping > > drm/i915: initialization/teardown for the aliasing ppgtt > > drm/i915: ppgtt binding/unbinding support > > drm/i915: ppgtt register definitions > > drm/i915: ppgtt debugfs info > > drm/i915: enable ppgtt > > > > I was under impression that I've sent a r-b for the series some time ago, > but perhaps it got missing in action. I have a small suggestion for the > module parameter patch (Patch7), but other than that, for the series: > Reviewed-by: Eugeni Dodonov > > > > Also, I was using this pretty extensively for the past few months, on > different SNB and IVB machines, so you can add the: > Tested-by: Eugeni Dodonov Slurped this in with v4 for the last patch, thanks for the review and testing. -Daniel -- Daniel Vetter Mail: daniel@ffwll.ch Mobile: +41 (0)79 365 57 48