From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Wilson Subject: Re: [RFC] Async flips Date: Wed, 31 Oct 2012 15:49:36 +0000 Message-ID: <84c8a8$6bko2f@orsmga001.jf.intel.com> References: <1351622029-2276-1-git-send-email-jbarnes@virtuousgeek.org> <20121031125324.GD3791@intel.com> <20121031082349.37933396@jbarnes-desktop> <20121031083909.7c2eecec@jbarnes-desktop> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by gabe.freedesktop.org (Postfix) with ESMTP id 4A31CA0C18 for ; Wed, 31 Oct 2012 08:50:19 -0700 (PDT) In-Reply-To: <20121031083909.7c2eecec@jbarnes-desktop> 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: Jesse Barnes , Daniel Vetter Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Wed, 31 Oct 2012 08:39:09 -0700, Jesse Barnes wrote: > On Wed, 31 Oct 2012 16:26:54 +0100 > Daniel Vetter wrote: > > > On Wed, Oct 31, 2012 at 4:23 PM, Jesse Barnes wrote: > > > > > >> On Tue, Oct 30, 2012 at 01:33:47PM -0500, Jesse Barnes wrote: > > >> > The hw supports async flips through the render ring, so why not expose it? > > >> > It gives us one more "tear me harder" option we can use in the DDX and > > >> > for other cases where simply flipping to the latest buffer is more > > >> > important than visual quality. > > >> > > >> The only reason I can see why anyone would really want async flips is > > >> when you're restricted to double buffering. With triple buffering you > > >> should be able to override the previous flip w/o tearing. > > >> > > >> Well, actually if you use the ring based flips, then you can't do the > > >> override. My atomic page flip code can do it because it's using mmio > > >> flips. There were also other reasons favoring mmio over ring. > > >> > > >> Once the atomic code is deemed ready, I would suggest we just nuke the > > >> ring based flip code (pun intended). > > > > > > Yeah, I agree. In fact one of the first versions of the flip code used > > > mmio, and I think it's a better way to go. > > > > How are we gonna sync up with outstanding rendering before issuing the > > flip? If the answer is involves enabling the render irq, I'm not gonna > > like it ;-) > > Why are you afraid of irqs when rendering is active? We'll already be > awake at those times anyway... Because it involves the driver stalling. -Chris -- Chris Wilson, Intel Open Source Technology Centre