From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH v3] drm/i915: Fix recursive calls to unmap Date: Tue, 17 Jan 2012 12:15:37 +0100 Message-ID: <20120117111537.GK4093@phenom.ffwll.local> References: <1320031005-8729-1-git-send-email-ben@bwidawsk.net> <20111103044730.GA19799@cloud01> <20111103152339.46fe6ce8@bwidawsk.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-we0-f177.google.com (mail-we0-f177.google.com [74.125.82.177]) by gabe.freedesktop.org (Postfix) with ESMTP id A5C869ECE1 for ; Tue, 17 Jan 2012 03:15:42 -0800 (PST) Received: by werm13 with SMTP id m13so1972216wer.36 for ; Tue, 17 Jan 2012 03:15:41 -0800 (PST) Content-Disposition: inline In-Reply-To: <20111103152339.46fe6ce8@bwidawsk.net> 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: Ben Widawsky Cc: Daniel Vetter , intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Thu, Nov 03, 2011 at 03:23:39PM -0700, Ben Widawsky wrote: > On Thu, 3 Nov 2011 20:19:23 +0000 > Dave Airlie wrote: > > > >> > > > >> > The solution here is to add a new flag to the call chain which gives the > > >> > routines the information they need to possibly defer actions which may > > >> > cause us to recurse. A macro has been defined to replace i915_gpu_idle > > >> > which defaults to the old behavior. > > >> > > > >> > Kudos to Chris for tracking this one down. > > >> > > >> So this fixes the non-VTd case, the VT-d case still hits a recursion > > >> here, for posterity its below. > > > > Okay I take that back, I got my EL6 kernel rock stable with the > > correct blend of backported bits. > > > > So ignore that backtrace, however I did get another IOMMU hang on my > > upstream kernel with gem_linear_blits, > > > > so this should be fine to merge but I'm guessing we have more > > debugging to do on the VT-d cases. > > > > Dave. > > Does it pass your original failing case? Hi Ben, Is v3 of this patch the right one to merge or do we still have some outstanding issues on this? Also, have you looked at the recent ilk dmar fallout, I think strict dmar iotlb flushing doesn't sit too well with our gpus. My snb dies almost immediately with that (even when I do not enable rc6 nor semaphores). Cheers, Daniel -- Daniel Vetter Mail: daniel@ffwll.ch Mobile: +41 (0)79 365 57 48