https://bugs.freedesktop.org/show_bug.cgi?id=86701
--- Comment #15 from Pekka Paalanen ppaala...@gmail.com ---
The patch series has landed in Mesa, particularly:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=cdcfe48fb0431184fabb40aa5a244d086f551df5
Feel free to test how the software GL on
From: Marta Lofstedt marta.lofst...@intel.com
v2 : Refine the restriction to Version == 30.
Signed-off-by: Marta Lofstedt marta.lofst...@intel.com
---
src/mesa/main/multisample.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/mesa/main/multisample.c
Patches 7 8:
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Tue, May 12, 2015 at 5:54 PM, Adam Jackson a...@redhat.com wrote:
This still doesn't do anything unless your driver exposes the flush
control extension.
Signed-off-by: Adam Jackson a...@redhat.com
---
Events can be added to an OpenCL command queue concurrently from
multiple threads, but pipe_context and pipe_screen objects
are not threadsafe. The threadsafe wrappers protect all pipe_screen
and pipe_context function calls with a mutex, so we can
safely use them with multiple threads.
CC: 10.5
These wrappers can be used by state trackers to ensure threadsafe access
to pipe_context and pipe_screen objects.
CC: 10.5 mesa-sta...@lists.freedesktop.org
---
src/gallium/SConscript | 1 +
src/gallium/drivers/threadsafe/Makefile.am | 11 +
On Wed, May 6, 2015 at 9:36 AM, Samuel Pitoiset
samuel.pitoi...@gmail.com wrote:
On 05/06/2015 02:53 AM, Robert Bragg wrote:
As we've learned more about the observability capabilities of Gen
graphics we've found that it's not enough to only try and configure the
OA unit from userspace
On Tue, May 12, 2015 at 5:18 AM, Francisco Jerez curroje...@riseup.net wrote:
Kenneth Graunke kenn...@whitecape.org writes:
On Monday, May 11, 2015 05:12:41 PM Matt Turner wrote:
I think set_* would be better.
Agreed. Reading this in action...
exec_all(bld.MOV(...))
sounds a bit funny
Petri Latval wrote some piglit test for the core
INTEL_performance_query behaviour (i.e not assuming anything about the
semantics of the counters themselves) that he sent to the list some
time last year, but unfortunately they fell through the cracks and
never landed.
The other week I revived
Tested-by: Mark Janes mark.a.ja...@intel.com
Tapani Pälli tapani.pa...@intel.com writes:
Commit dd5c825 changed the way how execution size for instructions
get set. Previously it was based on destination register width, now
it is set explicitly when emitting instructions.
Signed-off-by:
Signed-off-by: Alex Deucher alexander.deuc...@amd.com
---
include/pci_ids/radeonsi_pci_ids.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/include/pci_ids/radeonsi_pci_ids.h
b/include/pci_ids/radeonsi_pci_ids.h
index 464031c..197ce8e 100644
--- a/include/pci_ids/radeonsi_pci_ids.h
+++
For the series:
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Tue, May 12, 2015 at 7:35 PM, Alex Deucher alexdeuc...@gmail.com wrote:
Signed-off-by: Alex Deucher alexander.deuc...@amd.com
Cc: mesa-sta...@lists.freedesktop.org
---
include/pci_ids/radeonsi_pci_ids.h | 1 +
1 file
Signed-off-by: Alex Deucher alexander.deuc...@amd.com
Cc: mesa-sta...@lists.freedesktop.org
---
include/pci_ids/radeonsi_pci_ids.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/include/pci_ids/radeonsi_pci_ids.h
b/include/pci_ids/radeonsi_pci_ids.h
index ca81f76..464031c 100644
---
On Tue, May 12, 2015 at 4:24 AM, Tapani Pälli tapani.pa...@intel.com wrote:
Commit dd5c825 changed the way how execution size for instructions
get set. Previously it was based on destination register width, now
it is set explicitly when emitting instructions.
Signed-off-by: Tapani Pälli
On Mon 11 May 2015, Emil Velikov wrote:
The EGL 1.3, 1.4 and 1.5 spec explicitly mentions that providing
static symbols for functions provided by EGL extensions is not portable.
Considering that relatively recently we've seen a non-mesa desktop EGL
implementation, the fact that we opt for such
On Mon 11 May 2015, Emil Velikov wrote:
The only user (egl_gallium) is not longer around.
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
For the series,
Reviewed-by: Chad Versace chad.vers...@intel.com
___
mesa-dev mailing list
On Tue 12 May 2015, Chad Versace wrote:
On Mon 11 May 2015, Emil Velikov wrote:
The only user (egl_gallium) is not longer around.
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
For the series,
Reviewed-by: Chad Versace chad.vers...@intel.com
Um... except for patch 5, the one
From: Rob Clark robcl...@freedesktop.org
Whitelist adreno 306 (as found in msm8916/apq8016). Works pretty much
out of the box, although the smaller GMEM size requires more tiles to
fit 1920x1080, so bump up the max # of tiles as well.
Since it is just whitelist + trivial change, it makes sense
On Mon, 2015-05-11 at 12:37 +0200, Alejandro Piñeiro wrote:
Since commit c0cd5b var-data.binding was set only when explicit_binding
was false, thas was wrong, should be a test to true. This prevented
to use any binding point different to 0.
In any case, that if statement is not needed. Right
On Mon, May 11, 2015 at 4:11 PM, Samuel Pitoiset
samuel.pitoi...@gmail.com wrote:
Patches 1 and 2 look fine to me.
See my comments below for this one.
On 05/06/2015 02:53 AM, Robert Bragg wrote:
--- a/src/mesa/main/performance_query.c
+++ b/src/mesa/main/performance_query.c
@@ -1,5 +1,5
https://bugs.freedesktop.org/show_bug.cgi?id=90147
Brian Paul bri...@vmware.com changed:
What|Removed |Added
CC||julien.iso...@gmail.com
On Tue, May 12, 2015 at 7:55 AM, Francisco Jerez curroje...@riseup.net wrote:
This problem can easily be reproduced with a number of
ARB_shader_image_load_store piglit tests, which use a buffer object as
PBO for a pixel transfer operation and later on bind the same buffer
to the pipeline as
I like faster!
I don't really understand though where the wins come from.
softpipe can't have much pending work in the first place (maybe tile
cache?), llvmpipe potentially has the rasterizer threads I guess but I'm
not quite sure how this works...
Though I'm not really too familiar with the dri
Reviewed-by: Jason Ekstrand jason.ekstr...@intel.com
On May 12, 2015 7:55 AM, Francisco Jerez curroje...@riseup.net wrote:
This problem can easily be reproduced with a number of
ARB_shader_image_load_store piglit tests, which use a buffer object as
PBO for a pixel transfer operation and later
From: Roland Scheidegger srol...@vmware.com
All the functionality was pretty much there, just not tested.
Trivially fix up the missing pieces (take target info from view not
resource), and add some missing bits for cubes.
Also add some minimal debug validation to detect uninitialized target
On Tue, May 12, 2015 at 10:36 PM, srol...@vmware.com wrote:
From: Roland Scheidegger srol...@vmware.com
Some bits were already there for texture views but some were missing.
In particular for cube map views things needed to change a bit.
For simplicity I ended up removing the separate face
On 13 May 2015 at 12:36, srol...@vmware.com wrote:
From: Roland Scheidegger srol...@vmware.com
This was missing, and drivers relying on the target in the view could get
into quite some trouble.
Missing signoff,
otherwise
Reviewed-by: Dave Airlie airl...@redhat.com
---
From: Roland Scheidegger srol...@vmware.com
This was missing, and drivers relying on the target in the view could get
into quite some trouble.
---
src/gallium/auxiliary/util/u_blitter.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/auxiliary/util/u_blitter.c
From: Roland Scheidegger srol...@vmware.com
Some bits were already there for texture views but some were missing.
In particular for cube map views things needed to change a bit.
For simplicity I ended up removing the separate face addr bit (just use
the z bit) - cube arrays didn't use it already,
Commit dd5c825 changed the way how execution size for instructions
get set. Previously it was based on destination register width, now
it is set explicitly when emitting instructions.
Signed-off-by: Tapani Pälli tapani.pa...@intel.com
Reviewed-by: Francisco Jerez curroje...@riseup.net
Bugzilla:
For the series:
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Tue, May 12, 2015 at 12:43 AM, Emil Velikov emil.l.veli...@gmail.com wrote:
The only user (egl_gallium) is not longer around.
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com
---
src/egl/main/egldriver.c | 364
This is required for the I915_PARAM_REVISION macro. Previously this
define was directly copied into the Mesa source.
---
configure.ac | 2 +-
src/mesa/drivers/dri/i965/intel_screen.c | 5 -
2 files changed, 1 insertion(+), 6 deletions(-)
diff --git a/configure.ac
https://bugs.freedesktop.org/show_bug.cgi?id=90162
Tapani Pälli lem...@gmail.com changed:
What|Removed |Added
Status|NEW |ASSIGNED
https://bugs.freedesktop.org/show_bug.cgi?id=86701
--- Comment #16 from nerdopol...@verizon.net ---
Created attachment 115713
-- https://bugs.freedesktop.org/attachment.cgi?id=115713action=edit
backtrace of weston-desktop-shell
backtrace of weston-desktop-shell on qemu with Weston running
On Sat, 2015-04-18 at 12:26 +0200, Marek Olšák wrote:
On Fri, Apr 17, 2015 at 1:21 PM, Timothy Arceri t_arc...@yahoo.com.au wrote:
Hi all,
Last year I spent a whole bunch of time profiling Mesa looking for areas
where improvements could be made. Anyway I thought I'd point out a
couple
From: Marek Olšák marek.ol...@amd.com
These are just wrappers around the existing extension functions.
---
src/egl/main/eglapi.c | 45 +
1 file changed, 45 insertions(+)
diff --git a/src/egl/main/eglapi.c b/src/egl/main/eglapi.c
index 34a113b..e3b8ec2
Hi,
This patch series adds most things needed to enable EGL 1.5. The last missing
features are EGL_KHR_gl_colorspace and EGL_KHR_gl_texture_3D_image, though the
latter should be at least partially done I think.
Please review.
Marek
___
mesa-dev
From: Marek Olšák marek.ol...@amd.com
Declare the functions without the suffix, so that the core names are exported.
---
src/egl/main/eglapi.c | 20
1 file changed, 12 insertions(+), 8 deletions(-)
diff --git a/src/egl/main/eglapi.c b/src/egl/main/eglapi.c
index
From: Marek Olšák marek.ol...@amd.com
---
src/egl/main/eglapi.c | 14 +-
src/egl/main/eglapi.h | 2 +-
src/egl/main/eglsync.c | 2 +-
src/egl/main/eglsync.h | 2 +-
4 files changed, 16 insertions(+), 4 deletions(-)
diff --git a/src/egl/main/eglapi.c b/src/egl/main/eglapi.c
From: Marek Olšák marek.ol...@amd.com
---
include/EGL/eglmesaext.h | 28
1 file changed, 28 deletions(-)
diff --git a/include/EGL/eglmesaext.h b/include/EGL/eglmesaext.h
index 27cf7eb..80601f0 100644
--- a/include/EGL/eglmesaext.h
+++ b/include/EGL/eglmesaext.h
@@
From: Marek Olšák marek.ol...@amd.com
---
src/egl/main/eglcontext.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/egl/main/eglcontext.c b/src/egl/main/eglcontext.c
index 514b91a..e50b8fb 100644
--- a/src/egl/main/eglcontext.c
+++ b/src/egl/main/eglcontext.c
@@
https://bugs.freedesktop.org/show_bug.cgi?id=86701
--- Comment #19 from nerdopol...@verizon.net ---
As suggested as a test, before running Weston, I exported
vblank_mode=0
And this prevents the hangs
--
You are receiving this mail because:
You are the assignee for the bug.
From: Marek Olšák marek.ol...@amd.com
with the extension of keeping:
#define KHRONOS_APICALL __attribute__((visibility(default)))
And don't include mesa headers in egl.h.
---
include/EGL/egl.h | 562 +
include/EGL/eglext.h | 259
From: Marek Olšák marek.ol...@amd.com
---
src/egl/main/eglcontext.c | 30 ++
1 file changed, 30 insertions(+)
diff --git a/src/egl/main/eglcontext.c b/src/egl/main/eglcontext.c
index e50b8fb..e767f4b 100644
--- a/src/egl/main/eglcontext.c
+++
From: Marek Olšák marek.ol...@amd.com
---
src/egl/main/eglapi.c | 9 -
src/egl/main/egldisplay.h | 3 +--
2 files changed, 5 insertions(+), 7 deletions(-)
diff --git a/src/egl/main/eglapi.c b/src/egl/main/eglapi.c
index d9053dc..51c2e78 100644
--- a/src/egl/main/eglapi.c
+++
From: Marek Olšák marek.ol...@amd.com
---
src/egl/main/eglapi.c | 12
1 file changed, 12 insertions(+)
diff --git a/src/egl/main/eglapi.c b/src/egl/main/eglapi.c
index 60df297..544f7e4 100644
--- a/src/egl/main/eglapi.c
+++ b/src/egl/main/eglapi.c
@@ -1162,6 +1162,7 @@
From: Marek Olšák marek.ol...@amd.com
There's no driver support yet, because EGL_KHR_gl_colorspace isn't
implemented.
---
src/egl/main/eglapi.c | 15 +++
src/egl/main/egldisplay.h | 1 +
2 files changed, 16 insertions(+)
diff --git a/src/egl/main/eglapi.c
From: Marek Olšák marek.ol...@amd.com
---
src/egl/main/eglglobals.c | 10 --
src/egl/main/eglglobals.h | 9 -
2 files changed, 19 deletions(-)
diff --git a/src/egl/main/eglglobals.c b/src/egl/main/eglglobals.c
index 129bf29..884cff0 100644
--- a/src/egl/main/eglglobals.c
+++
From: Marek Olšák marek.ol...@amd.com
---
src/egl/drivers/dri2/platform_android.c | 4
src/egl/drivers/dri2/platform_drm.c | 4
src/egl/drivers/dri2/platform_wayland.c | 8
src/egl/drivers/dri2/platform_x11.c | 8
src/egl/main/eglapi.c | 7
From: Marek Olšák marek.ol...@amd.com
---
src/egl/main/eglapi.c | 38 ++
1 file changed, 38 insertions(+)
diff --git a/src/egl/main/eglapi.c b/src/egl/main/eglapi.c
index 6457798..34a113b 100644
--- a/src/egl/main/eglapi.c
+++ b/src/egl/main/eglapi.c
@@
From: Marek Olšák marek.ol...@amd.com
---
src/egl/drivers/dri2/egl_dri2.c | 4 ++--
src/egl/main/eglapi.c | 40
src/egl/main/eglapi.h | 4 ++--
src/egl/main/eglimage.h | 10 +-
src/egl/main/eglsync.c | 4
From: Marek Olšák marek.ol...@amd.com
---
src/egl/main/eglapi.c | 27 +++
1 file changed, 19 insertions(+), 8 deletions(-)
diff --git a/src/egl/main/eglapi.c b/src/egl/main/eglapi.c
index e3b8ec2..ff0217e 100644
--- a/src/egl/main/eglapi.c
+++ b/src/egl/main/eglapi.c
@@
On Tue 12 May 2015, Emil Velikov wrote:
The extension requires that the address of the core functions should be
available via eglGetProcAddress. Currently the list is guarded by
_EGL_GET_CORE_ADDRESSES, which was only set for the scons (windows)
build.
Unconditionally enable it for all the
https://bugs.freedesktop.org/show_bug.cgi?id=90147
--- Comment #11 from Julien Isorce julien.iso...@gmail.com ---
(In reply to Brian Paul from comment #10)
Julien, can you test the new patch?
It is missing a #else just before #error Unsupported platform, otherwise it
is working fine.
--
You
We could keep them in comments...
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
On 2015-05-12 17:01:17, Dave Airlie wrote:
From: Dave Airlie airl...@redhat.com
gcc 4.4.7 really doesn't like them, and they aren't standard
C++, they seem to be a gcc extension.
Signed-off-by: Dave
https://bugs.freedesktop.org/show_bug.cgi?id=86701
--- Comment #18 from nerdopol...@verizon.net ---
Hi. I realized that the hang happens when I hover over an icon on the desktop
shell, and it tries to show a tooltip
Output of weston running with WAYLAND_DEBUG=server
http://pastebin.com/0aQGBLFH
From: Dave Airlie airl...@redhat.com
gcc 4.4.7 really doesn't like them, and they aren't standard
C++, they seem to be a gcc extension.
Signed-off-by: Dave Airlie airl...@redhat.com
---
src/mesa/drivers/dri/i965/brw_cs.cpp | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff
https://bugs.freedesktop.org/show_bug.cgi?id=86701
Pekka Paalanen ppaala...@gmail.com changed:
What|Removed |Added
CC||axel.d...@ens.fr
https://bugs.freedesktop.org/show_bug.cgi?id=77449
Bug 77449 depends on bug 84155, which changed state.
Bug 84155 Summary: Incorrect fog of war effect in Wasteland 2
https://bugs.freedesktop.org/show_bug.cgi?id=84155
What|Removed |Added
Tom Stellard t...@stellard.net writes:
On Sat, May 09, 2015 at 04:42:29PM +0300, Francisco Jerez wrote:
Refactor ::trigger and ::abort to split out the operations that access
concurrently modified data members and require locking from the
recursive and possibly re-entrant part of these
Kenneth Graunke kenn...@whitecape.org writes:
On Monday, May 11, 2015 05:12:41 PM Matt Turner wrote:
On Tue, May 5, 2015 at 9:57 AM, Francisco Jerez curroje...@riseup.net
wrote:
Kenneth Graunke kenn...@whitecape.org writes:
On Saturday, May 02, 2015 06:29:30 PM Francisco Jerez wrote:
Am 13.05.2015 um 04:45 schrieb Ilia Mirkin:
On Tue, May 12, 2015 at 10:36 PM, srol...@vmware.com wrote:
From: Roland Scheidegger srol...@vmware.com
Some bits were already there for texture views but some were missing.
In particular for cube map views things needed to change a bit.
For
Thanks Tapani!
Reviewed-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
Can you add a test to piglit to check this case?
Sam
On 11/05/15 12:24, Tapani Pälli wrote:
Patch marks uniforms inside UBO properly referenced by stages.
Signed-off-by: Tapani Pälli tapani.pa...@intel.com
On Tuesday, May 12, 2015 09:23:38 PM Jordan Justen wrote:
On 2015-05-12 17:55:36, Kenneth Graunke wrote:
For scalar GS support, we either need to add a third constructor which
forth, right?
Did you miss removing the compute constructor prototype from brw_fs.h?
Reviewed-by: Jordan
https://bugs.freedesktop.org/show_bug.cgi?id=90162
--- Comment #3 from Martina Kollarova martina.kollar...@intel.com ---
I reproduced it with Mesa 10.6.0-devel (git-971be2b 2015-05-12
trusty-oibaf-ppa). I got the drivers from
https://launchpad.net/~oibaf/+archive/ubuntu/graphics-drivers
--
You
From: Marta Lofstedt marta.lofst...@intel.com
v3: only expose ARB_shader_atomic_counters enums
for gles 3.1 and GL core and remove functionality
related to obsolete:
extra_ARB_shader_atomic_counters_and_geometry_shader.
Signed-off-by: Marta Lofstedt marta.lofst...@intel.com
---
Not all contexts have geometry shaders. For example, ES 3.1 doesn't
have them. Or a pre-3.2 GL context. This needs to be checked for
MAX_GEOMETRY_ATOMIC_COUNTERS and the other one.
On Tue, May 12, 2015 at 10:29 AM, Marta Lofstedt
marta.lofst...@linux.intel.com wrote:
From: Marta Lofstedt
On 2015-05-12 17:55:36, Kenneth Graunke wrote:
For scalar GS support, we either need to add a third constructor which
forth, right?
Did you miss removing the compute constructor prototype from brw_fs.h?
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
takes the GS structures, or combine
On 05/13/2015 08:02 AM, Samuel Iglesias Gonsálvez wrote:
Thanks Tapani!
Reviewed-by: Samuel Iglesias Gonsalvez sigles...@igalia.com
Can you add a test to piglit to check this case?
I was hoping you had some test that hit this? :) I'll take a look if
existing tests can be modified easily
Yes, enabling it unconditionally would be better.
Marek
On Tue, May 12, 2015 at 1:30 AM, Emil Velikov emil.l.veli...@gmail.com wrote:
The extension requires that the address of the core functions should be
available via eglGetProcAddress. Currently the list is guarded by
These warnings have been detected by Clang 3.6.
codegen/nv50_ir_from_tgsi.cpp:1319:10: warning: struct 'Source' was
previously declared as a class [-Wmismatched-tags] const struct tgsi::Source
*code;
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
src/gallium/drivers/nouveau/codegen/nv50_ir_emit_nvc0.cpp | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_nvc0.cpp
b/src/gallium/drivers/nouveau/codegen/nv50_ir_emit_nvc0.cpp
index
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
src/gallium/drivers/nouveau/nv30/nvfx_fragprog.c | 40
1 file changed, 40 deletions(-)
diff --git a/src/gallium/drivers/nouveau/nv30/nvfx_fragprog.c
b/src/gallium/drivers/nouveau/nv30/nvfx_fragprog.c
index
This has been detected by Clang 3.6 which displays a warning message
because variable 'nv30' is uninitialized when used here. According to
the description of container_of, 'sample' must be initialized, otherwise
the result is undefined.
This patch gets rid of container_of() which is not required
Reviewed-by: Ilia Mirkin imir...@alum.mit.edu
On Tue, May 12, 2015 at 11:13 AM, Samuel Pitoiset
samuel.pitoi...@gmail.com wrote:
These warnings have been detected by Clang 3.6.
codegen/nv50_ir_from_tgsi.cpp:1319:10: warning: struct 'Source' was
previously declared as a class
On 05/12/2015 05:19 PM, Ilia Mirkin wrote:
On Tue, May 12, 2015 at 11:13 AM, Samuel Pitoiset
samuel.pitoi...@gmail.com wrote:
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
src/gallium/drivers/nouveau/nv30/nvfx_fragprog.c | 40
1 file changed, 40
Reviewed-by: Ilia Mirkin imir...@alum.mit.edu
On Tue, May 12, 2015 at 11:13 AM, Samuel Pitoiset
samuel.pitoi...@gmail.com wrote:
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
src/gallium/drivers/nouveau/nvc0/nvc0_state.c | 18 --
1 file changed, 18 deletions(-)
Reviewed-by: Ilia Mirkin imir...@alum.mit.edu
On Tue, May 12, 2015 at 11:13 AM, Samuel Pitoiset
samuel.pitoi...@gmail.com wrote:
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
src/gallium/drivers/nouveau/codegen/nv50_ir_emit_nvc0.cpp | 1 -
1 file changed, 1 deletion(-)
diff
On Tue, May 12, 2015 at 11:13 AM, Samuel Pitoiset
samuel.pitoi...@gmail.com wrote:
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
src/gallium/drivers/nouveau/nv30/nvfx_fragprog.c | 40
1 file changed, 40 deletions(-)
diff --git
On 05/12/2015 05:21 PM, Ilia Mirkin wrote:
On Tue, May 12, 2015 at 11:13 AM, Samuel Pitoiset
samuel.pitoi...@gmail.com wrote:
This has been detected by Clang 3.6 which displays a warning message
because variable 'nv30' is uninitialized when used here. According to
the description of
On Tue, May 12, 2015 at 11:13 AM, Samuel Pitoiset
samuel.pitoi...@gmail.com wrote:
This has been detected by Clang 3.6 which displays a warning message
because variable 'nv30' is uninitialized when used here. According to
the description of container_of, 'sample' must be initialized, otherwise
On 05/12/2015 05:24 PM, Ilia Mirkin wrote:
On Tue, May 12, 2015 at 11:19 AM, Samuel Pitoiset
samuel.pitoi...@gmail.com wrote:
-/* warning: this only works forward, and probably only if not inside any
IF */
-static void
-nv40_fp_bra(struct nvfx_fpc *fpc, unsigned target)
Please keep. This is
On Tue, May 12, 2015 at 11:19 AM, Samuel Pitoiset
samuel.pitoi...@gmail.com wrote:
-/* warning: this only works forward, and probably only if not inside any
IF */
-static void
-nv40_fp_bra(struct nvfx_fpc *fpc, unsigned target)
Please keep. This is the only docs for this stuff.
I didn't
On Tue, May 12, 2015 at 11:25 AM, Samuel Pitoiset
samuel.pitoi...@gmail.com wrote:
On 05/12/2015 05:24 PM, Ilia Mirkin wrote:
On Tue, May 12, 2015 at 11:19 AM, Samuel Pitoiset
samuel.pitoi...@gmail.com wrote:
-/* warning: this only works forward, and probably only if not inside
any
IF */
From: Marta Lofstedt marta.lofst...@intel.com
Signed-off-by: Marta Lofstedt marta.lofst...@intel.com
---
src/glsl/builtin_functions.cpp | 3 +--
src/glsl/builtin_types.cpp | 2 +-
src/glsl/glsl_lexer.ll | 13 +++--
src/glsl/glsl_parser_extras.h | 5 +
4 files changed,
This problem can easily be reproduced with a number of
ARB_shader_image_load_store piglit tests, which use a buffer object as
PBO for a pixel transfer operation and later on bind the same buffer
to the pipeline as shader image -- The problem is not exclusive to
images though, and is likely to
Hi Neil,
On 12/05/15 11:14, Neil Roberts wrote:
This is required for the I915_PARAM_REVISION macro. Previously this
define was directly copied into the Mesa source.
Tobias (tobijk on IRC) spotted a regression that was caused by the
commit introducing the define in libdrm. Would be nice to
On Tue, May 12, 2015 at 10:53 AM, Marta Lofstedt
marta.lofst...@linux.intel.com wrote:
From: Marta Lofstedt marta.lofst...@intel.com
Signed-off-by: Marta Lofstedt marta.lofst...@intel.com
---
src/glsl/builtin_functions.cpp | 3 +--
src/glsl/builtin_types.cpp | 2 +-
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
src/gallium/drivers/nouveau/nvc0/nvc0_state.c | 18 --
1 file changed, 18 deletions(-)
diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_state.c
b/src/gallium/drivers/nouveau/nvc0/nvc0_state.c
index dca06f4..63c3c52
Hi,
This series just fixes some compilation warnings reported by Clang for Nouveau.
Feel free to review.
Samuel Pitoiset (5):
nv30: remove unused nv40_fp_bra() and nvfx_fp_memcpy() functions
nv30: do not use container_of() when sample param is not initialized
nv50/ir: remove unused
On Tue, May 12, 2015 at 11:24 AM, Samuel Pitoiset
samuel.pitoi...@gmail.com wrote:
On 05/12/2015 05:21 PM, Ilia Mirkin wrote:
On Tue, May 12, 2015 at 11:13 AM, Samuel Pitoiset
samuel.pitoi...@gmail.com wrote:
This has been detected by Clang 3.6 which displays a warning message
because
On 05/12/2015 05:36 PM, Ilia Mirkin wrote:
On Tue, May 12, 2015 at 11:24 AM, Samuel Pitoiset
samuel.pitoi...@gmail.com wrote:
On 05/12/2015 05:21 PM, Ilia Mirkin wrote:
On Tue, May 12, 2015 at 11:13 AM, Samuel Pitoiset
samuel.pitoi...@gmail.com wrote:
This has been detected by Clang 3.6
The nv40_fp_bra() function in the same file is also unused but this is
the only place where the nv30/nv40 isa is documented.
Signed-off-by: Samuel Pitoiset samuel.pitoi...@gmail.com
---
src/gallium/drivers/nouveau/nv30/nvfx_fragprog.c | 17 +++--
1 file changed, 3 insertions(+), 14
Reviewed-by: Ilia Mirkin imir...@alum.mit.edu
On Tue, May 12, 2015 at 11:35 AM, Samuel Pitoiset
samuel.pitoi...@gmail.com wrote:
The nv40_fp_bra() function in the same file is also unused but this is
the only place where the nv30/nv40 isa is documented.
Signed-off-by: Samuel Pitoiset
From: Neil Roberts n...@linux.intel.com
This adds the framework for the GLX flush control extension. It is advertised
if the DRI driver implements the new __DRI2_FLUSH_CONTROL extension. The GLX
attributes are converted to the appropriate DRI attributes.
---
src/glx/dri2_glx.c | 19
From: Neil Roberts n...@linux.intel.com
Implementing the extension just involves copying the release behaviour value
from the context config to the GL context's constants. The upper layers of
Mesa handles the actual flushing.
---
src/mesa/drivers/dri/i965/brw_context.c | 20 +++-
Signed-off-by: Adam Jackson a...@redhat.com
---
src/glx/drisw_glx.c | 13 +++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git a/src/glx/drisw_glx.c b/src/glx/drisw_glx.c
index a07feb9..c2f8c4b 100644
--- a/src/glx/drisw_glx.c
+++ b/src/glx/drisw_glx.c
@@ -410,7 +410,7 @@
From: Neil Roberts n...@linux.intel.com
Previously the CreateContext method of __DriverApiRec took a set of arguments
to describe the attribute values from the window system API's
CreateContextAttribs function. As more attributes get added this could quickly
get unworkable and every new attribute
From: Neil Roberts n...@linux.intel.com
Adds a DRI extension which advertises that the driver can accept a new
__DRI_CTX_ATTRIB_RELEASE_BEHAVIOR attribute in the createContextAttribs
function. If the attribute is specified then it will be passed down to the
CreateContext function in the
From: Neil Roberts n...@linux.intel.com
It shouldn't be necessary to flush the context within the driver
implementation because the old context is explicitly flushed in
_mesa_make_current which is called a little further on. It is useful to
only have a single place that flushes when switching
This hides the flush control attribute from the CreateContextAttribs
request. Older X servers don't know about it and will throw an error,
and at least for purposes of testing it's not something that matters.
Signed-off-by: Adam Jackson a...@redhat.com
---
src/glx/create_context.c | 19
1 - 100 of 102 matches
Mail list logo