From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH 2/2] drm/i915/vlv: Replaced Blitter ring based flips with MMIO Flips for VLV. Date: Fri, 7 Feb 2014 15:47:17 +0100 Message-ID: <20140207144717.GY17001@phenom.ffwll.local> References: <1389266799-13317-1-git-send-email-akash.goel@intel.com> <1389266799-13317-3-git-send-email-akash.goel@intel.com> <20140109113135.GB31852@nuc-i3427.alporthouse.com> <8BF5CF93467D8C498F250C96583BC09CC3095F@BGSMSX103.gar.corp.intel.com> <8BF5CF93467D8C498F250C96583BC09CC6CDEA@BGSMSX103.gar.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-ee0-f49.google.com (mail-ee0-f49.google.com [74.125.83.49]) by gabe.freedesktop.org (Postfix) with ESMTP id 1B6B9FBAE2 for ; Fri, 7 Feb 2014 06:47:22 -0800 (PST) Received: by mail-ee0-f49.google.com with SMTP id d17so1602732eek.36 for ; Fri, 07 Feb 2014 06:47:22 -0800 (PST) Content-Disposition: inline In-Reply-To: <8BF5CF93467D8C498F250C96583BC09CC6CDEA@BGSMSX103.gar.corp.intel.com> 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: "Goel, Akash" Cc: "'intel-gfx@lists.freedesktop.org'" List-Id: intel-gfx@lists.freedesktop.org On Fri, Feb 07, 2014 at 11:59:29AM +0000, Goel, Akash wrote: > Please could you kindly elaborate here, it will help us to proceed further with this patch. As Chris said, instead of rolling your own code to track when flips are emitted to the ring, simply add a real request (with the add_request function) like the execbuf paths. Then add any additional trackin you need to our request structure. -Daniel > > Best Regards > Akash > > -----Original Message----- > From: Goel, Akash > Sent: Monday, January 13, 2014 3:17 PM > To: Chris Wilson > Cc: intel-gfx@lists.freedesktop.org > Subject: RE: [Intel-gfx] [PATCH 2/2] drm/i915/vlv: Replaced Blitter ring based flips with MMIO Flips for VLV. > > >> Rather exporting deep magic from i915_gem, just emit the request after the mmio flip and use the normal signalling mechanisms. There are other users who could also use a request after a flip. > > Sorry, couldn't understand your point. > > With Command streamer based flips, we could use the cross ring MBOX mechanism at Hw level, to ensure that buffer is flipped only when the rendering is completed. > > But with MMIO flips, need to ensure that we somehow introduce a wait for the rendering to complete, before updating the Display Surface Address register, to effect the flip. > > Best Regards > Akash > > -----Original Message----- > From: Chris Wilson [mailto:chris@chris-wilson.co.uk] > Sent: Thursday, January 09, 2014 5:02 PM > To: Goel, Akash > Cc: intel-gfx@lists.freedesktop.org > Subject: Re: [Intel-gfx] [PATCH 2/2] drm/i915/vlv: Replaced Blitter ring based flips with MMIO Flips for VLV. > > On Thu, Jan 09, 2014 at 04:56:39PM +0530, akash.goel@intel.com wrote: > > From: Akash Goel > > > > Using MMIO based flips now on VLV for Media power well residency optimization. > > The blitter ring is currently being used just for the command streamer > > based flip calls. For pure 3D workloads, with MMIO flips, there will > > be no use of blitter ring and this will ensure the 100% residency in D0i3 for Media well. > > The other alternative of having Render ring based flip calls is not > > being used, as that option adversly affects the performance (FPS) of > > certain 3D Apps > > Rather exporting deep magic from i915_gem, just emit the request after the mmio flip and use the normal signalling mechanisms. There are other users who could also use a request after a flip. > -Chris > > -- > Chris Wilson, Intel Open Source Technology Centre > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch