From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jesse Barnes Subject: Re: [PATCH] drm/i915: Disable full ppgtt by default Date: Thu, 6 Mar 2014 10:01:12 -0800 Message-ID: <20140306100112.37e6794c@jbarnes-desktop> References: <1394104461-13382-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 oproxy7-pub.mail.unifiedlayer.com (oproxy7-pub.mail.unifiedlayer.com [67.222.55.9]) by gabe.freedesktop.org (Postfix) with SMTP id CA8EFFAE88 for ; Thu, 6 Mar 2014 10:00:45 -0800 (PST) In-Reply-To: <1394104461-13382-1-git-send-email-daniel.vetter@ffwll.ch> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces@lists.freedesktop.org Errors-To: intel-gfx-bounces@lists.freedesktop.org To: Daniel Vetter Cc: Intel Graphics Development , Ben Widawsky List-Id: intel-gfx@lists.freedesktop.org On Thu, 6 Mar 2014 12:14:21 +0100 Daniel Vetter wrote: > There are too many oustanding issues: > > - Fence handling in the current code is broken. There's a patch series > from me, but it's blocked on and extended review (which includes > writing the testcases). > > - IOMMU mapping handling is broken, we need to properly refcount it - > currently it gets destroyed when the first vma is unbound, so way > too early. > > - There's a pending reset issue on snb. Since Mika's reset work and > full ppgtt have been pulled in in separate branches and ended up > intermittingly breaking each another it's unclear who's the exact > culprit here. > > - We still have persistent evidince of crazy recursion bugs through > vma_unbind and ppgtt_relase, e.g. > > https://bugs.freedesktop.org/show_bug.cgi?id=73383 > > This issue (and a few others meanwhile resolved) have blocked our > performance measuring/tuning group since 3 months. > > - Secure batch dispatching is broken. This is blocking Brad Volkin's > command checker work since 3 months. Do we have bugs and/or tasks filed for all these, or just the one? We need to make sure people are signed up to fix/review them, or it'll end up staying disabled forever, and then we'll be stuck without a command checker and some advanced features coming up... -- Jesse Barnes, Intel Open Source Technology Center