Op 08-02-12 20:37, Christian König schreef:
And while at it implement the rest of the feature
querying also.
Signed-off-by: Christian König
---
src/gallium/state_trackers/vdpau/mixer.c | 286 +-
src/gallium/state_trackers/vdpau/query.c | 12 +-
src/gall
Op 08-02-12 20:37, Christian König schreef:
Signed-off-by: Christian König
---
src/gallium/state_trackers/vdpau/mixer.c | 78 --
src/gallium/state_trackers/vdpau/query.c |1 +
src/gallium/state_trackers/vdpau/vdpau_private.h |9 ++-
3 files chang
On Sun, Feb 12, 2012 at 09:02:44AM -0500, nobled wrote:
> Judging by the patch that tried to change the tls_model:
> https://bugs.freedesktop.org/show_bug.cgi?id=35268
> It looks like we'd need to disable enable_asm if the tls_model
> attribute isn't supported, since it would probably end up defaul
On Mon, Feb 13, 2012 at 06:10:36PM +0800, Zhigang Gong wrote:
> On Sun, Feb 12, 2012 at 09:02:44AM -0500, nobled wrote:
>From ac92209ccf1d8877eb58bf0bfdd0ccc389200bac Mon Sep 17 00:00:00 2001
From: Zhigang Gong
Date: Mon, 13 Feb 2012 17:29:12 +0800
Subject: [PATCH 2/2] GLX_TLS: use TLS macros when
On 13.02.2012 01:55, Andy Furniss wrote:
Christian König wrote:
This is a shader based median filter, generally
used for noise reduction, it could still need some
improvements, but should usually work out of the box.
Hi Christian.
I haven't really tested these properly yet, but did find a sma
A filter strength of zero or one doesn't make any
sense. Thanks to Andy Furniss for pointing this out.
Signed-off-by: Christian König
---
src/gallium/auxiliary/vl/vl_median_filter.c |4 ++--
src/gallium/state_trackers/vdpau/mixer.c |8
src/gallium/state_trackers/vdp
The matrix filter wasn't freed on mixer destruction.
Signed-off-by: Christian König
---
src/gallium/state_trackers/vdpau/mixer.c |5 +
1 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/src/gallium/state_trackers/vdpau/mixer.c
b/src/gallium/state_trackers/vdpau/mixer.c
index
On 13.02.2012 09:00, Maarten Lankhorst wrote:
Is dynamically allocating vmixer->noise_reduction.filter really needed?
Wouldn't it make more sense to make it a static member, and making
the cleanup member set things to null again as needed.
And again, why dynamically allocate the filter.. make the
Tracing function entry/exits is a bit pointless
when VDPAU_TRACE=1 does the same thing.
Signed-off-by: Christian König
---
src/gallium/state_trackers/vdpau/decode.c | 10 --
src/gallium/state_trackers/vdpau/device.c |9 -
src/gallium/state_trackers/vdpau/mixer.c
Uses less code and looks at least a bit cleaner than mapping manually.
Signed-off-by: Christian König
---
src/gallium/state_trackers/vdpau/surface.c | 23 +--
1 files changed, 5 insertions(+), 18 deletions(-)
diff --git a/src/gallium/state_trackers/vdpau/surface.c
b/src/g
Signed-off-by: Christian König
---
src/gallium/auxiliary/vl/vl_compositor.c | 41 +++-
src/gallium/auxiliary/vl/vl_compositor.h | 13 ++-
src/gallium/state_trackers/vdpau/mixer.c | 19 ++-
src/gallium/state_trackers/xorg/xvmc/surface.c |
https://bugs.freedesktop.org/show_bug.cgi?id=45967
Brian Paul changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|
Hey Christian,
Op 13-02-12 13:09, Christian König schreef:
No, it's not really needed, but I had it this way in the first place
and you need a lot of "if (something_is_not_null) cleanup(something)"
in the destructor functions, and with this implementation we only need
to test if it's allocated
Op 13-02-12 14:36, Christian König schreef:
Tracing function entry/exits is a bit pointless
when VDPAU_TRACE=1 does the same thing.
Signed-off-by: Christian König
---
src/gallium/state_trackers/vdpau/decode.c | 10 --
src/gallium/state_trackers/vdpau/device.c |9
Op 13-02-12 14:36, Christian König schreef:
Signed-off-by: Christian König
---
src/gallium/auxiliary/vl/vl_compositor.c | 41 +++-
src/gallium/auxiliary/vl/vl_compositor.h | 13 ++-
src/gallium/state_trackers/vdpau/mixer.c | 19 ++-
src
Op 13-02-12 14:36, Christian König schreef:
Uses less code and looks at least a bit cleaner than mapping manually.
Signed-off-by: Christian König
---
src/gallium/state_trackers/vdpau/surface.c | 23 +--
1 files changed, 5 insertions(+), 18 deletions(-)
Can't test it ri
We only used it in a few places that can implemented differently.
---
src/mesa/state_tracker/st_cb_drawpixels.c | 13 +++--
src/mesa/state_tracker/st_cb_fbo.c| 13 +++--
src/mesa/state_tracker/st_cb_fbo.h|1 -
3 files changed, 10 insertions(+), 17 deletions(
It was only used for software buffers and easily computed.
---
src/mesa/state_tracker/st_cb_fbo.c | 13 ++---
src/mesa/state_tracker/st_cb_fbo.h |1 -
2 files changed, 6 insertions(+), 8 deletions(-)
diff --git a/src/mesa/state_tracker/st_cb_fbo.c
b/src/mesa/state_tracker/st_cb_fbo
The subsequent u_surface_default_template() call does that for us.
---
src/mesa/state_tracker/st_cb_fbo.c |1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/src/mesa/state_tracker/st_cb_fbo.c
b/src/mesa/state_tracker/st_cb_fbo.c
index 2e884a4..19bbdc5 100644
--- a/src/mesa/s
---
src/mesa/state_tracker/st_manager.c |4 +---
1 files changed, 1 insertions(+), 3 deletions(-)
diff --git a/src/mesa/state_tracker/st_manager.c
b/src/mesa/state_tracker/st_manager.c
index 7efc173..d54b7ed 100644
--- a/src/mesa/state_tracker/st_manager.c
+++ b/src/mesa/state_tracker/st_man
The st_renderbuffer_alloc_storage() function is used to allocate both
window-system buffers and user-created renderbuffers. The later kind
are never directly displayed so don't set PIPE_BIND_DISPLAY_TARGET for
those surfaces.
NOTE: This is a candidate for the 8.0 branch.
---
src/mesa/state_track
---
src/mesa/state_tracker/st_cb_drawpixels.c | 44 +---
1 files changed, 2 insertions(+), 42 deletions(-)
diff --git a/src/mesa/state_tracker/st_cb_drawpixels.c
b/src/mesa/state_tracker/st_cb_drawpixels.c
index 88e57b8..de4c189 100644
--- a/src/mesa/state_tracker/st_cb
Added in _mesa_pack_uint_24_8_depth_stencil_row(). This could be hit
by something like glDrawPixels(GL_DEPTH_STENCIL, GL_UNSIGNED_INT_24_8)
into a MESA_FORMAT_Z32_FLOAT_X24S8 buffer.
---
src/mesa/main/format_pack.c | 13 +
1 files changed, 13 insertions(+), 0 deletions(-)
diff --gi
---
src/gallium/include/pipe/p_state.h |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/gallium/include/pipe/p_state.h
b/src/gallium/include/pipe/p_state.h
index f6486f0..72ec04a 100644
--- a/src/gallium/include/pipe/p_state.h
+++ b/src/gallium/include/pipe/p_stat
---
src/mesa/state_tracker/st_cb_clear.c | 10 ++
1 files changed, 2 insertions(+), 8 deletions(-)
diff --git a/src/mesa/state_tracker/st_cb_clear.c
b/src/mesa/state_tracker/st_cb_clear.c
index a8365a7..1935833 100644
--- a/src/mesa/state_tracker/st_cb_clear.c
+++ b/src/mesa/state_trac
Previously, this function only handled 2D textures.
The fallback texture is used when we try to sample from an incomplete
texture object. GLSL says sampling an incomplete texture should return
(0,0,0,1).
---
src/mesa/main/mtypes.h |2 +-
src/mesa/main/shared.c |8 ++-
src/mesa/main/t
Just use _mesa_get_fallback_texture() instead.
---
src/mesa/state_tracker/st_atom_sampler.c |3 +-
src/mesa/state_tracker/st_atom_texture.c |3 +-
src/mesa/state_tracker/st_cb_texture.c | 51 --
src/mesa/state_tracker/st_cb_texture.h |4 --
4 files cha
Just use _mesa_framebuffer_renderbuffer().
---
src/mesa/state_tracker/st_cb_fbo.c | 15 +--
1 files changed, 1 insertions(+), 14 deletions(-)
diff --git a/src/mesa/state_tracker/st_cb_fbo.c
b/src/mesa/state_tracker/st_cb_fbo.c
index 5a3186d..ed51808 100644
--- a/src/mesa/state_trac
---
src/mesa/state_tracker/st_cb_fbo.c | 22 +++---
1 files changed, 3 insertions(+), 19 deletions(-)
diff --git a/src/mesa/state_tracker/st_cb_fbo.c
b/src/mesa/state_tracker/st_cb_fbo.c
index ed51808..300c3f0 100644
--- a/src/mesa/state_tracker/st_cb_fbo.c
+++ b/src/mesa/state
Christian König wrote:
A filter strength of zero or one doesn't make any
sense.
Low strengths work OK now, but this patch seems to have made denoise
stronger. Maybe I am looking at the wrong material, but more than 0.2
for sharpen or denoise looks too way much.
Benchmark wise sharpen seems
- Original Message -
> We only used it in a few places that can implemented differently.
> ---
> src/mesa/state_tracker/st_cb_drawpixels.c | 13 +++--
> src/mesa/state_tracker/st_cb_fbo.c| 13 +++--
> src/mesa/state_tracker/st_cb_fbo.h|1 -
> 3 fil
- Original Message -
> Added in _mesa_pack_uint_24_8_depth_stencil_row(). This could be hit
> by something like glDrawPixels(GL_DEPTH_STENCIL,
> GL_UNSIGNED_INT_24_8)
> into a MESA_FORMAT_Z32_FLOAT_X24S8 buffer.
> ---
> src/mesa/main/format_pack.c | 13 +
> 1 files changed
On 02/13/2012 09:44 AM, Jose Fonseca wrote:
- Original Message -
Added in _mesa_pack_uint_24_8_depth_stencil_row(). This could be hit
by something like glDrawPixels(GL_DEPTH_STENCIL,
GL_UNSIGNED_INT_24_8)
into a MESA_FORMAT_Z32_FLOAT_X24S8 buffer.
---
src/mesa/main/format_pack.c |
- Original Message -
> Previously, this function only handled 2D textures.
>
> The fallback texture is used when we try to sample from an incomplete
> texture object. GLSL says sampling an incomplete texture should
> return
> (0,0,0,1).
> ---
> src/mesa/main/mtypes.h |2 +-
> src
The series looks good to me.
Jose
- Original Message -
> ---
> src/mesa/state_tracker/st_cb_fbo.c | 22 +++---
> 1 files changed, 3 insertions(+), 19 deletions(-)
>
> diff --git a/src/mesa/state_tracker/st_cb_fbo.c
> b/src/mesa/state_tracker/st_cb_fbo.c
> index ed51808
On 02/13/2012 09:47 AM, Jose Fonseca wrote:
- Original Message -
Previously, this function only handled 2D textures.
The fallback texture is used when we try to sample from an incomplete
texture object. GLSL says sampling an incomplete texture should
return
(0,0,0,1).
---
src/mesa/
On 02/13/2012 05:19 PM, Brian Paul wrote:
> ---
> src/gallium/include/pipe/p_state.h |6 +++---
> 1 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/gallium/include/pipe/p_state.h
> b/src/gallium/include/pipe/p_state.h
> index f6486f0..72ec04a 100644
> --- a/src/gallium/in
This patch adds the pixel pack operations in glGetTexImage for
compressed textures with unsigned, normalized values. It also
fixes the failures in intel oglconform pxstore-gettex due to
following sub test cases:
- Test all mipmaps with byte swapping enabled
- Test all small mipmaps with all allo
This patch adds the pixel store operations in decompress_texture_image().
decompress_texture_image() is used in glGetTexImage() for compressed
textures with unsigned, normalized values.
It also fixes the failures in intel oglconform pxstore-gettex due to
following sub test cases:
- Test all mipm
https://bugs.freedesktop.org/show_bug.cgi?id=45420
Stephane Marchesin changed:
What|Removed |Added
Status|REOPENED|RESOLVED
Resolution|
https://bugs.freedesktop.org/show_bug.cgi?id=45292
Dennis Schridde changed:
What|Removed |Added
CC||devuran...@gmx.net
--
Configure bugma
On 02/13/2012 11:33 AM, Anuj Phogat wrote:
This patch adds the pixel store operations in decompress_texture_image().
decompress_texture_image() is used in glGetTexImage() for compressed
textures with unsigned, normalized values.
It also fixes the failures in intel oglconform pxstore-gettex due t
Reallocate/resize decompress FBO only if texture image width/height is
greater than existing decompress FBO width/height.
Signed-off-by: Anuj Phogat
---
src/mesa/drivers/common/meta.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/mesa/drivers/common/meta.c b/src/m
On Sat, 11 Feb 2012 11:30:00 -0800, Kenneth Graunke
wrote:
> On 02/11/2012 07:58 AM, Matt Turner wrote:
> > Hi,
> >
> > There are a few scientific software packages that require Mesa's GLw,
> > and with 8.0 we've split GLw out into a separate repository. Can we do
> > an 8.0 release of GLw? It pa
As suggested by José.
---
src/mesa/main/format_pack.c | 31 +++
src/mesa/main/format_unpack.c | 20 +++-
2 files changed, 30 insertions(+), 21 deletions(-)
diff --git a/src/mesa/main/format_pack.c b/src/mesa/main/format_pack.c
index e20e361..f8c0f
Rather than the one in st_texture_object. This sampler view really has
no connection to the one used for rendering.
---
src/mesa/state_tracker/st_gen_mipmap.c |6 +-
1 files changed, 5 insertions(+), 1 deletions(-)
diff --git a/src/mesa/state_tracker/st_gen_mipmap.c
b/src/mesa/state_tra
Similar to the previous commit.
---
src/mesa/state_tracker/st_cb_texture.c |7 +--
1 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/src/mesa/state_tracker/st_cb_texture.c
b/src/mesa/state_tracker/st_cb_texture.c
index c2011da..b0f6b14 100644
--- a/src/mesa/state_tracker/st_c
---
src/mesa/state_tracker/st_texture.h | 16
1 files changed, 0 insertions(+), 16 deletions(-)
diff --git a/src/mesa/state_tracker/st_texture.h
b/src/mesa/state_tracker/st_texture.h
index 3d2a636..b5c22ee 100644
--- a/src/mesa/state_tracker/st_texture.h
+++ b/src/mesa/state_t
Implement in terms of st_create_texture_sampler_view_format().
---
src/mesa/state_tracker/st_texture.h | 21 +
1 files changed, 9 insertions(+), 12 deletions(-)
diff --git a/src/mesa/state_tracker/st_texture.h
b/src/mesa/state_tracker/st_texture.h
index b5c22ee..0b3e0be 100
from st_get/create_texture_sampler_view_from stobj() functions.
No real value in these cases.
---
src/mesa/state_tracker/st_atom_texture.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/state_tracker/st_atom_texture.c
b/src/mesa/state_tracker/st_atom_texture.
On 02/13/2012 12:34 PM, Anuj Phogat wrote:
Reallocate/resize decompress FBO only if texture image width/height is
greater than existing decompress FBO width/height.
Signed-off-by: Anuj Phogat
---
src/mesa/drivers/common/meta.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff
On 02/13/2012 09:38 AM, Eric Anholt wrote:
On Sat, 11 Feb 2012 11:30:00 -0800, Kenneth Graunke
wrote:
On 02/11/2012 07:58 AM, Matt Turner wrote:
Hi,
There are a few scientific software packages that require Mesa's GLw,
and with 8.0 we've split GLw out into a separate repository. Can we do
an
In the table of of push/pop attributes, this one doesn't fall under
the enable group.
---
src/mesa/main/attrib.c |5 -
1 files changed, 0 insertions(+), 5 deletions(-)
diff --git a/src/mesa/main/attrib.c b/src/mesa/main/attrib.c
index 846da35..5b1db29 100644
--- a/src/mesa/main/attrib.c
+
Fixes (with the previous commit) piglit GL_ARB_multisample/pushpop.
---
src/mesa/main/attrib.c | 19 +--
1 files changed, 17 insertions(+), 2 deletions(-)
diff --git a/src/mesa/main/attrib.c b/src/mesa/main/attrib.c
index 5b1db29..7042312 100644
--- a/src/mesa/main/attrib.c
+++
On Sun, 12 Feb 2012 01:13:21 -0800, Kenneth Graunke
wrote:
> On 02/11/2012 01:13 PM, Eric Anholt wrote:
> > On Fri, 10 Feb 2012 22:24:08 -0800, Kenneth Graunke
> > wrote:
> >> We don't want to use brw->sampler.count here, as it includes samplers
> >> used only by the FS...which is most of them.
On Sun, 12 Feb 2012 23:29:14 -0800, Kenneth Graunke
wrote:
> Commit dc7f449d1ac53a66e6efb56ccf2a5953418a26ca introduced a new method
> for avoiding MOVs: try to rewrite the destination of the instruction
> that produced the RHS so it writes into the LHS.
>
> Unfortunately, this is not safe for t
56 matches
Mail list logo
