nv50,nvc0: Fix gallium nine regression regarding sampler bindings
The new approach is that samplers don't get unbound even if they won't be used
in a draw and we should just leave them be as well.
Fixes a regression in multiple windows games using gallium nine and nouveau.
v2: adjust num_samplers to keep track of the highest sampler bound
v3: rework how to set the new value of num_samplers
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106577
Fixes: 4d6fab245e
"cso: don't track the number of sampler states bound"
Signed-off-by: Karol Herbst <kherbst@redhat.com>
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
This commit is contained in:
parent
b6f095f7ce
commit
fc0139d283
|
@ -600,25 +600,23 @@ static inline void
|
|||
nv50_stage_sampler_states_bind(struct nv50_context *nv50, int s,
|
||||
unsigned nr, void **hwcso)
|
||||
{
|
||||
unsigned highest_found = 0;
|
||||
unsigned i;
|
||||
|
||||
assert(nr <= PIPE_MAX_SAMPLERS);
|
||||
for (i = 0; i < nr; ++i) {
|
||||
struct nv50_tsc_entry *old = nv50->samplers[s][i];
|
||||
|
||||
if (hwcso[i])
|
||||
highest_found = i;
|
||||
|
||||
nv50->samplers[s][i] = nv50_tsc_entry(hwcso[i]);
|
||||
if (old)
|
||||
nv50_screen_tsc_unlock(nv50->screen, old);
|
||||
}
|
||||
assert(nv50->num_samplers[s] <= PIPE_MAX_SAMPLERS);
|
||||
for (; i < nv50->num_samplers[s]; ++i) {
|
||||
if (nv50->samplers[s][i]) {
|
||||
nv50_screen_tsc_unlock(nv50->screen, nv50->samplers[s][i]);
|
||||
nv50->samplers[s][i] = NULL;
|
||||
}
|
||||
}
|
||||
|
||||
nv50->num_samplers[s] = nr;
|
||||
if (nr >= nv50->num_samplers[s])
|
||||
nv50->num_samplers[s] = highest_found + 1;
|
||||
|
||||
nv50->dirty_3d |= NV50_NEW_3D_SAMPLERS;
|
||||
}
|
||||
|
|
|
@ -464,11 +464,15 @@ nvc0_stage_sampler_states_bind(struct nvc0_context *nvc0,
|
|||
unsigned s,
|
||||
unsigned nr, void **hwcso)
|
||||
{
|
||||
unsigned highest_found = 0;
|
||||
unsigned i;
|
||||
|
||||
for (i = 0; i < nr; ++i) {
|
||||
struct nv50_tsc_entry *old = nvc0->samplers[s][i];
|
||||
|
||||
if (hwcso[i])
|
||||
highest_found = i;
|
||||
|
||||
if (hwcso[i] == old)
|
||||
continue;
|
||||
nvc0->samplers_dirty[s] |= 1 << i;
|
||||
|
@ -477,14 +481,8 @@ nvc0_stage_sampler_states_bind(struct nvc0_context *nvc0,
|
|||
if (old)
|
||||
nvc0_screen_tsc_unlock(nvc0->screen, old);
|
||||
}
|
||||
for (; i < nvc0->num_samplers[s]; ++i) {
|
||||
if (nvc0->samplers[s][i]) {
|
||||
nvc0_screen_tsc_unlock(nvc0->screen, nvc0->samplers[s][i]);
|
||||
nvc0->samplers[s][i] = NULL;
|
||||
}
|
||||
}
|
||||
|
||||
nvc0->num_samplers[s] = nr;
|
||||
if (nr >= nvc0->num_samplers[s])
|
||||
nvc0->num_samplers[s] = highest_found + 1;
|
||||
}
|
||||
|
||||
static void
|
||||
|
|
Loading…
Reference in New Issue