From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH v2 3/3] drm/i915: Track OACONTROL register enable/disable during parsing Date: Thu, 10 Apr 2014 08:43:40 +0200 Message-ID: <20140410064340.GG9262@phenom.ffwll.local> References: <1395945820-20376-4-git-send-email-bradley.d.volkin@intel.com> <1396027310-16951-1-git-send-email-bradley.d.volkin@intel.com> <20140410060721.GK15788@zhen-hp.sh.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-ee0-f41.google.com (mail-ee0-f41.google.com [74.125.83.41]) by gabe.freedesktop.org (Postfix) with ESMTP id CF3036E195 for ; Wed, 9 Apr 2014 23:43:42 -0700 (PDT) Received: by mail-ee0-f41.google.com with SMTP id t10so2633499eei.28 for ; Wed, 09 Apr 2014 23:43:41 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20140410060721.GK15788@zhen-hp.sh.intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Zhenyu Wang Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Thu, Apr 10, 2014 at 02:07:21PM +0800, Zhenyu Wang wrote: > On 2014.03.28 10:21:50 -0700, bradley.d.volkin@intel.com wrote: > > From: Brad Volkin > > > > There is some thought that the data from the performance counters enabled > > via OACONTROL should only be available to the process that enabled counting. > > To limit snooping, require that any batch buffer which sets OACONTROL to a > > non-zero value also sets it back to 0 before the end of the batch. > > > > I think this might be too strict although there's good point in this. > But it makes almost impossible to insert mi report count command to probe > GPU statistics from kind of third-party tool or lib. At least in the case of mesa you _must_ use mesa (or whatever your gl library is) to insert the MI_PERF cmd at just the rigth spots in the command stream and ofc flush all outstanding vertices and similar things. If you want a global perf sampling support then I think we need to wire up the time-based sampling the hw provides to the perf subsystem. And figure out how to coordinate between mesa wanting to use OA and perf (probably just reject mesa batches or something like that). > Could we have a i915.enable_cmd_parser config that can disable this? As is I don't see a compelling reason. Also if you require the users of your perf tuning tooling to use a module option, you're doing it wrong. This stuff should Just Work. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch