Hi,
AFAIR not enabling this makes LLVM generate really slow code in some
common cases. Maybe this is just a bug in LLVM/R600 triggered by unsafe
FP math optimization or some optimization is too eager. Other drivers do
fine with these types of optimization.
What's the impact on performance with un
On 18.02.2015 16:52, Grigori Goronzy wrote:
> Hi,
>
> AFAIR not enabling this makes LLVM generate really slow code in some
> common cases. Maybe this is just a bug in LLVM/R600 triggered by unsafe
> FP math optimization or some optimization is too eager. Other drivers do
> fine with these types of
On Tue, Feb 17, 2015 at 05:39:33PM -0800, Kenneth Graunke wrote:
> There's some debate about whether we should use Meta or BLORP,
> but either should run circles around the BLT engine.
>
> In particular, this means that Gen8+ will use the 3D engine for blits,
> like we do on Gen6-7.
>
> Improves
Am 2015-02-18 09:13, schrieb Michel Dänzer:
On 18.02.2015 16:52, Grigori Goronzy wrote:
Hi,
AFAIR not enabling this makes LLVM generate really slow code in some
common cases. Maybe this is just a bug in LLVM/R600 triggered by
unsafe
FP math optimization or some optimization is too eager. Other
On 18.02.2015 17:13, Michel Dänzer wrote:
> On 18.02.2015 16:52, Grigori Goronzy wrote:
>>
>> What's the impact on performance with unsafe FP math disabled at this time?
>
> I don't know. Correctness trumps performance.
FWIW, I couldn't seem to measure any significant difference with Unigine
Vall
Am Dienstag, 17. Februar 2015, 21:21:18 schrieb Matt Turner:
> On Mon, Feb 16, 2015 at 10:25 AM, Matt Turner wrote:
> > On Mon, Feb 16, 2015 at 1:36 AM, Marc Dietrich wrote:
> >> Avoid redefined symbol errors in clang. Based on a suggestion from
> >> Rafael à vila de EspÃndola in
> >> http://ll
> On Feb 17, 2015, at 11:52 PM, Grigori Goronzy wrote:
>
> Hi,
>
> AFAIR not enabling this makes LLVM generate really slow code in some
> common cases. Maybe this is just a bug in LLVM/R600 triggered by unsafe
> FP math optimization or some optimization is too eager. Other drivers do
> fine wit
> On Feb 18, 2015, at 1:15 AM, Michel Dänzer wrote:
>
> On 18.02.2015 17:13, Michel Dänzer wrote:
>> On 18.02.2015 16:52, Grigori Goronzy wrote:
>>>
>>> What's the impact on performance with unsafe FP math disabled at this time?
>>
>> I don't know. Correctness trumps performance.
>
> FWIW, I
From: Philipp Zabel
Since imx-drm has graduated from staging it seems to be a good idea to
recognize it by default in the libdrm tests.
Signed-off-by: Philipp Zabel
Signed-off-by: Lucas Stach
---
lst: added commit message + rebase to master
---
tests/kmstest/main.c | 1 +
tests/modetest/
https://bugs.freedesktop.org/show_bug.cgi?id=88618
Emil Velikov changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
Jordan Justen writes:
> An *untested* update for d9cd982d556be560af3bcbcdaf62b6b93eb934a5.
>
> A similar change was needed for CS to allow the piglit test
> tests/spec/arb_compute_shader/execution/simple-barrier-atomics.shader_test
> to pass.
>
> I think the previous change will fix cases that wr
Connor Abbott writes:
> On Tue, Feb 17, 2015 at 3:04 PM, Francisco Jerez
> wrote:
>> Jason Ekstrand writes:
>>
>>> On Mon, Feb 16, 2015 at 11:39 AM, Francisco Jerez
>>> wrote:
>>>
The round-robin allocation strategy is expected to decrease the amount
of false dependencies created by
https://bugs.freedesktop.org/show_bug.cgi?id=89203
--- Comment #1 from andre35...@yahoo.com ---
Created attachment 113615
--> https://bugs.freedesktop.org/attachment.cgi?id=113615&action=edit
dmesg
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee
https://bugs.freedesktop.org/show_bug.cgi?id=89203
Bug ID: 89203
Summary: Mesa 10.4.3 and up causes stuttering and frame drops
in a particular game
Product: Mesa
Version: 10.4
Hardware: Other
OS: All
https://bugs.freedesktop.org/show_bug.cgi?id=89203
Alex Deucher changed:
What|Removed |Added
Component|GLX |Drivers/Gallium/radeonsi
Assign
On 18 February 2015 at 00:46, Ilia Mirkin wrote:
> Wine maybe? (They're compat-only for now, although some work is being
> done to support core, but that might only be for their D3D10+ layer.)
The current plan for Wine is just to add support for core profiles.
There may be a case for hardware that
On 12/02/15 04:05, Laura Ekstrand wrote:
---
src/mapi/glapi/gen/ARB_direct_state_access.xml | 7 +++
src/mesa/main/bufferobj.c | 63 +++---
src/mesa/main/bufferobj.h | 9
src/mesa/main/tests/dispatch_sanity.cpp| 1
looks good to me.
Reviewed-by: Martin Peres
On 12/02/15 04:05, Laura Ekstrand wrote:
---
src/mapi/glapi/gen/ARB_direct_state_access.xml | 7 +++
src/mesa/main/bufferobj.c | 65 --
src/mesa/main/bufferobj.h | 5 +-
src/mes
On 12/02/15 04:05, Laura Ekstrand wrote:
v2: review from Ian Romanick
- Fix space in ARB_direct_state_access.xml.
- Remove "_mesa" from the name of buffer_data static fallback.
- Restore VBO_DEBUG and BOUNDS_CHECK.
- Fix beginning of comment to start on same line as /*
---
src/
On 02/17/2015 06:03 AM, Neil Roberts wrote:
> On Gen9+ the 1D miptree is laid out with all of the mipmap levels in a
> horizontal line.
> ---
> src/mesa/drivers/dri/i965/brw_tex_layout.c | 71
> --
> 1 file changed, 68 insertions(+), 3 deletions(-)
>
> diff --git a/sr
https://bugs.freedesktop.org/show_bug.cgi?id=79706
Bug 79706 depends on bug 86944, which changed state.
Bug 86944 Summary: glsl_parser_extras.cpp", line 1455: Error: Badly formed
expression. (Oracle Studio)
https://bugs.freedesktop.org/show_bug.cgi?id=86944
What|Removed
https://bugs.freedesktop.org/show_bug.cgi?id=86944
Ian Romanick changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On Wed, Feb 18, 2015 at 8:31 AM, Francisco Jerez wrote:
> Connor Abbott writes:
>
>> On Tue, Feb 17, 2015 at 3:04 PM, Francisco Jerez
>> wrote:
>>> Jason Ekstrand writes:
>>>
On Mon, Feb 16, 2015 at 11:39 AM, Francisco Jerez
wrote:
> The round-robin allocation strategy is e
On 02/17/2015 07:59 AM, Martin Peres wrote:
> Hey Laura,
>
> Thanks for this code. I'll be sending reviews throughout the week :)
>
> On 12/02/15 04:05, Laura Ekstrand wrote:
>> diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c
>> index 336feff..ce6f446 100644
>> --- a/src/mesa/mai
Use pipe_sampler_view_reference() instead of ordinary assignment.
Also add a new sanity check assertion.
Fixes piglit gl-1.0-drawpixels-color-index test crash. But note
that the test still fails.
Cc: "10.4, 10.5"
---
src/mesa/state_tracker/st_cb_drawpixels.c | 13 +
1 file changed,
Hi,
On 20 January 2015 at 21:49, Dave Airlie wrote:
> On 19 January 2015 at 21:00, Chris Wilson wrote:
>> In order to suport GLX_EXT_buffer_age in DRI2, we need to pass back the
>> last swap buffer count that the back buffer was defined for. For
>> simplicity, we can reuse an existing field in t
On 02/11/2015 06:05 PM, Laura Ekstrand wrote:
> ---
> src/mapi/glapi/gen/ARB_direct_state_access.xml | 8 +++
> src/mesa/main/tests/dispatch_sanity.cpp| 1 +
> src/mesa/main/teximage.c | 90
> ++
> src/mesa/main/teximage.h
On 02/17/2015 08:03 AM, Martin Peres wrote:
> Hey,
>
> Why not move these two patches before the one introducing BufferRange?
>
> Also, I wouldn't mind seeing them squashed together since the previous
> patch
> alone does not change any behaviour.
I was going to suggest the same thing. Specific
This patch is
Reviewed-by: Ian Romanick
On 02/11/2015 06:05 PM, Laura Ekstrand wrote:
> ---
> src/mesa/main/teximage.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c
> index 29c325b..336feff 100644
> --- a/src/mesa/m
This patch fixes a performance regression that causes our piglit CI
system to slow by 50%. If anyone has time to take a look at this and
provide a RB, it will improve the rate at which you get your piglit
results.
-Mark
Francisco Jerez writes:
> This fixes a regression in the running time of P
Ben Widawsky writes:
> I promise to look at this again in more detail tomorrow when I am more
> awake, but meanwhile, I'd be very much in favor of just setting
> mt->qpith for all miptree layout types.
I'm not sure what you mean. Do you mean that we should just always set
mt->qpitch to the actua
Reviewed-by: Matt Turner
Thanks Curro!
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
On 02/18/2015 10:57 AM, Mark Janes wrote:
> This patch fixes a performance regression that causes our piglit CI
> system to slow by 50%. If anyone has time to take a look at this and
> provide a RB, it will improve the rate at which you get your piglit
> results.
When you send a message like this
On Tue, Dec 9, 2014 at 2:52 AM, Eduardo Lima Mitev wrote:
> From: Samuel Iglesias Gonsalvez
>
> GLSL 1.50 and GLSL 4.40 specs, they both say the same in
> "Interface Blocks" section:
>
> "If no optional qualifier is used in a member-declaration, the qualification
> of
> the member includes all i
On Monday 19 January 2015, Chris Wilson wrote:
> Allow the DDXes to opt-in and handle swap-interval=0 requests for
> themselves, for example by using asynchronous pageflips, rather than
> forcing a blit. This has the important side-effect of also
> disambiguating CopyRegion calls to always be clien
On Mon, Dec 1, 2014 at 5:04 AM, Eduardo Lima Mitev wrote:
> From: Samuel Iglesias Gonsalvez
>
> Create a new search function to look for matching built-in functions by name
> and use it for built-in function redefinition or overload in GLSL ES 3.00.
>
> GLSL ES 3.0 spec, chapter 6.1 "Function Def
On 02/17/2015 03:27 AM, Tapani Pälli wrote:
> Signed-off-by: Tapani Pälli
> ---
> src/mapi/glapi/gen/ARB_program_interface_query.xml | 109
> +
> src/mapi/glapi/gen/gl_API.xml | 2 +
> src/mapi/glapi/gen/gl_genexec.py | 1 +
> src/me
On Tue, Jan 13, 2015 at 3:29 AM, Eduardo Lima Mitev wrote:
> This patch was updated and is pending review.
I think we're waiting on a new version with Tapani's comment addressed.
The patch looks right, FWIW.
___
mesa-dev mailing list
mesa-dev@lists.fre
On 02/17/2015 10:23 AM, Matt Turner wrote:
> On Tue, Feb 17, 2015 at 3:27 AM, Tapani Pälli wrote:
>> Here is a skeleton for the GL_ARB_program_interface_query API functions.
>> Adding the enum values makes it possible to start introducing changes in
>> the current shader query functions using thes
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
On 01/22/2015 10:06 AM, Emil Velikov wrote:
> On 15/12/14 12:08, Eduardo Lima Mitev wrote:
>> On 12/15/2014 12:21 PM, Emil Velikov wrote:
>>>
>>> Above you've mentioned "test failures were gathered ... against 10.3.3",
>>> which brings the question:
>>> Should we include those in either one of the
Hello list,
A candidate for the Mesa 10.4.5 release is now available. The current
patch queue is as follows:
- 17 queued
- 7 nominated (outstanding)
- and 0 rejected (obsolete) patches
In a nut shell this gives us Darwin build fixes and a few driver specific
patches - nouveau (nvc0), radeonsi
On Wednesday, February 18, 2015 12:00:39 PM Matt Turner wrote:
> On Mon, Dec 1, 2014 at 5:04 AM, Eduardo Lima Mitev wrote:
> > From: Samuel Iglesias Gonsalvez
> >
> > Create a new search function to look for matching built-in functions by name
> > and use it for built-in function redefinition or
On Wed, Feb 18, 2015 at 6:02 PM, Ian Romanick wrote:
> On 02/17/2015 06:03 AM, Neil Roberts wrote:
>> + /* When this layout is used the horizontal alignment is fixed at 64 and
>> the
>> +* hardware ignores the value given in the surface state
>> +*/
>> + const unsigned int align_w = 6
v2: Rebase on the nir_opcodes.h python code generation support.
v3: Use SSA values, and set an appropriate writemask on dot products.
v4: Make the arguments be SSA references as well. This lets you stack up
expressions in the arguments of other expressions, at the cost of
having to insert
v2: Try to patch up the scons bits.
---
src/glsl/Makefile.am | 2 ++
src/glsl/SConscript | 2 ++
src/mesa/SConscript | 2 ++
src/mesa/drivers/dri/common/Makefile.am | 2 ++
src/mesa/drivers/dri/i915/Makefile.am| 2 ++
src/mesa/driv
On Wed, Feb 18, 2015 at 3:34 PM, Erik Faye-Lund wrote:
> On Wed, Feb 18, 2015 at 6:02 PM, Ian Romanick wrote:
>> On 02/17/2015 06:03 AM, Neil Roberts wrote:
>>> + /* When this layout is used the horizontal alignment is fixed at 64 and
>>> the
>>> +* hardware ignores the value given in the
On Thu, Feb 19, 2015 at 1:12 AM, Matt Turner wrote:
> On Wed, Feb 18, 2015 at 3:34 PM, Erik Faye-Lund wrote:
>> On Wed, Feb 18, 2015 at 6:02 PM, Ian Romanick wrote:
>>> On 02/17/2015 06:03 AM, Neil Roberts wrote:
+ /* When this layout is used the horizontal alignment is fixed at 64
On Tuesday, February 17, 2015 02:25:05 PM Ian Romanick wrote:
> (Cross posting to mesa-dev.)
>
> On 02/17/2015 01:54 PM, Ilia Mirkin wrote:
> > On Tue, Feb 17, 2015 at 4:45 PM, Ilia Mirkin wrote:
> >> On Tue, Feb 17, 2015 at 4:34 PM, Ian Romanick wrote:
> >>> "Dependencies
> >>>
> >>> OpenGL
On Wed, Feb 18, 2015 at 6:43 PM, Eric Anholt wrote:
> v2: Rebase on the nir_opcodes.h python code generation support.
> v3: Use SSA values, and set an appropriate writemask on dot products.
> v4: Make the arguments be SSA references as well. This lets you stack up
> expressions in the argumen
https://bugs.freedesktop.org/show_bug.cgi?id=89199
Vinson Lee changed:
What|Removed |Added
Keywords||bisected
CC|
Fixes all of the failures in arb_direct_state_access/gettextureimage-targets
except for GL_TEXTURE_1D_ARRAY PBO. Previously, the miptree for meta pbo
uploads and downloads was not getting initialized correctly.
---
src/mesa/drivers/common/meta_tex_subimage.c | 8 ++--
src/mesa/drivers/dri/i9
Corrects the way that _mesa_meta_pbo_TexSubImage and
_mesa_meta_pbo_GetTexSubImage handle 1D_ARRAY textures. Fixes a failure in
the Piglit arb_direct_state_access/gettextureimage-targets test.
---
src/mesa/drivers/common/meta_tex_subimage.c | 60 -
1 file changed, 34 i
On Wed, Feb 18, 2015 at 07:13:54PM +, Neil Roberts wrote:
> Ben Widawsky writes:
>
> > I promise to look at this again in more detail tomorrow when I am more
> > awake, but meanwhile, I'd be very much in favor of just setting
> > mt->qpith for all miptree layout types.
>
> I'm not sure what
From: Marius Predut
On 32-bit, for floating point operations is used x86 FPU registers
instead SSE, reason for when reinterprets an integer as a float
result is unexpected (modify floats when they are written to memory).
The patch was checked with and without -O3 compiler flag.
Also, it add per
On Wed, Feb 18, 2015 at 6:03 AM, Sedat Dilek wrote:
> On Tue, Feb 17, 2015 at 11:00 AM, Sedat Dilek wrote:
>> On Tue, Feb 17, 2015 at 10:40 AM, Marc Dietrich wrote:
>>> This adds clang/gcc visibility macro detection to configure and
>>> util/macros.h.
>>> This is can be used to conveniently add
On Tue, Feb 17, 2015 at 11:00 AM, Sedat Dilek wrote:
> On Tue, Feb 17, 2015 at 10:40 AM, Marc Dietrich wrote:
>> This adds clang/gcc visibility macro detection to configure and
>> util/macros.h.
>> This is can be used to conveniently add e.g. a "HIDDEN" attribute to a
>> function.
>>
>> Signed-
On Wed, Feb 18, 2015 at 5:45 PM, Laura Ekstrand
wrote:
> Fixes all of the failures in
> arb_direct_state_access/gettextureimage-targets
> except for GL_TEXTURE_1D_ARRAY PBO. Previously, the miptree for meta pbo
> uploads and downloads was not getting initialized correctly.
> ---
> src/mesa/drive
Comparing the location field is equivalent and more efficient.
We'll also need this when we start using NIR for ARB programs, as our
NIR converter will set the location field correctly, but probably won't
use the GLSL names for these concepts.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers
backend_visitor now handles this, so we can delete the vec4_visitor
specific code.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_vec4.h | 3 ---
src/mesa/drivers/dri/i965/brw_vec4_gs_visitor.cpp | 2 +-
src/mesa/drivers/dri/i965/brw_vec4_visitor.cp
This is similar to _mesa_shader_stage_to_string(), but returns "VS"
instead of "vertex".
Signed-off-by: Kenneth Graunke
---
src/glsl/glsl_parser_extras.cpp | 17 +
src/glsl/glsl_parser_extras.h | 3 +++
2 files changed, 20 insertions(+)
diff --git a/src/glsl/glsl_parser_extra
We introduce three new fields in backend_visitor:
- debug_enabled: whether or not INTEL_DEBUG & DEBUG_
- stage_name: "vertex", "fragment", etc. for use in messages
- stage_abbrev: "VS", "FS", etc. for use in messages
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_shader.cpp | 3
Every VS system value has type D. We can always add this back if that
changes, but for now, it's extra typing.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_fs.h | 2 +-
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 7 +++
2 files changed, 4 insertions(+), 5 de
This is now trivial to do right.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
b/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
index 4f66b62..5
When compiling, we have a gl_shader_stage (MESA_SHADER_*) enum, and want
to know whether debugging is enabled for that stage. This allows us to
easily translate it into the corresponding debug flag.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/intel_debug.c | 13 +
s
Previously, the vec4 backend labeled shaders as "vec4" - now it uses the
specific names "VS" and "GS".
The FS backend now correctly prints "VS" for vertex shaders (rather than
"fs"). It also prints "FS" instead of "fs" for fragment shaders;
preserving that behavior didn't seem essential.
Signed-
These code paths can (or will) be used for other shader stages.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 4 ++--
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 11 ++-
2 files changed, 8 insertions(+), 7 deletions(-)
diff --git a/src/mesa/drivers/dri/i9
Series is
Reviewed-by: Jason Ekstrand
On Feb 18, 2015 9:00 PM, "Kenneth Graunke" wrote:
> This is now trivial to do right.
>
> Signed-off-by: Kenneth Graunke
> ---
> src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/mesa
https://bugs.freedesktop.org/show_bug.cgi?id=89218
Bug ID: 89218
Summary: lower_instructions.cpp:648:48: error: invalid suffix
'd' on floating constant
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS:
https://bugs.freedesktop.org/show_bug.cgi?id=89218
Ilia Mirkin changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On Wednesday 18 February 2015 15:08:19 Kenneth Graunke wrote:
> On Wednesday, February 18, 2015 12:00:39 PM Matt Turner wrote:
> > On Mon, Dec 1, 2014 at 5:04 AM, Eduardo Lima Mitev
wrote:
> > > From: Samuel Iglesias Gonsalvez
> > >
> > > Create a new search function to look for matching built-
On Wednesday 18 February 2015 12:00:39 Matt Turner wrote:
> On Mon, Dec 1, 2014 at 5:04 AM, Eduardo Lima Mitev wrote:
> > From: Samuel Iglesias Gonsalvez
> >
> > Create a new search function to look for matching built-in functions by
> > name and use it for built-in function redefinition or over
73 matches
Mail list logo
