From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Wilson Subject: Re: [PATCH 6/6] drm/i915: Really wait for pending flips in intel_pipe_set_base() Date: Wed, 13 Feb 2013 17:39:50 +0000 Message-ID: <20130213173950.GA16928@cantiga.alporthouse.com> References: <1359476018-31274-1-git-send-email-ville.syrjala@linux.intel.com> <1359476018-31274-7-git-send-email-ville.syrjala@linux.intel.com> <20130213154935.GL5813@phenom.ffwll.local> <20130213170623.GQ9135@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by gabe.freedesktop.org (Postfix) with ESMTP id C77CBE6671 for ; Wed, 13 Feb 2013 09:39:57 -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: Daniel Vetter Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Wed, Feb 13, 2013 at 06:11:00PM +0100, Daniel Vetter wrote: > I'm confused here - where can we extract parallelism in set_base > between waiting for pending flips and the pinning? And imo set_base > isn't really critical: It's officially a synchronous thing (we have a > vblank wait in there), and if we want to fix that imo the nuclear > pageflip should be the answer. Speaking of which I've had a patch to remove that extra synchronous wait for over a year... -Chris -- Chris Wilson, Intel Open Source Technology Centre