diff options
| author | Erik Faye-Lund <erik.faye-lund@collabora.com> | 2022-10-05 16:20:27 +0200 |
|---|---|---|
| committer | Marge Bot <emma+marge@anholt.net> | 2022-10-10 10:20:02 +0000 |
| commit | 55f6a2bb51a1fadcd68f9f09ccf4435a4e4ca269 (patch) | |
| tree | 6025c1e2b46f6e35b3402d5c0106a70556341aa1 /src/gallium/drivers/svga/svga_pipe_sampler.c | |
| parent | 4e7b9aaa066e0ea03dc408193211c423896e603c (diff) | |
gallium: normalized_coords -> unnormalized_coords
A lot of code zero-initializes pipe_sampler_state, and sets the states
the non-zero fields manually. This means that normalized_coords is the
"default" setting.
However, setting normalized_coords to true isn't allways allowed, and
we'd need to check PIPE_CAP_TEXRECT first. So it's not really the ideal
default here. There's recently been found quite a bit of bugs in this
area, where the state-tracker didn't properly lower texrects.
Let's switch this around to avoid more bugs like this in the future.
Reviewed-by: Adam Jackson <ajax@redhat.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/18971>
Diffstat (limited to 'src/gallium/drivers/svga/svga_pipe_sampler.c')
| -rw-r--r-- | src/gallium/drivers/svga/svga_pipe_sampler.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/gallium/drivers/svga/svga_pipe_sampler.c b/src/gallium/drivers/svga/svga_pipe_sampler.c index 3e0e26c7628..eb398459a9f 100644 --- a/src/gallium/drivers/svga/svga_pipe_sampler.c +++ b/src/gallium/drivers/svga/svga_pipe_sampler.c @@ -246,7 +246,7 @@ svga_create_sampler_state(struct pipe_context *pipe, cso->addressu = translate_wrap_mode(sampler->wrap_s); cso->addressv = translate_wrap_mode(sampler->wrap_t); cso->addressw = translate_wrap_mode(sampler->wrap_r); - cso->normalized_coords = sampler->normalized_coords; + cso->normalized_coords = !sampler->unnormalized_coords; cso->compare_mode = sampler->compare_mode; cso->compare_func = sampler->compare_func; |