On Sun, Nov 30, 2014 at 2:24 AM, Chris Forbes wrote:
> Fixes the piglit test: spec/glsl-es-3.00/compiler/undef-GL_ES.vert
>
> Signed-off-by: Chris Forbes
> ---
> src/glsl/glcpp/glcpp-parse.y | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/glsl/glcpp/glcpp-parse.y b
On Wed, Dec 3, 2014 at 11:31 PM, Ben Widawsky wrote:
> On Wed, Dec 03, 2014 at 11:27:55PM -0800, Ben Widawsky wrote:
>> On Wed, Dec 03, 2014 at 04:47:18PM -0800, Matt Turner wrote:
>> > ---
>> > The number of times I've wondered why piglit/gbm wasn't working...
>> >
>> > src/egl/drivers/dri2/egl_
On Wed, 2014-12-03 at 14:58 -0800, Jason Ekstrand wrote:
>
>
> On Wed, Dec 3, 2014 at 2:52 PM, Ian Romanick
> wrote:
> On 12/01/2014 03:04 AM, Iago Toral Quiroga wrote:
> > From: Jason Ekstrand
(...)
>
> >
> > +enum mesa_array_format_datatype {
>
On 04.12.2014 17:17, Iago Toral wrote:
On Wed, 2014-12-03 at 14:58 -0800, Jason Ekstrand wrote:
On Wed, Dec 3, 2014 at 2:52 PM, Ian Romanick
wrote:
On 12/01/2014 03:04 AM, Iago Toral Quiroga wrote:
> From: Jason Ekstrand
(...)
>
> +enum mesa_array_format_
On Thu, 2014-12-04 at 17:39 +0900, Michel Dänzer wrote:
> On 04.12.2014 17:17, Iago Toral wrote:
> > On Wed, 2014-12-03 at 14:58 -0800, Jason Ekstrand wrote:
> >> On Wed, Dec 3, 2014 at 2:52 PM, Ian Romanick
> >> wrote:
> >> On 12/01/2014 03:04 AM, Iago Toral Quiroga wrote:
> >>
On Wed, 2014-12-03 at 14:26 -0800, Jason Ekstrand wrote:
>
>
> On Wed, Dec 3, 2014 at 2:15 PM, Ian Romanick
> wrote:
> This patch is
>
> Reviewed-by: Ian Romanick
>
> Should it also be tagged for 10.4?
>
>
> Yes, it should.
Should I add "Cc: 10.4 "
On 02/12/2014 20:53, Mario Kleiner wrote :
Restores proper immediate tearing swap behaviour for
OpenGL bufferswap under DRI3/Present.
+ if (priv->swap_interval == 0)
+ options |= XCB_PRESENT_OPTION_ASYNC;
+
back->busy = 1;
Currently under DRI3 glx, you'll get triple
On 02/12/2014 20:53, Mario Kleiner wrote :
targetSBC == 0 is a special case, which asks the function
to block until all pending OpenGL bufferswap requests have
completed.
Currently the function just falls through for targetSBC == 0,
returning bogus results.
This breaks applications originally w
Le 04/12/2014 11:44, Axel Davy a écrit :
On 02/12/2014 20:53, Mario Kleiner wrote :
targetSBC == 0 is a special case, which asks the function
to block until all pending OpenGL bufferswap requests have
completed.
Currently the function just falls through for targetSBC == 0,
returning bogus resul
On 02/12/2014 20:53, Mario Kleiner wrote :
Prevent calls to glXGetSyncValuesOML() and glXWaitForMscOML()
from overwriting the (ust,msc) values of the last successfull
swapbuffers call (PresentPixmapCompleteNotify event), as
glXWaitForSbcOML() relies on those values corresponding to
the most recen
On 02/12/2014 20:53, Mario Kleiner wrote :
glXSwapBuffersMscOML() with target_msc=divisor=remainder=0 gets
translated into target_msc=divisor=0 but remainder=1 by the mesa
api. This is done for server DRI2 where there needs to be a way
to tell the server-side DRI2ScheduleSwap implementation if a
On Wednesday, December 03, 2014 10:09:59 AM Chris Forbes wrote:
> It would be nice if this could be added to the existing logic for the
> interaction between builtins and app-provided overloads -- or do we
> need to fail earlier than that?
>
This is the place where we store the information about
Jan Vesely writes:
> This way we get a warning when an enum value is not handled
>
> Signed-off-by: Jan Vesely
> ---
> src/gallium/state_trackers/clover/core/kernel.cpp | 45
> ++-
> 1 file changed, 20 insertions(+), 25 deletions(-)
>
> diff --git a/src/gallium/state_tracke
From: Marek Olšák
Discussion: https://bugs.freedesktop.org/show_bug.cgi?id=83510#c8
---
src/gallium/drivers/radeonsi/si_shader.c | 27 +++
1 file changed, 27 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
i
Hello all,
Kenneth kindly pointed out in an earlier thread [1] that it's quite
likely that drivers for older hardware are somewhat busted.
A quick look at my local piglit runs, shows that the classic swrast has
a staggering number of regressions ~130 and only ~20 fixes. These stats
come from the 1
On Dec 4, 2014 1:07 AM, "Iago Toral" wrote:
>
> On Wed, 2014-12-03 at 14:26 -0800, Jason Ekstrand wrote:
> >
> >
> > On Wed, Dec 3, 2014 at 2:15 PM, Ian Romanick
> > wrote:
> > This patch is
> >
> > Reviewed-by: Ian Romanick
> >
> > Should it also be tagged for 10.4?
> >
Hi Keith,
With the week of Thanksgiving behind us, can you let us know your plans
wrt this series. Would you have a chance to look at it any time soon, or
is it quite low on your list ?
Thanks
Emil
P.S. Not meant to be pushy but curious.
On 02/12/14 19:53, Mario Kleiner wrote:
> A slightly upda
Hmm I have to say I'm not really convinced of that solution. Because all
divs are lowered, this will screw the results of all divs (if the rcp
would come from a legacy arb_fp rcp that would be different and quite
possible some apps depending on it, problems like that are very common
for d3d9 apps t
https://bugs.freedesktop.org/show_bug.cgi?id=86980
Roland Scheidegger changed:
What|Removed |Added
CC||bri...@vmware.com
--- Comment #4 fr
https://bugs.freedesktop.org/show_bug.cgi?id=86980
--- Comment #5 from Brian Paul ---
Going from my notes, some of the catia tests use GL_NV_fragment_program[23]
features (which like, GL_NV_fragment_program_option, layer on
GL_ARB_fragment_program). And those tests fail with Mesa because viewper
Signed-off-by: Ilia Mirkin
---
https://www.opengl.org/registry/specs/EXT/texture_sRGB_decode.txt
Explicitly talks about ES, other drivers seem to expose it. Not sure about ES1.
src/mesa/main/extensions.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/main/extensio
Returning FLT_MAX instead of 0 also works, which is similar to another
hw instruction: V_RCP_CLAMP_F32.
The Unreal engine is a pretty big target with a lot of apps out there.
I'm afraid a driconf option isn't feasible.
Marek
On Thu, Dec 4, 2014 at 5:39 PM, Roland Scheidegger wrote:
> Hmm I have
I'm also concerned this sort of ad-hoc re-interpretation of opcode
semantics will come to bytes us later, as different state trackers might
want different semantics.
I think we might need to redefine TGSI_OPCODE_RCP opcode or introduce a
TGSI_OPCODE_RCP_LEGACY opcode.
Also, do we know exactl
https://bugs.freedesktop.org/show_bug.cgi?id=84186
Mathias Brodala changed:
What|Removed |Added
CC|i...@noctus.net |
--
You are receiving this mail becau
The problem is, while glsl was quite lenient in older versions wrt
precision or things like div by zero, starting with glsl 4.10 you are
_required_ to return +-/Inf for divs by zero. Hence if you use hacks
here to make some app run, you _will_ have to fix that properly in the
future in any case.
(T
On Wed, Dec 3, 2014 at 3:53 PM, Ben Widawsky
wrote:
> SKL moves the GS threadcount to dw8 from dw7, and no longer does the divide
> by 2
> thing.
>
> Only compile test.
>
> Cc: Kristian Høgsberg
> Signed-off-by: Ben Widawsky
> ---
> src/mesa/drivers/dri/i965/gen8_gs_state.c | 29 ++
Oh hm. Apparently EXT_sRGB is required for this [or ES3, but that's
not available on the driver I'm interested quite yet]. I'll see if
that can be added easily.
On Thu, Dec 4, 2014 at 12:42 PM, Ilia Mirkin wrote:
> Signed-off-by: Ilia Mirkin
> ---
>
> https://www.opengl.org/registry/specs/EXT/te
I agree that this should be fixed properly, but my time is limited at
the moment. We know there's a problem with our shader backend, because
there are issues with other games too. We can fix it later and we
certainly won't forget about it.
Note that r600g is using a lot of these non-standard legac
I certainly don't want to deter you from pushing a quick fix now.
If possible I'd just like any further information you might have
collected while debugging this issue in the hope we can figure out a
more comprehensive solution should be.
Because I'm sure other drivers (including VMware's svg
Am 04.12.2014 um 20:49 schrieb Marek Olšák:
> I agree that this should be fixed properly, but my time is limited at
> the moment. We know there's a problem with our shader backend, because
> there are issues with other games too. We can fix it later and we
> certainly won't forget about it.
>
> No
On 12/04/2014 11:48 AM, Axel Davy wrote:
Le 04/12/2014 11:44, Axel Davy a écrit :
On 02/12/2014 20:53, Mario Kleiner wrote :
targetSBC == 0 is a special case, which asks the function
to block until all pending OpenGL bufferswap requests have
completed.
Currently the function just falls through
On 12/04/2014 11:20 AM, Axel Davy wrote:
On 02/12/2014 20:53, Mario Kleiner wrote :
Restores proper immediate tearing swap behaviour for
OpenGL bufferswap under DRI3/Present.
+ if (priv->swap_interval == 0)
+ options |= XCB_PRESENT_OPTION_ASYNC;
+
back->busy = 1;
Cu
This is similar to the existing functions get_instance, get_array_instance,
etc. for getting a type singleton. The new get_sampler_instance() function
will be used by the upcoming shader cache.
---
src/glsl/glsl_types.cpp | 111
src/glsl/glsl_types.
From: Tapani Pälli
Shader binary cache requires this to be able to cache hash data from
the gl_shader_program structure.
Signed-off-by: Tapani Pälli
Reviewed-by: Paul Berry
Reviewed-by: Carl Worth
---
src/mesa/program/hash_table.h | 8
1 file changed, 8 insertions(+)
diff --git a/s
There is an internal implementation detail that the hash table
underlying the struct string_to_uint_map stores each value internally
as (value+1). The user needn't be very concerned with this (other than
knowing that a value of UINT_MAX cannot be stored) since put() adds 1
and get() subtracts 1.
H
On 11/04/2014 04:20 AM, Tapani Pälli wrote:
> Note that some of the GLSL specifications explicitly state this as
> compile error, some simply state that 'it is an error'.
>
> Signed-off-by: Tapani Pälli
> ---
> src/glsl/glsl_parser.yy | 13 -
> 1 file changed, 12 insertions(+), 1 del
On 11/29/2014 12:54 PM, Chris Forbes wrote:
> Fixes the piglit test: spec/glsl-es-3.00/compiler/undef-GL_ES.vert
>
> Signed-off-by: Chris Forbes
> ---
> src/glsl/glcpp/glcpp-parse.y | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/glsl/glcpp/glcpp-parse.y b/src/gls
Ugh, yes, I'll fix that and land it at the same time.
Thanks :)
On Fri, Dec 5, 2014 at 11:12 AM, Ian Romanick wrote:
> On 11/29/2014 12:54 PM, Chris Forbes wrote:
>> Fixes the piglit test: spec/glsl-es-3.00/compiler/undef-GL_ES.vert
>>
>> Signed-off-by: Chris Forbes
>> ---
>> src/glsl/glcpp/gl
On 12/02/2014 09:54 AM, Ilia Mirkin wrote:
> I believe the issue is that _mesa_TextureView should be checking the
> target internal format against _mesa_base_tex_format() and returning a
> GL_INVALID_VALUE error (? the spec conveniently lists errors as
> 'TODO') if it returns -1. Chris -- thoughts?
On 12/04/2014 02:00 PM, Carl Worth wrote:
> This is similar to the existing functions get_instance, get_array_instance,
> etc. for getting a type singleton. The new get_sampler_instance() function
> will be used by the upcoming shader cache.
> ---
> src/glsl/glsl_types.cpp | 111
> +++
On 12/04/2014 02:00 PM, Carl Worth wrote:
> There is an internal implementation detail that the hash table
> underlying the struct string_to_uint_map stores each value internally
> as (value+1). The user needn't be very concerned with this (other than
> knowing that a value of UINT_MAX cannot be st
On Wed, Dec 3, 2014 at 5:58 PM, Kenneth Graunke wrote:
> On Tuesday, December 02, 2014 10:34:49 AM Matt Turner wrote:
>> On Tue, Dec 2, 2014 at 3:50 AM, Kenneth Graunke
>> wrote:
>> > The "Pixel Shader Computed Depth Mode" value is entirely based on the
>> > shader program, so we can easily do i
On Thu, Dec 4, 2014 at 2:00 PM, Carl Worth wrote:
> + assert(!"Should not get here.");
> + return error_type;
Let's just use unreachable("not reached") here.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailm
---
Ken pointed this out in his review, and I fixed it in the FS, but apparently
forgot to fix it in the vec4 code.
src/mesa/drivers/dri/i965/brw_vec4_dead_code_eliminate.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_vec4_dead_code_eliminate
---
src/mesa/drivers/dri/i965/brw_fs.h | 8
1 file changed, 8 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.h
b/src/mesa/drivers/dri/i965/brw_fs.h
index b29b6b0..c7bc55c 100644
--- a/src/mesa/drivers/dri/i965/brw_fs.h
+++ b/src/mesa/drivers/dri/i965/brw_fs.h
@@ -119,6 +119
This is a revert of commit 4656c14e ("i965/fs: Change the type of
booleans to UD and emit correct immediates") plus some small additional
fixes, like casting ctx->Const.UniformBooleanTrue to int and changing UD
to D in the ir_unop_b2f cases. Note that it's safe to leave 0x3f80
as UD and as a li
Jason realized that we could fix the result of the CMP instruction on
Gen <= 5 by doing -(result & 1). Also do the resolves in the vec4
backend before use, rather than when the bool was created. The FS does
this and it saves some unnecessary resolves.
On Ironlake:
total instructions in shared pro
From: Roland Scheidegger
The original idea was to optimize away the condition by integrating it directly
into the CMP instruction. However, with native integers this requires an extra
I2F instruction. It is also fishy because the negation used didn't really honor
ieee754 float comparison rules, n
The GS has an interesting use for mul. It's essentially used as a fancy mov (in
fact, I am not sure why a mov isn't used). The documentation in the function has
a very good explanation from Paul on the mechanics.
CHV has some quirks with regard to multiplication. While the documentation is
somewha
The index_bias (aka base_vertex) applies to the downstream draw just as
much, since the actual index values are never modified.
Signed-off-by: Ilia Mirkin
Cc: "10.3 10.4"
---
src/gallium/auxiliary/indices/u_primconvert.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/auxiliary/
Signed-off-by: Ilia Mirkin
Cc: "10.4 10.3"
---
src/gallium/auxiliary/indices/u_primconvert.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary/indices/u_primconvert.c
b/src/gallium/auxiliary/indices/u_primconvert.c
index 539ca53..4632781 100644
--- a/src/
Signed-off-by: Ilia Mirkin
Cc: "10.3 10.4"
---
src/gallium/auxiliary/indices/u_primconvert.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/gallium/auxiliary/indices/u_primconvert.c
b/src/gallium/auxiliary/indices/u_primconvert.c
index 95a2478..539ca53 100644
--- a/src/gallium/auxili
This series is
Reviewed-by: Jason Ekstrand
On Thu, Dec 4, 2014 at 3:05 PM, Matt Turner wrote:
> Jason realized that we could fix the result of the CMP instruction on
> Gen <= 5 by doing -(result & 1). Also do the resolves in the vec4
> backend before use, rather than when the bool was created.
Matt Turner writes:
> ---
> src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 10 ++
> src/mesa/drivers/dri/i965/brw_vec4_generator.cpp | 10 ++
> 2 files changed, 20 insertions(+)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_fs_generator.cpp
> b/src/mesa/drivers/dri/i965/brw_
On 12/04/2014 03:05 PM, Matt Turner wrote:
> This is a revert of commit 4656c14e ("i965/fs: Change the type of
> booleans to UD and emit correct immediates") plus some small additional
Commit 4656c14e contained some shader-db data... does this pseudo-revert
have shader-db changes?
> fixes, like c
On 12/04/2014 03:05 PM, Matt Turner wrote:
> Jason realized that we could fix the result of the CMP instruction on
> Gen <= 5 by doing -(result & 1). Also do the resolves in the vec4
> backend before use, rather than when the bool was created. The FS does
> this and it saves some unnecessary resolv
On Thu, Dec 4, 2014 at 4:22 PM, Ian Romanick wrote:
> On 12/04/2014 03:05 PM, Matt Turner wrote:
>> This is a revert of commit 4656c14e ("i965/fs: Change the type of
>> booleans to UD and emit correct immediates") plus some small additional
>
> Commit 4656c14e contained some shader-db data... does
On Thu, Dec 4, 2014 at 4:26 PM, Ian Romanick wrote:
> On 12/04/2014 03:05 PM, Matt Turner wrote:
>> Jason realized that we could fix the result of the CMP instruction on
>> Gen <= 5 by doing -(result & 1). Also do the resolves in the vec4
>> backend before use, rather than when the bool was create
On 12/04/2014 05:01 PM, Ilia Mirkin wrote:
Signed-off-by: Ilia Mirkin
Cc: "10.3 10.4"
---
src/gallium/auxiliary/indices/u_primconvert.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/gallium/auxiliary/indices/u_primconvert.c
b/src/gallium/auxiliary/indices/u_primconvert.c
index 9
The PRMs say that
region must be a replicated scalar
(with HorzStride = VertStride = 0).
but apparently that doesn't actually apply to all generations. I did
notice when implementing the optimization later in this series that G45
and ILK needed this regioning.
---
src/mesa/drivers/dri/i96
---
src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 4
1 file changed, 4 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_generator.cpp
b/src/mesa/drivers/dri/i965/brw_fs_generator.cpp
index 4ae35ac..2f99d65 100644
--- a/src/mesa/drivers/dri/i965/brw_fs_generator.cpp
+++ b/src/m
The LINE instruction performs a multiply-add instruction (a * b + c)
where b and c are scalar arguments. It reads b and c from offsets in
src0 such that you can load them (it they're representable) as a
vector-float immediate with a single instruction.
Hurts some programs, but that'll all get bett
Safe from causing optimization loops, since we don't constant propagate
VF arguments.
(for this and the previous patch):
total instructions in shared programs: 4289075 -> 4271932 (-0.40%)
instructions in affected programs: 1616779 -> 1599636 (-1.06%)
---
src/mesa/drivers/dri/i965/brw_fs_cse.c
LGTM. Reviewed-by: Brian Paul
On 12/04/2014 04:25 PM, srol...@vmware.com wrote:
From: Roland Scheidegger
The original idea was to optimize away the condition by integrating it directly
into the CMP instruction. However, with native integers this requires an extra
I2F instruction. It is also
On Thu, Dec 4, 2014 at 3:37 PM, Ben Widawsky
wrote:
> The GS has an interesting use for mul. It's essentially used as a fancy mov
> (in
> fact, I am not sure why a mov isn't used). The documentation in the function
> has
> a very good explanation from Paul on the mechanics.
What do you mean? Th
On Thursday, December 04, 2014 03:37:17 PM Ben Widawsky wrote:
> The GS has an interesting use for mul. It's essentially used as a fancy mov
> (in
> fact, I am not sure why a mov isn't used). The documentation in the function
> has
> a very good explanation from Paul on the mechanics.
>
> CHV ha
Hello,
Building recent mesa git i found that most video players that use VDPAu
hardware decoding fail with
vl/vl_stubs.c:45:vl_video_buffer_formats: Assertion `0' failed.
Trace/breakpoint trap
for example trying to play a 1080p h264 video that worked with hw
decoding:
$ mpv /media/bigdata/
On Thursday, December 04, 2014 03:05:59 PM Matt Turner wrote:
> Jason realized that we could fix the result of the CMP instruction on
> Gen <= 5 by doing -(result & 1). Also do the resolves in the vec4
> backend before use, rather than when the bool was created. The FS does
> this and it saves some
On Thu, Dec 4, 2014 at 5:48 PM, Kenneth Graunke wrote:
> Some comments would be nice:
>
> /**
> * Resolve the result of a Gen4-5 CMP instruction to a proper boolean.
> *
> * CMP on Gen4-5 only sets the LSB of the result; the rest are undefined.
> * If we need a proper boolean value, we have to
Cuts an instruction from two shaders in Tesseract, by allowing the
(x+y) cmp 0 -> x cmp -y optimization to take place.
instructions in affected programs: 1198 -> 1194 (-0.33%)
---
src/glsl/opt_algebraic.cpp | 10 ++
1 file changed, 10 insertions(+)
diff --git a/src/glsl/opt_algebraic
On Fri, Nov 21, 2014 at 10:23 AM, Matt Turner wrote:
> Thoughts?
Ping^2
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Matt Turner writes:
> Cuts an instruction from two shaders in Tesseract, by allowing the
> (x+y) cmp 0 -> x cmp -y optimization to take place.
>
> instructions in affected programs: 1198 -> 1194 (-0.33%)
Reviewed-by: Eric Anholt
signature.asc
Description: PGP signature
___
https://bugs.freedesktop.org/show_bug.cgi?id=86939
Vinson Lee changed:
What|Removed |Added
Blocks||79706
--
You are receiving this mail becau
https://bugs.freedesktop.org/show_bug.cgi?id=79706
Vinson Lee changed:
What|Removed |Added
Depends on||86939
--
You are receiving this mail becau
https://bugs.freedesktop.org/show_bug.cgi?id=86944
Vinson Lee changed:
What|Removed |Added
Blocks||79706
--
You are receiving this mail becau
https://bugs.freedesktop.org/show_bug.cgi?id=79706
Vinson Lee changed:
What|Removed |Added
Depends on||86944
--
You are receiving this mail becau
https://bugs.freedesktop.org/show_bug.cgi?id=86980
Vinson Lee changed:
What|Removed |Added
Blocks||79706
--
You are receiving this mail becau
https://bugs.freedesktop.org/show_bug.cgi?id=79706
Vinson Lee changed:
What|Removed |Added
Depends on||86980
--
You are receiving this mail becau
Mario Kleiner writes:
> glXSwapBuffersMscOML() with target_msc=divisor=remainder=0 gets
> translated into target_msc=divisor=0 but remainder=1 by the mesa
> api. This is done for server DRI2 where there needs to be a way
> to tell the server-side DRI2ScheduleSwap implementation if a call
> to glX
Mario Kleiner writes:
> On 12/04/2014 11:48 AM, Axel Davy wrote:
>> Le 04/12/2014 11:44, Axel Davy a écrit :
>>> On 02/12/2014 20:53, Mario Kleiner wrote :
targetSBC == 0 is a special case, which asks the function
to block until all pending OpenGL bufferswap requests have
completed
Mario Kleiner writes:
> Restores proper immediate tearing swap behaviour for
> OpenGL bufferswap under DRI3/Present.
>
> Cc: "10.3 10.4"
>
> v2: Add Frank Binns signed off by for his original earlier
> patch from April 2014, which is identical to this one, and
> Chris Wilsons reviewed tag from M
Mario Kleiner writes:
> A slightly updated and extended series of the dri3/present fixes for Mesa i
> sent last week.
>
> Patch 1 and 2 are same as before. Patch 3 now has signed off by Frank Binns
> and reviewed by Chris Wilson. Patch 4 and 5 are additional fixes. The last
> one makes INTEL_swap
On 12/04/2014 04:33 PM, Matt Turner wrote:
> On Thu, Dec 4, 2014 at 4:26 PM, Ian Romanick wrote:
>> On 12/04/2014 03:05 PM, Matt Turner wrote:
>>> Jason realized that we could fix the result of the CMP instruction on
>>> Gen <= 5 by doing -(result & 1). Also do the resolves in the vec4
>>> backend
On 12/04/2014 06:06 PM, Matt Turner wrote:
> On Fri, Nov 21, 2014 at 10:23 AM, Matt Turner wrote:
>> Thoughts?
>
> Ping^2
Seems reasonable enough... any progress on the "common ternary pattern"
better? (I deleted the original message, so I reviewed the patch from
the list archive.)
> _
On 12/04/2014 04:37 PM, Matt Turner wrote:
> The LINE instruction performs a multiply-add instruction (a * b + c)
> where b and c are scalar arguments. It reads b and c from offsets in
> src0 such that you can load them (it they're representable) as a
if?
> vect
On 12/04/2014 04:37 PM, Matt Turner wrote:
> The LINE instruction performs a multiply-add instruction (a * b + c)
> where b and c are scalar arguments. It reads b and c from offsets in
> src0 such that you can load them (it they're representable) as a
> vector-float immediate with a single instruct
On Thu, Dec 04 2014, Ian Romanick wrote:
> Should this just get squashed with the previous commit?
Yes. I only hesitated because I wrote the second and not the first, (so
I didn't want to lose authorship in the history).
But it's not useful to have an iterate in the history that actually does
the
On Thu, Dec 4, 2014 at 7:34 PM, Brian Paul wrote:
> On 12/04/2014 05:01 PM, Ilia Mirkin wrote:
>>
>> Signed-off-by: Ilia Mirkin
>> Cc: "10.3 10.4"
>> ---
>> src/gallium/auxiliary/indices/u_primconvert.c | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/src/gallium/auxiliary/indice
On Thu, Dec 4, 2014 at 6:54 PM, Ian Romanick wrote:
> On 12/04/2014 04:33 PM, Matt Turner wrote:
>> On Thu, Dec 4, 2014 at 4:26 PM, Ian Romanick wrote:
>>> On 12/04/2014 03:05 PM, Matt Turner wrote:
Jason realized that we could fix the result of the CMP instruction on
Gen <= 5 by doing
On Thu, Dec 04 2014, Ian Romanick wrote:
> Shouldn't type be glsl_base_type instead of unsigned?
Yes, thanks for the catch. Fixed.
>> + if (shadow)
>> +return (array ? sampler2DArrayShadow_type : sampler2DShadow_type
>> +);
>
> It's weird that the closing paren ended up here.
On Thu, Dec 04 2014, Matt Turner wrote:
> On Thu, Dec 4, 2014 at 2:00 PM, Carl Worth wrote:
>> + assert(!"Should not get here.");
>> + return error_type;
>
> Let's just use unreachable("not reached") here.
That's a good idea. I copied this line from the surrounding code in the
same file, (the
On Thu, Dec 04, 2014 at 05:08:21PM -0800, Matt Turner wrote:
> On Thu, Dec 4, 2014 at 3:37 PM, Ben Widawsky
> wrote:
> > The GS has an interesting use for mul. It's essentially used as a fancy mov
> > (in
> > fact, I am not sure why a mov isn't used). The documentation in the
> > function has
>
On Thu, Dec 04 2014, Carl Worth wrote:
> So I think I'll follow up with a separate patch to clean these up.
Before I do that, I noticed the following in util/macros.h:
/**
* Unreachable macro. Useful for suppressing "control reaches end of
non-void
* function" warnings.
On Thu, Dec 04, 2014 at 05:17:05PM -0800, Kenneth Graunke wrote:
> On Thursday, December 04, 2014 03:37:17 PM Ben Widawsky wrote:
> > The GS has an interesting use for mul. It's essentially used as a fancy mov
> > (in
> > fact, I am not sure why a mov isn't used). The documentation in the
> > fun
On Thu, Dec 04, 2014 at 07:48:06PM -0800, Ben Widawsky wrote:
> On Thu, Dec 04, 2014 at 05:08:21PM -0800, Matt Turner wrote:
> > On Thu, Dec 4, 2014 at 3:37 PM, Ben Widawsky
> > wrote:
> > > The GS has an interesting use for mul. It's essentially used as a fancy
> > > mov (in
> > > fact, I am not
On 12/03/2014 05:42 PM, Kenneth Graunke wrote:
> BRW_NEW_VERTICES is flagged every time we draw a primitive. Having
> the brw_vs_prog atom depend on BRW_NEW_VERTICES meant that we had to
> compute the VS program key and do a program cache lookup for every
> single primitive. This is painfully exp
On Thu, Dec 4, 2014 at 7:54 PM, Carl Worth wrote:
> On Thu, Dec 04 2014, Carl Worth wrote:
>> So I think I'll follow up with a separate patch to clean these up.
>
> Before I do that, I noticed the following in util/macros.h:
>
> /**
> * Unreachable macro. Useful for suppressing "c
fs_generator no longer knows what stage it's generating code for, so
we have to set the debug name of the shader from the call site.
Signed-off-by: Kristian Høgsberg
---
src/mesa/drivers/dri/i965/brw_blorp_blit_eu.cpp | 4 +++-
src/mesa/drivers/dri/i965/brw_fs.cpp| 17 --
The scalar vertex shader will use the ATTR register file for vertex
attributes. This patch adds support for the ATTR file to fs_visitor.
Signed-off-by: Kristian Høgsberg
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 10 +-
src/mesa/drivers/dri/i965/brw_fs.h | 3 +++
This patch uses the previous refactoring to add a new run_vs() method
that generates vertex shader code using the scalar visitor and
optimizer.
Signed-off-by: Kristian Høgsberg
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 111 +-
src/mesa/drivers/dri/i965/brw_fs.h | 21 +
1 - 100 of 120 matches
Mail list logo
