From 7d6aef6fa5b9871d46c9f2d1cf06e70fef69adfb Mon Sep 17 00:00:00 2001 From: Eric Anholt Date: Tue, 17 Jul 2018 13:21:58 -0700 Subject: [PATCH] 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. --- src/gallium/drivers/v3d/v3d_bufmgr.c | 11 +++++------ 1 file 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 { -- 2.30.2