https://bugs.freedesktop.org/show_bug.cgi?id=80848
--- Comment #14 from Juha-Pekka Heikkilä ---
I paste the contents here for both xcb.pc files as they're short.
Directories/files where they point are valid, I don't see anything wrong in
these.
/opt/lib/pkgconfig/xcb.pc:
prefix=/opt
exec_prefix=
https://bugs.freedesktop.org/show_bug.cgi?id=81834
--- Comment #1 from Christoph Haag ---
In case other people look at this bug, there is another backtrace where it runs
into a failed assertion: https://bugs.freedesktop.org/show_bug.cgi?id=75276#c32
You are right, after reverting f4b0ab7afd83c81
https://bugs.freedesktop.org/show_bug.cgi?id=81834
--- Comment #2 from Christoph Haag ---
Created attachment 103628
--> https://bugs.freedesktop.org/attachment.cgi?id=103628&action=edit
backtrace of effects demo with f4b0ab7afd83c811329211eae8167c9bf238870c
reverted
The effects cave demo is ha
https://bugs.freedesktop.org/show_bug.cgi?id=81834
--- Comment #3 from Michel Dänzer ---
(In reply to comment #2)
> The effects cave demo is having yet another crash.
[...]
> Might be because of the revert or not...
I don't think it's related to the revert. Looks like the kernel fails to
provide
Hi,
running the piglit tests on my implementation of geometry shaders for
Sandy Bridge produces a GPU hang for the following test:
./glsl-1.50-geometry-primitive-id-restart GL_TRIANGLE_STRIP_ADJACENCY
ffs
That test checks primitive restarts but the hang seems to be unrelated
to that, since it ha
Signed-off-by: Petri Latvala
---
src/glsl/tests/test_case_generator.py | 4
1 file changed, 4 insertions(+)
diff --git a/src/glsl/tests/test_case_generator.py
b/src/glsl/tests/test_case_generator.py
index 5df8420..6633702 100644
--- a/src/glsl/tests/test_case_generator.py
+++ b/src/glsl/te
optimization-test loops over directories that contain a file named
create_test_cases.py. For each occurrence, it enters the directory
that contains the file and executes it, and is then supposed to return
to the previous directory.
The name .. was hardcoded in the script, and should be
../.. inste
Signed-off-by: Petri Latvala
---
src/glsl/test_optpass.cpp | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/glsl/test_optpass.cpp b/src/glsl/test_optpass.cpp
index e4878bf..ef62752 100644
--- a/src/glsl/test_optpass.cpp
+++ b/src/glsl/test_optpass.cpp
@@ -131,6 +131,8 @@ do_optimization(
Already-existing and used declare_temp() and assign_x() functions can
then be implemented in terms of the general functions.
Signed-off-by: Petri Latvala
---
src/glsl/tests/test_case_generator.py | 19 ---
1 file changed, 16 insertions(+), 3 deletions(-)
diff --git a/src/glsl/te
Signed-off-by: Petri Latvala
---
src/glsl/tests/test_case_generator.py | 15 +++
1 file changed, 15 insertions(+)
diff --git a/src/glsl/tests/test_case_generator.py
b/src/glsl/tests/test_case_generator.py
index 6633702..5f550df 100644
--- a/src/glsl/tests/test_case_generator.py
+++
Sort expression operands when possible so that building expected IR
sexps doesn't need to know which ordering will be produced by an
optimization pass.
Signed-off-by: Petri Latvala
---
src/glsl/tests/compare_ir | 4 ++--
src/glsl/tests/sexps.py | 37 +
2 fi
Move the IR sexp builder helpers and test script creation parts of
tests/lower_jumps/create_test_cases.py into tests/test_case_generator.py
No functional changes.
Signed-off-by: Petri Latvala
---
src/glsl/tests/lower_jumps/create_test_cases.py | 336 +++-
src/glsl/tests/test
Continuing the tale of fixing
https://bugs.freedesktop.org/show_bug.cgi?id=76861
Instead of making an IR expression type for mid3, I implemented an
optimization pass that optimizes general min/max expression trees, by
dropping operands that are proven to never be selected as the final
result.
Thi
tests/minmax/create_test_cases.py generates the following tests:
multiple_min*.opt_test:
Construct a tree of min expressions for all permutations of a var_ref
and three constants. They should all optimize to a single min with
the variable and the smallest constant.
multiple_max*.opt_test:
Same
Add an optimization pass that drops min/max expression operands that
can be proven to not contribute to the final result. The algorithm is
similar to alpha-beta pruning on a minmax search, from the field of
AI.
This optimization pass can optimize min/max expressions where operands
are min/max expr
On Mon, Jul 28, 2014 at 8:47 PM, Anuj Phogat wrote:
> Fixes Khronos GLES3 CTS test:
> dynamic_expression_array_access_vertex
>
> Cc:
> Signed-off-by: Anuj Phogat
> ---
> I don't have a test case though it might be useful to also include
> the check for VS_OPCODE_URB_WRITE here?
>
> src/mesa/dri
Same comment as for 1/2: adding SHADER_OPCODE_GEN4_SCRATCH_WRITE to
the negaative list in can_do_source_mods should fix this.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Hi list,
I was starting to look into getting cppamp-driver-ng working with
mesa/clover, and I quickly ran into a question...
cppamp-driver-ng explicitly lists which platforms are supported in its
OpenCL back-end's source code. I went to add in the Mesa/Clover
entries, and discovered that we're ac
On 07/19/2014 09:16 AM, Paul wrote:
Brian Paul vmware.com> writes:
On 07/11/2014 01:47 AM, Florian Link wrote:
Hi everyone,
I read a post from 2012 that MESA supports geometry shaders. I tried a
current MESA 10.2.x release, but MESA GL_VERSION returns OpenGL version
3.0 (not 3.2) and the
GL
https://bugs.freedesktop.org/show_bug.cgi?id=81834
--- Comment #4 from Christoph Haag ---
(In reply to comment #3)
> (In reply to comment #2)
> > The effects cave demo is having yet another crash.
> [...]
> > Might be because of the revert or not...
>
> I don't think it's related to the revert.
https://bugs.freedesktop.org/show_bug.cgi?id=80848
--- Comment #15 from Emil Velikov ---
(In reply to comment #14)
> I paste the contents here for both xcb.pc files as they're short.
> Directories/files where they point are valid, I don't see anything wrong in
> these.
>
AFAICT they look good an
https://bugs.freedesktop.org/show_bug.cgi?id=81881
Priority: medium
Bug ID: 81881
Assignee: mesa-dev@lists.freedesktop.org
Summary: Scons+MSVC 12.0 shared targets build fails
Severity: normal
Classification: Unclassified
OS:
https://bugs.freedesktop.org/show_bug.cgi?id=81881
Yury Blokhin changed:
What|Removed |Added
CC||ultrab...@gmail.com
--
You are receiving
On Tuesday, July 29, 2014 10:12:23 AM Iago Toral Quiroga wrote:
> Hi,
>
> running the piglit tests on my implementation of geometry shaders for
> Sandy Bridge produces a GPU hang for the following test:
>
> ./glsl-1.50-geometry-primitive-id-restart GL_TRIANGLE_STRIP_ADJACENCY
> ffs
>
> That test
On Monday, July 28, 2014 06:32:47 PM Jason Ekstrand wrote:
> GL_SAMPLE_SHADING is specified as a valid pname for glGet in the
> GL_ARB_sample_shading extension. It seems as if we forgot to add it to the
> table of pnames.
>
> Signed-off-by: Jason Ekstrand
> ---
> src/mesa/main/get_hash_params.p
On 29/07/14 16:31, Brian Paul wrote:
On 07/19/2014 09:16 AM, Paul wrote:
Brian Paul vmware.com> writes:
On 07/11/2014 01:47 AM, Florian Link wrote:
Hi everyone,
I read a post from 2012 that MESA supports geometry shaders. I tried a
current MESA 10.2.x release, but MESA GL_VERSION returns O
https://bugs.freedesktop.org/show_bug.cgi?id=81881
Emil Velikov changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On 24/07/14 01:43, Emil Velikov wrote:
> The kms_swrast driver has a separate InitScreen hook for its DriverAPI
> from the rest of the DRI2 drivers, all of which capable of buffer
> sharing. As such we no longer need to dive through the pipe-driver and
> winsys layers in order to determine if the d
On 25/07/14 07:34, Pekka Paalanen wrote:
> On Fri, 25 Jul 2014 09:14:40 +0300
> Pekka Paalanen wrote:
>
[snip]
> Hm, reading Kenneth's comments about variable naming at
> http://lists.freedesktop.org/archives/mesa-dev/2014-July/064103.html
> I wonder, is there real benefit for different driver lo
https://bugs.freedesktop.org/show_bug.cgi?id=81881
--- Comment #2 from Yury Blokhin ---
Yes, this fix helped, thanx!
--
You are receiving this mail because:
You are the assignee for the bug.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
htt
master indeed works without having to set the environment variable (at
least with a simple test program targeting OpenGL 3.3 core using a geometry
shader). OpenGL Extensions Viewer, however, still sees only OpenGL 3.0
without the environment variable set. Although, it would be nice to find
out what
On Tue, Jul 29, 2014 at 7:18 AM, Matt Turner wrote:
>
> On Mon, Jul 28, 2014 at 8:47 PM, Anuj Phogat wrote:
> > Fixes Khronos GLES3 CTS test:
> > dynamic_expression_array_access_vertex
> >
> > Cc:
> > Signed-off-by: Anuj Phogat
> > ---
> > I don't have a test case though it might be useful to a
See https://code.google.com/p/chromium/issues/detail?id=308715 for more
info.
Review URL: https://codereview.chromium.org/27740005
Patch authored by hu...@chromium.org, as per
https://android.googlesource.com/platform/external/chromium_org/third_party/mesa/src/+/009d829daee69743d77f4eaea702633a2f
Considering the way we've been consolidating things it makes
sense to add the final two (aux and tests) in here.
Signed-off-by: Emil Velikov
---
src/Makefile.am | 10 +-
src/gallium/Makefile.am | 16
2 files changed, 17 insertions(+), 9 deletions(-)
diff --git a
nouveau uses STL for a while now thus we need to include
external/stlport/libstlport.mk in order to get the build
at least partially working.
v2: Use the installed libdrm headers over the DRM_TOP ones.
Cc: "10.1 10.2"
Signed-off-by: Emil Velikov
---
src/gallium/drivers/nouveau/Android.mk|
For all everyone willing to give the freedreno driver
a go they can now build it under Android.
Cc: "10.1 10.2"
Cc: Rob Clark
Cc: freedr...@lists.freedesktop.org
Signed-off-by: Emil Velikov
---
src/egl/main/Android.mk | 6 ++
src/gallium/targets/egl-static/Android.mk | 3
Yet another makefile less to worry about.
Signed-off-by: Emil Velikov
---
configure.ac| 1 -
src/Makefile.am | 3 +-
src/gallium/Makefile.am | 25 ---
src/gallium/targets/Makefile.am | 71 -
4 files
Saves us a few lines and brings us closer to the automake build.
Drop DRM_TOP as it's not longer used.
Signed-off-by: Emil Velikov
---
Android.mk | 1 -
src/mesa/drivers/dri/Android.mk | 3 +--
src/mesa/drivers/dri/i915/Android.mk | 3 +--
src/mesa/drivers/dri/i965
Rather than having two separate almost empty and identical makefiles,
compact them thus improving the configure and build time.
Additionally this makes the automake build symmetrical to the scons
and android one.
Signed-off-by: Emil Velikov
---
configure.ac| 3 +-
src/Makef
Saves us a few lines and brings us closer to the automake build.
Signed-off-by: Emil Velikov
---
src/gallium/winsys/i915/drm/Android.mk | 5 +
src/gallium/winsys/intel/drm/Android.mk | 5 +
src/gallium/winsys/svga/drm/Android.mk | 3 +--
3 files changed, 3 insertions(+), 10 deletions(-
If the user/builder has a setup capable of using sse4.1 it's their
responsibility to enable it.
Let's unconditionally include main/streaming-load-memcpy.c, as it
already features a ifdef __SSE4_1__ check and add a stub implementation
for people that don't have -msse4.1 enabled at compile-time.
Th
From: "Myles C. Maxfield"
Android includes a fix to asm/posix_types.h inside
/ndk/toolchains/x86-4.7/prebuilt/linux-x86_64/lib/gcc/
i686-linux-android/4.7/include-fixed/asm/posix_types.h. That
file #include_next's the real asm/posix_types.h, but then fixes
(redefines) the __FD_ZERO macro. However
From: Paulo Sergio Travaglia
- link against libdrm_radeon
- link the r600 driver against libstlport
- linkin the newly added libmesa_pipe_radeon library
required by r600 and radeonsi drivers
v2: Include pipe_radeon after pipe_r600/radeonsi.
Cc: "10.1 10.2"
[Emil Velikov] Split up and add co
Hello list,
Here is an update+expansion of a lovely series that cleans up our builds
and as an added bonus it makes things build for android on almost every
driver but
- radeonsi - driver requires llvm but the build is not there yet.
- nouveau - bionic loves tr1/* C++ headers + broken drm_gral
One step closer to the way we handle automake builds.
Signed-off-by: Emil Velikov
---
src/loader/Android.mk | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/src/loader/Android.mk b/src/loader/Android.mk
index 6890af4..1c6fdae 100644
--- a/src/loader/Android.mk
+++ b/src/
The latter lacks various functionality used by mesa/glsl.
Cc: "10.1 10.2"
Signed-off-by: Emil Velikov
---
src/glsl/Android.mk | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/glsl/Android.mk b/src/glsl/Android.mk
index 8a39426..7b1fa7e 100644
--- a/src/glsl/Android.mk
+++
No longer needed as of last commit.
Signed-off-by: Emil Velikov
---
src/mesa/drivers/dri/i915/Android.mk | 3 +--
src/mesa/drivers/dri/i915/Makefile.sources | 4
src/mesa/drivers/dri/i965/Android.mk | 3 +--
src/mesa/drivers/dri/i965/Makefile.sources | 7 +--
4 files changed
From: Paulo Sergio Travaglia
- include the correct folders
- add a new buildscript for the common radeon folder
v2: Use the installed libdrm headers over the DRM_TOP ones.
Cc: "10.1 10.2"
[Emil Velikov] Split up and add commit message.
Signed-off-by: Emil Velikov
---
src/gallium/Android.mk
Rather than having the sources list duplicated across all three
build systems, define it once and use it whenever needed.
Signed-off-by: Emil Velikov
---
src/egl/main/Android.mk | 23 ---
src/egl/main/Makefile.am | 40 +++-
src/e
Android build never really installs the headers, as such we need to
explicitly add their location in the source tree otherwise it will
fail to find them.
v2: Android now installs the headers, so let's use that ;)
Signed-off-by: Emil Velikov
---
src/gallium/targets/egl-static/Android.mk | 4 +---
The variable i915_C_FILES changed to i915_FILES with commit
34d4216e641 back in mesa 9.1/9.2. Yet we've missed to update the
the android build, essentially creating an dummy/empty driver that
can never work.
Cc: "10.1 10.2"
Signed-off-by: Emil Velikov
---
src/mesa/drivers/dri/i915/Android.mk |
Will allow us to nuke an include or two from the drivers.
Signed-off-by: Emil Velikov
---
src/mesa/drivers/dri/Android.mk | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/drivers/dri/Android.mk b/src/mesa/drivers/dri/Android.mk
index 5cb6d7b..1ffa845 100644
--- a/src/mesa/drivers/dri
Trying to get rid of the hardcoded dependency of DRM_TOP which
expects that mesa is localted in /external/drm. Will
Signed-off-by: Emil Velikov
---
src/egl/drivers/dri2/Android.mk | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/egl/drivers/dri2/Android.mk b/src/egl/driver
Both share the identical dependencies, as such we can simplify
the scons script.
Signed-off-by: Emil Velikov
---
src/gallium/SConscript | 42 +++---
1 file changed, 11 insertions(+), 31 deletions(-)
diff --git a/src/gallium/SConscript b/src/gallium/SConscript
Rather than including two extra folders only for two headers,
just prefix the headers and be done with it.
Cc: "10.1 10.2"
Cc: Rob Clark
Cc: freedr...@lists.freedesktop.org
Signed-off-by: Emil Velikov
Reviewed-by: Rob Clark
---
src/gallium/drivers/freedreno/Makefile.am| 2 --
src/gall
To be closer to its automake counterpart.
Signed-off-by: Emil Velikov
---
src/gallium/Android.mk | 14 ++
1 file changed, 10 insertions(+), 4 deletions(-)
diff --git a/src/gallium/Android.mk b/src/gallium/Android.mk
index 767361a..bf99e4f 100644
--- a/src/gallium/Android.mk
+++ b/sr
Upstream Android (system/core) has dropped these formats with commit
6bac41f1bf9(get rid of HAL pixelformats 5551 and ) yet does not
mention why.
These formats never really worked so we're safe to drop them as well.
Identical commit is available in the android-x86 external/mesa repo
comm
For a while the nouveau pipe driver has been a static library
and it has been using STL for even longer.
Correct add the link and cleanup the gallium_DRIVERS.
Cc: "10.1 10.2"
Signed-off-by: Emil Velikov
---
src/egl/main/Android.mk | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
d
Recent versions of bionic has picked up support for these functions,
leading to build issues due to the redefition of the symbols.
Note: wrapping things in #ifdef does not cut it :\
Identical patch is available in chromium, android-x86 and perhaps other
projects.
commit 66c1c789ce3407472de9e
For all the people interested in testing the freedreno driver on
their Android devices. The next commit will hook these up within
the libEGL driver (via the gallium-egl backend).
There may be some rough edges but those can be sorted when a
willing builder/tester comes along.
v2:
- s/freefreno/fr
Signed-off-by: Emil Velikov
---
src/mesa/Makefile.am | 10 --
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/src/mesa/Makefile.am b/src/mesa/Makefile.am
index d59af99..e7d55c9 100644
--- a/src/mesa/Makefile.am
+++ b/src/mesa/Makefile.am
@@ -120,20 +120,18 @@ endif
libmesa_
One makefile less, with the potential of further compacting the
automake build.
Signed-off-by: Emil Velikov
---
configure.ac | 1 -
src/Makefile.am| 1 -
src/gallium/Makefile.am| 49 +
src/gallium/state_t
ERROR is a #define in the MSVC WinGDI.h header file.
Add the _TOKEN suffix as we do for a few other lexer tokens.
---
src/glsl/glcpp/glcpp-lex.l |2 +-
src/glsl/glcpp/glcpp-parse.y |6 +++---
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/glsl/glcpp/glcpp-lex.l b/src/
From: Ian Romanick
Returns the type without any arrays.
This will be used in later patches in this series.
Signed-off-by: Ian Romanick
Suggested-by: Timothy Arceri
Cc: Timothy Arceri
---
src/glsl/glsl_types.h | 12
1 file changed, 12 insertions(+)
diff --git a/src/glsl/glsl_ty
From: Ian Romanick
Signed-off-by: Ian Romanick
Reviewed-by: Matt Turner [v1]
Cc: Timothy Arceri
---
src/glsl/ast_to_hir.cpp| 3 +--
src/glsl/glsl_types.cpp| 3 +--
src/glsl/link_uniforms.cpp | 23 ---
src/glsl/link_varyings.cpp | 6 ++
4 files changed, 12 in
Reviewed-by: Ian Romanick
On 07/22/2014 02:07 PM, Alon Levy wrote:
> Remove incorrect struct prefix, ir_variable is a class
>
> Signed-off-by: Alon Levy
> ---
> src/glsl/opt_dead_builtin_varyings.cpp | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/glsl/opt_dead_bu
On Tue, Jul 29, 2014 at 3:54 PM, Emil Velikov wrote:
> If the user/builder has a setup capable of using sse4.1 it's their
> responsibility to enable it.
>
> Let's unconditionally include main/streaming-load-memcpy.c, as it
> already features a ifdef __SSE4_1__ check and add a stub implementation
>
sdc_offset is produced and consumed in the same function, so there's no
need to store it in the context, nor pass pointers to it through various
call chains.
Saves 128 bytes per brw_stage_state structure, and makes the code
clearer as well.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dr
Instead of stuffing bits directly into the brw_sampler_state structure,
we now store them in local variables, then use brw_emit_sampler_state()
to assemble the packet. This separates the decision about what values
to use from the actual packet emission, which makes the code more
reusable across ge
This was the only actual difference between Gen4-6 and Gen7+ in terms of
the values we program. The rest was just mechanical structure
rearrangement.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_sampler_state.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
d
This copies a few changes from gen7_upload_sampler_state_table; the next
patch will delete that function.
Gen7+ has per-stage sampler state pointer update packets, so we emit
them as soon as we emit a new table for a stage. On Gen6 and earlier,
we have a single packet, so we delay until we've cha
The code in brw_sampler_state.c now handles all generations; we don't
need the extra Gen7+ only code anymore.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/Makefile.sources | 1 -
src/mesa/drivers/dri/i965/brw_sampler_state.c | 11 +-
src/mesa/drivers/dri/i965/brw_state.h
A while back, I wrote a patch to skip border color upload, and got
annoyed by having to write it twice - once for the Gen4-6 code, and
once for Gen7+. So I decided to clean that up.
23 patches and a bug fix later, I've unified the code, deleting
gen7_sampler_state.c entirely. There's now one cop
Although the Gen4-6 and Gen7+ variants used different structure types,
they didn't use any of the fields - only the size, which is identical.
So both decoders did exactly the same thing.
Someday we should implement useful decoders for SAMPLER_STATE.
Signed-off-by: Kenneth Graunke
---
src/mesa/d
This simplifies the code, removes use of the old structures, and also
allows us to combine the Gen6 and Gen7+ code.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_blorp.h| 4 ++
src/mesa/drivers/dri/i965/gen6_blorp.cpp | 65 ++--
src/mesa/driver
This simply assembles all the SAMPLER_STATE fields into their proper bit
locations. Making it work on all generations was easy enough; some of
the fields are even in the same place.
Not used by anything yet, but will be soon. I made it non-static so
BLORP can use it too.
Signed-off-by: Kenneth
The border color is only needed when using the GL_CLAMP_TO_BORDER or
(deprecated) GL_CLAMP wrap modes; all others ignore it, including the
common GL_CLAMP_TO_EDGE and GL_REPEAT wrap modes.
In those cases, we can skip uploading it entirely, saving a bit of space
in the batchbuffer. Instead, we jus
We'll use these to replace the existing structures.
I've adopted the convention that "BRW" applies to all hardware, and
"GENX" applies starting with generation X, but might be replaced by some
later generation.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_defines.h | 54
This is the last user of the structure. Hardcoding 16 is poor style,
but this code is already a senseless mess, so it's not that much worse.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_draw.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/src/mesa/
The Gen4-6 and Gen7+ code is virtually identical, but both use different
structure types. Switching to use a uint32_t pointer and operate on the
number of DWords will make it possible to share code.
It turns out that SURFACE_STATE is the same number of DWords on every
platform currently; it will
brw_upload_sampler_state_table now handles all generations, so we don't
need the vtable mechanism either.
There's still a lot of code duplication; the next patches will address
that.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_context.c| 3 --
src/mesa/drivers/dri/
We've moved to using bitshifts (like we did for surface state); nothing
uses the structures anymore.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_structs.h | 99 -
1 file changed, 99 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_stru
The old one has been inaccurate for years.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_sampler_state.c | 13 ++---
1 file changed, 6 insertions(+), 7 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_sampler_state.c
b/src/mesa/drivers/dri/i965/brw_sampler_sta
It's just an array of four floats, and we have an array of four floats,
so this is literally just a memcpy...but with custom structs and strange
macros to give the appearance of doing something more.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_sampler_state.c | 9 +++--
When the driver was originally written, it only supported texturing in
the pixel shader backend; vertex and geometry shader texturing came much
later. Originally, the pixel shader was referred to as "WM" (the
Windowizer/Masker unit). So, this code happened to only be relevant for
the WM stage, at
This makes it easy to tell that they're grouped together, and also
improves gdb printing.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_defines.h | 16 +---
1 file changed, 9 insertions(+), 7 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_defines.h
b/src
Now that gen7_sampler_state.c is gone, everything is once again in a
single file.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_sampler_state.c | 4 ++--
src/mesa/drivers/dri/i965/brw_state.h | 6 --
2 files changed, 2 insertions(+), 8 deletions(-)
diff --git a/sr
The "Min/Mag State Not Equal" bit is supposed to be set when the min/mag
filters or address rounding modes differ. BLORP uses identical min/mag
settings, so the bit should be unset.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/gen6_blorp.cpp | 2 --
1 file changed, 2 deletions(-
Other than this, brw_update_sampler_state only deals with a single
SAMPLER_STATE structure, and doesn't need to know which position it is
in the table. The caller takes care of dealing with multiple surface
states.
Pushing this up a level allows us to drop the ss_index parameter.
Signed-off-by:
This was copied from the Gen4-6 code, but is unused.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/gen7_sampler_state.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/gen7_sampler_state.c
b/src/mesa/drivers/dri/i965/gen7_sampler_
These are the last users of struct gen7_sampler_state.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_fs_generator.cpp| 2 +-
src/mesa/drivers/dri/i965/brw_vec4_generator.cpp | 2 +-
src/mesa/drivers/dri/i965/gen8_fs_generator.cpp | 3 +--
src/mesa/drivers/dri/i965/gen8_
On Fri, Jun 6, 2014 at 9:13 PM, Matt Turner wrote:
> On Fri, Jun 6, 2014 at 4:57 PM, Anuj Phogat wrote:
>> Fixes gles3 Khronos CTS test: texture_storage_texture_targets
>>
>> Cc:
>> Signed-off-by: Anuj Phogat
>> ---
>> src/mesa/main/texstorage.c | 7 +++
>> 1 file changed, 7 insertions(+)
On Fri, Jun 6, 2014 at 4:57 PM, Anuj Phogat wrote:
> Fixes gles3 Khronos CTS test: texture_storage_texture_internal_formats
>
> Cc:
> Signed-off-by: Anuj Phogat
> ---
> src/mesa/main/texstorage.c | 18 ++
> 1 file changed, 18 insertions(+)
>
> diff --git a/src/mesa/main/texstor
If we're going to make a utility function declared in glformats.h, let's
move the function body to glformats.c.
On Fri, Jun 6, 2014 at 4:57 PM, Anuj Phogat wrote:
> Cc:
> Signed-off-by: Anuj Phogat
> ---
> src/mesa/main/glformats.h | 3 +++
> src/mesa/main/teximage.c | 12 ++--
> 2
I think you're missing the SRGB and SLUMINANCE formats. Other than that, I
think that's all of them.
On Fri, Jun 6, 2014 at 4:57 PM, Anuj Phogat wrote:
> Cc:
> Signed-off-by: Anuj Phogat
> ---
> src/mesa/main/glformats.c | 67
> +++
> src/mesa/mai
You should fix the commit message to be _mesa_is_enum_format_snorm (you
forgot the beginning underscore). Other than that, looks good.
Reviewed-by: Jason Ekstrand
On Fri, Jun 6, 2014 at 4:57 PM, Anuj Phogat wrote:
> Cc:
> Signed-off-by: Anuj Phogat
> ---
> src/mesa/main/glformats.c | 37 +
Reviewed-by: Jason Ekstrand
On Fri, Jun 6, 2014 at 4:57 PM, Anuj Phogat wrote:
> Cc:
> Signed-off-by: Anuj Phogat
> ---
> src/mesa/main/teximage.c | 9 -
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c
> index b86
On 30/07/14 00:24, Matt Turner wrote:
> On Tue, Jul 29, 2014 at 3:54 PM, Emil Velikov
> wrote:
>> If the user/builder has a setup capable of using sse4.1 it's their
>> responsibility to enable it.
>>
>> Let's unconditionally include main/streaming-load-memcpy.c, as it
>> already features a ifdef
On Fri, Jun 6, 2014 at 4:57 PM, Anuj Phogat wrote:
> Function is utilized by next patch in the series.
>
> Cc:
> Signed-off-by: Anuj Phogat
> ---
> src/mesa/main/glformats.c | 18 ++
> src/mesa/main/glformats.h | 3 +++
> 2 files changed, 21 insertions(+)
>
> diff --git a/src/
Looks good to me.
Reviewed-by: Jason Ekstrand
On Fri, Jun 6, 2014 at 4:57 PM, Anuj Phogat wrote:
> Fixes many failures in gles3 Khronos CTS test: packed_pixels
>
> Cc:
> Signed-off-by: Anuj Phogat
> ---
> src/mesa/main/teximage.c | 15 +++
> 1 file changed, 15 insertions(+)
>
>
1 - 100 of 120 matches
Mail list logo
