diff options
| author | Eric Anholt <[email protected]> | 2018-07-17 13:21:58 -0700 |
|---|---|---|
| committer | Eric Anholt <[email protected]> | 2018-07-20 11:11:29 -0700 |
| commit | 7d6aef6fa5b9871d46c9f2d1cf06e70fef69adfb (patch) | |
| tree | fde32cfe141790028f379ba5f2fd6aad84e5c521 | |
| parent | 5d11094db15d6a418e58eeed9695186ac9f5f5a9 (diff) | |
v3d: Reduce the stale BO reclamation spam with dump_stats set.
This was obviously meant to be when we were actually freeing a BO, not
just when there was at least one BO in the list.
| -rw-r--r-- | src/gallium/drivers/v3d/v3d_bufmgr.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/src/gallium/drivers/v3d/v3d_bufmgr.c b/src/gallium/drivers/v3d/v3d_bufmgr.c index f7a851f9d43..1b4acc6fca8 100644 --- a/src/gallium/drivers/v3d/v3d_bufmgr.c +++ b/src/gallium/drivers/v3d/v3d_bufmgr.c @@ -238,14 +238,13 @@ free_stale_bos(struct v3d_screen *screen, time_t time) list_for_each_entry_safe(struct v3d_bo, bo, &cache->time_list, time_list) { - if (dump_stats && !freed_any) { - fprintf(stderr, "Freeing stale BOs:\n"); - v3d_bo_dump_stats(screen); - freed_any = true; - } - /* If it's more than a second old, free it. */ if (time - bo->free_time > 2) { + if (dump_stats && !freed_any) { + fprintf(stderr, "Freeing stale BOs:\n"); + v3d_bo_dump_stats(screen); + freed_any = true; + } v3d_bo_remove_from_cache(cache, bo); v3d_bo_free(bo); } else { |