Matt Turner writes:
> Optimizes
>
> cmp.ge.f0(8) null g45<8,8,1>F 0F
> (+f0) sel(8)g50<1>F g40<8,8,1>F g10<8,8,1>F
> cmp.ge.f0(8) null g45<8,8,1>F 0F
> (+f0) sel(8)g51<1>F g41<8,8,1>F g11<8,8,1>F
> cmp.ge.f0(8) null g45<8,8,1>F 0F
> (+f0) se
Matt Turner writes:
> Since compare instructions write the flag register, they should not be
> considered dead even if their destination is never read. Instead of
> removing them if found to be dead, set their destination to null to free
> a register.
>
> ---
> src/mesa/drivers/dri/i965/brw_fs.c
Matt Turner writes:
> We'd like to CSE some instructions, like CMP, that often have null
> destinations. Instead of replacing them with MOVs to null, just don't
> emit the MOV.
> ---
> src/mesa/drivers/dri/i965/brw_fs_cse.cpp | 42
> +++-
> 1 file changed, 25 inserti
The idea of the original order was that you'd dead code eliminate accesses
to push constants. But I've never seen a case of that (nor has
shader-db), while we frequently see sparse accesses of large constant
arrays that would overflow into pull constants.
Cuts pull constant use on csgo, serious s
On 10/28/2013 11:02 AM, Carl Worth wrote:
[snip]
Would you mind adding the "mesa: " prefix to the commit title?
> @@ -241,18 +234,20 @@ _mesa_DeleteQueries(GLsizei n, const GLuint *ids)
>return;
> }
>
> - /* No query objects can be active at this time! */
> - if (ctx->Query.Curr
There had been some discussion at XDC about Mesa 10.0 release plans, but
I don't think anyone committed to anything. The plan I pitched was to
create the 10.0 branch on 10/31 (feature freeze) and release on 11/27.
There are a lot of patches on the mailing list in need of review
(Curro's ARB_shade
On 28 October 2013 19:50, Anuj Phogat wrote:
>
>
>
> On Mon, Oct 28, 2013 at 6:14 PM, Paul Berry wrote:
>
>> On 25 October 2013 16:45, Anuj Phogat wrote:
>>
>>> - Enable GEN6_WM_MSDISPMODE_PERSAMPLE, GEN6_WM_POSOFFSET_SAMPLE,
>>> GEN6_WM_OMASK_TO_RENDER_TARGET as per extension's specification.
On Mon, Oct 28, 2013 at 6:14 PM, Paul Berry wrote:
> On 25 October 2013 16:45, Anuj Phogat wrote:
>
>> - Enable GEN6_WM_MSDISPMODE_PERSAMPLE, GEN6_WM_POSOFFSET_SAMPLE,
>> GEN6_WM_OMASK_TO_RENDER_TARGET as per extension's specification.
>> - Only enable one of GEN6_WM_8_DISPATCH_ENABLE or
>> GE
On 28 October 2013 18:14, Paul Berry wrote:
> On 25 October 2013 16:45, Anuj Phogat wrote:
>
>> - Enable GEN6_WM_MSDISPMODE_PERSAMPLE, GEN6_WM_POSOFFSET_SAMPLE,
>> GEN6_WM_OMASK_TO_RENDER_TARGET as per extension's specification.
>> - Only enable one of GEN6_WM_8_DISPATCH_ENABLE or
>> GEN6_WM_1
On 25 October 2013 16:44, Anuj Phogat wrote:
> Patches listed below implement the GL_ARB_sample_shading extension
> on Intel hardware >= gen6. I verified the implementation with a
> number of piglit tests, currently under review on piglit mailing
> list. Observed no piglit, gles3 CTS regressions
On 25 October 2013 16:45, Anuj Phogat wrote:
> - Enable GEN7_WM_MSDISPMODE_PERSAMPLE, GEN7_WM_POSOFFSET_SAMPLE,
> GEN7_WM_OMASK_TO_RENDER_TARGET as per extension's specification.
> - Only enable one of GEN7_WM_8_DISPATCH_ENABLE or
> GEN7_WM_16_DISPATCH_ENABLE
> when GEN7_WM_MSDISPMODE_PERSAMP
On 25 October 2013 16:45, Anuj Phogat wrote:
> - Enable GEN6_WM_MSDISPMODE_PERSAMPLE, GEN6_WM_POSOFFSET_SAMPLE,
> GEN6_WM_OMASK_TO_RENDER_TARGET as per extension's specification.
> - Only enable one of GEN6_WM_8_DISPATCH_ENABLE or
> GEN6_WM_16_DISPATCH_ENABLE
> when GEN6_WM_MSDISPMODE_PERSAMP
Before we were only checking the st->vertex_array_out_of_memory flag
after updating array state. But if there's two consecutive glDrawArrays
calls and the first one is skipped because of OOM, the second one should
be skipped too.
Cc: 9.2
---
src/mesa/state_tracker/st_draw.c |7 ---
1 fi
Kenneth Graunke writes:
> This adds the basic driver hooks to allocate/free the brw variant.
> It doesn't contain any additional information yet, but it will soon.
>
> v2: Use the new _mesa_init_transform_feedback_object helper function
> (requested by Eric and Ian).
Reviewed-by: Eric Anholt
On Mon, Oct 28, 2013 at 3:23 PM, Paul Berry wrote:
> On 25 October 2013 16:45, Anuj Phogat wrote:
>
>> V2:
>>- Update comments
>>- Make changes to support simd16 mode.
>>- Add compute_sample_id variables in brw_wm_prog_key
>>- Add a special backend instruction to compute sample_i
Chris Forbes writes:
> Signed-off-by: Chris Forbes
This function should probably have a comment:
/**
* Alpha test support for when we compile it into the shader instead
* of using the normal fixed function alpha test.
*/
> +void
> +fs_visitor::emit_alpha_test()
> +{
> + this->current_ann
On 25 October 2013 16:45, Anuj Phogat wrote:
> V2:
>- Update comments
>- Make changes to support simd16 mode.
>- Add compute_sample_id variables in brw_wm_prog_key
>- Add a special backend instruction to compute sample_id.
>
> Signed-off-by: Anuj Phogat
> ---
> src/mesa/drivers/
Check if the new buffer object has the same name as the current
buffer object before looking it up.
---
src/mesa/main/varray.c |7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/mesa/main/varray.c b/src/mesa/main/varray.c
index 5b5b8b2..93581ea 100644
--- a/src/mesa/m
---
src/mesa/main/get.c | 24
src/mesa/main/get_hash_params.py |3 +++
src/mesa/main/varray.c |9 +
3 files changed, 36 insertions(+)
diff --git a/src/mesa/main/get.c b/src/mesa/main/get.c
index b39fdfb..caa163e 100644
--- a/src/me
* Instead of assuming the displaytarget is the same
stride / colorspace as the destination, lets
actually check the source bitmap.
* Fixes random stride issues in rendering
---
src/gallium/winsys/sw/hgl/bitmap_wrapper.cpp | 20
src/gallium/winsys/sw/hgl/bitmap_wrapper.h
Kenneth Graunke writes:
> With Linux 3.12, register writes work on Ivybridge and Baytrail, but not
> Haswell. That will be fixed in a future kernel revision, at which point
> this extension will automatically be enabled.
>
> Signed-off-by: Kenneth Graunke
> ---
> src/mesa/drivers/dri/i965/inte
On 25 October 2013 16:45, Anuj Phogat wrote:
> Thsi function is used to test if we need to do per sample shading or
> per fragment shading.
>
s/Thsi/This/
>
> Signed-off-by: Anuj Phogat
> ---
> src/mesa/program/program.c | 31 +++
> src/mesa/program/program.h | 3
This will become derived state as part of the ARB_vertex_attrib_binding
support.
---
src/mesa/main/api_arrayelt.c | 36 ++---
src/mesa/main/api_validate.c |6 ++--
src/mesa/main/arrayobj.c | 26 +++
src/mesa/main/attrib.c |4 +--
sr
update_array() and update_array_format() are changed to update the new
attrib and binding states, and the client arrays become derived state.
---
src/mesa/main/api_arrayelt.c |7 +
src/mesa/main/arrayobj.c | 63 --
src/mesa/main/arrayobj.h |5 +
src/mesa/main/attrib.c |
Split out the code for updating the array format into a new function
called update_array_format(). This function will be called by both
update_array() and the new glVertexAttrib*Format() entry points in
ARB_vertex_attrib_binding.
---
src/mesa/main/varray.c | 144 ++
* Use more consistant data sources
* Fix improper color space assignments
* Remove unnecessary comments and code
* Drop unnecessary round_up function (this was leftover
from moving winsys code out of renderer)
---
.../targets/haiku-softpipe/GalliumContext.cpp | 27
..
---
src/mesa/main/enable.c | 36 ++---
src/mesa/main/get.c |8 ++---
src/mesa/main/get_hash_params.py | 66 +++---
src/mesa/main/getstring.c| 18 +--
src/mesa/main/varray.c | 12 +++
5
On 25 October 2013 16:45, Anuj Phogat wrote:
> V2:
>- Update comments
>- Use fs_reg(0x) in AND instruction to get the 16 bit
> sample_mask.
>- Add a special backend instructions to compute sample_mask.
>- Add a new variable uses_omask in brw_wm_prog_data.
>
> Signed-off-b
---
src/mesa/main/varray.h |6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/mesa/main/varray.h b/src/mesa/main/varray.h
index a75cb7d..8a9487c 100644
--- a/src/mesa/main/varray.h
+++ b/src/mesa/main/varray.h
@@ -51,8 +51,10 @@ _mesa_update_array_max_element(struct g
On 10/28/2013 04:52 AM, Pohjolainen, Topi wrote:
> On Mon, Oct 28, 2013 at 01:14:55PM +0200, Pohjolainen, Topi wrote:
>> On Fri, Oct 25, 2013 at 10:35:48PM -0700, Kenneth Graunke wrote:
>>> Implementing the GetTransformFeedbackVertexCount() driver hook allows
>>> the VBO module to call us with the
Paul Berry writes:
> On 18 October 2013 15:44, Eric Anholt wrote:
>
>> This only operates on constant/uniform values for now, because otherwise
>> I'd
>> have to deal with killing my available CSE entries when assignments happen,
>> and getting even this working in the tree ir was painful enough
This is a step in doing scheduling as described in Muchnick (p538). A
difference is that our latency function is only specific to one
instruction (it doesn't describe, for example, the different latency
between WAR of a send's arguments and RAW of a send's destination), but
that's changeable later
On 25 October 2013 16:44, Anuj Phogat wrote:
> New functions added by GL_ARB_sample_shading:
> glMinSampleShadingARB()
>
> New enums:
> GL_SAMPLE_SHADING_ARB
> GL_MIN_SAMPLE_SHADING_VALUE_ARB
>
> V2: Update comments.
> Create new GL4x.xml.
> Remove redundant code in get.c.
> Update th
---
src/mesa/main/api_validate.c |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/mesa/main/api_validate.c b/src/mesa/main/api_validate.c
index d31f593..f285c97 100644
--- a/src/mesa/main/api_validate.c
+++ b/src/mesa/main/api_validate.c
@@ -120,7 +120,7 @@ check_va
On 25 October 2013 16:45, Anuj Phogat wrote:
> V2:
>- Update comments.
>- Make changes to support simd16 mode.
>- Add compute_pos_offset variable in brw_wm_prog_key.
>- Add variable uses_omask in brw_wm_prog_data.
>
> Signed-off-by: Anuj Phogat
> ---
> src/mesa/drivers/dri/i965/
---
src/mapi/glapi/gen/ARB_vertex_attrib_binding.xml | 58 ++
src/mapi/glapi/gen/Makefile.am |1 +
src/mapi/glapi/gen/gl_API.xml|6 ++-
src/mesa/main/tests/dispatch_sanity.cpp | 12 ++---
src/mesa/main/varray.c
On 10/28/2013 10:54 AM, Anuj Phogat wrote:
> On Mon, Oct 28, 2013 at 10:29 AM, Ian Romanick wrote:
>> On 10/25/2013 04:44 PM, Anuj Phogat wrote:
>>> New functions added by GL_ARB_sample_shading:
>>> glMinSampleShadingARB()
>>>
>>> New enums:
>>> GL_SAMPLE_SHADING_ARB
>>> GL_MIN_SAMPLE_SHADING_VALU
Chris Forbes writes:
> Signed-off-by: Chris Forbes
> ---
> src/mesa/drivers/dri/i965/brw_wm.c | 12
> src/mesa/drivers/dri/i965/brw_wm.h | 2 ++
> 2 files changed, 14 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_wm.c
> b/src/mesa/drivers/dri/i965/brw_wm.c
> index
This extension decouples the vertex attributes from the vertex buffer
bindings, making it possible to change buffers without having to
respecify the formats. The mapping between attributes and vertex
buffer bindings also becomes configurable.
This series adds new state structs and changes the gl*
When faced with a million instructions that all became candidates at the
same time (none of which individually reduce register pressure), the ones
on the critical path are more likely to be the ones that will free up some
candidates soon.
shader-db:
total instructions in shared programs: 1681070 -
---
src/mesa/main/ffvertex_prog.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/main/ffvertex_prog.c b/src/mesa/main/ffvertex_prog.c
index 08665c6..be6ac0f 100644
--- a/src/mesa/main/ffvertex_prog.c
+++ b/src/mesa/main/ffvertex_prog.c
@@ -228,7 +228,7 @@ static vo
On 10/26/2013 12:36 AM, Vinson Lee wrote:
> logic_op is of type GLenum (unsigned int).
>
> Fixes "Macro compares unsigned to 0" defect reported by Coverity.
>
> Signed-off-by: Vinson Lee
> ---
> src/mesa/drivers/dri/i915/intel_blit.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>
On 28 October 2013 16:26, Anuj Phogat wrote:
>
>
>
> On Mon, Oct 28, 2013 at 3:23 PM, Paul Berry wrote:
>
>> On 25 October 2013 16:45, Anuj Phogat wrote:
>>
>>> V2:
>>>- Update comments
>>>- Make changes to support simd16 mode.
>>>- Add compute_sample_id variables in brw_wm_prog_key
On 28/10/13 08:07, Adrian Negreanu wrote:
> From: Adrian Negreanu
>
> Fixes linker error:
> ld: .../libmesa_dri_common_intermediates/libmesa_dri_common.a(dri_util.o):
> in function globalDriverAPI:dri_util.c(.data.rel+0x0): error: undefined
> reference to 'driDriverAPI'
>
Hi Adrian,
To be
On 10/28/2013 12:02 PM, Carl Worth wrote:
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA256
There is nothing in the OpenGL specification which prevents the user from
calling glGenQueries to generate a new query object while another object is
active. Neither is there anything in the Mesa implementa
Carl Worth writes:
> Eric Anholt writes:
>> Fixes the new piglit GL_ARB_texture_buffer_object/data-sync
>>
>> Cc: "9.2"
>
> Hi Eric,
>
> I recently pushed this out to the 9.2 branch. But I also noticed that my
> testing didn't include the new test mentioned above.
>
> Do you expect to be able t
Matt Turner writes:
> On Fri, Oct 25, 2013 at 2:49 PM, Eric Anholt wrote:
>> I noticed this in a shader in Unigine Heaven that was spilling. While it
>> doesn't really reduce register pressure, it shaves a few instructions
>> anyway (7955 -> 7882).
>>
>> v2: Fix turning "0 >> x" into "x" instea
On 10/28/2013 04:14 AM, Pohjolainen, Topi wrote:
> On Fri, Oct 25, 2013 at 10:35:48PM -0700, Kenneth Graunke wrote:
>> Implementing the GetTransformFeedbackVertexCount() driver hook allows
>> the VBO module to call us with the right number of vertices.
>>
>> The hardware doesn't directly count the
This will be used by the ARB_vertex_attrib_binding implementation.
This reverts commit db38e9a0e179441f59274f6f2a751912c29872e2.
---
src/mesa/main/enable.c |2 ++
src/mesa/main/mtypes.h |3 +++
src/mesa/main/state.c |1 +
src/mesa/main/varray.c |4
4 files changed, 10 inserti
On 10/28/2013 10:53 AM, Ian Romanick wrote:
> On 10/26/2013 01:33 PM, Kenneth Graunke wrote:
>> Currently, we emit one SO_DECL structure per output, so we use the index
>> in the Outputs[] array as the index into the so_decl[] array as well.
>>
>> In order to support the fake "gl_SkipComponents[123
On Tue, Oct 29, 2013 at 10:48 AM, Eric Anholt wrote:
> Chris Forbes writes:
>
>> Signed-off-by: Chris Forbes
>
> This function should probably have a comment:
>
> /**
> * Alpha test support for when we compile it into the shader instead
> * of using the normal fixed function alpha test.
> */
On 10/28/2013 10:57 AM, Ian Romanick wrote:
> On 10/26/2013 01:33 PM, Kenneth Graunke wrote:
>> This depends on ARB_transform_feedback2, so I've predicated it on the
>> ability to do register writes.
>>
>> It also depends on ARB_transform_feedback3, which is the only reason we
>> couldn't expose it
On 10/28/2013 03:08 PM, Eric Anholt wrote:
> Kenneth Graunke writes:
>
>> With Linux 3.12, register writes work on Ivybridge and Baytrail, but not
>> Haswell. That will be fixed in a future kernel revision, at which point
>> this extension will automatically be enabled.
>>
>> Signed-off-by: Kenn
This only operates on constant/uniform values for now, because otherwise I'd
have to deal with killing my available CSE entries when assignments happen,
and getting even this working in the tree ir was painful enough.
As is, it has the following effect in shader-db:
total instructions in shared p
The first six patches have been on the list previously, but it got
confusing when they changed order and some were dropped.
The last nine patches implement a new SEL-generating peephole, a dead
flow control elimination pass, and some silly algebraic optimizations
that surprisingly are seen in real
Only one program's instruction count is changed, but a shader in Tropics
is also affected.
instructions in affected programs: 162 -> 159 (-1.85%)
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 14 ++
1 file changed, 14 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
total instructions in shared programs: 1287130 -> 1285860 (-0.10%)
instructions in affected programs: 93529 -> 92259 (-1.36%)
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 22 ++
1 file changed, 22 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
b/src/mesa/dr
Helps a lot of Steam games.
total instructions in shared programs: 1287432 -> 1287130 (-0.02%)
instructions in affected programs: 21447 -> 21145 (-1.41%)
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 8
1 file changed, 8 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
total instructions in shared programs: 1287488 -> 1287457 (-0.00%)
instructions in affected programs: 1745 -> 1714 (-1.78%)
---
src/mesa/drivers/dri/i965/brw_fs_sel_peephole.cpp | 38 +--
1 file changed, 21 insertions(+), 17 deletions(-)
diff --git a/src/mesa/drivers/dri/i
The pass previously searched only backward from an ENDIF (and ELSE) to
find MOVs with the same destination. This commit extends the pass to
also search forward from the IF (and ELSE) to find matching MOVs which
it can replace with SEL instructions before the IF.
E.g., the pass can now optimize
Removes if/endif and if/else/endif.
total instructions in shared programs: 1293990 -> 1288172 (-0.45%)
instructions in affected programs: 95390 -> 89572 (-6.10%)
---
src/mesa/drivers/dri/i965/Makefile.sources | 1 +
src/mesa/drivers/dri/i965/brw_fs.cpp | 1 +
src/mesa/
fs_visitor::try_replace_with_sel optimizes only if statements whose
"then" and "else" bodies contain a single MOV instruction. It also did
could not handle constant arguments, since they cause an extra MOV
immediate to be generated (since we haven't run constant propagation,
there are more than the
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 30 ++
src/mesa/drivers/dri/i965/brw_fs.h | 1 +
2 files changed, 31 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
b/src/mesa/drivers/dri/i965/brw_fs.cpp
index b2eac6c..28d369a 100644
--- a/src/mesa/drive
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 1 +
src/mesa/drivers/dri/i965/brw_fs.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
b/src/mesa/drivers/dri/i965/brw_fs.cpp
index b985251..b2eac6c 100644
--- a/src/mesa/drivers/dri/i965/brw_fs.cpp
+++ b/src
Optimizes
cmp.ge.f0(8) null g45<8,8,1>F 0F
(+f0) sel(8)g50<1>F g40<8,8,1>F g10<8,8,1>F
cmp.ge.f0(8) null g45<8,8,1>F 0F
(+f0) sel(8)g51<1>F g41<8,8,1>F g11<8,8,1>F
cmp.ge.f0(8) null g45<8,8,1>F 0F
(+f0) sel(8)g52<1>F g42<8,8,1>F g12
Since compare instructions write the flag register, they should not be
considered dead even if their destination is never read. Instead of
removing them if found to be dead, set their destination to null to free
a register.
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 6 --
1 file changed, 4 ins
We'd like to CSE some instructions, like CMP, that often have null
destinations. Instead of replacing them with MOVs to null, just don't
emit the MOV.
---
src/mesa/drivers/dri/i965/brw_fs_cse.cpp | 42 +++-
1 file changed, 25 insertions(+), 17 deletions(-)
diff --git a
---
src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp | 16
1 file changed, 4 insertions(+), 12 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp
b/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp
index 84b74ff..9cc752e 100644
--- a/s
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 13 +
src/mesa/drivers/dri/i965/brw_fs.h | 3 +++
2 files changed, 16 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
b/src/mesa/drivers/dri/i965/brw_fs.cpp
index be301e2..7e7ced0 100644
--- a/src/mesa/drivers/dri/i965/brw
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 8
src/mesa/drivers/dri/i965/brw_fs.h | 1 +
2 files changed, 9 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
b/src/mesa/drivers/dri/i965/brw_fs.cpp
index a3268fb..be301e2 100644
--- a/src/mesa/drivers/dri/i965/brw_fs.cpp
++
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA256
There is nothing in the OpenGL specification which prevents the user from
calling glGenQueries to generate a new query object while another object is
active. Neither is there anything in the Mesa implementation which prevents
this. So remove the INVA
On 10/28/2013 10:57 AM, Kenneth Graunke wrote:
> On 10/28/2013 10:50 AM, Ian Romanick wrote:
>> On 10/26/2013 01:33 PM, Kenneth Graunke wrote:
>>> This extension is written a bit strangely. Although it introduces the
>>> concept of multiple transform feedback streams, it doesn't actually
>>> provi
On 10/26/2013 01:33 PM, Kenneth Graunke wrote:
> This depends on ARB_transform_feedback2, so I've predicated it on the
> ability to do register writes.
>
> It also depends on ARB_transform_feedback3, which is the only reason we
> couldn't expose it previously.
Dare I even ask if we have tests?
>
On 10/28/2013 10:50 AM, Ian Romanick wrote:
> On 10/26/2013 01:33 PM, Kenneth Graunke wrote:
>> This extension is written a bit strangely. Although it introduces the
>> concept of multiple transform feedback streams, it doesn't actually
>> provide more than a single stream.
>>
>> The ARB_gpu_shade
On Mon, Oct 28, 2013 at 10:29 AM, Ian Romanick wrote:
> On 10/25/2013 04:44 PM, Anuj Phogat wrote:
>> New functions added by GL_ARB_sample_shading:
>> glMinSampleShadingARB()
>>
>> New enums:
>> GL_SAMPLE_SHADING_ARB
>> GL_MIN_SAMPLE_SHADING_VALUE_ARB
>>
>> V2: Update comments.
>> Create new G
On 10/26/2013 01:33 PM, Kenneth Graunke wrote:
> Currently, we emit one SO_DECL structure per output, so we use the index
> in the Outputs[] array as the index into the so_decl[] array as well.
>
> In order to support the fake "gl_SkipComponents[1234]" varyings from
> ARB_transform_feedback3, we'l
On 10/26/2013 01:33 PM, Kenneth Graunke wrote:
> This is a bit shorter.
>
> Signed-off-by: Kenneth Graunke
With the one change suggested below,
Reviewed-by: Ian Romanick
> ---
> src/mesa/drivers/dri/i965/gen7_sol_state.c | 8
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> di
On 10/26/2013 01:33 PM, Kenneth Graunke wrote:
> ARB_transform_feedback3 allows applications to insert blank space
> between interleaved varyings by adding fake 1, 2, 3, or 4-component
> varyings named gl_SkipComponents[1234].
>
> Mesa's core data structures don't explicitly track these, instead s
On 10/26/2013 01:33 PM, Kenneth Graunke wrote:
> This extension is written a bit strangely. Although it introduces the
> concept of multiple transform feedback streams, it doesn't actually
> provide more than a single stream.
>
> The ARB_gpu_shader5 extension is what introduces the ability to wri
On 10/27/2013 07:24 AM, Daniel Vetter wrote:
> On Sat, Oct 26, 2013 at 01:07:48PM -0700, Kenneth Graunke wrote:
>> These patches add some missing flushing, which appears to help. I'm
>> still getting GPU hangs, but they're much less frequent, and now have
>> an IPEHR of MI_SEMAPHORE_MBOX. I suspe
On 10/25/2013 04:45 PM, Anuj Phogat wrote:
> New builtins added by GL_ARB_sample_shading:
> in vec2 gl_SamplePosition
> in int gl_SampleID
> in int gl_NumSamples
> out int gl_SampleMask[]
>
> V2: - Use SWIZZLE_ for STATE_NUM_SAMPLES.
> - Use "result.samplemask" in arb_output_attrib_string.
On 10/25/2013 04:45 PM, Anuj Phogat wrote:
> Thsi function is used to test if we need to do per sample shading or
> per fragment shading.
>
> Signed-off-by: Anuj Phogat
With the one change mentioned below, this patch is
Reviewed-by: Ian Romanick
> ---
> src/mesa/program/program.c | 31 ++
On 10/25/2013 04:44 PM, Anuj Phogat wrote:
> New functions added by GL_ARB_sample_shading:
> glMinSampleShadingARB()
>
> New enums:
> GL_SAMPLE_SHADING_ARB
> GL_MIN_SAMPLE_SHADING_VALUE_ARB
>
> V2: Update comments.
> Create new GL4x.xml.
> Remove redundant code in get.c.
> Update the
Anuj Phogat writes:
> Setting bilinear_filter flag in case of multisample blits with
> GL_LINEAR filter causes incorrect behavior in translate_dst_to_src()
> function. This broke Modern Warfare (1, 2 and 3) on SNB, IVB and HSW.
...
> Cc: mesa-sta...@lists.freedesktop.org
Hi Anuj,
From my reading
I just ran a quick.tests run on evergreen without any regressions.
Patch looks good to me, and doesn't seem to cause any regressions on
the hardware I have available to test with.
--Aaron
On Tue, Oct 22, 2013 at 11:28 AM, Tom Stellard wrote:
> From: Tom Stellard
>
> Without DataLayout, a lot o
Eric Anholt writes:
> Fixes the new piglit GL_ARB_texture_buffer_object/data-sync
>
> Cc: "9.2"
Hi Eric,
I recently pushed this out to the 9.2 branch. But I also noticed that my
testing didn't include the new test mentioned above.
Do you expect to be able to push the new test to piglit soon? I
On Sun, Oct 27, 2013 at 7:42 AM, Paul Berry wrote:
> On 18 October 2013 17:24, Anuj Phogat wrote:
>>
>> Setting bilinear_filter flag in case of multisample blits with
>> GL_LINEAR filter causes incorrect behavior in translate_dst_to_src()
>> function. This broke Modern Warfare (1, 2 and 3) on SNB
Carl Worth writes:
> If someone could rebase this against 9.2 and email the result to
> mesa-stable, I would be more than happy to merge it.
Last ping from me on this one. If a new patch appears in the next day or
two, this could get in to 9.2.3
Otherwise, not having heard anything, I'll be drop
On 10/25/2013 01:53 PM, Rob Clark wrote:
From: Rob Clark
Add 'start' parameter to generator/translator.
Signed-off-by: Rob Clark
---
src/gallium/auxiliary/indices/u_indices.c | 6 --
src/gallium/auxiliary/indices/u_indices.h | 4 +++-
src/gallium/auxiliary/indices/
Just a bunch of minor nits...
On 10/25/2013 01:53 PM, Rob Clark wrote:
From: Rob Clark
A convenient front end to indices generate/translate code, for emulating
primitives which are not supported natively by the driver.
This handles saving/restoring index buffer state, etc.
Signed-off-by: Rob
- Original Message -
> From: Roland Scheidegger
>
> The layer coming from GS needs to be clamped (not sure if that's actually
> the correct error behavior but we need something) as the number can be higher
> than the amount of layers in the fb. However, this code was using the layer
> c
On 10/27/2013 08:02 AM, Rico Schüller wrote:
Signed-off-by: Rico Schüller
---
src/mesa/main/texparam.c | 12 ++--
1 Datei geändert, 6 Zeilen hinzugefügt(+), 6 Zeilen entfernt(-)
Pushed. Thanks.
-Brian
___
mesa-dev mailing list
mesa-dev@l
https://bugs.freedesktop.org/show_bug.cgi?id=70932
Brian Paul changed:
What|Removed |Added
Assignee|mesa-dev@lists.freedesktop. |sitewranglers@lists.freedes
I ran this through a piglit CL test run on my 7850, no test fixes or
regressions.
--Aaron
On Tue, Oct 22, 2013 at 11:28 AM, Tom Stellard wrote:
> From: Tom Stellard
>
> Without DataLayout, a lot of optimization passes aren't run and the ones
> that are don't work as well.
> ---
> src/gallium/d
On 23 October 2013 03:31, Timothy Arceri wrote:
> The main purpose of this patch is to increase readability of
> the array code by introducing is_unsized_array() to glsl_types.
> Some redundent is_array() checks are also removed, and small number
> of other related clean ups.
>
> The introduction
Tested-by: Andreas Boll
The test passes now, no piglit regressions on rv770
Thanks,
Andreas.
2013/10/27 Vadim Girlin :
> ---
> cc: Andreas Boll
>
> Andreas, this patch should fix the issue with SB on RV770 that you
> reported on IRC (assert with interpolation-mixed.shader_test).
>
> There are
On Mon, Oct 28, 2013 at 01:14:55PM +0200, Pohjolainen, Topi wrote:
> On Fri, Oct 25, 2013 at 10:35:48PM -0700, Kenneth Graunke wrote:
> > Implementing the GetTransformFeedbackVertexCount() driver hook allows
> > the VBO module to call us with the right number of vertices.
> >
> > The hardware does
On Fri, Oct 25, 2013 at 10:35:48PM -0700, Kenneth Graunke wrote:
> Implementing the GetTransformFeedbackVertexCount() driver hook allows
> the VBO module to call us with the right number of vertices.
>
> The hardware doesn't directly count the number of vertices written by
> SOL, so we instead use
On 10/26/2013 07:42 AM, Matt Turner wrote:
On Thu, Oct 24, 2013 at 1:28 AM, Tapani Pälli wrote:
Hello;
These patches introduce GL_OES_get_program_binary extension support for Mesa.
There are already stub functions for this extension, patches add the missing
functionality part. This is based on
From: Adrian Negreanu
Fixes linker error:
ld: .../libmesa_dri_common_intermediates/libmesa_dri_common.a(dri_util.o):
in function globalDriverAPI:dri_util.c(.data.rel+0x0): error: undefined
reference to 'driDriverAPI'
Signed-off-by: Adrian Negreanu
---
src/mesa/drivers/dri/Android.mk
100 matches
Mail list logo
