diff options
author | Ilia Mirkin <[email protected]> | 2017-01-10 22:07:53 -0500 |
---|---|---|
committer | Ilia Mirkin <[email protected]> | 2017-01-12 20:39:19 -0500 |
commit | eb60a89bc3ac2b43faf52d06e05670bbbca7292d (patch) | |
tree | 8a910c40646ee6b3e14bb26d48c341cef38e587c /install-gallium-links.mk | |
parent | 89458366585c34879b70110758bb4fd3acb62ce0 (diff) |
nouveau: take extra push space into account for pushbuf_space calls
Ever since a long time ago when I messed around with fences, I ensure
that after a PUSH_SPACE call there is enough space to write a fence out
into the pushbuf.
However the PUSH_SPACE macro is not all-knowing, and so sometimes we
have to invoke nouveau_pushbuf_space manually with the relocs/pushes
args set. If we don't take the extra allocation from PUSH_SPACE into
account, then we will end up accidentally flushing when the code was not
expecting a flush. This can lead to various runtime and rendering
failures.
The amount of extra allocation isn't that important - it has to be at
least 8 based on the current nouveau_winsys.h setting, but even more
won't hurt. I just rounded up to powers of 2.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=99354
Cc: "12.0 13.0" <[email protected]>
Signed-off-by: Ilia Mirkin <[email protected]>
Acked-by: Ben Skeggs <[email protected]>
Diffstat (limited to 'install-gallium-links.mk')
0 files changed, 0 insertions, 0 deletions