diff options
| author | Rob Clark <[email protected]> | 2019-03-10 11:08:10 -0400 |
|---|---|---|
| committer | Rob Clark <[email protected]> | 2019-03-10 11:40:06 -0400 |
| commit | 941adcef036c1a427c66ab08d041a008431c1e8c (patch) | |
| tree | c183a77e0204f2d5af1997c8c11bd9572a6dd6ad | |
| parent | db944999a1bad94471966862cc38212019165937 (diff) | |
freedreno/a6xx: more bcolor fixes
Non-zero offset wasn't working, which breaks a bunch of
dEQP-GLES31.functional.texture.border_clamp.formats.* when doing sharded
deqp runs (because order of tests changes, resulting in different
texture state bound.. deqp doesn't really clean up it's gl state between
tests very well)
Previously, if additional textures were bound, due to using too small of
a bcolor_entry size, the last 32bytes of the bcolor_entry would be
overwritten.
Signed-off-by: Rob Clark <[email protected]>
| -rw-r--r-- | src/gallium/drivers/freedreno/a6xx/fd6_emit.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/gallium/drivers/freedreno/a6xx/fd6_emit.c b/src/gallium/drivers/freedreno/a6xx/fd6_emit.c index 95ac616a76c..b48a0d68fde 100644 --- a/src/gallium/drivers/freedreno/a6xx/fd6_emit.c +++ b/src/gallium/drivers/freedreno/a6xx/fd6_emit.c @@ -162,10 +162,10 @@ struct PACKED bcolor_entry { uint32_t rgb10a2; uint32_t z24; /* also s8? */ uint16_t srgb[4]; /* appears to duplicate fp16[], but clamped, used for srgb */ - uint8_t __pad1[24]; + uint8_t __pad1[56]; }; -#define FD6_BORDER_COLOR_SIZE 0x60 +#define FD6_BORDER_COLOR_SIZE sizeof(struct bcolor_entry) #define FD6_BORDER_COLOR_UPLOAD_SIZE (2 * PIPE_MAX_SAMPLERS * FD6_BORDER_COLOR_SIZE) static void |