A really nice piece of work, thx allot.
Both patches reviewed and pushed upstream.
Cheers,
Christian.
Am 13.02.2014 21:32, schrieb Grigori Goronzy:
---
src/gallium/auxiliary/Makefile.sources | 3 +-
src/gallium/auxiliary/vl/vl_deint_filter.c | 491 +
On 02/13/2014 06:09 AM, sathishkumar sivagurunathan wrote:
[snip]
Mobile 4 Series Chipset Integrated Graphics Controller = i915
The kernel driver for that hardware is i915.ko, but the 3D driver is
i965 (i965_dri.so).
--Ken
signature.asc
Description: OpenPGP digital signature
On 2014-02-13 14:13:05 +, Emil Velikov wrote:
On 13/02/14 13:20, Kusanagi Kouichi wrote:
libselinux provides pkgconfig file since 2.0.89 (2009-10-29).
Can you check how many of the currently supported distros include that
version or later ?
I was nicely surprised when I did a similar
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 4a195bd..837c5b0 100644
--- a/src/glx/glxext.c
+++ b/src/glx/glxext.c
@@ -686,7 +686,8 @@ static
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/glx/indirect_glx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/glx/indirect_glx.c b/src/glx/indirect_glx.c
index 28b8cd0..306bf5b 100644
--- a/src/glx/indirect_glx.c
+++ b/src/glx/indirect_glx.c
@@
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/glx/glxcmds.c | 13 -
1 file changed, 8 insertions(+), 5 deletions(-)
diff --git a/src/glx/glxcmds.c b/src/glx/glxcmds.c
index 38a5262..4d8d0c2 100644
--- a/src/glx/glxcmds.c
+++ b/src/glx/glxcmds.c
@@ -1392,13
_eglInitResource() was used to memset entire _EGLSync by
writing more than size of pointed target. This does work
as long as Resource is the first element in _EGLSync,
this patch fixes such dependency.
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/egl/main/eglsync.c | 3
Check calloc return values in hash_table_insert() and
hash_table_replace()
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/mesa/program/prog_hash_table.c | 16 ++--
1 file changed, 10 insertions(+), 6 deletions(-)
diff --git
Resend of the earlier glx patches with the issue pointed out by Petri fixed
as well as two additions.
Patch number five is a bit special. hash_table_insert() and
hash_table_replace() don't really have a way to report errors and I did not
want to go changing the api since these are called from
Requested by Marek.
Cc: 10.1 mesa-sta...@lists.freedesktop.org
---
src/gallium/auxiliary/pipebuffer/pb_bufmgr.h |4 ++--
src/gallium/auxiliary/pipebuffer/pb_bufmgr_cache.c |8
src/gallium/winsys/radeon/drm/radeon_drm_winsys.c |2 +-
Sure. If you don't mind, I'll just do that as a follow-on patch...
-Brian
On 02/13/2014 07:11 PM, Marek Olšák wrote:
Please, can the size factor be a float?
Thanks,
Marek
On Fri, Feb 14, 2014 at 2:21 AM, Brian Paul bri...@vmware.com wrote:
From: Thomas Hellstrom thellst...@vmware.com
In
https://bugs.freedesktop.org/show_bug.cgi?id=74988
Priority: medium
Bug ID: 74988
Assignee: mesa-dev@lists.freedesktop.org
Summary: Buffer overrun (segfault) decompressing ETC2 texture
in GLBenchmark 3.0 Manhattan
Severity:
On 02/13/2014 03:18 PM, Fabian Bieler wrote:
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 v2 with the comment by Ian Romanick regarding incorrect usage of the
GLSL
Decompressing ETC2 textures was causing intermitent segfault
by copying resulting 4x4 texel block to the destination texture
regardless of the size of the destination texture. Issue found
via application crash in GLBenchmark 3.0's Manhattan test.
Signed-off-by: Courtney Goeltzenleuchter
Forgot to mention. This patch addresses bug
#74988https://bugs.freedesktop.org/show_bug.cgi?id=74988
.
No piglit regressions.
Courtney
On Fri, Feb 14, 2014 at 8:52 AM, Courtney Goeltzenleuchter
court...@lunarg.com wrote:
Decompressing ETC2 textures was causing intermitent segfault
by
Initialize field to eliminate valgrind warning.
Signed-off-by: Courtney Goeltzenleuchter court...@lunarg.com
---
src/mesa/main/texcompress_etc.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/main/texcompress_etc.c b/src/mesa/main/texcompress_etc.c
index f9234b0..97adc86 100644
---
Initialize field to eliminate valgrind warning.
Signed-off-by: Courtney Goeltzenleuchter court...@lunarg.com
---
src/mesa/main/texcompress_etc.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/main/texcompress_etc.c b/src/mesa/main/texcompress_etc.c
index f9234b0..97adc86 100644
---
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Fri, Feb 14, 2014 at 3:47 PM, Brian Paul bri...@vmware.com wrote:
Requested by Marek.
Cc: 10.1 mesa-sta...@lists.freedesktop.org
---
src/gallium/auxiliary/pipebuffer/pb_bufmgr.h |4 ++--
In doing some testing we’ve noticed that trying to draw pixel aligned textures does not work very well with linear filtering in llvmpipe.Here’s an example of the problem.Imagine wanting to paint a 100x100 texture. After being scaled by 100 the texture coordinates will end up as:0.5, 1.5, 2.5, 3.5,
Am 14.02.2014 18:07, schrieb Jeff Muizelaar:
In doing some testing we’ve noticed that trying to draw pixel aligned
textures does not work very well with linear filtering in llvmpipe.
Here’s an example of the problem.
Imagine wanting to paint a 100x100 texture. After being scaled by 100
https://bugs.freedesktop.org/show_bug.cgi?id=74251
--- Comment #10 from Andreas Boll andreas.boll@gmail.com ---
Should be fixed with
http://cgit.freedesktop.org/mesa/mesa/commit/?id=c6dbcf10dff1f8343a26081f5489ef732ebb5460
--
You are receiving this mail because:
You are the assignee for
https://bugs.freedesktop.org/show_bug.cgi?id=74717
Andreas Boll andreas.boll@gmail.com changed:
What|Removed |Added
Assignee|mesa-dev@lists.freedesktop.
https://bugs.freedesktop.org/show_bug.cgi?id=73900
Andreas Boll andreas.boll@gmail.com changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
On Feb 14, 2014, at 1:00 PM, Roland Scheidegger srol...@vmware.com wrote:
Am 14.02.2014 18:07, schrieb Jeff Muizelaar:
I'll need to take another look and run some tests, though I've got some
quick comments:
@@ -1031,16 +1082,28 @@ lp_build_sample_image_linear(struct
On Thu, Feb 6, 2014 at 6:28 PM, Jordan Justen jordan.l.jus...@intel.com wrote:
v4:
* Merge with recent compute shader parsing of
input layout qualifiers
v3:
* Fix major brokenness of dual instance mode operation
using Paul's suggestions
* Update parsing to allow separate primitive
On 02/14/2014 08:05 AM, Courtney Goeltzenleuchter wrote:
Initialize field to eliminate valgrind warning.
There are a couple other fields that aren't set it all paths (e.g.,
flipped). I want to suggest just memseting the whole structure, but
it's not obvious to me how it's used throughout the
https://bugs.freedesktop.org/show_bug.cgi?id=74251
Marek Olšák mar...@gmail.com changed:
What|Removed |Added
Status|NEW |RESOLVED
On 02/14/2014 07:52 AM, Courtney Goeltzenleuchter wrote:
Decompressing ETC2 textures was causing intermitent segfault
by copying resulting 4x4 texel block to the destination texture
regardless of the size of the destination texture. Issue found
via application crash in GLBenchmark 3.0's
On Thu, Feb 13, 2014 at 07:56:26AM -0800, Matt Arsenault wrote:
On Feb 7, 2014, at 7:46 AM, Tom Stellard t...@stellard.net wrote:
From: Tom Stellard thomas.stell...@amd.com
---
lib/Target/R600/AMDGPUISelDAGToDAG.cpp | 48
++
Am 14.02.2014 19:59, schrieb Jeff Muizelaar:
On Feb 14, 2014, at 1:00 PM, Roland Scheidegger srol...@vmware.com
mailto:srol...@vmware.com wrote:
Am 14.02.2014 18:07, schrieb Jeff Muizelaar:
I'll need to take another look and run some tests, though I've got some
quick comments:
@@
So far it's happened to be that we're only ever calling
intel_miptree_blit() (up/downsampling) from the ReadBuffer, but I stumbled
over a null ReadBuffer case when debugging later parts of the series.
---
src/mesa/drivers/dri/i965/brw_blorp_blit.cpp | 7 ---
1 file changed, 4 insertions(+), 3
The only DRI2 vs DRI3 delta was just how to decide about frontbuffer-ness
for doing the upsample.
---
src/mesa/drivers/dri/i965/brw_context.c | 26 ---
src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 102 --
src/mesa/drivers/dri/i965/intel_mipmap_tree.h | 19
Mostly mt-target == 2D_MS just results in a few checks that we don't try
to allocate multiple LODs and don't try to do slice copies with them. But
with the introduction of binding renderbuffers to textures, we need more
consistency.
---
src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 8 +---
Compare this VS to the one for the post-130 case. Fixes piglit
glsl-lod-bias, and presumably tons of other code (I haven't done a full
piglit run on swrast).
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=74911
---
src/mesa/drivers/common/meta.c | 2 +-
1 file changed, 1 insertion(+), 1
This lets us simplify our shaders, and rely on GLES-prohibited
functionality (like ARB_texture_multisample) when writing these
driver-internal functions.
---
src/mesa/drivers/common/meta.c | 39 ---
src/mesa/drivers/common/meta.h | 3 +++
2 files changed, 19
Pretty silly to pass in values dereferenced out of one of the arguments.
---
src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 35 +--
1 file changed, 11 insertions(+), 24 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c
It often confused people because it was unclear on whether it was the
physical or logical, and people needed the other one as well. We can
recompute it trivially using the minify() macro, clarifying which value is
being used and making getting the other value obvious.
---
Since only window system renderbuffers can have a singlesample_mt, this
lets us drop a bunch of sanity checking to make sure that we're just a
renderbuffer-like thing.
---
src/mesa/drivers/dri/i965/brw_context.c | 20 ++-
src/mesa/drivers/dri/i965/intel_fbo.c | 91 ++-
Am 14.02.2014 23:35, schrieb Roland Scheidegger:
Am 14.02.2014 19:59, schrieb Jeff Muizelaar:
On Feb 14, 2014, at 1:00 PM, Roland Scheidegger srol...@vmware.com
mailto:srol...@vmware.com wrote:
Am 14.02.2014 18:07, schrieb Jeff Muizelaar:
I'll need to take another look and run some tests,
On 02/14/2014 03:00 PM, Eric Anholt wrote:
Compare this VS to the one for the post-130 case. Fixes piglit
glsl-lod-bias, and presumably tons of other code (I haven't done a full
piglit run on swrast).
Looks like a good fix. If we had NV_explicit_attrib_location[1], we
could easily unify a
This is a nice improvement.
Reviewed-by: Chris Forbes chr...@ijw.co.nz
On Sat, Feb 15, 2014 at 12:00 PM, Eric Anholt e...@anholt.net wrote:
It often confused people because it was unclear on whether it was the
physical or logical, and people needed the other one as well. We can
recompute it
We generate steaming piles of these for the centroid workaround, and this
quickly cleans them up.
total instructions in shared programs: 1591228 - 1590047 (-0.07%)
instructions in affected programs: 26111 - 24930 (-4.52%)
GAINED:0
LOST:
Improves performance of a dolphin emulator trace I had laying around by
3.60131% +/- 0.995887% (n=128).
---
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 23 ++-
1 file changed, 18 insertions(+), 5 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
The code was removed early last year.
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 6 --
1 file changed, 6 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
b/src/mesa/drivers/dri/i965/brw_fs.cpp
index 4f5558b..d35928e 100644
--- a/src/mesa/drivers/dri/i965/brw_fs.cpp
+++
Section 4.3.1, page 220, of OpenGL 3.3 specification explains
the error conditions for glreadPixels():
If the format is DEPTH_STENCIL, then values are taken from
both the depth buffer and the stencil buffer. If there is
no depth buffer or if there is no stencil buffer, then the
From: Jeff Muizelaar jmuizel...@mozilla.com
Similar to the other cases, shift some weight/coord calculations to int
space. This should be slightly faster (on x86 sse it should actually safe one
instruction, and generally int instructions are cheaper).
---
From: Jeff Muizelaar jmuizel...@mozilla.com
Similar to the other cases, shift some weight/coord calculations to int
space. This should be slightly faster (on x86 sse it should actually safe one
instruction, and generally int instructions are cheaper).
---
From: Jeff Muizelaar jmuizel...@mozilla.com
The previous method for converting coords to ints was sligthly inaccurate
(effectively losing 1bit from the 8bit lerp weight). This is probably
especially noticeable when trying to draw a pixel-aligned texture.
As an example, for a 100x100 texture after
From: Roland Scheidegger srol...@vmware.com
The previous code used coords which were calculated as
(int) (f_coord * tex_size * 256) 8.
This is not only unnecessarily complex but can give the wrong texel due to
rounding (as that uses truncation, not round to nearest) if the pixel is less
than
On 02/14/2014 03:00 PM, Eric Anholt wrote:
Compare this VS to the one for the post-130 case. Fixes piglit
glsl-lod-bias, and presumably tons of other code (I haven't done a full
piglit run on swrast).
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=74911
---
FWIW I've just cleaned 1/3 and 3/3 up a little and splitted it off into
two patches (I really want to be able to track any changes this might
cause separately), and on x86 sse I actually managed to shave off one
instruction by using lp_build_iround() too :-).
2/3 is more of the same just for the
On 02/14/2014 03:00 PM, Eric Anholt wrote:
Pretty silly to pass in values dereferenced out of one of the arguments.
---
src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 35
+--
1 file changed, 11 insertions(+), 24 deletions(-)
diff --git
This drops the MOVs for header setup, which are totally mis-scheduled.
total instructions in shared programs: 1590047 - 1589331 (-0.05%)
instructions in affected programs: 43729 - 43013 (-1.64%)
GAINED:0
LOST: 0
glb27-trex:
x
Fixes failing Khronos CTS test packed_depth_stencil_init.test
Cc: mesa-sta...@lists.freedesktop.org
Signed-off-by: Anuj Phogat anuj.pho...@gmail.com
---
src/mesa/main/texparam.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/mesa/main/texparam.c b/src/mesa/main/texparam.c
index
On 02/14/2014 03:00 PM, Eric Anholt wrote:
Since only window system renderbuffers can have a singlesample_mt, this
lets us drop a bunch of sanity checking to make sure that we're just a
renderbuffer-like thing.
---
src/mesa/drivers/dri/i965/brw_context.c | 20 ++-
On 02/14/2014 03:00 PM, Eric Anholt wrote:
[snip]
@@ -666,78 +667,6 @@ intel_miptree_create_for_bo(struct brw_context *brw,
return mt;
}
-
-/**
- * For a singlesample DRI2 buffer, this simply wraps the given region with a
miptree.
- *
- * For a multisample DRI2 buffer, this wraps
On 02/14/2014 03:00 PM, Eric Anholt wrote:
It often confused people because it was unclear on whether it was the
physical or logical, and people needed the other one as well. We can
recompute it trivially using the minify() macro, clarifying which value is
being used and making getting the
On 02/14/2014 04:48 PM, Eric Anholt wrote:
The code was removed early last year.
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 6 --
1 file changed, 6 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
b/src/mesa/drivers/dri/i965/brw_fs.cpp
index 4f5558b..d35928e 100644
Am 15.02.2014 01:54, schrieb srol...@vmware.com:
From: Jeff Muizelaar jmuizel...@mozilla.com
The previous method for converting coords to ints was sligthly inaccurate
(effectively losing 1bit from the 8bit lerp weight). This is probably
especially noticeable when trying to draw a
https://bugs.freedesktop.org/show_bug.cgi?id=75010
Priority: medium
Bug ID: 75010
Assignee: mesa-dev@lists.freedesktop.org
Summary: clang: error: unknown argument:
'-fstack-protector-strong'
Severity: blocker
Signed-off-by: Kusanagi Kouichi sl...@ac.auone-net.jp
---
configure.ac | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/configure.ac b/configure.ac
index 00a0eaa..5c58928 100644
--- a/configure.ac
+++ b/configure.ac
@@ -506,11 +506,12 @@ AC_ARG_ENABLE([selinux],
On 02/14/2014 04:49 PM, Anuj Phogat wrote:
Section 4.3.1, page 220, of OpenGL 3.3 specification explains
the error conditions for glreadPixels():
If the format is DEPTH_STENCIL, then values are taken from
both the depth buffer and the stencil buffer. If there is
no depth buffer
On 02/14/2014 05:31 PM, Anuj Phogat wrote:
Fixes failing Khronos CTS test packed_depth_stencil_init.test
Cc: mesa-sta...@lists.freedesktop.org
Signed-off-by: Anuj Phogat anuj.pho...@gmail.com
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
---
src/mesa/main/texparam.c | 3 +++
1 file
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 02/14/2014 05:09 PM, Kenneth Graunke wrote:
On 02/14/2014 03:00 PM, Eric Anholt wrote:
Compare this VS to the one for the post-130 case. Fixes piglit
glsl-lod-bias, and presumably tons of other code (I haven't done
a full piglit run on
On 02/14/2014 07:04 PM, Ian Romanick wrote:
On 02/14/2014 05:09 PM, Kenneth Graunke wrote:
On 02/14/2014 03:00 PM, Eric Anholt wrote:
Compare this VS to the one for the post-130 case. Fixes piglit
glsl-lod-bias, and presumably tons of other code (I haven't done
a full piglit run on swrast).
Grigori,
I just tried this out on a few interlaced videos I have, and it works
really quite well! One thing I did notice is that when playing back
content with mplayer (and -vo vdpau:deint=3) in full screen mode, the
black bars are now stale/uninitialized textures. This does not happen
if I don't
Reviewed-by: Matt Turner matts...@gmail.com
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Reviewed-by: Matt Turner matts...@gmail.com
I'll commit later, unless someone does it first.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
68 matches
Mail list logo