On Fri, 07 Feb 2014 01:20:39 +
Emil Velikov emil.l.veli...@gmail.com wrote:
The argument is this setup worked, and was supported, for 10 years
now. You don't break supported setups without good reason.
FWIW I'm in favour of not breaking existing setups without any serious
reason.
https://bugs.freedesktop.org/show_bug.cgi?id=69101
Mike Lothian m...@fireburn.co.uk changed:
What|Removed |Added
Status|RESOLVED|REOPENED
For those interested Ian's talk is available here:
http://video.fosdem.org/2014/H1301_Cornil/Saturday/Three_Years_Experience_with_a_Treelike_Shader_IR.webm
On Fri, 2014-02-07 at 00:34 -0500, Connor Abbott wrote:
Hi,
So I believe that we can all agree that the tree-based representation
that
On Fri, Feb 7, 2014 at 3:55 AM, Michel Dänzer mic...@daenzer.net wrote:
On Mon, 2014-02-03 at 18:07 +0100, Marek Olšák wrote:
Do you mean something like:
write_domains = VRAM
read_domains = VRAM | GTT
I don't think it's perfect.
That's what I meant, and of course it's not perfect. :)
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/glx/glxcmds.c | 28
1 file changed, 20 insertions(+), 8 deletions(-)
diff --git a/src/glx/glxcmds.c b/src/glx/glxcmds.c
index b5377c2..38a5262 100644
--- a/src/glx/glxcmds.c
+++
1..5 are previously submitted patches which seem to have been lost in
the list, just resubmitting them here. Other patches are new.
Juha-Pekka Heikkila (9):
glx: Add missing null checks in glxcmds.c
glx: Add missing null check in __glXCloseDisplay
glsl: Fix null access on file read error
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/glx/glx_pbuffer.c | 19 +++
1 file changed, 15 insertions(+), 4 deletions(-)
diff --git a/src/glx/glx_pbuffer.c b/src/glx/glx_pbuffer.c
index 183fbaa..411d6e5 100644
--- a/src/glx/glx_pbuffer.c
+++
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 4
1 file changed, 4 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
b/src/mesa/drivers/dri/i965/brw_fs.cpp
index c290c28..217892a 100644
---
_eglInitResource() was used to memset entire _EGLContext by
writing more than size of pointed target. This does work
as long as Resource is the first element in _EGLContext,
this patch fixes such dependency.
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/glsl/main.cpp | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/glsl/main.cpp b/src/glsl/main.cpp
index 5ea57d5..3a0f812 100644
--- a/src/glsl/main.cpp
+++ b/src/glsl/main.cpp
@@ -232,7 +232,7 @@
ir_constant::ir_constant(const struct glsl_type,
const ir_constant_data *) was copying too much memory.
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/glsl/ir.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/glsl/ir.cpp b/src/glsl/ir.cpp
index
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/glx/glxext.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/glx/glxext.c b/src/glx/glxext.c
index 2711e57..4a195bd 100644
--- a/src/glx/glxext.c
+++ b/src/glx/glxext.c
@@ -269,7 +269,8 @@
_eglInitResource() was used to memset entire _EGLSurface by
writing more than size of pointed target. This does work
as long as Resource is the first element in _EGLSurface,
this patch fixes such dependency.
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/glx/clientinfo.c | 4
1 file changed, 4 insertions(+)
diff --git a/src/glx/clientinfo.c b/src/glx/clientinfo.c
index e90fd79..8f3fdeb 100644
--- a/src/glx/clientinfo.c
+++ b/src/glx/clientinfo.c
@@ -113,6 +113,10 @@
Signed-off-by: Erik Faye-Lund kusmab...@gmail.com
---
src/gallium/auxiliary/tgsi/tgsi_exec.c | 26 --
1 file changed, 4 insertions(+), 22 deletions(-)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_exec.c
b/src/gallium/auxiliary/tgsi/tgsi_exec.c
index 3d37eaa..96809cd
In the specification text of NV_vertex_program1_1, the upper
limit of the RCC instruction is written as 1.884467e+19 in
scientific notation, but as 0x5F80 in binary. But the binary
version translates to 1.84467e+19 rather than 1.884467e+19 in
scientific notation.
Since the lower-limit equals
On Thu, Feb 6, 2014 at 5:30 PM, Brian Paul bri...@vmware.com wrote:
On 02/06/2014 09:09 AM, Erik Faye-Lund wrote:
In the specification text of NV_vertex_program1_1, the upper
limit of the RCC instruction is written as 1.884467e+19 in
scientific notation, but as 0x5F80 in binary. But the
https://bugs.freedesktop.org/show_bug.cgi?id=69101
--- Comment #21 from Alexander Mezin mezin.alexan...@gmail.com ---
Yes, unigine valley renders fully transparent window with OpenGL compositing,
and everything works fine with XRender.
Also, looks like first launched windowed application still
On 02/07/2014 05:43 AM, Juha-Pekka Heikkila wrote:
1..5 are previously submitted patches which seem to have been lost in
the list, just resubmitting them here. Other patches are new.
Juha-Pekka Heikkila (9):
glx: Add missing null checks in glxcmds.c
glx: Add missing null check in
On Fri, Feb 7, 2014 at 4:15 PM, Brian Paul bri...@vmware.com wrote:
On 02/07/2014 05:45 AM, Erik Faye-Lund wrote:
Signed-off-by: Erik Faye-Lund kusmab...@gmail.com
---
src/gallium/auxiliary/tgsi/tgsi_exec.c | 26 --
1 file changed, 4 insertions(+), 22 deletions(-)
Am 07.02.2014 04:37, schrieb Dave Airlie:
On Fri, Feb 7, 2014 at 12:36 PM, Roland Scheidegger srol...@vmware.com
wrote:
Am 07.02.2014 02:56, schrieb Ilia Mirkin:
On Thu, Feb 6, 2014 at 8:52 PM, Dave Airlie airl...@gmail.com wrote:
From: Dave Airlie airl...@redhat.com
This just adds the TG4
From: Tom Stellard thomas.stell...@amd.com
---
lib/Target/R600/AMDGPUISelDAGToDAG.cpp | 48 ++
lib/Target/R600/SIISelLowering.cpp | 29
lib/Target/R600/SIISelLowering.h | 1 -
test/CodeGen/R600/add.ll | 10 +++
https://bugs.freedesktop.org/show_bug.cgi?id=72895
Alex Deucher ag...@yahoo.com changed:
What|Removed |Added
Summary|Missing trees in flightgear |Missing trees in
On Fri, Feb 7, 2014 at 12:34 AM, Connor Abbott cwabbo...@gmail.com wrote:
Hi,
So I believe that we can all agree that the tree-based representation
that GLSL IR currently uses for shaders needs to go. For the benefit
of those that didn't watch Ian Romanick's talk at FOSDEM, I'll
reiterate
On Thu, Feb 6, 2014 at 7:08 PM, Joey Reid joeyisdama...@gmail.com wrote:
the folder:
http://www.x.org/docs/AMD/
has an outdated version of Southern Islands Series Instruction Set
Architecture. The latest version can be found here:
On Fri, Feb 7, 2014 at 6:34 AM, Connor Abbott cwabbo...@gmail.com wrote:
- It turns out that the original advantage of a tree-based IR, to be
able to automatically generate pattern-matching code for optimizing
certain code patterns, only really matters for CPU's with weird
instruction sets
If st_GetTexImage() is to decompress the texture, avoid the fallback
path even if prefer_blit_based_texture_transfer = false. For drivers
that returned PIPE_CAP_PREFER_BLIT_BASED_TEXTURE_TRANSFER = 0, we
were always taking the fallback path for texture decompression rather
than rendering a quad.
The CSO stuff looks good. We want the ability to save and restore the
first viewport, but not the other ones.
Marek
On Tue, Jan 21, 2014 at 7:09 AM, Dave Airlie airl...@gmail.com wrote:
From: Dave Airlie airl...@redhat.com
this just ties the mesa code to the pre-existing gallium interface,
Am 07.02.2014 16:49, schrieb Alex Deucher:
On Fri, Feb 7, 2014 at 12:34 AM, Connor Abbott cwabbo...@gmail.com wrote:
Hi,
So I believe that we can all agree that the tree-based representation
that GLSL IR currently uses for shaders needs to go. For the benefit
of those that didn't watch Ian
FYI, I found the patch which compiles some shader variants in glLinkProgram:
http://cgit.freedesktop.org/~mareko/mesa/commit/?h=precompileid=8dc586297e4902dfc14cf8001b8202db4fdc9447
Marek
On Tue, Jan 21, 2014 at 12:56 AM, Marek Olšák mar...@gmail.com wrote:
It was pretty simple IIRC. You just
Cool. Do you think something like this is check-in-able behind a
ST_PRECOMPILE=1 env var (or something that fits well into the current
env var scheme, dunno what it would be)?
On Fri, Feb 7, 2014 at 12:46 PM, Marek Olšák mar...@gmail.com wrote:
FYI, I found the patch which compiles some shader
On Fri, Feb 07, 2014 at 05:20:12PM +0100, Christian König wrote:
Am 07.02.2014 16:49, schrieb Alex Deucher:
On Fri, Feb 7, 2014 at 12:34 AM, Connor Abbott cwabbo...@gmail.com wrote:
Hi,
So I believe that we can all agree that the tree-based representation
that GLSL IR currently uses for
On Fri, Feb 07, 2014 at 10:49:01AM -0500, Alex Deucher wrote:
On Fri, Feb 7, 2014 at 12:34 AM, Connor Abbott cwabbo...@gmail.com wrote:
Hi,
So I believe that we can all agree that the tree-based representation
that GLSL IR currently uses for shaders needs to go. For the benefit
of those
On Fri, Feb 07, 2014 at 10:31:20AM -0800, Matt Arsenault wrote:
I didn't think to try this. Where is the address folding happening?
There are TableGen patterns that do the folding. I recently added
several new ones: r200932-r200935.
-Tom
On 02/07/2014 07:46 AM, Tom Stellard wrote:
From:
The bits which normally contain the source register descriptions
actually contain the JIP/UIP jump targets, which we already printed.
Interpreting JIP/UIP as source registers results in some really creepy
looking output, like IF statements with acc14.40,1,0UD sources.
Signed-off-by: Kenneth
The WHILE instruction doesn't have UIP. It only has JIP.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/mesa/drivers/dri/i965/gen8_disasm.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/gen8_disasm.c
https://bugs.freedesktop.org/show_bug.cgi?id=67676
--- Comment #5 from Eric Anholt e...@anholt.net ---
Daniel: the problem is that normal apps use the alpha, but they don't fill it
with values that should be used to blend window contents over other windows.
So you must not return the rgba visual
Kenneth Graunke kenn...@whitecape.org writes:
The bits which normally contain the source register descriptions
actually contain the JIP/UIP jump targets, which we already printed.
Interpreting JIP/UIP as source registers results in some really creepy
looking output, like IF statements with
On Fri, Feb 7, 2014 at 11:20 AM, Christian König
deathsim...@vodafone.de wrote:
Am 07.02.2014 16:49, schrieb Alex Deucher:
On Fri, Feb 7, 2014 at 12:34 AM, Connor Abbott cwabbo...@gmail.com
wrote:
Hi,
So I believe that we can all agree that the tree-based representation
that GLSL IR
Am 07.02.2014 17:39, schrieb Brian Paul:
If st_GetTexImage() is to decompress the texture, avoid the fallback
path even if prefer_blit_based_texture_transfer = false. For drivers
that returned PIPE_CAP_PREFER_BLIT_BASED_TEXTURE_TRANSFER = 0, we
were always taking the fallback path for texture
On 02/07/2014 07:49 AM, Alex Deucher wrote:
On Fri, Feb 7, 2014 at 12:34 AM, Connor Abbott cwabbo...@gmail.com wrote:
Hi,
So I believe that we can all agree that the tree-based representation
that GLSL IR currently uses for shaders needs to go. For the benefit
of those that didn't watch Ian
On 02/06/2014 09:34 PM, Connor Abbott wrote:
Hi,
So I believe that we can all agree that the tree-based representation
that GLSL IR currently uses for shaders needs to go. For the benefit
of those that didn't watch Ian Romanick's talk at FOSDEM, I'll
reiterate some of the problems with it
On 02/07/2014 08:06 AM, Alex Deucher wrote:
On Thu, Feb 6, 2014 at 7:08 PM, Joey Reid joeyisdama...@gmail.com wrote:
the folder:
http://www.x.org/docs/AMD/
has an outdated version of Southern Islands Series Instruction Set
Architecture. The latest version can be found here:
On Fri, Feb 07, 2014 at 02:52:15PM -0500, Rob Clark wrote:
On Fri, Feb 7, 2014 at 11:20 AM, Christian König
deathsim...@vodafone.de wrote:
Am 07.02.2014 16:49, schrieb Alex Deucher:
On Fri, Feb 7, 2014 at 12:34 AM, Connor Abbott cwabbo...@gmail.com
wrote:
Hi,
So I believe that we
This makes it obvious which number is which.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/mesa/drivers/dri/i965/gen8_disasm.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/gen8_disasm.c
On 02/07/2014 05:45 AM, Erik Faye-Lund wrote:
Signed-off-by: Erik Faye-Lund kusmab...@gmail.com
---
src/gallium/auxiliary/tgsi/tgsi_exec.c | 26 --
1 file changed, 4 insertions(+), 22 deletions(-)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_exec.c
https://bugs.freedesktop.org/show_bug.cgi?id=67676
Daniel Stone dan...@fooishbar.org changed:
What|Removed |Added
CC||dan...@fooishbar.org
On Fri, Feb 7, 2014 at 3:13 PM, Ian Romanick i...@freedesktop.org wrote:
On 02/06/2014 09:34 PM, Connor Abbott wrote:
Hi,
So I believe that we can all agree that the tree-based representation
that GLSL IR currently uses for shaders needs to go. For the benefit
of those that didn't watch Ian
https://bugs.freedesktop.org/show_bug.cgi?id=72895
--- Comment #15 from Barto mister.free...@laposte.net ---
I tried with flightgear 3.0.0 release candidate,
and the bug is still here,
I created a bugreport in order to warn the flightgear developpers :
On Fri, Feb 07, 2014 at 11:46:55AM +0900, Michel Dänzer wrote:
On Don, 2014-02-06 at 09:40 -0800, Tom Stellard wrote:
From: Tom Stellard thomas.stell...@amd.com
DS instructions that access local memory can only uses addresses that
are less than or equal to the value of M0. When M0 is
Broadwell's 3DSTATE_CLEAR_PARAMS packet expects a floating point value
regardless of format. This means we need to stop converting it to
UNORM.
Storing the value as float would make sense, but since we already have a
uint32_t field, this patch continues shoehorning it into that. In a
sense,
Doh, yes because GL has ARB_texture_gather then has stuff hidden away
in ARB_gpu_shader5 I forgot to add the extra bits which I suppose we should
do.
So I've reposted with the component selection in src1 now.
Hmm seems a bit excessive to use an extra reg for that (gather4 but only
in
Kenneth Graunke kenn...@whitecape.org writes:
Broadwell's 3DSTATE_CLEAR_PARAMS packet expects a floating point value
regardless of format. This means we need to stop converting it to
UNORM.
Storing the value as float would make sense, but since we already have a
uint32_t field, this patch
https://bugs.freedesktop.org/show_bug.cgi?id=72895
--- Comment #16 from Fredrik Höglund fred...@kde.org ---
Created attachment 93635
-- https://bugs.freedesktop.org/attachment.cgi?id=93635action=edit
patch
Thanks, but I think I've found the cause of the problem.
Can you confirm that this
https://bugs.freedesktop.org/show_bug.cgi?id=72895
--- Comment #17 from Ilia Mirkin imir...@alum.mit.edu ---
(In reply to comment #16)
Created attachment 93635 [details] [review]
patch
Thanks, but I think I've found the cause of the problem.
Can you confirm that this patch fixes it for
On 04/02/14 19:24, Tom Stellard wrote:
From: Tom Stellard thomas.stell...@amd.com
Linking with LLVM static libraries is easily broken by changes to
the llvm-config program or when LLVM adds, removes, or changes library
components. Keeping up with these changes requires a lot of maintanence
https://bugs.freedesktop.org/show_bug.cgi?id=72895
--- Comment #18 from Barto mister.free...@laposte.net ---
(In reply to comment #16)
Created attachment 93635 [details] [review]
patch
Thanks, but I think I've found the cause of the problem.
Can you confirm that this patch fixes it for
On Tue, Jan 28, 2014 at 1:50 PM, Paul Berry stereotype...@gmail.com wrote:
On 22 January 2014 09:16, Connor Abbott cwabbo...@gmail.com wrote:
This patch introduces all the changes to the IR that are necessary for
representing programs in the SSA form. This consists of a new variable
mode, the
https://bugs.freedesktop.org/show_bug.cgi?id=72895
--- Comment #19 from Benjamin Bellec b.bel...@gmail.com ---
(In reply to comment #16)
Created attachment 93635 [details] [review]
patch
Thanks, but I think I've found the cause of the problem.
Can you confirm that this patch fixes it for
https://bugs.freedesktop.org/show_bug.cgi?id=72895
--- Comment #20 from Fredrik Höglund fred...@kde.org ---
(In reply to comment #18)
(In reply to comment #16)
Can you confirm that this patch fixes it for you?
yes your patch solves the bug, thank you very much
No problem - the apitrace
https://bugs.freedesktop.org/show_bug.cgi?id=72895
--- Comment #21 from Ian Romanick i...@freedesktop.org ---
Comment on attachment 93635
-- https://bugs.freedesktop.org/attachment.cgi?id=93635
patch
Review of attachment 93635:
-
Am 07.02.2014 23:25, schrieb Dave Airlie:
Doh, yes because GL has ARB_texture_gather then has stuff hidden away
in ARB_gpu_shader5 I forgot to add the extra bits which I suppose we should
do.
So I've reposted with the component selection in src1 now.
Hmm seems a bit excessive to use an
https://bugs.freedesktop.org/show_bug.cgi?id=72895
--- Comment #22 from Fredrik Höglund fred...@kde.org ---
(In reply to comment #21)
Comment on attachment 93635 [details] [review]
patch
Review of attachment 93635 [details] [review]:
On 02/07/2014 04:44 AM, Juha-Pekka Heikkila wrote:
ir_constant::ir_constant(const struct glsl_type,
const ir_constant_data *) was copying too much memory.
The code looks correct as-is to me. This copies one ir_constant_data
union to another... they're declared as the same type, and they have
We don't need to allocate all the state related to GL_ARB_debug_output
until some aspect of that extension is actually needed.
The sizeof(gl_debug_state) is huge (~285KB on 64-bit systems), not even
counting the 54(!) hash tables and lists that it contains. This change
reduces the size of
---
src/mesa/main/errors.c | 50 ++--
1 file changed, 23 insertions(+), 27 deletions(-)
diff --git a/src/mesa/main/errors.c b/src/mesa/main/errors.c
index 28357e0..f84a3c1 100644
--- a/src/mesa/main/errors.c
+++ b/src/mesa/main/errors.c
@@ -346,9
Whitespace changes, 78-column rewrapping, comment clean-ups, add
some braces, etc.
---
src/mesa/main/errors.c | 125
1 file changed, 84 insertions(+), 41 deletions(-)
diff --git a/src/mesa/main/errors.c b/src/mesa/main/errors.c
index
Mesa 10.1 release candidate 1 is now available for testing. The current
plan of record is to have the 10.1 release on Friday, February 28th.
The tag in the GIT repository for Mesa 10.1-rc1 is 'mesa-10.1-rc1'.
Mesa 10.1 release candidate 1 is available for download at
Previously, we would program the sample count based on the blorp
operation's dst sample count -- which is zero for a HiZ op; we'd also
ignore the difference between physical and logical surface size for
the depth surface.
That kindof worked -- the two errors almost cancel out -- but produced
In commit eeed49f5f290793870c60b5b635b977a732a1eb4, Mark accidentally
renamed MESA_FORMAT_S8_Z24 to MESA_FORMAT_Z24_UNORM_X8_UINT and
MESA_FORMAT_X8_Z24 to MESA_FORMAT_Z24_UNORM_S8_UINT, reversing their
sense. The commit message was correct, but what sed commands actually
got run didn't match
From: Gregory Hainaut gregory.hain...@gmail.com
This was originally included in another patch, but it was split out by
Ian Romanick.
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
---
src/mesa/main/pipelineobj.c | 24
1 file changed, 24 insertions(+)
diff --git
From: Ian Romanick ian.d.roman...@intel.com
This adds the necessary bits for both the API and the GLSL compiler.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
src/glsl/glsl_parser_extras.cpp | 1 +
src/glsl/glsl_parser_extras.h | 7 +++
src/mesa/main/extensions.c | 1 +
From: Gregory Hainaut gregory.hain...@gmail.com
Implement IsProgramPipeline based on the VAO code.
This was originally included in another patch, but it was split out by
Ian Romanick.
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
---
src/mesa/main/mtypes.h | 2 ++
I'm taking a patch from Paul's notebook, and I'm going to try land a
giant patch series in a small number of more manageable chunks.
GL_ARB_separate_shader_objects has been work-in-progress for about 10
months. This represents about half of the final patch series. The next
block of patches will
From: Ian Romanick ian.d.roman...@intel.com
GL_ARB_separate_shader_objects adds the ability to specify location
layouts for interstage inputs and outputs.
In addition, this extension makes 'in' and 'out' generally available for
shader inputs and outputs. This mimics the behavior of
From: Gregory Hainaut gregory.hain...@gmail.com
This will allow the guts of the implementation to be shared with
_mesa_CreateShaderProgramv.
This was originally included in another patch, but it was split out by
Ian Romanick.
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
---
From: Gregory Hainaut gregory.hain...@gmail.com
This was originally included in another patch, but it was split out by
Ian Romanick.
v2 (idr):
* Trivial reformatting.
* Remove GL_COMPUTE_SHADER. Compute shaders don't participate in pipeline
objects anyway. Suggested by Matt Turner.
v3
From: Gregory Hainaut gregory.hain...@gmail.com
This was originally included in another patch, but it was split out by
Ian Romanick.
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
---
src/mesa/main/shaderapi.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git
From: Gregory Hainaut gregory.hain...@gmail.com
V1:
* Extend gl_shader_state as pipeline object state
* Add a new container gl_pipeline_shader_state that contains
binding point of the previous object
* Update mesa init/free shader state due to the extension of
the attibute
* Add an
From: Gregory Hainaut gregory.hain...@gmail.com
Implement GenProgramPipelines based on the VAO code.
This was originally included in another patch, but it was split out by
Ian Romanick.
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
---
src/mesa/main/pipelineobj.c | 30
From: Ian Romanick ian.d.roman...@intel.com
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
src/mesa/main/context.c | 102
1 file changed, 34 insertions(+), 68 deletions(-)
diff --git a/src/mesa/main/context.c b/src/mesa/main/context.c
From: Gregory Hainaut gregory.hain...@gmail.com
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
---
src/mesa/main/mtypes.h| 4
src/mesa/main/shaderapi.c | 8
2 files changed, 12 insertions(+)
diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h
index
From: Gregory Hainaut gregory.hain...@gmail.com
Future patches will use this function outside shaderapi.c.
This was originally included in another patch, but it was split out by
Ian Romanick.
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
---
src/mesa/main/shaderapi.c | 6 +++---
From: Gregory Hainaut gregory.hain...@gmail.com
Implement DeleteProgramPipelines based on the VAO code.
This was originally included in another patch, but it was split out by
Ian Romanick.
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
---
src/mesa/main/pipelineobj.c | 33
From: Gregory Hainaut gregory.hain...@gmail.com
This was originally included in another patch, but it was split out by
Ian Romanick.
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
---
src/mesa/main/mtypes.h| 6 ++
src/mesa/main/shaderapi.c | 25 +
2 files
brw_depthbuffer_format is not very reusable at the moment, since it
uses global state (ctx-DrawBuffer) to access a particular depth buffer.
For HiZ on Broadwell, I need a function which simply converts the
formats. However, at least one existing user of brw_depthbuffer_format
really wants the
Broadwell's 3DSTATE_WM_HZ_OP packet makes this much easier.
Instead of programming the whole pipeline, we simply have to emit the
depth/stencil packets, a state override, and a pipe control. Then
arrange for the state to be put back. This is easily done from a single
function.
Signed-off-by:
The existing code followed the vtable function signature, which is not a
great fit: many of the parameters are unused, and the function still
inspects global state, making it less reusable.
This patch refactors the depth buffer packet emission code into a new
function which takes exactly the
HiZ buffers still don't exist, but when they do, we'll set them up.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/mesa/drivers/dri/i965/gen8_depth_state.c | 25 +
1 file changed, 17 insertions(+), 8 deletions(-)
diff --git
They work fine now, too.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/mesa/drivers/dri/i965/brw_clear.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_clear.c
b/src/mesa/drivers/dri/i965/brw_clear.c
index 1964572..7c2e5cc 100644
It appears to work fine.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/mesa/drivers/dri/i965/brw_context.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_context.c
b/src/mesa/drivers/dri/i965/brw_context.c
index 021287e..b9251a2
We're going to need these to implement HiZ.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/mesa/drivers/dri/i965/brw_defines.h | 25 +
1 file changed, 25 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_defines.h
Broadwell's HiZ Resolve operation still has the restriction that the
rectangle primitive must be 8x4 aligned. So I believe we still need
this.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 2 +-
1 file changed, 1 insertion(+), 1
On Fri, Feb 7, 2014 at 10:22 PM, Andrew Guertin li...@dolphinling.net wrote:
Hi,
I updated mesa and started getting some bad behavior in League of Legends
(played through wine). After starting a game, upon hovering any of the skill
buttons, instead of an info tooltip appearing, the whole
94 matches
Mail list logo