diff options
author | Jason Ekstrand <[email protected]> | 2017-08-19 15:03:39 -0700 |
---|---|---|
committer | Jason Ekstrand <[email protected]> | 2017-08-20 20:14:49 -0700 |
commit | c366943ebffefc4a8acb3fdd08dd45934098595b (patch) | |
tree | 781147350375d95a26c1beac02d47c4480fd11c3 /src/mesa/drivers/dri/i965/brw_bufmgr.c | |
parent | cadcd89278edcda8aba28a38da2ea6690207b38b (diff) |
i965/bufmgr: s/BO_ALLOC_FOR_RENDER/BO_ALLOC_BUSY/
"Alloc for render" is a terrible name for a flag because it means
basically nothing. What the flag really does is allocate a busy BO
which someone theorized at one point in time would be more efficient if
you're planning to immediately render to it. If the flag really means
"alloc a busy BO" we should just call it that.
Reviewed-by: Kenneth Graunke <[email protected]>
Diffstat (limited to 'src/mesa/drivers/dri/i965/brw_bufmgr.c')
-rw-r--r-- | src/mesa/drivers/dri/i965/brw_bufmgr.c | 15 |
1 files changed, 7 insertions, 8 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_bufmgr.c b/src/mesa/drivers/dri/i965/brw_bufmgr.c index 829f0b1d386..5b4e784ae24 100644 --- a/src/mesa/drivers/dri/i965/brw_bufmgr.c +++ b/src/mesa/drivers/dri/i965/brw_bufmgr.c @@ -256,20 +256,19 @@ bo_alloc_internal(struct brw_bufmgr *bufmgr, struct bo_cache_bucket *bucket; bool alloc_from_cache; uint64_t bo_size; - bool for_render = false; + bool busy = false; bool zeroed = false; - if (flags & BO_ALLOC_FOR_RENDER) - for_render = true; + if (flags & BO_ALLOC_BUSY) + busy = true; if (flags & BO_ALLOC_ZEROED) zeroed = true; - /* FOR_RENDER really means "I'm ok with a busy BO". This doesn't really - * jive with ZEROED as we have to wait for it to be idle before we can - * memset. Just disallow that combination. + /* BUSY does doesn't really jive with ZEROED as we have to wait for it to + * be idle before we can memset. Just disallow that combination. */ - assert(!(for_render && zeroed)); + assert(!(busy && zeroed)); /* Round the allocated size up to a power of two number of pages. */ bucket = bucket_for_size(bufmgr, size); @@ -290,7 +289,7 @@ bo_alloc_internal(struct brw_bufmgr *bufmgr, retry: alloc_from_cache = false; if (bucket != NULL && !list_empty(&bucket->head)) { - if (for_render && !zeroed) { + if (busy && !zeroed) { /* Allocate new render-target BOs from the tail (MRU) * of the list, as it will likely be hot in the GPU * cache and in the aperture for us. If the caller |