Ian,
I am sure I comes across trouble for the following code.
(function main
(signature void
(parameters
)
(
(loop ((declare () int i@0x8d19434)) ((constant int (0)) )
((constant int (32)) ) ((constant int (1)) ) (
(call foo ((var_ref sampler2d@0x8eef134) (var_ref
m
>
> Do either of you guys plan to implement support for this extension? The
> value to developers is obviously increased if more drivers support the
> extension. This extension was born from feedback that I received from
> people at FOSDEM and from various game developers at Game Developer
> Conf
This patch should work when checking than no OQAP is used before beeing queued,
assuming that a value in OQAP is consumed
and cannot be read twice. However I'm not sure I cover all LDS instructions
that queues a value, I only use LDS_RET_READ in switch case.
Vincent
- Mail original -
On 23 October 2013 10:51, Eric Anholt wrote:
> Paul Berry writes:
>
> > When a geometry shader is present, the fragment shader gl_PrimitiveID
> > input acts like an ordinary varying, receiving data from the gs
> > gl_PrimitiveID output. When there's no geometry shader, we have to
> > ask the fi
When a geometry shader is present, the fragment shader gl_PrimitiveID
input acts like an ordinary varying, receiving data from the gs
gl_PrimitiveID output. When there's no geometry shader, we have to
ask the fixed function SF hardware to provide the primitive ID to the
fragment shader instead.
P
https://bugs.freedesktop.org/show_bug.cgi?id=34495
Alex Deucher changed:
What|Removed |Added
Assignee|dri-devel@lists.freedesktop |mesa-dev@lists.freedesktop.
Looks good.
A future improvement could be querying PIPE_CAP_MAX_TEXTURE_2D_LEVELS instead
of using a constant width/height.
Jose
- Original Message -
> Used to test rasterization, because we often breakdown on
> subdivision of triangles with long edges.
>
> Signed-off-by: Zack Rusin
On 10/24/2013 01:13 PM, Courtney Goeltzenleuchter wrote:
If a user set MESA_INFO and the OpenGL application uses a
3.0 or later context then the MESA_INFO debug output will have
an error when it queries for extensions using the deprecated
enum GL_EXTENSIONS. Passing context argument allows code
t
https://bugs.freedesktop.org/show_bug.cgi?id=70864
Priority: medium
Bug ID: 70864
Assignee: mesa-dev@lists.freedesktop.org
Summary: classic drivers needlessly link to libdrm_intel /
libdrm_nouveau / libdrm_radeon
Severity: no
https://bugs.freedesktop.org/show_bug.cgi?id=70864
--- Comment #1 from Fabio Pedretti ---
It looks every classic driver (including swrast) now includes all the classic
drivers, they are more or less a copy of mesa_dri_drivers.so.
--
You are receiving this mail because:
You are the assignee for
https://bugs.freedesktop.org/show_bug.cgi?id=69437
U. Artie Eoff changed:
What|Removed |Added
Status|RESOLVED|VERIFIED
--- Comment #8 from U. Artie Eo
https://bugs.freedesktop.org/show_bug.cgi?id=70864
Matt Turner changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=70864
--- Comment #3 from Fabio Pedretti ---
Thanks but it doesn't look they are hardlinks anyway, the md5sum all differ,
also their size is slightly different.
--
You are receiving this mail because:
You are the assignee for the bug.
___
https://bugs.freedesktop.org/show_bug.cgi?id=34495
--- Comment #74 from hapoofesg...@goingon.ir ---
S(In reply to comment #73)
> (In reply to comment #71)
> > (In reply to comment #70)
> > > I wonder if the fix has been committed? I am using Debian testing with the
> > > mesa 9.1-7 package provide
https://bugs.freedesktop.org/show_bug.cgi?id=70864
--- Comment #4 from Johannes Obermayr ---
(In reply to comment #2)
> The point of mega drivers was to link all of the (classic) drivers into a
> single file.
... to waste memory on runtime if you make use of packages provided by
distributions wh
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
calculation from the scene, and this was act
https://bugs.freedesktop.org/show_bug.cgi?id=70864
--- Comment #5 from Matt Turner ---
(In reply to comment #3)
> Thanks but it doesn't look they are hardlinks anyway, the md5sum all differ,
> also their size is slightly different.
Strange, that's not what I see on my system:
mattst88@work-Thin
Reviewed-by: Ian Romanick
On 10/01/2013 07:15 PM, Francisco Jerez wrote:
> v2: Represent atomics as GLSL intrinsics.
> ---
> src/glsl/builtin_functions.cpp | 58
> +
> src/glsl/builtin_variables.cpp | 15 +++
> src/glsl/glcpp/glcpp-parse.y|
Reviewed-by: Ian Romanick
On 10/01/2013 07:15 PM, Francisco Jerez wrote:
> And use it to forbid comparisons of opaque operands. According to the
> GL 4.2 specification:
>
>> Except for array indexing, structure member selection, and
>> parentheses, opaque variables are not allowed to be operand
On 10/25/2013 03:20 AM, Dave Airlie wrote:
>>
>> Do either of you guys plan to implement support for this extension? The
>> value to developers is obviously increased if more drivers support the
>> extension. This extension was born from feedback that I received from
>> people at FOSDEM and from
On 10/08/2013 10:24 AM, Adam Jackson wrote:
> Module: Mesa
> Branch: master
> Commit: d101204c23ba2f593881ede357309f3924cd
> URL:
> http://cgit.freedesktop.org/mesa/mesa/commit/?id=d101204c23ba2f593881ede357309f3924cd
>
> Author: Adam Jackson
> Date: Fri Oct 4 09:25:51 2013 -0400
>
On 21/10/13 23:23, Bryan Cain wrote:
> This is a port of 4da54c91d24da ("nvc0: implement multisample textures") to
> nv50.
>
> When coupled with the patch to only report 16 texture samplers (to fix
> crashes), all of the Piglit tests in spec/arb_texture_multisample pass.
>
Hello Bryan,
Big thank
_XReply returns 1 on success, but indirect_bind_context returns 0 on
success.
Signed-off-by: Adam Jackson
---
src/glx/indirect_glx.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/glx/indirect_glx.c b/src/glx/indirect_glx.c
index d27b019..28b8cd0 100644
--- a/src
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/u_indices_gen.py | 21 +++-
From: Rob Clark
This patchset (compared to RFC I sent previously) changes u_primconvert
to just be a front-end to the u_indices stuff. It handles binding/
restoring new index buffer state, etc. So driver using it just has
to put this at the top of their pipe->draw_vbo():
if (prim_needs_emula
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 Clark
---
src/gallium/auxiliary/Makefile.sources| 1 +
src/ga
From: Rob Clark
Use u_primconvert to convert unsupported primitives into supported
primitive plus index buffer.
Signed-off-by: Rob Clark
---
src/gallium/drivers/freedreno/a2xx/fd2_context.c | 24 ++-
src/gallium/drivers/freedreno/a3xx/fd3_context.c | 12 +-
src/galliu
On 10/25/2013 12:14 PM, Adam Jackson wrote:
> _XReply returns 1 on success, but indirect_bind_context returns 0 on
> success.
>
> Signed-off-by: Adam Jackson
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=70486
Reviewed-and-tested-by: Ian Romanick
The other way to fix this would be to
On 10/21/2013 05:55 PM, Eric Anholt wrote:
[snip]
> This interface means synchronizing with the GPU, which sucks when we
> have the ability to actually do DTFB in the hardware pipeline (Indirect
> Parameter Enable of 3DPRIMITIVE).
It's not that simple.
The 3DPRIMITIVE indirect registers require y
On 10/22/2013 04:30 AM, Marek Olšák wrote:
> On Fri, Oct 18, 2013 at 8:09 AM, Kenneth Graunke
> wrote:
>> DrawTransformFeedback() needs to obtain the number of vertices written
>> to a particular stream during the last Begin/EndTransformFeedback block.
>> The new driver hook returns exactly that
On 10/25/2013 10:14 AM, srol...@vmware.com wrote:
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 th
On Fri, 2013-10-25 at 12:59 -0700, Ian Romanick wrote:
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=70486
> Reviewed-and-tested-by: Ian Romanick
Pushed, thanks and sorry.
- ajax
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http
Erik Faye-Lund writes:
> Why is this tagged as i965/fs, when everything seems to happen in the
> glsl-optimizer?
>
> On Thu, Oct 24, 2013 at 5:53 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 sha
Am 25.10.2013 22:33, schrieb Brian Paul:
> On 10/25/2013 10:14 AM, srol...@vmware.com wrote:
>> 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 a
On 25.10.2013 20:35, Emil Velikov wrote:
> On 21/10/13 23:23, Bryan Cain wrote:
>> This is a port of 4da54c91d24da ("nvc0: implement multisample textures") to
>> nv50.
>>
>> When coupled with the patch to only report 16 texture samplers (to fix
>> crashes), all of the Piglit tests in spec/arb_textu
Thanks, Marek. Could someone with commit access pick this up? Let me
know if you'd like me to reformat/resend/create a git tree/whatever.
-ilia
On Sun, Oct 13, 2013 at 9:16 AM, Marek Olšák wrote:
> For the series:
>
> Reviewed-by: Marek Olšák
>
> Marek
>
> On Sun, Oct 13, 2013 at 3:43 AM, Ili
On 10/25/2013 01:35 PM, Emil Velikov wrote:
> On 21/10/13 23:23, Bryan Cain wrote:
>> This is a port of 4da54c91d24da ("nvc0: implement multisample textures") to
>> nv50.
>>
>> When coupled with the patch to only report 16 texture samplers (to fix
>> crashes), all of the Piglit tests in spec/arb_te
Kenneth Graunke writes:
> On 10/21/2013 05:55 PM, Eric Anholt wrote:
> [snip]
>> This interface means synchronizing with the GPU, which sucks when we
>> have the ability to actually do DTFB in the hardware pipeline (Indirect
>> Parameter Enable of 3DPRIMITIVE).
>
> It's not that simple.
>
> The 3
Paul Berry writes:
> When a geometry shader is present, the fragment shader gl_PrimitiveID
> input acts like an ordinary varying, receiving data from the gs
> gl_PrimitiveID output. When there's no geometry shader, we have to
> ask the fixed function SF hardware to provide the primitive ID to th
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" instead of "0" (caught by Erik
Faye-Lund).
---
src/glsl/opt_algebraic.cpp | 10 ++
1
While ir_builder is slightly less efficient, we're only increasing the
work when there's actual optimization being done, and it's way more
readable code.
---
src/glsl/opt_algebraic.cpp | 40 ++--
1 file changed, 10 insertions(+), 30 deletions(-)
diff --git a/sr
Matt and I had each screwed up these common required patterns recently, in
ways that wouldn't have been noticed for a long time if not for code
review. Just enforce it in the caller so that we don't rely on code
review catching these bugs.
---
src/glsl/opt_algebraic.cpp | 117 +++-
On 10/25/2013 04:11 PM, Christoph Bumiller wrote:
> On 25.10.2013 20:35, Emil Velikov wrote:
>> On 21/10/13 23:23, Bryan Cain wrote:
>>> This is a port of 4da54c91d24da ("nvc0: implement multisample textures") to
>>> nv50.
>>>
>>> When coupled with the patch to only report 16 texture samplers (to f
On 17 October 2013 04:42, Timothy Arceri wrote:
> ---
> src/glsl/ast_to_hir.cpp | 27 ++-
> 1 file changed, 14 insertions(+), 13 deletions(-)
>
> diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp
> index dfa32d9..f96ed53 100644
> --- a/src/glsl/ast_to_hir.cpp
On 25.10.2013 23:51, Bryan Cain wrote:
> On 10/25/2013 04:11 PM, Christoph Bumiller wrote:
>> On 25.10.2013 20:35, Emil Velikov wrote:
>>> On 21/10/13 23:23, Bryan Cain wrote:
This is a port of 4da54c91d24da ("nvc0: implement multisample textures") to
nv50.
When coupled with the
On 10/25/2013 05:05 PM, Christoph Bumiller wrote:
> On 25.10.2013 23:51, Bryan Cain wrote:
>> On 10/25/2013 04:11 PM, Christoph Bumiller wrote:
>>> On 25.10.2013 20:35, Emil Velikov wrote:
On 21/10/13 23:23, Bryan Cain wrote:
> This is a port of 4da54c91d24da ("nvc0: implement multisample
On Fri, Oct 25, 2013 at 10:28 PM, Kenneth Graunke wrote:
> On 10/22/2013 04:30 AM, Marek Olšák wrote:
>> On Fri, Oct 18, 2013 at 8:09 AM, Kenneth Graunke
>> wrote:
>>> DrawTransformFeedback() needs to obtain the number of vertices written
>>> to a particular stream during the last Begin/EndTrans
I'll do it in a moment.
Marek
On Fri, Oct 25, 2013 at 11:25 PM, Ilia Mirkin wrote:
> Thanks, Marek. Could someone with commit access pick this up? Let me
> know if you'd like me to reformat/resend/create a git tree/whatever.
>
> -ilia
>
> On Sun, Oct 13, 2013 at 9:16 AM, Marek Olšák wrote:
>>
Reviewed-by: Aaron Watry
I have tested this on a Radeon 5400 (Cedar), and I just sent a few
generated tests to the piglit list.
--Aaron
On Wed, Oct 23, 2013 at 6:28 PM, Tom Stellard wrote:
> From: Tom Stellard
>
> ---
> lib/Target/R600/AMDGPUISelLowering.cpp | 1 +
> test/CodeGen/R600/llvm.
On 10/17/2013 11:09 PM, Kenneth Graunke wrote:
> Here's my implementation of ARB_transform_feedback2. I believe it's
> complete; it passes all of our Piglit tests and a lot of Intel's
> oglconform tests.
>
> This should work out of the box on Ivybridge and Baytrail. It won't
> work on Haswell at
On Sun, Oct 20, 2013 at 11:57 AM, Christian König
wrote:
> Hi Marek,
>
> I've just send out a v6 of the patch, please take a second look. Most things
> are fixed now, but there are still a couple of open issues:
>
>
>> 3) There should also probably be some checking for
>> GL_ARB_texture_non_power_
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 with these patches
on SNB, IVB & HSW.
These patches can
This patch implements the common support code required for the
GL_ARB_sample_shading extension.
V2: Move GL_ARB_sample_shading to ARB extension list.
Signed-off-by: Anuj Phogat
Reviewed-by: Ian Romanick
Reviewed-by: Ken Graunke
---
src/glsl/glcpp/glcpp-parse.y| 3 +++
src/glsl/glsl_pa
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 API_XML list in Makefile.am.
Add extra_gl40_ARB_sample_shadin
Number of samples will be required in fragment shader program by new
GLSL builtin uniform "gl_NumSamples".
V2: Use "state.numsamples" in place of "state.num.samples"
Use _NEW_BUFFERS flag in place of _NEW_MULTISAMPLE
Signed-off-by: Anuj Phogat
Reviewed-by: Ian Romanick
Reviewed-by: Ken Grau
Thsi function is used to test if we need to do per sample shading or
per fragment shading.
Signed-off-by: Anuj Phogat
---
src/mesa/program/program.c | 31 +++
src/mesa/program/program.h | 3 +++
2 files changed, 34 insertions(+)
diff --git a/src/mesa/program/program
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.
- Add comment to explain the size of gl_SampleMask[] a
This is required while adding builtin system value vec{2, 3, 4}
variables. For example:
(declare (sys) vec2 gl_SamplePosition)
Without this patch above glsl ir splits in to:
(declare (temporary) float gl_SamplePosition_x)
(declare (temporary) float gl_SamplePosition_y)
Signed-off-by: Anuj Phogat
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/brw_context.h | 1 +
src/mesa/drivers/dri/i965/brw_fs.cpp
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/dri/i965/brw_defines.h| 1 +
src/mesa/drivers/dri/i965/
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-by: Anuj Phogat
---
src/mesa/drivers/dri/i965/brw_context.h
- 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_PERSAMPLE is enabled.
Refer SNB PRM Vol. 2, Part 1, Page 279 f
- 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_PERSAMPLE is enabled. Refer IVB PRM Vol. 2, Part 1,
Page 288 f
Signed-off-by: Anuj Phogat
Reviewed-by: Paul Berry
Reviewed-by: Ken Graunke
---
src/mesa/drivers/dri/i965/intel_extensions.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
b/src/mesa/drivers/dri/i965/intel_extensions.c
index 803d090..88201bd 10
On 10/25/2013 03:26 PM, Marek Olšák wrote:
[snip]
>> At least for Ivybridge, I think I want this software path 100% of the
>> time. We may want to remove the stall on Haswell as a later optimization.
>
> I'd like to have a dedicated flag for this fallback like we have
> Const.PrimitiveRestartInSo
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 the 'more automatic' shader cache wo
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" instead of "0" (caught by Erik
>
On 10/25/2013 02:49 PM, Eric Anholt wrote:
> Matt and I had each screwed up these common required patterns recently, in
> ways that wouldn't have been noticed for a long time if not for code
> review. Just enforce it in the caller so that we don't rely on code
> review catching these bugs.
> ---
>
Both Gallium and i965 subclass gl_transform_feedback_object, which
requires implementing a custom NewTransformFeedback hook. Creating a
helper function to initialize the fields avoids code duplication and
divergence.
Signed-off-by: Kenneth Graunke
Cc: Eric Anholt
Cc: Marek Olšák
---
src/mesa/
This picks up a missing obj->EverBound = GL_FALSE line, and will catch
any new fields that get added in the future.
Signed-off-by: Kenneth Graunke
Cc: Marek Olšák
---
src/mesa/state_tracker/st_cb_xformfb.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
Compile tested with --with-gal
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).
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i
The ARB_transform_feedback2 extension introduces the ability to pause
and resume transform feedback sessions. Although only one can be active
at a time, it's possible to switch between multiple transform feedback
objects while paused.
In order to facilitate this, we need to save/restore the SO_WR
DrawTransformFeedback() needs to obtain the number of vertices written
to a particular stream during the last Begin/EndTransformFeedback block.
The new driver hook returns exactly that information.
Gallium drivers already implement this by passing the transform feedback
object to the drawing funct
Renaming it makes it obvious that it isn't used, and the assertion
verifies that the VBO module never passes us such an object.
Signed-off-by: Kenneth Graunke
Reviewed-by: Ian Romanick
Reviewed-by: Eric Anholt
---
src/mesa/drivers/dri/i965/brw_draw.c | 4 +++-
src/mesa/drivers/dri/i965/brw_dra
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 the SO_NUM_PRIMS_WRITTEN(n) counters and multiply
by the number of vertice
75 matches
Mail list logo
