- Jan 24, 2015
-
-
Emil Velikov authored
Signed-off-by:
Emil Velikov <emil.l.velikov@gmail.com>
-
Emil Velikov authored
Signed-off-by:
Emil Velikov <emil.l.velikov@gmail.com>
-
- Jan 23, 2015
-
-
When the shader does indirect addressing on the constants, we allocate a temporary constant buffer to which we copy the constants from the app given user constants and the constants filled in the shader. This patch makes this buffer be allocated once. Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Signed-off-by:
Tiziano Bacocco <tizbac2@gmail.com> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit f8a74410)
-
Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit e0f75044)
-
Reviewed-by:
Tiziano Bacocco <tizbac2@gmail.com> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit b9cbea9d)
-
Relative addressing needs the constant buffer to get all the correct constants, even those defined by the shader. The code to copy the shader constants to the constant buffer was enabled only for debug build. Enable it always. Cc: "10.4" <mesa-stable@lists.freedesktop.org> Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
David Heidelberg <david@ixit.cz> Signed-off-by:
Axel Davy <axel.davy@ens.fr> (cherry picked from commit a7219870)
-
Since constant indirect adressing is not allowed for ps, we can remove our code to handle that. Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit 4b7a9cfd)
-
relative adressing for constants is possible only for vs float constants. Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit 9690bf33)
-
Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit bce94ce8)
-
Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit 9e23b64c)
-
Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit 09eb1e90)
-
Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
David Heidelberg <david@ixit.cz> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit f19e6993)
-
Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit 3676ab02)
-
Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
David Heidelberg <david@ixit.cz> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit 2b9f079a)
-
Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit fdff111d)
-
The fix is that this line: "src[s] = tx->regs.vT[s];" is wrong if s doesn't start from 0. Instead access tx->regs.vT directly when needed. Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit 78652106) Conflicts: src/gallium/state_trackers/nine/nine_shader.c
-
Not filling them correctly results in bad padding and later crash. Reviewed-by:
David Heidelberg <david@ixit.cz> Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit b1259544)
-
texcoord for ps < 1_4 should clamp between 0 and 1 the values. texcrd (texcoord ps 1_4) does not clamp and can be used with two modifiers _dw and _dz that means the channels are divided by w or z. Implement those in shared code, since the same modifiers can be used for texld ps 1_4. v2: replace DIV by RCP + MUL v3: Remove an useless MOV Reviewed-by:
Tiziano Bacocco <tizbac2@gmail.com> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit 5399119f) Conflicts: src/gallium/state_trackers/nine/nine_shader.c
-
- Jan 22, 2015
-
-
Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit 6378d749)
-
This is wine (and windows) behaviour. Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit 018407b5)
-
Signed-off-by:
Axel Davy <axel.davy@ens.fr> Signed-off-by:
Tiziano Bacocco <tizbac2@gmail.com> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit 3ca67f88)
-
Previous implementation didn't work well with nested loops. Instead of using several address registers, put a0 and aL into normal registers, and copy them to one address register when we need to use them. Wine tests loop_index_test() and nested_loop_test() now pass correctly. Fixes r600g crash while loading Bioshock - bug https://bugs.freedesktop.org/show_bug.cgi?id=85696 Tested-by:
David Heidelberg <david@ixit.cz> Reviewed-by:
Tiziano Bacocco <tizbac2@gmail.com> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit 6a8e5e48)
-
We should take the absolute value of the input. Also return -FLT_MAX instead of -Inf for an input of 0. Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
David Heidelberg <david@ixit.cz> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit c9aa9a0a)
-
When the input's xyz are 0.0, the output should be 0.0. This is due to the fact that Inf * 0 = 0 for dx9. To handle this case, cap the result of RSQ to FLT_MAX. We have FLT_MAX * 0 = 0. Reviewed-by:
David Heidelberg <david@ixit.cz> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit f5e8e3fb)
-
We should use the absolute value of the input as input to ureg_RSQ. Moreover, an input of 0.0 should return FLT_MAX. Reviewed-by:
David Heidelberg <david@ixit.cz> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit 2487f735)
-
POW doesn't match directly TGSI, since we should take the absolute value of src0. Fixes black textures in some games Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
David Heidelberg <david@ixit.cz> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit c12f8c20)
-
Cc: "10.4" <mesa-stable@lists.freedesktop.org> Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
David Heidelberg <david@ixit.cz> Signed-off-by:
Axel Davy <axel.davy@ens.fr> (cherry picked from commit e0dd9ca9)
-
Let's say we have c1 and c2 declared in the shader and c0 given by the app Then here we would have read c0, c1 and c2 given by the app, instead of the correct c0, c1, c2. This correction fixes several issues in some games. Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit 53dc992f)
-
According to docs and Wine, these two vs outputs have to be saturated. Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
David Heidelberg <david@ixit.cz> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit 9fb58a74)
-
Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit a2148381)
-
The shader code is already behaving as if they are floats when the the card doesn't support integers Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit d08c7b0b)
-
Convert them to shader booleans at earlier stage. Previous code is fine, but later patch will make integers being converted at earlier stage, so do the same for booleans Reviewed-by:
Tiziano Bacocco <tizbac2@gmail.com> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit d9d18fe3)
-
Adds ATI1 and ATI2 support to nine. They map to PIPE_FORMAT_RGTC1_UNORM and PIPE_FORMAT_RGTC2_UNORM, but need special handling. Reviewed-by:
David Heidelberg <david@ixit.cz> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Signed-off-by:
Xavier Bouchoux <xavierb@gmail.com> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit 77f0ecf9)
-
According to msdn, we must act as if user didn't ask srgb if we don't support it. Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
David Heidelberg <david@ixit.cz> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit b0b54303)
-
Buffers in the MANAGED pool are supposed to have the content in a ram buffer, a copy in VRAM if there is enough memory (driver manages memory and decide when to delete the buffer in VRAM). This is not implemented properly in nine, and a VRAM copy is going to be created when the RAM memory is filled, and the VRAM copy will get synced with the RAM memory updates. Due to some issues (in the implementation or in app logic), it can happen we try to create a sampler view of the resource while we haven't created the VRAM resource. This hack creates the resource when we hit this case, which prevents crashing, but doesn't help with the resource content. This fixes several games crashing at launch. Acked-by:
Axel Davy <axel.davy@ens.fr> Acked-by:
David Heidelberg <david@ixit.cz> Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Signed-off-by:
Stanislaw Halik <sthalik@misaki.pl> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit 82810d3b)
-
While previous code was having the correct behaviour in general, this new code is more readable (without checking all gallium formats manually) and has a more defined behaviour for depth stencil resources. Reviewed-by:
Tiziano Bacocco <tizbac2@gmail.com> Reviewed-by:
David Heidelberg <david@ixit.cz> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit 47280d77)
-
Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
David Heidelberg <david@ixit.cz> Signed-off-by:
Axel Davy <axel.davy@ens.fr> (cherry picked from commit 0abfb80d)
-
The implicit swapchains are destroyed when the device instance is destroyed. However for non-implicit swapchains, it is not the case, and the application can have kept an reference on the swapchain buffers to reuse them. Fixes problems with battle.net launcher. Cc: "10.4" <mesa-stable@lists.freedesktop.org> Tested-by:
Nick Sarnie <commendsarnex@gmail.com> Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
David Heidelberg <david@ixit.cz> Signed-off-by:
Axel Davy <axel.davy@ens.fr> (cherry picked from commit 0d2c22e6)
-
This->surfaces contains the surfaces associated to the levels and faces. This->surfaces[6*Level] is what we want here, since it gives us a face descriptor for the level 'Level'. Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
David Heidelberg <david@ixit.cz> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Signed-off-by:
Xavier Bouchoux <xavierb@gmail.com> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit 92321611)
-
Use same similar settings as u_sampler_view_default_template Reviewed-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
David Heidelberg <david@ixit.cz> Signed-off-by:
Axel Davy <axel.davy@ens.fr> Cc: "10.4" <mesa-stable@lists.freedesktop.org> (cherry picked from commit 18c7e702)
-