diff options
| author | Ben Widawsky <ben@bwidawsk.net> | 2016-11-29 13:22:17 -0800 |
|---|---|---|
| committer | Ben Widawsky <ben@bwidawsk.net> | 2017-03-09 16:10:21 -0800 |
| commit | 97d943500aea16c5700cab0a44b1959d13bfab02 (patch) | |
| tree | aeb8d0bd80729b803cfe1e960ae775aab263652f | |
| parent | fc12aaff1a42c57e5fd92612880a649f72feb0ba (diff) | |
i965: Remove scanout restriction from lossless compression
v2: Try to keep the assert as recommended by Topi. This requires
modifying the num_samples check to be <= 1 because internally created
buffers set num_samples = 0.
v3: Buffers are proactively marked as scanout, often, and so checking
is_scanout in whether or not the buffer supports non-msrt fast clears
will return false. To avoid this, only check buffers which are destined
to use ccs (is a scanout buffer, and has an "mcs" buffer). Chad found
this issue.
v4: Use a better assertion based off of change in last patch. (Topi)
v5: Remove the assert entirely
Cc: Topi Pohjolainen <topi.pohjolainen@intel.com>
Cc: Chad Versace <chadversary@chromium.org>
Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
Acked-by: Daniel Stone <daniels@collabora.com>
| -rw-r--r-- | src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c index 1961a49592..97672df472 100644 --- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c +++ b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c @@ -740,8 +740,7 @@ intel_miptree_create(struct brw_context *brw, const bool lossless_compression_disabled = INTEL_DEBUG & DEBUG_NO_RBC; const bool is_lossless_compressed = unlikely(!lossless_compression_disabled) && - brw->gen >= 9 && !mt->is_scanout && - intel_miptree_supports_lossless_compressed(brw, mt); + brw->gen >= 9 && intel_miptree_supports_lossless_compressed(brw, mt); if (is_lossless_compressed) { assert((mt->aux_disable & INTEL_AUX_DISABLE_CCS) == 0); |