On 12.08.2015 19:32, Marek Olšák wrote:
> On Wed, Aug 12, 2015 at 11:30 AM, Michel Dänzer wrote:
>> On 11.08.2015 01:56, Marek Olšák wrote:
>>> Reviewed-by: Marek Olšák
>>
>> Thanks Marek. Unfortunately, I remembered in the meantime that this
>> patch alone won't help for r300g, because it doesn'
Here's bunch of changes to texture storage with the fixes from review.
Thanks;
Tapani Pälli (3):
mesa: expose dimension check for glTex*Storage functions
mesa: validate size parameters for glTexStorage*Multisample
mesa: set correct error for non-renderable multisample textures
src/mesa/ma
v2: same common error on gles31 and desktop OpenGL
(spotted by Erik Faye-Lund)
Signed-off-by: Marta Lofstedt
Signed-off-by: Tapani Pälli
---
src/mesa/main/teximage.c | 13 ++---
1 file changed, 10 insertions(+), 3 deletions(-)
diff --git a/src/mesa/main/teximage.c b/src/mesa/main/t
v2: code cleanup
Signed-off-by: Tapani Pälli
---
src/mesa/main/teximage.c | 24
1 file changed, 24 insertions(+)
diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c
index d35dc12..add7438 100644
--- a/src/mesa/main/teximage.c
+++ b/src/mesa/main/teximage.c
This is done so that following patch can use it to verify dimensions
for multisample variants of glTex*Storage.
v2: move function to header, use bool instead GLboolean
Signed-off-by: Tapani Pälli
---
src/mesa/main/texstorage.c | 2 +-
src/mesa/main/texstorage.h | 21 +
2 fi
On Friday, June 26, 2015 05:50:52 PM Topi Pohjolainen wrote:
> This is stimulating the shader binary re-use logic in i965 buffer
> object uploading. Without relaxing the current constraints there
> will be two identical copies in the cache. With the introduced
> three patches the logic begins to sh
Reviewed-by: Edward O'Callaghan
--
Edward O'Callaghan
edward.ocallag...@koparo.com
On Wed, Aug 12, 2015, at 08:45 PM, Marek Olšák wrote:
> From: Marek Olšák
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=74329
>
> v2: add a CAP for half floats
> drivers should not expose
On Thursday, June 25, 2015 03:45:36 PM Topi Pohjolainen wrote:
> and simplify the interface to take directly the size and to return
> the offset. The routine does nothing more than allocate, it doesn't
> upload anything.
>
> CC: Kenneth Graunke
> Signed-off-by: Topi Pohjolainen
> ---
> src/mesa
Reviewed-by: Tapani Palli
On 08/10/2015 07:31 PM, Neil Roberts wrote:
Since 4d7e0fa8c731776 this file is generated by the configure script.
---
src/gallium/targets/opencl/.gitignore | 1 +
1 file changed, 1 insertion(+)
create mode 100644 src/gallium/targets/opencl/.gitignore
diff --git a
On 08/10/2015 07:50 PM, Brian Paul wrote:
On 08/10/2015 02:06 AM, Tapani Pälli wrote:
This is done so that following patch can use it to verify dimenstions
for multisample variants of glTex*Storage.
Signed-off-by: Tapani Pälli
---
src/mesa/main/texstorage.c | 22 +-
sr
https://bugs.freedesktop.org/show_bug.cgi?id=91254
--- Comment #11 from Tomasz C. ---
In 10.6.4 this bug is still present
Can not you just go back patch that causes the problem?
--
You are receiving this mail because:
You are the assignee for the bug.
___
On Aug 12, 2015 4:41 AM, "Francisco Jerez" wrote:
>
> Francisco Jerez writes:
>
> > Neil Roberts writes:
> >
> >> Ok, that makes sense, thanks for the detailed explanation. I think
> >> you're right, given the extra restrictions avoiding the integer dword
> >> multiplication seems like the simpl
On Aug 12, 2015 4:21 PM, "Eric Anholt" wrote:
>
> Jason Ekstrand writes:
>
> > On Wed, Aug 12, 2015 at 11:55 AM, Eric Anholt wrote:
> >> NIR instruction count results on i965:
> >> total instructions in shared programs: 1261954 -> 1261937 (-0.00%)
> >> instructions in affected programs: 455
From: Marcos Paulo de Souza
Signed-off-by: Marcos Paulo de Souza
Suggested-by: Ilia Mirkin
---
src/gallium/auxiliary/tgsi/tgsi_text.c | 6 +-
src/gallium/drivers/nouveau/nouveau_compiler.c | 4
2 files changed, 9 insertions(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary
On Wed, Aug 12, 2015 at 03:09:44PM +0300, Joonas Lahtinen wrote:
> Add a comment about reinforcing command order so that
> 3DSTATE_BINDING_TABLE_POINTER_* commands are after
> 3DSTATE_CONSTANT_* commands for SKL & BXT, otherwise the
> GPU might hang.
>
> Changing the BLORP code is not relevant (wh
Please change those to be %u, all those are GLuint's. With that fixed,
Reviewed-by: Ilia Mirkin
On Wed, Aug 12, 2015 at 3:03 AM, Timothy Arceri wrote:
> ---
> src/glsl/linker.cpp | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/src/glsl/linker.cpp b/src/glsl/linker
Jason Ekstrand writes:
> On Wed, Aug 12, 2015 at 11:55 AM, Eric Anholt wrote:
>> NIR instruction count results on i965:
>> total instructions in shared programs: 1261954 -> 1261937 (-0.00%)
>> instructions in affected programs: 455 -> 438 (-3.74%)
>>
>> One in yofrankie, two in tropics. App
From: Nanley Chery
This function's cases for non-generic compressed formats duplicate
the GL to MESA translation in _mesa_glenum_to_compressed_format().
This patch replaces the switch cases with a call to the translation
function. There are no behavioral changes except for the RGB[A]4 formats:
Hi,
On 2015-08-12 18:05, Fredrik Höglund wrote:
> On Thursday 06 August 2015, Daniel Scharrer wrote:
>> CC: "10.6"
>
> I think the commit message should say which queries are added.
>
>>
>> ---
>>
>> v2: added CC for 10.6
>> renamed _mesa_tex_target_to_index to tex_target_to_index
>> mov
On Wed, Aug 12, 2015 at 6:59 PM, Thomas Helland
wrote:
> ... that are not mandated by the current openGL specifications
>
> Signed-off-by: Thomas Helland
> Reviewed-by: Edward O'Callaghan
> Reviewed-by: Tapani Pälli
> ---
> Tapani, Edward: Does your review still stand?
> I decided to be kind an
... that are not mandated by the current openGL specifications
Signed-off-by: Thomas Helland
Reviewed-by: Edward O'Callaghan
Reviewed-by: Tapani Pälli
---
Tapani, Edward: Does your review still stand?
I decided to be kind and not rename the file. That way we don't
break things for the mesamatri
My comment about making ASTC format detection simpler is a little
subjective. I'm planning to remove it from the final patch if it's
accepted. What this change allows in general is detecting compressed
formats belonging to a certain category of compressed formats.
Regards,
Nanley
On Tue, Aug 11,
I'd just add a 2015 block and a 2014 block.
On Thu, Aug 13, 2015 at 9:36 AM, Ilia Mirkin wrote:
> On Wed, Aug 12, 2015 at 5:23 PM, Thomas Helland
> wrote:
>> 2015-08-12 18:56 GMT+02:00 Kenneth Graunke :
>>> On Wednesday, August 12, 2015 06:32:50 PM Thomas Helland wrote:
2015-08-12 17:48 GMT
Reviewed-by: Ilia Mirkin
On Wed, Aug 12, 2015 at 5:29 PM, Rob Clark wrote:
> From: Rob Clark
>
> Signed-off-by: Rob Clark
> ---
> src/gallium/auxiliary/nir/tgsi_to_nir.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/src/gallium/auxiliary/nir/tgsi_to_nir.c
> b/src/gallium/auxili
On Wed, Aug 12, 2015 at 5:23 PM, Thomas Helland
wrote:
> 2015-08-12 18:56 GMT+02:00 Kenneth Graunke :
>> On Wednesday, August 12, 2015 06:32:50 PM Thomas Helland wrote:
>>> 2015-08-12 17:48 GMT+02:00 Ilia Mirkin :
>>> > On Tue, Aug 11, 2015 at 1:48 PM, Thomas Helland
>>> > wrote:
>>> >> Signed-of
From: Rob Clark
Signed-off-by: Rob Clark
---
src/gallium/auxiliary/nir/tgsi_to_nir.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/gallium/auxiliary/nir/tgsi_to_nir.c
b/src/gallium/auxiliary/nir/tgsi_to_nir.c
index 4130697..93dfb80 100644
--- a/src/gallium/auxiliary/nir/tgsi_to_ni
2015-08-12 18:56 GMT+02:00 Kenneth Graunke :
> On Wednesday, August 12, 2015 06:32:50 PM Thomas Helland wrote:
>> 2015-08-12 17:48 GMT+02:00 Ilia Mirkin :
>> > On Tue, Aug 11, 2015 at 1:48 PM, Thomas Helland
>> > wrote:
>> >> Signed-off-by: Thomas Helland
>> >> ---
>> >> This adds a section for t
On Wed, Aug 12, 2015 at 1:58 PM, Matt Turner wrote:
> If an immediate is written to multiple channels, we can load it in a
> single writemasked MOV.
>
> total instructions in shared programs: 6285144 -> 6261991 (-0.37%)
> instructions in affected programs: 718991 -> 695838 (-3.22%)
> helped:
If an immediate is written to multiple channels, we can load it in a
single writemasked MOV.
total instructions in shared programs: 6285144 -> 6261991 (-0.37%)
instructions in affected programs: 718991 -> 695838 (-3.22%)
helped:5762
---
The shader-db numbers are
On Wed, Aug 12, 2015 at 11:55 AM, Eric Anholt wrote:
> There are so many flags in textures, that the CSE pass would have a hard
> time referencing the correct set when figuring out if two texture ops are
> the same. By zeroing, we can avoid that fragility.
> ---
Presumably you didn't see that th
On Wed, Aug 12, 2015 at 11:55 AM, Eric Anholt wrote:
> NIR instruction count results on i965:
> total instructions in shared programs: 1261954 -> 1261937 (-0.00%)
> instructions in affected programs: 455 -> 438 (-3.74%)
>
> One in yofrankie, two in tropics. Apparently i965 had also optimized
As I'm guessing that it's unlikely that the VMWare implementation will
get upstreamed before the Mesa 11.0 branch point, how do people feel
about including the below implementation *without* the extension
enable? That way people will still be able to run bioshock infinite
without resorting to patch
NIR instruction count results on i965:
total instructions in shared programs: 1261954 -> 1261937 (-0.00%)
instructions in affected programs: 455 -> 438 (-3.74%)
One in yofrankie, two in tropics. Apparently i965 had also optimized all
of these out anyway.
Reviewed-by: Kenneth Graunke
---
sr
There are so many flags in textures, that the CSE pass would have a hard
time referencing the correct set when figuring out if two texture ops are
the same. By zeroing, we can avoid that fragility.
---
src/glsl/nir/nir.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/glsl
On Wed, Aug 12, 2015 at 1:58 PM, Matt Turner wrote:
> On Tue, Aug 11, 2015 at 6:51 PM, Ilia Mirkin wrote:
>> ---
>> src/glsl/builtin_functions.cpp | 32
>> +-
>> src/glsl/glcpp/glcpp-parse.y | 3 +++
>> src/glsl/glsl_parser_extras.cpp
On Wed, Aug 12, 2015 at 8:36 AM, Frank Binns wrote:
> The EGL 1.4 spec states for eglCreateContext:
>
> "attribute EGL_CONTEXT_CLIENT_VERSION is only valid when the current
> rendering API is EGL_OPENGL_ES_API"
>
> Additionally, if the EGL_KHR_create_context EGL extension is suppo
On Tue, Aug 11, 2015 at 6:51 PM, Ilia Mirkin wrote:
> ---
> src/glsl/builtin_functions.cpp | 32
> +-
> src/glsl/glcpp/glcpp-parse.y | 3 +++
> src/glsl/glsl_parser_extras.cpp| 1 +
> src/glsl/glsl_parser_extras.h |
On Wed, Aug 12, 2015 at 9:48 AM, Richard Yao wrote:
> Appears to fix shader compilation. Tested by starting the client and observing
> that the screen was correct after the trailers ran when previously, it was
> blank. Play tested on amd64.
>
> This was suggested by "Kuuchan" on the Steam forums:
Subject: [PATCH] i915: BINDING_TABLE_POINTER_* after CONSTANT_* for SKL & BXT
The driver you're modifying is i965. :)
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
On Wed, Aug 12, 2015 at 8:35 AM, Frank Binns wrote:
> When a buffer is provided to eglGetConfigs it's supposed to set the value
> of the num_config parameter to the total number of configs that have been
> copied into this buffer. For some reason the EGL spec doesn't consider it
> to be an error t
Appears to fix shader compilation. Tested by starting the client and observing
that the screen was correct after the trailers ran when previously, it was
blank. Play tested on amd64.
This was suggested by "Kuuchan" on the Steam forums:
https://steamcommunity.com/app/200390/discussions/0/540731690
Add a comment about reinforcing command order so that
3DSTATE_BINDING_TABLE_POINTER_* commands are after
3DSTATE_CONSTANT_* commands for SKL & BXT, otherwise the
GPU might hang.
Changing the BLORP code is not relevant (where the order
is "wrong"), as it is not used for GEN8 or up.
Cc: Mika Kuoppa
On 08/12/2015 10:07 AM, Daniel Scharrer wrote:
On 2015-08-12 17:14, Brian Paul wrote:
Another question below...
On 08/12/2015 08:31 AM, Daniel Scharrer wrote:
On 2015-08-06 15:46, Brian Paul wrote:
On 08/06/2015 07:44 AM, Daniel Scharrer wrote:
CC: "10.6"
---
v2: added CC for 10.6
On Wednesday, August 12, 2015 06:32:50 PM Thomas Helland wrote:
> 2015-08-12 17:48 GMT+02:00 Ilia Mirkin :
> > On Tue, Aug 11, 2015 at 1:48 PM, Thomas Helland
> > wrote:
> >> Signed-off-by: Thomas Helland
> >> ---
> >> This adds a section for the extensions nvidia has chosen to
> >> call the "GL
2015-08-12 17:48 GMT+02:00 Ilia Mirkin :
> On Tue, Aug 11, 2015 at 1:48 PM, Thomas Helland
> wrote:
>> Signed-off-by: Thomas Helland
>> ---
>> This adds a section for the extensions nvidia has chosen to
>> call the "GL ARB 2015 Extensions" unveiled at SIGGRAPH.
>
> There are ARB extensions releas
On 07/08/15 23:13, Chris Wilson wrote:
Since we always flush the intel_batchbuffer before calling
intel_front_flush(), simply more that call into intel_front_flush()
more -> move. I will pick up the review tomorrow but I did not want to
forget about this typo.
itself.
Signed-off-by: Chris Wils
On 2015-08-12 17:14, Brian Paul wrote:
> Another question below...
>
> On 08/12/2015 08:31 AM, Daniel Scharrer wrote:
>> On 2015-08-06 15:46, Brian Paul wrote:
>>> On 08/06/2015 07:44 AM, Daniel Scharrer wrote:
CC: "10.6"
---
v2: added CC for 10.6
renamed _mesa
On Thursday 06 August 2015, Daniel Scharrer wrote:
> CC: "10.6"
I think the commit message should say which queries are added.
>
> ---
>
> v2: added CC for 10.6
> renamed _mesa_tex_target_to_index to tex_target_to_index
> moved declaration of variable before code
> added missing sp
On Tue, Aug 11, 2015 at 1:48 PM, Thomas Helland
wrote:
> Signed-off-by: Thomas Helland
> ---
> This adds a section for the extensions nvidia has chosen to
> call the "GL ARB 2015 Extensions" unveiled at SIGGRAPH.
There are ARB extensions released every year (or more often, not
sure)... we don't
2015-08-12 16:15 GMT+02:00 Ilia Mirkin :
> On Wed, Aug 12, 2015 at 9:07 AM, Thomas Helland
> wrote:
>> V2: -Change to "not started" for most entries
>> -Add status for multisample_2d_array
>> -Change shader_multisample_interpolation to "not_stared"
>>
>> Signed-off-by: Thomas Helland
>> R
>2015-08-12 15:59 GMT+02:00 Gustaw Smolarczyk :
> Hello. I've just spotted a simple typo.
>
> 2015-08-12 15:07 GMT+02:00 Thomas Helland :
>>
>> V2: -Change to "not started" for most entries
>> -Add status for multisample_2d_array
>> -Change shader_multisample_interpolation to "not_stared"
>
When a buffer is provided to eglGetConfigs it's supposed to set the value
of the num_config parameter to the total number of configs that have been
copied into this buffer. For some reason the EGL spec doesn't consider it
to be an error to pass this function a buffer while specifying its size to
be
The EGL 1.4 spec states for eglCreateContext:
"attribute EGL_CONTEXT_CLIENT_VERSION is only valid when the current
rendering API is EGL_OPENGL_ES_API"
Additionally, if the EGL_KHR_create_context EGL extension is supported
(this is mandatory in EGL 1.5) then the EGL_CONTEXT_MAJOR_
Reviewed-by: Jason Ekstrand
On Wed, Aug 12, 2015 at 8:22 AM, Oded Gabbay wrote:
> Because we build here an array format, we don't need to swap the
> bytes for big endian.
> If it isn't an array format, the bytes will be swapped in
> _mesa_format_convert.
>
> v2: remove temp variable
>
> Signed-o
Because we build here an array format, we don't need to swap the
bytes for big endian.
If it isn't an array format, the bytes will be swapped in
_mesa_format_convert.
v2: remove temp variable
Signed-off-by: Oded Gabbay
Cc: Jason Ekstrand
Cc: Iago Toral
Cc: "10.5 10.6"
---
src/mesa/main/glfor
Another question below...
On 08/12/2015 08:31 AM, Daniel Scharrer wrote:
On 2015-08-06 15:46, Brian Paul wrote:
On 08/06/2015 07:44 AM, Daniel Scharrer wrote:
CC: "10.6"
---
v2: added CC for 10.6
renamed _mesa_tex_target_to_index to tex_target_to_index
moved declaration of varia
Reviewed-by: Iago Toral Quiroga
El 2015-08-12 10:24, Tapani Pälli escribió:
Extension spec originally required 2^24 but 2^27 is the minimum value
required by OpenGL 4.5 and OpenGL ES 3.1 specifications.
Fixes:
ES31-CTS.shader_storage_buffer_object.basic-max
Signed-off-by: Tapani Pälli
---
On Aug 12, 2015 3:55 AM, "Oded Gabbay" wrote:
>
> Because we build here an array format, we don't need to swap the
> bytes for big endian.
> If it isn't an array format, the bytes will be swapped in
> _mesa_format_convert.
>
> Signed-off-by: Oded Gabbay
> Cc: Jason Ekstrand
> Cc: Iago Toral
> C
On 2015-08-06 15:46, Brian Paul wrote:
> On 08/06/2015 07:44 AM, Daniel Scharrer wrote:
>> CC: "10.6"
>>
>> ---
>>
>> v2: added CC for 10.6
>> renamed _mesa_tex_target_to_index to tex_target_to_index
>> moved declaration of variable before code
>> added missing spaces in ternary ope
On Wed, Aug 12, 2015 at 9:07 AM, Thomas Helland
wrote:
> V2: -Change to "not started" for most entries
> -Add status for multisample_2d_array
> -Change shader_multisample_interpolation to "not_stared"
>
> Signed-off-by: Thomas Helland
> Reviewed-by: Tapani Pälli
> ---
> docs/GL3.txt | 2
Hello. I've just spotted a simple typo.
2015-08-12 15:07 GMT+02:00 Thomas Helland :
>
> V2: -Change to "not started" for most entries
> -Add status for multisample_2d_array
> -Change shader_multisample_interpolation to "not_stared"
>
> Signed-off-by: Thomas Helland
> Reviewed-by: Tapani P
On Tue, Aug 11, 2015 at 7:02 PM, Marek Olšák wrote:
> From: Marek Olšák
>
> The value was copied from r300g, which uses 1/12 subpixels, but this hw
> uses 1/16 subpixels.
>
> Should fix piglit: gl-1.4-polygon-offset (formerly a glean test)
> (untested, ported from radeonsi)
For the series:
Rev
V2: -Change to "not started" for most entries
-Add status for multisample_2d_array
-Change shader_multisample_interpolation to "not_stared"
Signed-off-by: Thomas Helland
Reviewed-by: Tapani Pälli
---
docs/GL3.txt | 25 +
1 file changed, 25 insertions(+)
diff --g
2015-08-12 7:45 GMT+02:00 Tapani Pälli :
>
>
> On 08/11/2015 08:48 PM, Thomas Helland wrote:
>>
>> Signed-off-by: Thomas Helland
>> ---
>> These are listed to the best of my knowledge, looking
>> at the nvidia driver dropped yesterday, and a glance at the
>> "what changed according to the ARB vari
On Wed, Aug 12, 2015 at 1:55 PM, Oded Gabbay wrote:
> Because we build here an array format, we don't need to swap the
> bytes for big endian.
> If it isn't an array format, the bytes will be swapped in
> _mesa_format_convert.
>
> Signed-off-by: Oded Gabbay
> Cc: Jason Ekstrand
> Cc: Iago Toral
On Wed, 2015-08-12 at 13:47 +0300, Francisco Jerez wrote:
> Timothy Arceri writes:
>
> > On Mon, 2015-07-27 at 16:01 +0300, Francisco Jerez wrote:
> > > This fixes the spec@arb_shader_image_load_store@invalid index bounds
> > > piglit tests on IVB, which were causing a GPU hang and then a crash
>
Scanout buffers on nv30 must always be non-swizzled and have special
width alignment constraints.
These constrains have been taken from the xf86-video-nouveau
src/nv_accel_common.c: nouveau_allocate_surface() function.
nouveau_allocate_surface() applies these width constraints only when a
tiled a
v2: Add compute mode flag to sampler state setup (Marek).
Drop branches which avoid reference counting (Marek).
Simplify unset branch condition (Marek).
---
src/gallium/drivers/r600/evergreen_compute.c | 25 ---
src/gallium/drivers/r600/evergreen_state.c | 46
On 8/11/15, Marek Olšák wrote:
> Do compute sampler states need RADEON_CP_PACKET3_COMPUTE_MODE as well?
Oops, I've forgotten to add it. My mistake.
I'm not sure if needed (I have a working test which uses linear
filtering), but the SET_SAMPLER packet too has a header with the
graphics/compute fie
On 08/12/2015 03:00 PM, Ilia Mirkin wrote:
On Wed, Aug 12, 2015 at 2:34 AM, Tapani Pälli wrote:
Hi Ilia;
Some comments below;
On 08/12/2015 04:51 AM, Ilia Mirkin wrote:
---
src/glsl/builtin_functions.cpp | 32
+-
src/glsl/glcpp/glcpp-parse.y
On Wed, Aug 12, 2015 at 2:34 AM, Tapani Pälli wrote:
> Hi Ilia;
>
> Some comments below;
>
>
> On 08/12/2015 04:51 AM, Ilia Mirkin wrote:
>>
>> ---
>> src/glsl/builtin_functions.cpp | 32
>> +-
>> src/glsl/glcpp/glcpp-parse.y | 3 +++
>> s
Francisco Jerez writes:
> Neil Roberts writes:
>
>> Ok, that makes sense, thanks for the detailed explanation. I think
>> you're right, given the extra restrictions avoiding the integer dword
>> multiplication seems like the simplest solution.
>>
>> I can't say that I fully understand what the d
Neil Roberts writes:
> Ok, that makes sense, thanks for the detailed explanation. I think
> you're right, given the extra restrictions avoiding the integer dword
> multiplication seems like the simplest solution.
>
> I can't say that I fully understand what the documentation is trying to
> say he
The code is heavily inspired from Francisco Jerez's code supporting the
image_load_store extension.
I moved out image_types[] out of builtin_builder::add_image_function to
share it with the newly-added builtin_builder::add_image_size_functions.
Backends willing to support this builtin should hand
Because we build here an array format, we don't need to swap the
bytes for big endian.
If it isn't an array format, the bytes will be swapped in
_mesa_format_convert.
Signed-off-by: Oded Gabbay
Cc: Jason Ekstrand
Cc: Iago Toral
Cc: "10.5 10.6"
---
src/mesa/main/glformats.c | 3 ---
1 file cha
> -Original Message-
> From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On
> Behalf Of Francisco Jerez
> Sent: Wednesday, August 12, 2015 12:45 PM
> To: Marta Lofstedt; mesa-dev@lists.freedesktop.org
> Subject: Re: [Mesa-dev] [PATCH] mesa/main: Add
> GL_IMAGE_FORMAT_COMPATIBIL
Timothy Arceri writes:
> On Mon, 2015-07-27 at 16:01 +0300, Francisco Jerez wrote:
>> This fixes the spec@arb_shader_image_load_store@invalid index bounds
>> piglit tests on IVB, which were causing a GPU hang and then a crash
>> due to the invalid binding table index result of the array index
>>
Marta Lofstedt writes:
> From: Marta Lofstedt
>
> According to Open GL ES 3.1 specification, section 8.10.2.
> GL_IMAGE_FORMAT_COMPATIBILITY_TYPE should be supported by
> glGetTexParameterfv.
>
> Signed-off-by: Marta Lofstedt
> ---
> src/mesa/main/texparam.c | 6 ++
> 1 file changed, 6 ins
From: Marek Olšák
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=74329
v2: add a CAP for half floats
drivers should not expose the CAPs if they don't support the formats
---
src/gallium/docs/source/screen.rst | 6 ++
src/gallium/drivers/freedreno/freedreno_screen.c
On Wed, Aug 12, 2015 at 11:30 AM, Michel Dänzer wrote:
> On 11.08.2015 01:56, Marek Olšák wrote:
>> Reviewed-by: Marek Olšák
>
> Thanks Marek. Unfortunately, I remembered in the meantime that this
> patch alone won't help for r300g, because it doesn't support
> PIPE_FORMAT_x8R8G8B8_UNORM. It shou
From: Marta Lofstedt
According to Open GL ES 3.1 specification, section 8.10.2.
GL_IMAGE_FORMAT_COMPATIBILITY_TYPE should be supported by
glGetTexParameterfv.
Signed-off-by: Marta Lofstedt
---
src/mesa/main/texparam.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/mesa/main/texp
Cc: Francisco Jerez
---
This patch works for direct indexing of images, but I'm having some trouble
getting indirect to work.
For example for:
tests/spec/arb_arrays_of_arrays/execution/image_store/basic-imageStore
-non-const-uniform-index.shader_test
Which has and image writeonly uniform i
Cc: Francisco Jerez
---
src/glsl/link_uniforms.cpp | 77 +-
1 file changed, 49 insertions(+), 28 deletions(-)
diff --git a/src/glsl/link_uniforms.cpp b/src/glsl/link_uniforms.cpp
index 4757fd5..f8a6620 100644
--- a/src/glsl/link_uniforms.cpp
+++ b/src/
On 11.08.2015 01:56, Marek Olšák wrote:
> Reviewed-by: Marek Olšák
Thanks Marek. Unfortunately, I remembered in the meantime that this
patch alone won't help for r300g, because it doesn't support
PIPE_FORMAT_x8R8G8B8_UNORM. It should help at least for r600g though,
not sure about nouveau.
--
E
Reviewed-by: Timothy Arceri
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
On 12.08.2015 08:02, Marek Olšák wrote:
> From: Marek Olšák
The series is
Reviewed-by: Michel Dänzer
--
Earthling Michel Dänzer | http://www.amd.com
Libre software enthusiast | Mesa and X developer
__
Extension spec originally required 2^24 but 2^27 is the minimum value
required by OpenGL 4.5 and OpenGL ES 3.1 specifications.
Fixes:
ES31-CTS.shader_storage_buffer_object.basic-max
Signed-off-by: Tapani Pälli
---
src/mesa/main/context.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
---
src/glsl/linker.cpp | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/glsl/linker.cpp b/src/glsl/linker.cpp
index d7efea5..e18754a 100644
--- a/src/glsl/linker.cpp
+++ b/src/glsl/linker.cpp
@@ -2880,8 +2880,9 @@ check_image_resources(struct gl_context *ctx, struct
g
88 matches
Mail list logo
