diff options
| author | Ben Widawsky <benjamin.widawsky@intel.com> | 2014-11-12 18:49:16 -0800 |
|---|---|---|
| committer | Ben Widawsky <benjamin.widawsky@intel.com> | 2014-11-13 10:48:24 -0800 |
| commit | f14a35f9dc932a7aff80134cacfef60e73320e89 (patch) | |
| tree | 89309ac24f827faca0e97e2f26c024c96285676c | |
| parent | 0d924738d9d86a19b9bffa01ba9f29f2f697bb27 (diff) | |
i965: Always enable VF statistics
Every other unit in the geometry pipeline automatically enables
statistics gathering. This part of the pipe has been controlled by the
DEBUG_STATS variable, but this is asymmetric. This dates back to the
original implementation, and I am not sure if there is a reason for it.
I need access to these stats to implement ARB_pipeline_statistics_query.
Eric wrote it, and Ken touched it last. Do you have any opposition?
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=86145
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_misc_state.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_misc_state.c b/src/mesa/drivers/dri/i965/brw_misc_state.c index 99fcddc6a21..2c408148ad8 100644 --- a/src/mesa/drivers/dri/i965/brw_misc_state.c +++ b/src/mesa/drivers/dri/i965/brw_misc_state.c @@ -929,8 +929,7 @@ brw_upload_invariant_state(struct brw_context *brw) const uint32_t _3DSTATE_VF_STATISTICS = is_965 ? GEN4_3DSTATE_VF_STATISTICS : GM45_3DSTATE_VF_STATISTICS; BEGIN_BATCH(1); - OUT_BATCH(_3DSTATE_VF_STATISTICS << 16 | - (unlikely(INTEL_DEBUG & DEBUG_STATS) ? 1 : 0)); + OUT_BATCH(_3DSTATE_VF_STATISTICS << 16 | 1); ADVANCE_BATCH(); } |