diff options
| author | Chris Wilson <chris@chris-wilson.co.uk> | 2013-09-24 13:44:59 +0100 |
|---|---|---|
| committer | Chris Wilson <chris@chris-wilson.co.uk> | 2013-09-25 09:40:35 +0100 |
| commit | 53b43673efed4d82b6dd498e9b76892e3e5b13e6 (patch) | |
| tree | 4aed4a621d3c3f6c562e02603c5d4f3a1e337e02 | |
| parent | ad0390068832ad4727371902fe41a85a53de1894 (diff) | |
sna: Add some DBG for early flushes
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
| -rw-r--r-- | src/sna/kgem.c | 34 |
1 files changed, 26 insertions, 8 deletions
diff --git a/src/sna/kgem.c b/src/sna/kgem.c index b0e0477e..d46f9671 100644 --- a/src/sna/kgem.c +++ b/src/sna/kgem.c @@ -4571,8 +4571,10 @@ bool kgem_check_bo(struct kgem *kgem, ...) if (bo->exec) continue; - if (needs_semaphore(kgem, bo)) + if (needs_semaphore(kgem, bo)) { + DBG(("%s: flushing for required semaphore\n", __FUNCTION__)); return false; + } num_pages += num_pages(bo); num_exec++; @@ -4587,8 +4589,10 @@ bool kgem_check_bo(struct kgem *kgem, ...) if (!num_pages) return true; - if (kgem_flush(kgem, flush)) + if (kgem_flush(kgem, flush)) { + DBG(("%s: opportunistic flushing\n", __FUNCTION__)); return false; + } if (kgem->aperture > kgem->aperture_low && kgem_ring_is_idle(kgem, kgem->ring)) { @@ -4641,18 +4645,25 @@ bool kgem_check_bo_fenced(struct kgem *kgem, struct kgem_bo *bo) return true; } - if (needs_semaphore(kgem, bo)) + if (needs_semaphore(kgem, bo)) { + DBG(("%s: flushing for required semaphore\n", __FUNCTION__)); return false; + } - if (kgem_flush(kgem, bo->flush)) + if (kgem_flush(kgem, bo->flush)) { + DBG(("%s: opportunistic flushing\n", __FUNCTION__)); return false; + } if (kgem->nexec >= KGEM_EXEC_SIZE(kgem) - 1) return false; if (kgem->aperture > kgem->aperture_low && - kgem_ring_is_idle(kgem, kgem->ring)) + kgem_ring_is_idle(kgem, kgem->ring)) { + DBG(("%s: current aperture usage (%d) is greater than low water mark (%d)\n", + __FUNCTION__, kgem->aperture, kgem->aperture_low)); return false; + } if (kgem->aperture + num_pages(bo) > kgem->aperture_high) return false; @@ -4703,8 +4714,10 @@ bool kgem_check_many_bo_fenced(struct kgem *kgem, ...) continue; } - if (needs_semaphore(kgem, bo)) + if (needs_semaphore(kgem, bo)) { + DBG(("%s: flushing for required semaphore\n", __FUNCTION__)); return false; + } assert_tiling(kgem, bo); num_pages += num_pages(bo); @@ -4731,12 +4744,17 @@ bool kgem_check_many_bo_fenced(struct kgem *kgem, ...) } if (num_pages) { - if (kgem_flush(kgem, flush)) + if (kgem_flush(kgem, flush)) { + DBG(("%s: opportunistic flushing\n", __FUNCTION__)); return false; + } if (kgem->aperture > kgem->aperture_low && - kgem_ring_is_idle(kgem, kgem->ring)) + kgem_ring_is_idle(kgem, kgem->ring)) { + DBG(("%s: current aperture usage (%d) is greater than low water mark (%d)\n", + __FUNCTION__, kgem->aperture, kgem->aperture_low)); return false; + } if (num_pages + kgem->aperture > kgem->aperture_high) return false; |