diff options
| author | Chris Wilson <chris@chris-wilson.co.uk> | 2013-10-23 08:40:43 +0100 |
|---|---|---|
| committer | Chris Wilson <chris@chris-wilson.co.uk> | 2013-10-23 08:40:43 +0100 |
| commit | aff67ada6cd2cb1f8adb5bec27a27fbc0f2884cc (patch) | |
| tree | 782ef1a4b7f7cf749bdc4d6bcfc80daf0daa9da5 /src | |
| parent | 11e8b299e11ff846e1c8090f7cbd34b2af850d85 (diff) | |
sna: Tidy opportunistic flushing
Rearrange the common code so that it should be tail-call optimised.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Diffstat (limited to 'src')
| -rw-r--r-- | src/sna/kgem.c | 49 |
1 files changed, 17 insertions, 32 deletions
diff --git a/src/sna/kgem.c b/src/sna/kgem.c index 74af4218..d60c430c 100644 --- a/src/sna/kgem.c +++ b/src/sna/kgem.c @@ -4651,15 +4651,17 @@ static bool aperture_check(struct kgem *kgem, unsigned num_pages) static inline bool kgem_flush(struct kgem *kgem, bool flush) { if (kgem->nreloc == 0) - return false; + return true; if (container_of(kgem, struct sna, kgem)->flags & SNA_POWERSAVE) - return false; + return true; if (kgem->flush == flush && kgem->aperture < kgem->aperture_low) - return false; + return true; - return kgem_ring_is_idle(kgem, kgem->ring); + DBG(("%s: opportunistic flushing? flush=%d,%d, aperture=%d/%d, idle?=%d\n", + __FUNCTION__, kgem->flush, flush, kgem->aperture, kgem->aperture_low, kgem_ring_is_idle(kgem, kgem->ring))); + return !kgem_ring_is_idle(kgem, kgem->ring); } bool kgem_check_bo(struct kgem *kgem, ...) @@ -4701,19 +4703,13 @@ bool kgem_check_bo(struct kgem *kgem, ...) return false; } - if (kgem_flush(kgem, flush)) { - DBG(("%s: opportunistic flushing, aperture %d/%d\n", - __FUNCTION__, kgem->aperture, kgem->aperture_low)); - return false; - } - if (num_pages + kgem->aperture > kgem->aperture_high) { DBG(("%s: final aperture usage (%d) is greater than high water mark (%d)\n", __FUNCTION__, num_pages + kgem->aperture, kgem->aperture_high)); return aperture_check(kgem, num_pages); } - return true; + return kgem_flush(kgem, flush); } bool kgem_check_bo_fenced(struct kgem *kgem, struct kgem_bo *bo) @@ -4753,12 +4749,6 @@ bool kgem_check_bo_fenced(struct kgem *kgem, struct kgem_bo *bo) return false; } - if (kgem_flush(kgem, bo->flush)) { - DBG(("%s: opportunistic flushing, aperture %d/%d\n", - __FUNCTION__, kgem->aperture, kgem->aperture_low)); - return false; - } - if (kgem->aperture + num_pages(bo) > kgem->aperture_high) { DBG(("%s: final aperture usage (%d) is greater than high water mark (%d)\n", __FUNCTION__, num_pages(bo) + kgem->aperture, kgem->aperture_high)); @@ -4780,7 +4770,7 @@ bool kgem_check_bo_fenced(struct kgem *kgem, struct kgem_bo *bo) return false; } - return true; + return kgem_flush(kgem, bo->flush); } bool kgem_check_many_bo_fenced(struct kgem *kgem, ...) @@ -4840,24 +4830,19 @@ bool kgem_check_many_bo_fenced(struct kgem *kgem, ...) return false; } - if (num_pages) { - if (kgem->nexec + num_exec >= KGEM_EXEC_SIZE(kgem)) - return false; + if (num_pages == 0) + return true; - if (kgem_flush(kgem, flush)) { - DBG(("%s: opportunistic flushing, aperture %d/%d\n", - __FUNCTION__, kgem->aperture, kgem->aperture_low)); - return false; - } + if (kgem->nexec + num_exec >= KGEM_EXEC_SIZE(kgem)) + return false; - if (num_pages + kgem->aperture > kgem->aperture_high) { - DBG(("%s: final aperture usage (%d) is greater than high water mark (%d)\n", - __FUNCTION__, num_pages + kgem->aperture, kgem->aperture_high)); - return aperture_check(kgem, num_pages); - } + if (num_pages + kgem->aperture > kgem->aperture_high) { + DBG(("%s: final aperture usage (%d) is greater than high water mark (%d)\n", + __FUNCTION__, num_pages + kgem->aperture, kgem->aperture_high)); + return aperture_check(kgem, num_pages); } - return true; + return kgem_flush(kgem, flush); } uint32_t kgem_add_reloc(struct kgem *kgem, |