Signed-off-by: Ilia Mirkin
---
This applies on top of Dave Airlie's r600g-texture-gather branch. Ran piglit
with -t gather, passed all 1057 tests. Can't say I fully understand what all
the arguments to handleTEX in the Coverter are but... seems to work. Will
probably require some care for nvc0 su
On 02/19/2014 06:05 PM, Ian Romanick wrote:
> On 02/19/2014 05:40 PM, Kenneth Graunke wrote:
>> brw_init_state() calls brw_upload_initial_gpu_state(). If hardware
>> contexts are enabled (brw->hw_ctx != NULL), this will upload some
>> initial invariant state for the GPU. Without hardware contexts
On Thu, Feb 20, 2014 at 7:03 AM, Kenneth Graunke wrote:
> On 02/19/2014 02:27 PM, Ian Romanick wrote:
>> On 02/19/2014 12:08 PM, Kenneth Graunke wrote:
>>> On 02/18/2014 09:48 PM, Chia-I Wu wrote:
Since 73bc6061f5c3b6a3bb7a8114bb2e1ab77d23cfdb, Z16 support is
not advertised for OpenGL ES
https://bugs.freedesktop.org/show_bug.cgi?id=71870
Tapani Pälli changed:
What|Removed |Added
Status|RESOLVED|VERIFIED
--- Comment #46 from Tapani Päll
Am 19.02.2014 11:04, schrieb Kenneth Graunke:
> According to the "Point Multisample Rasterization" of the OpenGL
> specification (3.0 or later), smooth points are supposed to be enabled
> implicitly when multisampling, regardless of the GL_POINT_SMOOTH flag.
>
> However, if GL_POINT_SPRITE is enab
On Mit, 2014-02-19 at 15:09 -0800, Tom Stellard wrote:
> ---
> src/gallium/drivers/radeonsi/si_shader.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/radeonsi/si_shader.c
> b/src/gallium/drivers/radeonsi/si_shader.c
> index 54270cd..9fed751 100644
>
On Don, 2014-02-20 at 00:53 +0100, Francisco Jerez wrote:
> Tom Stellard writes:
>
> > diff --git a/src/gallium/drivers/r600/evergreen_compute.c
> > b/src/gallium/drivers/r600/evergreen_compute.c
> > index 70efe5c..efd7143 100644
> > --- a/src/gallium/drivers/r600/evergreen_compute.c
> > +++ b/s
On Mit, 2014-02-19 at 15:09 -0800, Tom Stellard wrote:
> ---
> src/gallium/auxiliary/util/u_math.h | 10 ++
> 1 file changed, 10 insertions(+)
>
> diff --git a/src/gallium/auxiliary/util/u_math.h
> b/src/gallium/auxiliary/util/u_math.h
> index b5e0663..49f8bda 100644
> --- a/src/gallium/
On 02/19/2014 05:40 PM, Kenneth Graunke wrote:
> brw_init_state() calls brw_upload_initial_gpu_state(). If hardware
> contexts are enabled (brw->hw_ctx != NULL), this will upload some
> initial invariant state for the GPU. Without hardware contexts, we
> rely on this state being uploaded via atom
On 02/19/2014 02:49 PM, Adel Gadllah wrote:
> Hi,
>
> The attached patch adds support for the GLX_EXT_buffer_age extension,
> which is mostly used by compositors for efficient sub screen updates.
>
> The extension should not be reported as supported when running DRI2 but
> it seems to show up whe
brw_init_state() calls brw_upload_initial_gpu_state(). If hardware
contexts are enabled (brw->hw_ctx != NULL), this will upload some
initial invariant state for the GPU. Without hardware contexts, we
rely on this state being uploaded via atoms that subscribe to the
BRW_NEW_CONTEXT bit.
Commit 46
We're disabling GL_MULTISAMPLE, so we didn't need to worry about a lot of
that state. But to do MSAA to MSAA blits, we need to start handling more
state.
v2: Fix pasteo caught by Kenneth.
---
src/mesa/drivers/common/meta.c | 40 +---
src/mesa/drivers/common/me
We haven't been executing this code before the meta-blit case, because
we've been flagging the miptree as validated at texstorage time, and never
having to revalidate.
---
src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 15 ++-
src/mesa/drivers/dri/i965/intel_mipmap_tree.h | 2 ++
2 f
Note that this doesn't handle GL_EXT_multisample_scaled_blit yet. The
i965 code for that extension bakes in knowledge of the sample positions
(well, knowledge of the sample positions aligned to a lower-resolution
grid), which we would have to do at runtime somehow for meta.
---
src/mesa/drivers/c
Blending of values would occur when doing GL_LINEAR filtering with
scaling, and in an upcoming commit when doing MSAA resolves.
---
src/mesa/drivers/common/meta_blit.c | 30 +-
1 file changed, 25 insertions(+), 5 deletions(-)
diff --git a/src/mesa/drivers/common/meta_b
Compared to i965, the code generated doesn't use the AVG instruction. But
I'm not sure that multisampled integer resolves are really that important
to worry about.
---
src/mesa/drivers/common/meta.h | 10 ++
src/mesa/drivers/common/meta_blit.c | 68 +
These are non-stretched, non-resolving blits, so it's just a matter of
sampling once from our gl_SampleID and storing that to our color/depth.
---
src/mesa/drivers/common/meta.h | 6 +-
src/mesa/drivers/common/meta_blit.c | 147
2 files changed, 104 inse
Reviewed-by: Ian Romanick
On 02/19/2014 04:28 PM, Kenneth Graunke wrote:
> According to the latest documentation, any PIPE_CONTROL with the
> "Command Streamer Stall" bit set must also have another bit set,
> with five different options:
>
>- Render Target Cache Flush
>- Depth Cache Flus
On Wed, Feb 19, 2014 at 5:49 PM, Adel Gadllah wrote:
> Hi,
>
> The attached patch adds support for the GLX_EXT_buffer_age extension, which
> is mostly used by compositors for efficient sub screen updates.
>
> The extension should not be reported as supported when running DRI2 but it
> seems to sho
From: Leo Liu
Signed-off-by: Leo Liu
---
src/gallium/state_trackers/omx/vid_enc.c | 39
src/gallium/state_trackers/omx/vid_enc.h | 7 --
2 files changed, 29 insertions(+), 17 deletions(-)
diff --git a/src/gallium/state_trackers/omx/vid_enc.c
b/src/gallium
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 02/19/2014 12:08 PM, Kenneth Graunke wrote:
> On 02/18/2014 09:48 PM, Chia-I Wu wrote:
>> Since 73bc6061f5c3b6a3bb7a8114bb2e1ab77d23cfdb, Z16 support is
>> not advertised for OpenGL ES contexts due to the terrible
>> performance. It is still enable
Hi,
The attached patch adds support for the GLX_EXT_buffer_age extension, which is mostly used by compositors for efficient
sub screen updates.
The extension should not be reported as supported when running DRI2 but it seems to show up when I try to disable it
with LIBGL_DRI3_DISABLE ... not
Am 19.02.2014 20:09, schrieb Ian Romanick:
I'm hoping that Tapani or Darius will verify that this patch actually
fixes the problem. That's why people CC other people on patches. :)
On 02/18/2014 10:19 AM, Ian Romanick wrote:
From: Ian Romanick
Section 3.3 (Preprocessor) of the GLSL 1.30 spec
According to the latest documentation, any PIPE_CONTROL with the
"Command Streamer Stall" bit set must also have another bit set,
with five different options:
- Render Target Cache Flush
- Depth Cache Flush
- Stall at Pixel Scoreboard
- Post-Sync Operation
- Depth Stall
I chose "St
Tom Stellard writes:
> The offsets will be stored in the handles parameter. This makes
> it possible to use sub-buffers.
>
> v2:
> - Style fixes
> - Add support for constant sub-buffers
> - Store handles in device byte order
> ---
> src/gallium/drivers/r600/evergreen_compute.c | 10 +
Tom Stellard writes:
> ---
> src/gallium/auxiliary/util/u_math.h | 10 ++
> 1 file changed, 10 insertions(+)
>
> diff --git a/src/gallium/auxiliary/util/u_math.h
> b/src/gallium/auxiliary/util/u_math.h
> index b5e0663..49f8bda 100644
> --- a/src/gallium/auxiliary/util/u_math.h
> +++ b/s
On Wed, Feb 19, 2014 at 3:32 PM, Ilia Mirkin wrote:
> On Wed, Feb 19, 2014 at 6:09 PM, Tom Stellard wrote:
>> +/**
>> + * Reverse byte order of a 64bit word.
>> + */
>> +static INLINE uint64_t
>> +util_bswap64(uint64_t n)
>> +{
>> + return ((uint64_t)util_bswap32(n & 0x) << 32) |
>> +
On Wed, Feb 19, 2014 at 6:09 PM, Tom Stellard wrote:
> ---
> src/gallium/auxiliary/util/u_math.h | 10 ++
> 1 file changed, 10 insertions(+)
>
> diff --git a/src/gallium/auxiliary/util/u_math.h
> b/src/gallium/auxiliary/util/u_math.h
> index b5e0663..49f8bda 100644
> --- a/src/gallium/au
On Wed, Feb 19, 2014 at 6:09 PM, Tom Stellard wrote:
> ---
> src/gallium/auxiliary/util/u_math.h | 10 ++
> 1 file changed, 10 insertions(+)
For the series:
Reviewed-by: Alex Deucher
>
> diff --git a/src/gallium/auxiliary/util/u_math.h
> b/src/gallium/auxiliary/util/u_math.h
> index
---
src/gallium/drivers/radeonsi/si_shader.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
index 54270cd..9fed751 100644
--- a/src/gallium/drivers/radeonsi/si_shader.c
+++ b/src/gallium/drive
On 02/19/2014 01:44 PM, Jordan Justen wrote:
> On Fri, Feb 7, 2014 at 10:00 PM, Ian Romanick wrote:
>> From: Gregory Hainaut
>>
>> This was originally included in another patch, but it was split out by
>> Ian Romanick.
>>
>> Reviewed-by: Ian Romanick
>> ---
>> src/mesa/main/pipelineobj.c | 24 +
---
src/gallium/auxiliary/util/u_math.h | 10 ++
1 file changed, 10 insertions(+)
diff --git a/src/gallium/auxiliary/util/u_math.h
b/src/gallium/auxiliary/util/u_math.h
index b5e0663..49f8bda 100644
--- a/src/gallium/auxiliary/util/u_math.h
+++ b/src/gallium/auxiliary/util/u_math.h
@@ -7
The offsets will be stored in the handles parameter. This makes
it possible to use sub-buffers.
v2:
- Style fixes
- Add support for constant sub-buffers
- Store handles in device byte order
---
src/gallium/drivers/r600/evergreen_compute.c | 10 +-
src/gallium/drivers/radeonsi/
Signed-off-by: Fabian Bieler
Reviewed-by: Brian Paul
Reviewed-by: Ian Romanick
---
src/glsl/geom-outlining-150.c | 26 --
1 file changed, 8 insertions(+), 18 deletions(-)
diff --git a/src/glsl/geom-outlining-150.c b/src/glsl/geom-outlining-150.c
index 0bc20f0..3dffa16 1
v2: Don't remove ShaderSupported() test. It sets up some function pointers for
the CompileShader framework.
Signed-off-by: Fabian Bieler
Reviewed-by: Brian Paul
Reviewed-by: Ian Romanick
---
src/glsl/gsraytrace.cpp | 24 +---
1 file changed, 9 insertions(+), 15 deletions(-)
NV_transform_feedback is not supported by mesa.
Use transform feedback from core OpenGL 3.0.
This necessitates binding the transform feedback varyings before linking the
shader.
Signed-off-by: Fabian Bieler
Reviewed-by: Brian Paul
Reviewed-by: Ian Romanick
---
src/glsl/gsraytrace.cpp | 72 +++
This commit prepares the transition from extension to core geometry shaders.
(Core geometry shaders require GLSL version 1.5 or later.)
This includes using generic vertex attributes instead of built-ins.
Signed-off-by: Fabian Bieler
Reviewed-by: Brian Paul
Reviewed-by: Ian Romanick
---
src/gls
v2: Remove redundant 'core' in GLSL version statement.
Signed-off-by: Fabian Bieler
Reviewed-by: Brian Paul
Reviewed-by: Ian Romanick
---
src/glsl/gsraytrace.cpp | 34 ++
1 file changed, 26 insertions(+), 8 deletions(-)
diff --git a/src/glsl/gsraytrace.cpp b/sr
To get an OpenGL core profile context freeglut 2.6 or later is required.
Note that in spite of it's name HAVE_FREEGLUT is only defined if freeglut 2.6
(released in 2009) or later ist found.
Signed-off-by: Fabian Bieler
Reviewed-by: Brian Paul
Reviewed-by: Ian Romanick
---
configure.ac | 6 +++
Signed-off-by: Fabian Bieler
Reviewed-by: Brian Paul
Reviewed-by: Ian Romanick
---
src/glsl/geom-outlining-150.c | 13 +
1 file changed, 13 insertions(+)
diff --git a/src/glsl/geom-outlining-150.c b/src/glsl/geom-outlining-150.c
index 3dffa16..2e2a54a 100644
--- a/src/glsl/geom-out
Bind the transform feedback buffer before drawing into it und unbind it
afterwards.
Signed-off-by: Fabian Bieler
Reviewed-by: Brian Paul
Reviewed-by: Ian Romanick
---
src/glsl/gsraytrace.cpp | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/glsl/gsraytrace.cpp b/src/gls
The hardcoded numbers are a few lines off at the moment.
Keeping track of the numbers through further modifications is inconvenient.
The __LINE__ "constant" takes care of this automatically.
v2: Don't set source-string-number to line number.
Signed-off-by: Fabian Bieler
Reviewed-by: Brian Paul
Hello!
As mesa only supports geometry shaders in core profile contexts this patchset
adjusts the gsraytrace and the geom-outlining-150 demos to use the core
profile.
This is v3 with some comments by Ian Romanick adressed.
The series is reviewed by Brian Paul and Ian Romanick.
As I don't have gi
The freeglut header only defines the extensions to request an OpenGL core
profile context if freeglut.h (rather than glut.h) is included.
Note that the header is installed to include/GL/freeglut.h on OS X, too.
Signed-off-by: Fabian Bieler
Reviewed-by: Brian Paul
Reviewed-by: Ian Romanick
---
Use a vbo for vertex data instead of client-side arrays.
Also bind a vertex array object.
This is necessary for the switch to a core profile context.
Signed-off-by: Fabian Bieler
Reviewed-by: Brian Paul
Reviewed-by: Ian Romanick
---
src/glsl/geom-outlining-150.c | 25 ++---
Signed-off-by: Fabian Bieler
Reviewed-by: Brian Paul
Reviewed-by: Ian Romanick
---
src/glsl/gsraytrace.cpp | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/glsl/gsraytrace.cpp b/src/glsl/gsraytrace.cpp
index c21c667..f156fdc 100644
--- a/src/glsl/gsraytrace.cpp
+++ b/
On 02/19/2014 01:32 PM, Anuj Phogat wrote:
> On Wed, Feb 19, 2014 at 2:04 AM, Kenneth Graunke
> wrote:
>> This is a straight port from gen7_wm_state.c; I haven't looked into
>> whether we can do both.
>>
> Verified that restriction still holds true in BDW.
> See 3D Pipeline Stages > Pixel > Pixel
On 02/19/2014 03:03 PM, Kenneth Graunke wrote:
> Currently, all platforms have Z16 performance issues. On Haswell and
> later, we could potentially implement the PMA stall optimization, which
> I believe would reduce(?) the problem. I'm not sure if it would
> eliminate it though.
>
> I think the
https://bugs.freedesktop.org/show_bug.cgi?id=71870
Ian Romanick changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On Tue, Feb 18, 2014 at 5:28 PM, Ian Romanick wrote:
> On 02/18/2014 03:36 PM, Anuj Phogat wrote:
>> On Tue, Feb 18, 2014 at 11:01 AM, Ian Romanick wrote:
>>> On 02/10/2014 05:29 PM, Anuj Phogat wrote:
GLSL 1.50 spec says:
"If gl_FragCoord is redeclared in any fragment shader in a pr
On 02/19/2014 02:27 PM, Ian Romanick wrote:
> On 02/19/2014 12:08 PM, Kenneth Graunke wrote:
>> On 02/18/2014 09:48 PM, Chia-I Wu wrote:
>>> Since 73bc6061f5c3b6a3bb7a8114bb2e1ab77d23cfdb, Z16 support is
>>> not advertised for OpenGL ES contexts due to the terrible
>>> performance. It is still ena
On Tue, Feb 18, 2014 at 05:50:19PM +0100, Francisco Jerez wrote:
> Tom Stellard writes:
>
> > From: Tom Stellard
> >
> > After pipe_loader_release() is called, if any of the pipe_* objects
> > try to call into the gallium API the application will segfault.
> >
> > The only time devices are delet
On Wed, Feb 19, 2014 at 09:20:22PM +, Dorrington, Albert wrote:
> Currently clGetDeviceInfo() returns an empty string when queried for
> CL_DEVICE_EXTENSIONS.
>
> Looking through both the Mesa and LLVM/Clang code I see references to the
> following extensions:
> cl_khr_fp64
> cl_khr_int64_ba
Kenneth Graunke writes:
> On 02/18/2014 09:48 PM, Chia-I Wu wrote:
>> Since 73bc6061f5c3b6a3bb7a8114bb2e1ab77d23cfdb, Z16 support is not advertised
>> for OpenGL ES contexts due to the terrible performance. It is still enabled
>> for desktop GL because it was believed GL 3.0+ requires Z16.
>>
>
I replied to 7 & 14. Series:
Reviewed-by: Jordan Justen
On Fri, Feb 7, 2014 at 10:00 PM, Ian Romanick wrote:
> 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-
On Fri, Feb 7, 2014 at 10:00 PM, Ian Romanick wrote:
> From: Gregory Hainaut
>
> This was originally included in another patch, but it was split out by
> Ian Romanick.
>
> Reviewed-by: Ian Romanick
> ---
> src/mesa/main/pipelineobj.c | 24
> 1 file changed, 24 insertion
On Wed, Feb 19, 2014 at 2:04 AM, Kenneth Graunke wrote:
> This is a straight port from gen7_wm_state.c; I haven't looked into
> whether we can do both.
>
Verified that restriction still holds true in BDW.
See 3D Pipeline Stages > Pixel > Pixel Shader Thread Generation >
Pixel Grouping (Dispatch Si
On Wed, Feb 19, 2014 at 2:04 AM, Kenneth Graunke wrote:
> These enumerations are simply log2 of the number of multisamples shifted
> by a bit, so we can calculate them using ffs() in a lot less code.
>
> Suggested by Eric Anholt.
>
> Signed-off-by: Kenneth Graunke
> ---
> src/mesa/drivers/dri/i9
Currently clGetDeviceInfo() returns an empty string when queried for
CL_DEVICE_EXTENSIONS.
Looking through both the Mesa and LLVM/Clang code I see references to the
following extensions:
cl_khr_fp16
cl_khr_fp64
cl_khr_int64_base_atomics
cl_khr_int64_extended_atomics
cl_khr_gl_sha
On 02/19/2014 11:09 AM, Ian Romanick wrote:
> I'm hoping that Tapani or Darius will verify that this patch actually
> fixes the problem. That's why people CC other people on patches. :)
I have the game, and I can confirm that the lighting is awfully broken
with master, and looks correct after thi
On 02/19/2014 11:12 AM, Eric Anholt wrote:
> Kenneth Graunke writes:
>
>> On 02/18/2014 01:38 PM, Eric Anholt wrote:
>>> Kenneth Graunke writes:
>> [snip]
diff --git a/src/mesa/drivers/dri/i965/gen8_depth_state.c
b/src/mesa/drivers/dri/i965/gen8_depth_state.c
index f30ff28..3fa20
On Fri, Feb 7, 2014 at 10:00 PM, Ian Romanick wrote:
> From: Gregory Hainaut
>
> 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 exten
On 02/18/2014 09:48 PM, Chia-I Wu wrote:
> Since 73bc6061f5c3b6a3bb7a8114bb2e1ab77d23cfdb, Z16 support is not advertised
> for OpenGL ES contexts due to the terrible performance. It is still enabled
> for desktop GL because it was believed GL 3.0+ requires Z16.
>
> It turns out only GL 3.0 requir
Kenneth Graunke writes:
> ---
> src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 6 ++
> 1 file changed, 6 insertions(+)
>
> I'm mostly sending this out as a placeholder. Ultimately, we want to get
> MCS working. I'm not sure whether it would be valuable to push this (with
> a proper commit
Kenneth Graunke writes:
> This is a straight port from gen7_wm_state.c; I haven't looked into
> whether we can do both.
>
> v2: Actually do it right.
>
> Signed-off-by: Kenneth Graunke
> @@ -205,13 +201,39 @@ upload_ps_state(struct brw_context *brw)
> else
>dw6 |= GEN7_PS_POSOFFSET_
Kenneth Graunke writes:
> v2: Also set the "oMask Present to Render Target" bit, which is required
> for shaders that write oMask. Otherwise the hardware won't expect
> the extra data.
>
> Signed-off-by: Kenneth Graunke
> ---
> src/mesa/drivers/dri/i965/gen8_ps_state.c | 16 +++
On 19/02/14 18:12, Kai Wasserbäch wrote:
> The referenced commit set the with_dri_drivers variable to "yes" in the
> auto case, which is an unknown classic DRI driver and leads to a FTBFS.
>
Thanks for the patch Kai
The issue has been reported already[1] and a slightly more appropriate
patch has
Paul Berry writes:
> On 15 January 2014 14:01, Francisco Jerez wrote:
>
>> Paul Berry writes:
>>
>> > On 2 December 2013 11:31, Francisco Jerez wrote:
>> >
>> >> Until now it was only being taken into account in the VEC4 back-end
>> >> but not in the FS back-end. Do it in both cases.
>> >> --
Kenneth Graunke writes:
> On 02/18/2014 01:38 PM, Eric Anholt wrote:
>> Kenneth Graunke writes:
> [snip]
>>> diff --git a/src/mesa/drivers/dri/i965/gen8_depth_state.c
>>> b/src/mesa/drivers/dri/i965/gen8_depth_state.c
>>> index f30ff28..3fa20c8 100644
>>> --- a/src/mesa/drivers/dri/i965/gen8_de
I'm hoping that Tapani or Darius will verify that this patch actually
fixes the problem. That's why people CC other people on patches. :)
On 02/18/2014 10:19 AM, Ian Romanick wrote:
> From: Ian Romanick
>
> Section 3.3 (Preprocessor) of the GLSL 1.30 spec (and later) and the
> GLSL ES spec (all
This patch didn't apply to the 10.1 branch. I've picked most things to
the 10.1 branch except this series. Could you put a branch up somewhere
and send me a pull request? I'm sure you'd like to have these in the
release, and I don't want to mess them up. :)
2e0c90847f16a9cf2a40436beacb65c65535f
Chia-I Wu writes:
> Since 73bc6061f5c3b6a3bb7a8114bb2e1ab77d23cfdb, Z16 support is not advertised
> for OpenGL ES contexts due to the terrible performance. It is still enabled
> for desktop GL because it was believed GL 3.0+ requires Z16.
>
> It turns out only GL 3.0 requires Z16, and that is co
The referenced commit set the with_dri_drivers variable to "yes" in the
auto case, which is an unknown classic DRI driver and leads to a FTBFS.
CC: Emil Velikov
Signed-off-by: Kai Wasserbäch
---
configure.ac | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/configure.ac b/conf
From: Christian König
Signed-off-by: Christian König
---
src/gallium/state_trackers/omx/vid_dec_h264.c | 4
1 file changed, 4 insertions(+)
diff --git a/src/gallium/state_trackers/omx/vid_dec_h264.c
b/src/gallium/state_trackers/omx/vid_dec_h264.c
index 5f4a261..7f1c2fa 100644
--- a/src/g
From: Leo Liu
Signed-off-by: Leo Liu
---
src/gallium/state_trackers/omx/vid_enc.c | 38
src/gallium/state_trackers/omx/vid_enc.h | 7 --
2 files changed, 29 insertions(+), 16 deletions(-)
diff --git a/src/gallium/state_trackers/omx/vid_enc.c
b/src/gallium
https://bugs.freedesktop.org/show_bug.cgi?id=75098
--- Comment #4 from Christian Prochaska
---
(In reply to comment #3)
> Created attachment 94319 [details] [review]
> configure: use shared-glapi when more than one gl* API is used
>
> Hmm forcing shared-glapi whenever more than one gl* api is u
https://bugs.freedesktop.org/show_bug.cgi?id=75212
--- Comment #1 from Emil Velikov ---
There are a couple of solutions for this [1] [2]. I would prefer the latter as
I've never been a fan of black/white listing.
Eric are you leaning towards either solution ? Can we get an ack or your
thoughts i
https://bugs.freedesktop.org/show_bug.cgi?id=75212
Priority: medium
Bug ID: 75212
CC: e...@anholt.net, lem...@gmail.com
Assignee: mesa-dev@lists.freedesktop.org
Summary: Mesa selects wrong DRI driver
Severity: normal
Classifi
Signed-off-by: Timothy Arceri
---
src/mesa/main/errors.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/mesa/main/errors.c b/src/mesa/main/errors.c
index 5f4eac6..c00c796 100644
--- a/src/mesa/main/errors.c
+++ b/src/mesa/main/errors.c
@@ -575,6 +575,11 @@ validate_params(struct gl_
On Broadwell, g0.5 contains the "Scratch Space Pointer"; using OR
puts some bits of that into "ignored" sections of our message header.
While this doesn't hurt, it's also not terribly /useful/. Using MOV
is sufficient to set the only interesting bits in this part of the
message header.
Signed-of
I can't find any documentation to explain what ought to be done here, so
I simply guessed based on the pattern I observed in the 4x/8x cases.
It appears to work, but it could be totally wrong.
I was able to find the Sandybridge PRM quote from the comments in the
latest documentation: Shared Functi
The enumerations are just log2(num_samples) shifted by 3, which we can
easily compute via ffs().
This also makes it reusable for Broadwell, which has 2x MSAA.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/gen7_wm_surface_state.c | 10 --
1 file changed, 4 insertions(+), 6
We need to advertise 8x, 4x, and 2x multisamples. Previously, we only
claimed to support 0/1 samples.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_context.c | 6 ++
src/mesa/drivers/dri/i965/intel_screen.c | 5 -
2 files changed, 10 insertions(+), 1 deletion(-)
Thi
We already set the number of samples, but were missing the MSAA layout
mode. Reusing gen7_surface_msaa_bits makes it easy to set both.
This also lets us drop the Gen8 surface_num_multisamples function.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/gen8_surface_state.c | 16 ++---
This is a straight port from gen7_wm_state.c; I haven't looked into
whether we can do both.
v2: Actually do it right.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/gen8_ps_state.c | 38 ---
1 file changed, 30 insertions(+), 8 deletions(-)
diff --git a
The meaning and effects of this bit are surprisingly complicated.
See Rasterization > Windower > Multisampling > Multisample ModesState.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_defines.h | 1 +
src/mesa/drivers/dri/i965/gen8_sf_state.c | 4
2 files changed, 5 ins
According to the "Point Multisample Rasterization" of the OpenGL
specification (3.0 or later), smooth points are supposed to be enabled
implicitly when multisampling, regardless of the GL_POINT_SMOOTH flag.
However, if GL_POINT_SPRITE is enabled, you get square points no matter
what. Core context
v2: Also set the "oMask Present to Render Target" bit, which is required
for shaders that write oMask. Otherwise the hardware won't expect
the extra data.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/gen8_ps_state.c | 16 +++-
1 file changed, 15 insertions(+)
I made a few changes which I think simplify the code a bit compared to
the Gen7 implementation, but which are largely pointless.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_fs.h | 3 +++
src/mesa/drivers/dri/i965/gen8_fs_generator.cpp | 36 +
---
src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 6 ++
1 file changed, 6 insertions(+)
I'm mostly sending this out as a placeholder. Ultimately, we want to get
MCS working. I'm not sure whether it would be valuable to push this (with
a proper commit message) in the meantime.
diff --git
Largely cut and paste from Gen7; it works the same way.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_fs.h | 4
src/mesa/drivers/dri/i965/gen8_fs_generator.cpp | 29 -
2 files changed, 32 insertions(+), 1 deletion(-)
diff --git a/src
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/gen8_ps_state.c | 18 ++
1 file changed, 18 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/gen8_ps_state.c
b/src/mesa/drivers/dri/i965/gen8_ps_state.c
index e93668e..57bf053 100644
--- a/src/mesa/drivers/dri/i965
These enumerations are simply log2 of the number of multisamples shifted
by a bit, so we can calculate them using ffs() in a lot less code.
Suggested by Eric Anholt.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/gen8_multisample_state.c | 26 +++---
1 file changed
On 02/18/2014 01:38 PM, Eric Anholt wrote:
> Kenneth Graunke writes:
[snip]
>> diff --git a/src/mesa/drivers/dri/i965/gen8_depth_state.c
>> b/src/mesa/drivers/dri/i965/gen8_depth_state.c
>> index f30ff28..3fa20c8 100644
>> --- a/src/mesa/drivers/dri/i965/gen8_depth_state.c
>> +++ b/src/mesa/drive
94 matches
Mail list logo
