Optimize clamp(x, 0, 1), clamp(x, 0.0, b), where b < 1.0,
and clamp(x, b, 1.0), where b > 0.0 as saturate operations.
Shader-db output:
helped: shaders/0ad/9.shader_test fs16: 38 -> 37 (-2.63%)
helped: shaders/0ad/9.shader_test fs8:38 -> 37 (-2.63%)
helped: s
Signed-off-by: Abdiel Janulgue
---
src/glsl/opt_algebraic.cpp | 4
1 file changed, 4 insertions(+)
diff --git a/src/glsl/opt_algebraic.cpp b/src/glsl/opt_algebraic.cpp
index 8d7609d..0d694b6 100644
--- a/src/glsl/opt_algebraic.cpp
+++ b/src/glsl/opt_algebraic.cpp
@@ -589,6 +589,10 @@ ir_alg
Signed-off-by: Abdiel Janulgue
---
src/glsl/opt_algebraic.cpp | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/glsl/opt_algebraic.cpp b/src/glsl/opt_algebraic.cpp
index 0d694b6..ee52de7 100644
--- a/src/glsl/opt_algebraic.cpp
+++ b/src/glsl/opt_algebraic.cpp
@@ -593,6 +593,12 @@ ir_a
Signed-off-by: Abdiel Janulgue
---
src/glsl/opt_algebraic.cpp | 32
1 file changed, 32 insertions(+)
diff --git a/src/glsl/opt_algebraic.cpp b/src/glsl/opt_algebraic.cpp
index 9d55392..8d7609d 100644
--- a/src/glsl/opt_algebraic.cpp
+++ b/src/glsl/opt_algebraic.c
On Sunday 22 June 2014, Ilia Mirkin wrote:
> Fredrik,
>
> What happened to this series?
Ian suggested that we add a build-time dependency on current GL headers,
but I haven't gotten around to that yet.
It's not needed for patches 1-3 though, so those could be pushed.
Fredrik
_
Signed-off-by: Abdiel Janulgue
---
src/glsl/ir_optimization.h | 1 +
src/glsl/lower_instructions.cpp | 29 +
2 files changed, 30 insertions(+)
diff --git a/src/glsl/ir_optimization.h b/src/glsl/ir_optimization.h
index c63921c..a831adb 100644
--- a/src/glsl/ir_op
Signed-off-by: Abdiel Janulgue
---
src/glsl/ir.cpp | 2 ++
src/glsl/ir.h| 1 +
src/glsl/ir_validate.cpp | 1 +
3 files changed, 4 insertions(+)
diff --git a/src/glsl/ir.cpp b/src/glsl/ir.cpp
index 8fed768..7682e1e 100644
--- a/src/glsl/ir.cpp
+++ b/src/glsl/ir.cpp
@@ -251,6
This patch series add the plumbing to support the GLSL IR instruction
saturate().
Previously, saturate is implemented as min/max instructions. Most GPUs, however,
can probably perform saturate for free. With these changes, we can allow
saturate
to be optimized as a single instruction.
In additio
Now that we have the ir_binop_saturate implemented as a single
instruction, generate the correct simplified expression.
Signed-off-by: Abdiel Janulgue
---
src/glsl/ir_builder.cpp | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/src/glsl/ir_builder.cpp b/src/glsl/ir_builde
Signed-off-by: Abdiel Janulgue
---
src/glsl/ir_constant_expression.cpp | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/glsl/ir_constant_expression.cpp
b/src/glsl/ir_constant_expression.cpp
index 8afe8f7..b885a2f 100644
--- a/src/glsl/ir_constant_expression.cpp
+++ b/src/glsl/ir_con
Signed-off-by: Abdiel Janulgue
---
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 4
1 file changed, 4 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
b/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
index ee52c07..2112edf 100644
--- a/src/mesa/drivers/dri/i965/
Needed when vertex programs doesn't allow saturate
Signed-off-by: Abdiel Janulgue
---
src/mesa/program/ir_to_mesa.cpp| 5 -
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 6 +-
2 files changed, 9 insertions(+), 2 deletions(-)
diff --git a/src/mesa/program/ir_to_mesa.cpp b/src/
Now that sature is implemented natively as instruction,
we can cut down on uneeded functionality.
Signed-off-by: Abdiel Janulgue
---
src/mesa/drivers/dri/i965/brw_fs.h | 1 -
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 37
2 files changed, 38 deletions(
On Tuesday 24 June 2014, Carl Worth wrote:
> Brian Paul writes:
> >> On Friday 28 February 2014, Fredrik Höglund wrote:
> >> > It's possible that this patch fixes a segfault in FlightGear (see bug
> >> 73504),
> >> > so I think it's a candidate for the 10.1 branch, but maybe not for
> >> 10.1.0.
>
Now that sature is implemented natively as an instruction,
we can cut down on uneeded functionality.
Signed-off-by: Abdiel Janulgue
---
src/mesa/drivers/dri/i965/brw_vec4.h | 1 -
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 21 -
2 files changed, 22 deletions(
Signed-off-by: Abdiel Janulgue
---
src/mesa/drivers/dri/i965/brw_fs_channel_expressions.cpp | 1 +
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 4
2 files changed, 5 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_channel_expressions.cpp
b/src/mesa/drivers/dri/i96
Now that sature is implemented natively as instruction,
we can cut down on uneeded functionality
Signed-off-by: Abdiel Janulgue
---
src/mesa/program/ir_to_mesa.cpp| 48
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 51 --
2 file
Signed-off-by: Abdiel Janulgue
---
src/mesa/program/ir_to_mesa.cpp| 6 ++
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 6 ++
2 files changed, 12 insertions(+)
diff --git a/src/mesa/program/ir_to_mesa.cpp b/src/mesa/program/ir_to_mesa.cpp
index 17ccb64..4f32b7d 100644
--- a/sr
On Mon, Jun 23, 2014 at 3:52 PM, Ilia Mirkin wrote:
> On Mon, Jun 23, 2014 at 3:50 PM, Roland Scheidegger
> wrote:
>> Am 23.06.2014 21:14, schrieb Ilia Mirkin:
>>> On Mon, Jun 23, 2014 at 12:01 PM, Roland Scheidegger
>>> wrote:
Am 23.06.2014 17:18, schrieb Ilia Mirkin:
> On Mon, Jun 2
On Mon, Jun 23, 2014 at 01:30:14PM -0700, Matt Turner wrote:
> The emit_math?_gen? functions serve to implement workarounds for the
> math instruction, none of which exist on Gen8+.
There are still several restrictions with the math instruction on gen8.
Looking at the existing code, I realize I do
Hi all,
Recently, I've tried to enable my opengles application run on
tizen-ivi, tizen-ivi use wayland to display, how to enable the opengles part
show on it?
My latest patch crashed again, the following sequences describe what I
do on for the opengles parts:
1. wl_d
On 24/06/14 00:31, Carl Worth wrote:
> Adrian Negreanu writes:
>> I used commit bc8b07a6 as reference, and only the droid_display_vtbl had
>> this issue.
> ...
> CC: "10.1 10.2"
>
Believe that I'm the one "to blame" for adding the stable tag on the series.
>> - .create_pixmap_surface = dri2_
Hi folks,
I'm nearly done preparing the 10.1 and 10.2 branches for the next stable
releases, (which will also be the final release on the 10.1 branch).
The status of my 10.2 branch can be examined here:
http://cworth.org/~cworth/mesa-stable-queue/
I'm pretty happy with what it looks lik
On Monday, June 23, 2014 07:02:59 PM Neil Roberts wrote:
> Previously the blorp blitter would only be used if the format is identical
or
> there is only a difference between whether there is an alpha component or
not.
> This patch makes it also allow the blorp blitter if the only difference is
t
Hi folks,
I'm nearly done preparing the 10.1 and 10.2 branches for the next stable
releases, (which will also be the final release on the 10.1 branch).
The status of my 10.2 branch can be examined here:
http://cworth.org/~cworth/mesa-stable-queue/
I'm pretty happy with what it looks lik
On Mon, Jun 23, 2014 at 7:25 PM, Carl Worth wrote:
> Tobias Klausmann writes:
>> Some operations (e.g. OP_MUL/OP_MAD/OP_EXTBF might have a subop set.
>> After folding, make sure that it is cleared
>>
>> Signed-off-by: Tobias Klausmann
>> Reviewed-by: Ilia Mirkin
> Cc: "10.1 10.2"
>
> Hi Tobias
Grigori Goronzy writes:
> It's about as broken as on later UVD revisions.
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=66452
> Cc: "10.1 10.2"
This patch isn't picking over cleanly to 10.1. I didn't investigate what
happened (a file rename?) so perhaps the backport is actually triv
Adrian Negreanu writes:
> I used commit bc8b07a6 as reference, and only the droid_display_vtbl had this
> issue.
...
CC: "10.1 10.2"
> - .create_pixmap_surface = dri2_fallback_pixmap_surface,
> + .create_pixmap_surface = dri2_fallback_create_pixmap_surface,
This patch isn't picking cleanly
Tobias Klausmann writes:
> Some operations (e.g. OP_MUL/OP_MAD/OP_EXTBF might have a subop set.
> After folding, make sure that it is cleared
>
> Signed-off-by: Tobias Klausmann
> Reviewed-by: Ilia Mirkin
Cc: "10.1 10.2"
Hi Tobias and Ilia,
This patch isn't picking cleanly over to the 10.1 br
It did not. But, I just ran all of those tests with and without this patch,
and there were no regressions. So, I think we're good.
Pushed - thanks for the review!
On Thursday, May 29, 2014 07:32:21 PM Chris Forbes wrote:
> Did this test run include forcing ARB_gpu_shader5 on?
>
> On Thu, May
Roland Scheidegger writes:
> Am 07.06.2014 20:38, schrieb Johannes Obermayr:
>> From: Takashi Iwai
>>
>> Fix the crash of "gnome-control-center info" invocation on QEMU where
>> zero height is passed at init.
...
>> Cc: "10.2"
...
> Reviewed-by: Roland Scheidegger
Johannes, Roland,
Presumabl
Anuj Phogat writes:
> Fixes piglit tests in spec/glsl-es-3.00/compile:
...
> Cc:
For this change, (and other similar changes in the series, as well as
subsequent pre-processor changes that I am writing), I'm deciding to
generally not pick these over to the stable branch.
For one, my series is g
On 06/21/2014 06:59 PM, Marek Olšák wrote:
> That's right. A uniform won't work with ARB_draw_indirect unless you
> lower it to direct draws, which would be very bad if it was applied to
> all drivers.
>
> Radeonsi indeed supports BaseVertex and BaseInstance as system values
> in the vertex shader.
Brian Paul writes:
>> On Friday 28 February 2014, Fredrik Höglund wrote:
>> > It's possible that this patch fixes a segfault in FlightGear (see bug
>> 73504),
>> > so I think it's a candidate for the 10.1 branch, but maybe not for
>> 10.1.0.
>>
>> Ping?
>
> I took a look. I guess we don't have mu
Well, with Roland's fix to the draw module, llvmpipe works with my
tests as well. Is it likely that r600 will work as-is, or do you think
it'll require some fixes? I can try to find someone with the requisite
hw...
On Mon, Jun 23, 2014 at 4:57 PM, Marek Olšák wrote:
> The cap can be removed later
On Mon, Jun 23, 2014 at 4:13 PM, Ilia Mirkin wrote:
> On Mon, Jun 23, 2014 at 4:08 PM, wrote:
>> From: Roland Scheidegger
>>
>> The old logic would let all negative values go through unclamped, with
>> potentially disastrous results (probably trying to fetch viewport values
>> from random memor
Series is
Reviewed-by: Ian Romanick
On 06/23/2014 01:30 PM, Matt Turner wrote:
> ---
> src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 8
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
> b/src/mesa/drivers/dri/i965/b
On 23/06/14 21:21, Aaron Watry wrote:
> On Mon, Jun 23, 2014 at 1:54 PM, Emil Velikov
> wrote:
>> On 23/06/14 19:31, Aaron Watry wrote:
>>> Using /usr/local as PREFIX and an empty /usr/local/lib/vdpau directory
>>> to start, after make install of mesa I end up with
>>> /usr/local/lib/vdpau/ conta
On Mon, Jun 23, 2014 at 5:01 PM, Tobias Klausmann
wrote:
> previously, if we had something like:
>
> gl_ViewportIndex = idx;
> for(int i = 0; i < gl_in.length(); i++) {
>gl_Position = gl_in[i].gl_Position;
>EmitVertex();
> }
> EndPrimitive();
>
> we failed to set the right ViewportIndex.
>
previously, if we had something like:
gl_ViewportIndex = idx;
for(int i = 0; i < gl_in.length(); i++) {
gl_Position = gl_in[i].gl_Position;
EmitVertex();
}
EndPrimitive();
we failed to set the right ViewportIndex.
To resolve this, save the ViewportIndex and store it to the right register on
LGTM.
Jose
From: srol...@vmware.com
Sent: 23 June 2014 21:08
To: Jose Fonseca; mesa-dev@lists.freedesktop.org; imir...@alum.mit.edu
Cc: Roland Scheidegger; 10.1 10.2
Subject: [PATCH] draw: (trivial) fix clamping of viewport index
From: Roland Scheidegger
The cap can be removed later if needed.
Marek
On Mon, Jun 23, 2014 at 2:50 PM, Ilia Mirkin wrote:
> On Mon, Jun 23, 2014 at 7:55 AM, Roland Scheidegger
> wrote:
>> Am 23.06.2014 12:49, schrieb Roland Scheidegger:
>>> Am 22.06.2014 17:10, schrieb Ilia Mirkin:
Signed-off-by: Ilia Mirkin
>>
On Sun, Jun 22, 2014 at 04:05:49PM +0200, Francisco Jerez wrote:
> Bruno Jimenez writes:
>
> > On Sat, 2014-06-21 at 17:39 +0200, Francisco Jerez wrote:
> >[...]
> >> The implementation of PIPE_TRANSFER_MAP_DIRECTLY introduced in PATCH 10
> >> has somewhat worrying semantics: A mapping with this
The emit_math?_gen? functions serve to implement workarounds for the
math instruction, none of which exist on Gen8+.
---
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
b/src/
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
b/src/mesa/drivers/dri/i965/brw_fs.cpp
index d7b969e..185a1f6 100644
--- a/src/mesa/drivers/dri/i965/brw_fs.cpp
+++ b/src/mesa/drivers/dri/i965/b
---
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
b/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
index 7fd8c2b..f3316f8 100644
--- a/src/mesa/drivers/dri/i965/brw_vec4_vis
On Mon, Jun 23, 2014 at 1:54 PM, Emil Velikov wrote:
> On 23/06/14 19:31, Aaron Watry wrote:
>> Using /usr/local as PREFIX and an empty /usr/local/lib/vdpau directory
>> to start, after make install of mesa I end up with
>> /usr/local/lib/vdpau/ containing:
>>
>> awatry@ws-awatry:/usr/local/lib/vd
On Mon, Jun 23, 2014 at 4:08 PM, wrote:
> From: Roland Scheidegger
>
> The old logic would let all negative values go through unclamped, with
> potentially disastrous results (probably trying to fetch viewport values
> from random memory locations). GL has undefined rendering for vp indices
> ou
From: Roland Scheidegger
The old logic would let all negative values go through unclamped, with
potentially disastrous results (probably trying to fetch viewport values
from random memory locations). GL has undefined rendering for vp indices
outside valid range but that's a bit too undefined...
(
On Mon, Jun 23, 2014 at 3:50 PM, Roland Scheidegger wrote:
> Am 23.06.2014 21:14, schrieb Ilia Mirkin:
>> On Mon, Jun 23, 2014 at 12:01 PM, Roland Scheidegger
>> wrote:
>>> Am 23.06.2014 17:18, schrieb Ilia Mirkin:
On Mon, Jun 23, 2014 at 11:06 AM, Roland Scheidegger
wrote:
> Am
Am 23.06.2014 21:14, schrieb Ilia Mirkin:
> On Mon, Jun 23, 2014 at 12:01 PM, Roland Scheidegger
> wrote:
>> Am 23.06.2014 17:18, schrieb Ilia Mirkin:
>>> On Mon, Jun 23, 2014 at 11:06 AM, Roland Scheidegger
>>> wrote:
Am 23.06.2014 16:43, schrieb Ilia Mirkin:
> On Mon, Jun 23, 2014 at
I'm afraid that I cannot predict the future ("is always going to be.."). IMHO
we should keep the major, for when the loader gains different codepaths for
v1.0 vs v2.0 etc. Adding the mesa version will easily allow us to check for
broken/manged installs and possibly other issues.
Don't plan on touc
If the version is always going to be 1.0.0, it's pointless. If it's
going to be equal to the Mesa version, then it might find some use.
Marek
On Mon, Jun 23, 2014 at 9:12 PM, Emil Velikov wrote:
> The vdpau loader requires only major, and falls back to an unversioned SO.
> Thus IMHO we should ke
Emil Velikov wrote:
On 23/06/14 19:58, Andy Furniss wrote:
I still don't get libvdpau_gallium installed.
Not sure what made you believe that libvdpau_gallium should be installed, but
that truly is not the case.
Ahh, it was looking at the output of make install, which usually flies
past an
On Mon, Jun 23, 2014 at 12:06 PM, Neil Roberts wrote:
> Matt Turner writes:
>
>> We typically write
>>
>> Bugzilla: https:///
>>
>> Reviewed-by: Matt Turner
>>
>> Do you have commit access?
>
> Thanks for the review. I do have commit access so I've pushed the patch
> with the suggested chang
On Mon, Jun 23, 2014 at 12:01 PM, Roland Scheidegger wrote:
> Am 23.06.2014 17:18, schrieb Ilia Mirkin:
>> On Mon, Jun 23, 2014 at 11:06 AM, Roland Scheidegger
>> wrote:
>>> Am 23.06.2014 16:43, schrieb Ilia Mirkin:
On Mon, Jun 23, 2014 at 9:51 AM, Ilia Mirkin wrote:
> On Mon, Jun 23,
The vdpau loader requires only major, and falls back to an unversioned SO.
Thus IMHO we should keep at least the major.
In the future we might want to move to a versioning scheme similar to nvidia,
which lacks the revision number, thus we'll be OK with the BSD guys :)
libvdpau_nvidia.so -> libvdp
Matt Turner writes:
> We typically write
>
> Bugzilla: https:///
>
> Reviewed-by: Matt Turner
>
> Do you have commit access?
Thanks for the review. I do have commit access so I've pushed the patch
with the suggested change to the commit message.
I've also started to try and look at why tha
On 23/06/14 19:58, Andy Furniss wrote:
> Emil Velikov wrote:
>> Previous two commits, resolved the symlink generation required
>> by the versioning of the library and incorrectly changed the
>> way hardlinks are created by linking the ones from the build
>> tree. If the device used for building dif
On Mon, Jun 23, 2014 at 2:15 PM, Tobias Klausmann
wrote:
> previously, if we had something like:
>
> gl_ViewportIndex = idx;
> for(int i = 0; i < gl_in.length(); i++) {
>gl_Position = gl_in[i].gl_Position;
>EmitVertex();
> }
> EndPrimitive();
>
> we failed to set the right ViewportIndex.
>
Emil Velikov wrote:
Previous two commits, resolved the symlink generation required
by the versioning of the library and incorrectly changed the
way hardlinks are created by linking the ones from the build
tree. If the device used for building differs from the one
set as destination linking will f
On 23/06/14 19:31, Aaron Watry wrote:
> Using /usr/local as PREFIX and an empty /usr/local/lib/vdpau directory
> to start, after make install of mesa I end up with
> /usr/local/lib/vdpau/ containing:
>
> awatry@ws-awatry:/usr/local/lib/vdpau$ ls -l
> total 26944
> lrwxrwxrwx 1 root root 22 J
On Sat, Jun 21, 2014 at 06:33:17PM +0200, Francisco Jerez wrote:
> Tom Stellard writes:
>
> > From: Matt Arsenault
> >
> > If there were only warnings, they would not be added to the log.
> > Also fixes valgrind use after free errors.
> > ---
> > src/gallium/state_trackers/clover/core/compiler.
On a different note, do we really need the suffixes .1.0.0 etc.? It's
not like the version is going to change.
Marek
On Mon, Jun 23, 2014 at 8:31 PM, Aaron Watry wrote:
> Using /usr/local as PREFIX and an empty /usr/local/lib/vdpau directory
> to start, after make install of mesa I end up with
>
On Mon, Jun 23, 2014 at 12:10 PM, Ilia Mirkin wrote:
> On Mon, Jun 23, 2014 at 1:07 PM, Aaron Watry wrote:
>> On my machine, ${PREFIX}/lib/vdpau contains:
>> libvdpau_gallium.so.1 -> libvdpau_r600.so.1.0.0
>> libvdpau_r600.so*
>> libvdpau_radeonsi.so*
>>
>> Note that libvdpau_gallium.so.1 is only
On Mon, Jun 23, 2014 at 11:02 AM, Neil Roberts wrote:
> Previously the blorp blitter would only be used if the format is identical or
> there is only a difference between whether there is an alpha component or not.
> This patch makes it also allow the blorp blitter if the only difference is the
>
Using /usr/local as PREFIX and an empty /usr/local/lib/vdpau directory
to start, after make install of mesa I end up with
/usr/local/lib/vdpau/ containing:
awatry@ws-awatry:/usr/local/lib/vdpau$ ls -l
total 26944
lrwxrwxrwx 1 root root 22 Jun 23 13:25 libvdpau_r600.so ->
libvdpau_r600.so.1.0
previously, if we had something like:
gl_ViewportIndex = idx;
for(int i = 0; i < gl_in.length(); i++) {
gl_Position = gl_in[i].gl_Position;
EmitVertex();
}
EndPrimitive();
we failed to set the right ViewportIndex.
To resolve this, save the ViewportIndex and store it to the right register on
On 23/06/14 18:07, Aaron Watry wrote:
> On my machine, ${PREFIX}/lib/vdpau contains:
> libvdpau_gallium.so.1 -> libvdpau_r600.so.1.0.0
> libvdpau_r600.so*
> libvdpau_radeonsi.so*
>
> Note that libvdpau_gallium.so.1 is only created when I force an
> ldconfig on my system (until then, I just have
>
Previously the blorp blitter would only be used if the format is identical or
there is only a difference between whether there is an alpha component or not.
This patch makes it also allow the blorp blitter if the only difference is the
ordering of the RGB components (ie, RGB or BGR).
This is parti
On 06/21/2014 08:36 AM, Roland Scheidegger wrote:
> Am 21.06.2014 03:00, schrieb Ian Romanick:
>> This patch series fixes bugs in the i965 w.r.t. several uses of
>> gl_VertexID. OpenGL (desktop and ES) have the following expectations of
>> gl_VertexID:
>>
>> 1. When used with BaseVertex drawing co
On 06/22/2014 04:32 AM, Marek Olšák wrote:
> If you define enums in a struct or class, it then works like a
> namespace. E.g. the proper form to use such an enum would be:
>
> ctx->Const.VertexID = gl_constants::gl_VertexID_native;
I'd swear that I tried that... I know I tried a bunch of other
va
Previous two commits, resolved the symlink generation required
by the versioning of the library and incorrectly changed the
way hardlinks are created by linking the ones from the build
tree. If the device used for building differs from the one
set as destination linking will fail.
Reported-by: And
On Mon, Jun 23, 2014 at 9:48 AM, Brian Paul wrote:
> From: Fredrik Höglund
>
> Updated patch for refactored glxinfo/wglinfo code by Brian Paul.
There were more patches that added more things...
http://patchwork.freedesktop.org/project/mesa/list/?q=glxinfo
BTW, is the intention of glxinfo -l to
Emil Velikov wrote:
Yes I had a few copy/paste typos that were causing make install to fall short
when generating the (sym|hard)links. Should be fixed with commit 11e46a32aed.
Let me know if latest master work for you.
No, it fails to install anything to do with libvdpau_gallium* which is
pr
On Mon, Jun 23, 2014 at 1:07 PM, Aaron Watry wrote:
> On my machine, ${PREFIX}/lib/vdpau contains:
> libvdpau_gallium.so.1 -> libvdpau_r600.so.1.0.0
> libvdpau_r600.so*
> libvdpau_radeonsi.so*
>
> Note that libvdpau_gallium.so.1 is only created when I force an
> ldconfig on my system (until then,
On my machine, ${PREFIX}/lib/vdpau contains:
libvdpau_gallium.so.1 -> libvdpau_r600.so.1.0.0
libvdpau_r600.so*
libvdpau_radeonsi.so*
Note that libvdpau_gallium.so.1 is only created when I force an
ldconfig on my system (until then, I just have
libvdpau_[r600|radeonsi]*)
For some reason, while the
On 23/06/14 16:10, Andy Furniss wrote:
> Emil Velikov wrote:
>> Hi all,
>>
>> These patches add support for building (grouping) the various targets
>> per API, meaning that only one library will be created for e.g.
>> vdpau (libvdpau_gallium) with individual ones (libvdpau_r600) being a
>> hardlin
Am 23.06.2014 17:18, schrieb Ilia Mirkin:
> On Mon, Jun 23, 2014 at 11:06 AM, Roland Scheidegger
> wrote:
>> Am 23.06.2014 16:43, schrieb Ilia Mirkin:
>>> On Mon, Jun 23, 2014 at 9:51 AM, Ilia Mirkin wrote:
On Mon, Jun 23, 2014 at 9:39 AM, Ilia Mirkin wrote:
> If multiple viewports are
On Mon, Jun 23, 2014 at 11:24 AM, Tobias Klausmann
wrote:
Please add a brief description of what your change does and how it
achieves this. [Let me know if you're not comfortable writing that,
and I can compose it for you.]
Among other things, note that it fixes some piglit tests. And instead
of
Signed-off-by: Tobias Klausmann
---
.../drivers/nouveau/codegen/nv50_ir_driver.h | 1 +
.../drivers/nouveau/codegen/nv50_ir_from_tgsi.cpp | 27 --
2 files changed, 26 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_driver.h
b/sr
On Mon, Jun 23, 2014 at 11:06 AM, Roland Scheidegger wrote:
> Am 23.06.2014 16:43, schrieb Ilia Mirkin:
>> On Mon, Jun 23, 2014 at 9:51 AM, Ilia Mirkin wrote:
>>> On Mon, Jun 23, 2014 at 9:39 AM, Ilia Mirkin wrote:
If multiple viewports are supported, that implies the presence of a GS
On Mon, 2014-06-23 at 10:39 -0400, Jan Vesely wrote:
> The dst pointer needs to be initialized after any calls to
> compute_memory_grow_pool, as the function might change the pool->vbo pointer.
You are completely right. Good catch.
Reviewed-by: Bruno Jiménez
Sorry for any inconvenience this ma
Am 23.06.2014 16:43, schrieb Ilia Mirkin:
> On Mon, Jun 23, 2014 at 9:51 AM, Ilia Mirkin wrote:
>> On Mon, Jun 23, 2014 at 9:39 AM, Ilia Mirkin wrote:
>>> If multiple viewports are supported, that implies the presence of a GS
>>> and layered rendering, so we can enable ARB_fragment_layer_viewport
Emil Velikov wrote:
Hi all,
These patches add support for building (grouping) the various targets
per API, meaning that only one library will be created for e.g.
vdpau (libvdpau_gallium) with individual ones (libvdpau_r600) being a
hardlink to it.
How is this supposed to work from a users poi
On Mon, Jun 23, 2014 at 3:17 AM, Maarten Lankhorst
wrote:
> op 21-06-14 14:12, Ilia Mirkin schreef:
>> On Tue, Jun 17, 2014 at 2:34 AM, Maarten Lankhorst
>> wrote:
>>> nv30 seems to not support dma objects with offset, so simply extend the
>>> query_heap to cover the
>>> entire notifier, and use
On Mon, Jun 23, 2014 at 9:51 AM, Ilia Mirkin wrote:
> On Mon, Jun 23, 2014 at 9:39 AM, Ilia Mirkin wrote:
>> If multiple viewports are supported, that implies the presence of a GS
>> and layered rendering, so we can enable ARB_fragment_layer_viewport as
>> well.
>>
>> Signed-off-by: Ilia Mirkin
The dst pointer needs to be initialized after any calls to
compute_memory_grow_pool, as the function might change the pool->vbo pointer.
This fixes crashes and assertion failures in two gegl tests.
Signed-off-by: Jan Vesely
CC: Bruno Jimenez
CC: Tom Stellard
---
src/gallium/drivers/r600/comp
On Mon, Jun 23, 2014 at 9:39 AM, Ilia Mirkin wrote:
> If multiple viewports are supported, that implies the presence of a GS
> and layered rendering, so we can enable ARB_fragment_layer_viewport as
> well.
>
> Signed-off-by: Ilia Mirkin
> ---
>
> Untested on r600, but nv50/nvc0/llvmpipe seem to p
From: Fredrik Höglund
Updated patch for refactored glxinfo/wglinfo code by Brian Paul.
Signed-off-by: Brian Paul
---
src/xdemos/glinfo_common.c | 22 ++
1 file changed, 22 insertions(+)
diff --git a/src/xdemos/glinfo_common.c b/src/xdemos/glinfo_common.c
index e6517d7..4
I happened to add some of this to glxinfo last week. I'll rebase
Fredrik's patch on the latest (refactored) glxinfo...
-Brian
On 06/22/2014 09:55 AM, Ilia Mirkin wrote:
Fredrik,
What happened to this series?
-ilia
On Thu, Apr 10, 2014 at 10:09 AM, Brian Paul wrote:
On 04/09/2014 07:51
On 06/22/2014 10:57 AM, Daniel Manjarres wrote:
Author: Daniel Manjarres
Date: Sun Jun 22 09:47:58 2014 -0700
glx: Fix glxUseXFont for glxWindow and glxPixmaps
The current implementation of glxUseXFont requires creating
a temporary pixmap and graphics context, which requires a
Signed-off-by: Ilia Mirkin
---
src/gallium/drivers/nouveau/nvc0/nvc0_program.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_program.c
b/src/gallium/drivers/nouveau/nvc0/nvc0_program.c
index 667fbc8..ad8d1b0 100644
--- a/src/gallium/drivers/nouveau/n
If multiple viewports are supported, that implies the presence of a GS
and layered rendering, so we can enable ARB_fragment_layer_viewport as
well.
Signed-off-by: Ilia Mirkin
---
Untested on r600, but nv50/nvc0/llvmpipe seem to pass basic testing.
docs/GL3.txt | 2 +-
Signed-off-by: Ilia Mirkin
Reviewed-by: Marek Olšák
---
src/mesa/state_tracker/st_program.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/mesa/state_tracker/st_program.c
b/src/mesa/state_tracker/st_program.c
index 26eb978..1df411c 100644
--- a/src/mesa/state_tracker/st_program.c
On Mon, Jun 23, 2014 at 1:27 AM, Kenneth Graunke wrote:
> On i965, enabling and disabling the GS is not free: you have to do a
> full pipeline stall, reconfigure the URB and push constant space, and
> emit a bunch of state. Most clears aren't layered, so the GS isn't
> needed in the common case.
On Mon, Jun 23, 2014 at 7:55 AM, Roland Scheidegger wrote:
> Am 23.06.2014 12:49, schrieb Roland Scheidegger:
>> Am 22.06.2014 17:10, schrieb Ilia Mirkin:
>>> Signed-off-by: Ilia Mirkin
>>> ---
>>> docs/GL3.txt| 2 +-
>>> docs/relnotes/10.3.html
Am 23.06.2014 12:49, schrieb Roland Scheidegger:
> Am 22.06.2014 17:10, schrieb Ilia Mirkin:
>> Signed-off-by: Ilia Mirkin
>> ---
>> docs/GL3.txt| 2 +-
>> docs/relnotes/10.3.html | 2 +-
>> src/gallium/drivers/nouveau/nvc0/nvc0_program.
Am 22.06.2014 17:10, schrieb Ilia Mirkin:
> Signed-off-by: Ilia Mirkin
> ---
> docs/GL3.txt| 2 +-
> docs/relnotes/10.3.html | 2 +-
> src/gallium/drivers/nouveau/nvc0/nvc0_program.c | 2 ++
> src/gallium/drivers/nouveau/nvc0/nvc0_screen
Reviewed-by: Chris Forbes
Have you got a case where this makes a noticeable difference to performance?
On Mon, Jun 23, 2014 at 5:27 PM, Kenneth Graunke wrote:
> On i965, enabling and disabling the GS is not free: you have to do a
> full pipeline stall, reconfigure the URB and push constant spac
1 - 100 of 105 matches
Mail list logo