drm/etnaviv: use submit exec_state for perfmon sampling
authorLucas Stach <l.stach@pengutronix.de>
Fri, 24 Nov 2017 14:19:16 +0000 (15:19 +0100)
committerLucas Stach <l.stach@pengutronix.de>
Tue, 2 Jan 2018 16:33:01 +0000 (17:33 +0100)
The GPU exec state may have changed at the time when the perfmon sampling
is done, as it reflects the state of the last submission, not the current
GPU execution state.

So for proper sampling we must use the submit exec_state.

Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
drivers/gpu/drm/etnaviv/etnaviv_gpu.c
drivers/gpu/drm/etnaviv/etnaviv_perfmon.c
drivers/gpu/drm/etnaviv/etnaviv_perfmon.h

index 675c31be86a75bdf39a418ddb319331164975de6..afc4b6c5fbf56ff29be9731e5459662cd79df718 100644 (file)
@@ -1324,7 +1324,7 @@ static void sync_point_perfmon_sample(struct etnaviv_gpu *gpu,
                const struct etnaviv_perfmon_request *pmr = submit->pmrs + i;
 
                if (pmr->flags == flags)
-                       etnaviv_perfmon_process(gpu, pmr);
+                       etnaviv_perfmon_process(gpu, pmr, submit->exec_state);
        }
 }
 
index 768f5aafdd186f8fb97b5b0d1f957128f98e8e36..26dddfc41aacb5ee43708a7893988d14959350b8 100644 (file)
@@ -479,9 +479,9 @@ int etnaviv_pm_req_validate(const struct drm_etnaviv_gem_submit_pmr *r,
 }
 
 void etnaviv_perfmon_process(struct etnaviv_gpu *gpu,
-       const struct etnaviv_perfmon_request *pmr)
+       const struct etnaviv_perfmon_request *pmr, u32 exec_state)
 {
-       const struct etnaviv_pm_domain_meta *meta = &doms_meta[gpu->exec_state];
+       const struct etnaviv_pm_domain_meta *meta = &doms_meta[exec_state];
        const struct etnaviv_pm_domain *dom;
        const struct etnaviv_pm_signal *sig;
        u32 *bo = pmr->bo_vma;
index 35dce194cb0042eff1349ba06ae3237515a638bb..c1653c64ab6b1e954fca5d70154dc348c05a50be 100644 (file)
@@ -44,6 +44,6 @@ int etnaviv_pm_req_validate(const struct drm_etnaviv_gem_submit_pmr *r,
        u32 exec_state);
 
 void etnaviv_perfmon_process(struct etnaviv_gpu *gpu,
-       const struct etnaviv_perfmon_request *pmr);
+       const struct etnaviv_perfmon_request *pmr, u32 exec_state);
 
 #endif /* __ETNAVIV_PERFMON_H__ */