From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Widawsky Subject: Re: [PATCH 00/25] i915 HW context support Date: Thu, 7 Jun 2012 09:17:49 -0700 Message-ID: <20120607091749.792965a5@bwidawsk.net> References: <1338846185-10571-1-git-send-email-ben@bwidawsk.net> <4FCE9830.90201@whitecape.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <4FCE9830.90201@whitecape.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: mesa-dev-bounces+gcvmd-mesa3d-493=gmane.org@lists.freedesktop.org Errors-To: mesa-dev-bounces+gcvmd-mesa3d-493=gmane.org@lists.freedesktop.org To: Kenneth Graunke Cc: Mesa Dev , intel-gfx@lists.freedesktop.org, DRI Devel List-Id: intel-gfx@lists.freedesktop.org On Tue, 05 Jun 2012 16:37:20 -0700 Kenneth Graunke wrote: > On 06/04/2012 02:42 PM, Ben Widawsky wrote: > > Setting myself up for a late night crying session once again. Most of the > > people reading this probably know the history and reasons for the patches. If > > not, you can search the intel-gfx mailing list to try to learn more. I won't > > recap the whole thing here, and instead let the patches speak for themselves. > > > > Instead a brief review of what's here, and what's happened. Mostly, > > these badly need testing and review. I've carried these around for so > > long now, and seen so many different failures, I'm quite paranoid they > > still aren't perfect. Also, I've spent almost all of the time working on > > this in the kernel, so there is bound to be simple errors in the other > > stuff. > > > > I've run these on various workloads and saw nothing worth mentioning. > > > > > > 0-16: kernel patches > > > 17-21: libdrm patches (wait render patch is temporary) > > Patches 17-21 look fine. > Reviewed-by: Kenneth Graunke > > > 22-24: intel-gpu-tools > > 25: mesa > > Patch 25 looks fine too. Feel free to apply some combination of: > Reviewed-by: Kenneth Graunke > Signed-off-by: Kenneth Graunke > > I do like Paul's comment updates, so I suggest merging those. Ken, would you do these for me on patch 25? It's really your patch anyhow, I just made ever so slight modifications :-) > > > kernel > > git://people.freedesktop.org/~bwidawsk/drm-intel context_support_rev2 > > > > libdrm > > git://people.freedesktop.org/~bwidawsk/drm context_support_rev2 > > > > i-g-t > > git://people.freedesktop.org/~bwidawsk/intel-gpu-tools context_support -- Ben Widawsky, Intel Open Source Technology Center