Am 17.03.2013 20:59, schrieb Christoph Bumiller:
On 17.03.2013 18:04, Christoph Bumiller wrote:
On 17.03.2013 16:30, Christian König wrote:
Am 15.03.2013 18:58, schrieb Christoph Bumiller:
On 15.03.2013 13:08, Christian König wrote:
Am 14.03.2013 15:53, schrieb Christoph Bumiller:
On
From: Christian König
Instead of emitting each temporary separately, emit them in a chunk.
v2: keep separate function for emitting temps
Signed-off-by: Christian König
---
src/gallium/auxiliary/tgsi/tgsi_ureg.c | 30 +-
1 file changed, 13 insertions(+), 17
From: Christian König
Don't bother with free temporaries, just allocate them at
the end and also emit them in their own declaration.
Signed-off-by: Christian König
---
src/gallium/auxiliary/tgsi/tgsi_ureg.c | 55
src/gallium/auxiliary/tgsi/tgsi_ureg.h |
From: Christian König
Signed-off-by: Christian König
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 23 ++-
1 file changed, 10 insertions(+), 13 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
index
From: Christian König
Instead of allocating everything as temporaries, use the
new array allocation functions.
v2: fix bug in simplify_cmp, declare arrays on demand
Signed-off-by: Christian König
---
src/mesa/main/mtypes.h |1 +
src/mesa/state_tracker
From: Christian König
They shouldn't be necessary any more.
Signed-off-by: Christian König
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 36 +++-
1 file changed, 3 insertions(+), 33 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src
From: Christian König
Nobody seems to be using it, and only nv50 had a partial implementation.
Signed-off-by: Christian König
---
src/gallium/auxiliary/tgsi/tgsi_build.c| 19 -
src/gallium/auxiliary/tgsi/tgsi_dump.c | 38 --
src/gallium/auxiliary/tgsi
From: Christian König
Remember which declarations are declared as "arrays" and so
can be indirectly addressed. ArrayIDs start at 1, cause for
compatibility reasons zero is treaded as no array present.
Signed-off-by: Christian König
---
src/gallium/auxiliary/tgsi/tgsi_dump.c
From: Christian König
To further improve the optimization of source and destination
indirect addressing we need the ability to store a reference
to the declaration of the addressed operands.
Since most of the fields in tgsi_src_register doesn't apply for
an indirect addressing operand re
From: Christian König
v2: further improve the text with comments from Christoph Bumiller.
Signed-off-by: Christian König
---
src/gallium/docs/source/tgsi.rst | 18 ++
1 file changed, 18 insertions(+)
diff --git a/src/gallium/docs/source/tgsi.rst b/src/gallium/docs/source
Am 18.03.2013 15:12, schrieb Roland Scheidegger:
Am 18.03.2013 14:36, schrieb Christian König:
From: Christian König
To further improve the optimization of source and destination
indirect addressing we need the ability to store a reference
to the declaration of the addressed operands.
Since
Am 20.03.2013 11:43, schrieb Michel Dänzer:
From: Michel Dänzer
Fixes random failures with piglit glsl-max-varyings.
NOTE: This is a candidate for the 9.1 branch.
Signed-off-by: Michel Dänzer
Reviewed-by: Christian König
---
src/gallium/drivers/radeonsi/si_state_draw.c | 5 +
1
From: Christian König
Signed-off-by: Christian König
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 32 ++--
1 file changed, 25 insertions(+), 7 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
b/src/gallium/drivers/radeonsi
From: Christian König
Now the backend handles that itself.
Signed-off-by: Christian König
---
src/gallium/drivers/radeonsi/radeonsi_shader.c |9 -
1 file changed, 9 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
b/src/gallium/drivers/radeonsi
From: Christian König
Signed-off-by: Christian König
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 18 ++
1 file changed, 10 insertions(+), 8 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
b/src/gallium/drivers/radeonsi/radeonsi_shader.c
From: Christian König
Signed-off-by: Christian König
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 67 ++--
1 file changed, 51 insertions(+), 16 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
b/src/gallium/drivers/radeonsi
From: Christian König
Signed-off-by: Christian König
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 57 +++-
1 file changed, 45 insertions(+), 12 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
b/src/gallium/drivers/radeonsi
Am 21.03.2013 15:10, schrieb Alex Deucher:
On Thu, Mar 21, 2013 at 7:38 AM, Christian König
wrote:
From: Christian König
Signed-off-by: Christian König
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 57 +++-
1 file changed, 45 insertions(+), 12 deletions
Am 21.03.2013 15:06, schrieb Michel Dänzer:
When I tried this earlier, something broke. Unfortunately, I can't seem
to remember or dig up if it was a piglit test or e.g. one of the sampler
demos in mesa/demos/src/glsl/. Did you test the latter with this change?
Similar concern for patch 4, the r
/show_bug.cgi?id=62012 [radeonsi]
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=58150 [r600g]
NOTE: This is a candidate for the Mesa stable branch.
Signed-off-by: Michel Dänzer
Reviewed-by: Christian König
---
lib/Target/R600/R600ISelLowering.cpp | 11 ---
lib/Target/R600
Am 21.03.2013 17:20, schrieb Alex Deucher:
On Thu, Mar 21, 2013 at 12:09 PM, Christian König
wrote:
Am 21.03.2013 15:10, schrieb Alex Deucher:
On Thu, Mar 21, 2013 at 7:38 AM, Christian König
wrote:
From: Christian König
Signed-off-by: Christian König
---
src/gallium/drivers/radeonsi
Am 21.03.2013 17:17, schrieb Michel Dänzer:
On Don, 2013-03-21 at 17:12 +0100, Christian König wrote:
Am 21.03.2013 15:06, schrieb Michel Dänzer:
When I tried this earlier, something broke. Unfortunately, I can't seem
to remember or dig up if it was a piglit test or e.g. one of the sa
From: Christian König
Avoid creating arrays if we replace indirect addressing anyway.
Signed-off-by: Christian König
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp |4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa
From: Christian König
Even when we have arrays it is possible for simplify_cmp
to work on temps, just not on arrays.
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=62696
Signed-off-by: Christian König
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp |2 +-
1 file changed, 1 insertion
Am 25.03.2013 17:01, schrieb j.gli...@gmail.com:
From: Jerome Glisse
Same as on r600, trace cs execution by writting cs offset after each
states, this allow to pin point lockup inside command stream and
narrow down the scope of lockup investigation.
Signed-off-by: Jerome Glisse
Could your r
Am 25.03.2013 17:50, schrieb Jerome Glisse:
On Mon, Mar 25, 2013 at 12:38 PM, Christian König
wrote:
Am 25.03.2013 17:01, schrieb j.gli...@gmail.com:
From: Jerome Glisse
Same as on r600, trace cs execution by writting cs offset after each
states, this allow to pin point lockup inside
Am 25.03.2013 18:15, schrieb j.gli...@gmail.com:
From: Jerome Glisse
Same as on r600, trace cs execution by writting cs offset after each
states, this allow to pin point lockup inside command stream and
narrow down the scope of lockup investigation.
v2: Use WRITE_DATA packet instead of WRITE_M
From: Christian König
Now the backend handles that itself.
Signed-off-by: Christian König
---
src/gallium/drivers/radeonsi/radeonsi_shader.c |9 -
1 file changed, 9 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
b/src/gallium/drivers/radeonsi
From: Christian König
Signed-off-by: Christian König
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 32 ++--
1 file changed, 25 insertions(+), 7 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
b/src/gallium/drivers/radeonsi
From: Christian König
Signed-off-by: Christian König
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 18 ++
1 file changed, 10 insertions(+), 8 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
b/src/gallium/drivers/radeonsi/radeonsi_shader.c
From: Christian König
The include isn't needed and the file has moved with LLVM master.
Signed-off-by: Christian König
---
src/gallium/drivers/radeon/radeon_llvm_emit.cpp |1 -
1 file changed, 1 deletion(-)
diff --git a/src/gallium/drivers/radeon/radeon_llvm_emit.cpp
b/src/ga
From: Christian König
Signed-off-by: Christian König
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 67 ++--
1 file changed, 51 insertions(+), 16 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
b/src/gallium/drivers/radeonsi
From: Christian König
Signed-off-by: Christian König
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 57 +++-
1 file changed, 45 insertions(+), 12 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
b/src/gallium/drivers/radeonsi
Am 26.03.2013 12:53, schrieb Michel Dänzer:
On Die, 2013-03-26 at 11:56 +0100, Christian König wrote:
From: Christian König
The include isn't needed and the file has moved with LLVM master.
Signed-off-by: Christian König
Reviewed-by: Michel Dänzer
The rest of the series seems unch
From: Christian König
Signed-off-by: Christian König
---
src/gallium/drivers/radeonsi/radeonsi_pipe.c |2 +-
src/gallium/drivers/radeonsi/radeonsi_shader.c | 35 ++--
src/gallium/drivers/radeonsi/radeonsi_shader.h |6 +++-
src/gallium/drivers/radeonsi
From: Christian König
This should be used by both SI and R600.
Signed-off-by: Christian König
---
src/gallium/drivers/r600/r600_llvm.c | 12
.../drivers/radeon/radeon_setup_tgsi_llvm.c| 12
2 files changed, 12 insertions(+), 12 deletions
From: Christian König
This works different than on R600, we need to add the start instance manually.
Signed-off-by: Christian König
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 13 -
src/gallium/drivers/radeonsi/radeonsi_shader.h | 12 +++-
src/gallium/drivers
From: Christian König
Signed-off-by: Christian König
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 66 +++-
src/gallium/drivers/radeonsi/radeonsi_shader.h | 23 +
src/gallium/drivers/radeonsi/si_state.c| 32 +++-
src/gallium/drivers
Am 26.03.2013 14:42, schrieb Jerome Glisse:
On Tue, Mar 26, 2013 at 6:22 AM, Christian König
wrote:
Am 25.03.2013 18:15, schrieb j.gli...@gmail.com:
From: Jerome Glisse
Same as on r600, trace cs execution by writting cs offset after each
states, this allow to pin point lockup inside
an.
Marek
On Tue, Mar 26, 2013 at 3:00 PM, Christian König
wrote:
Am 26.03.2013 14:42, schrieb Jerome Glisse:
On Tue, Mar 26, 2013 at 6:22 AM, Christian König
wrote:
Am 25.03.2013 18:15, schrieb j.gli...@gmail.com:
From: Jerome Glisse
Same as on r600, trace cs execution by writting
Am 26.03.2013 15:56, schrieb Michel Dänzer:
On Die, 2013-03-26 at 14:51 +0100, Christian König wrote:
From: Christian König
Signed-off-by: Christian König
[...]
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.h
b/src/gallium/drivers/radeonsi/radeonsi_shader.h
index 9dae742
Am 26.03.2013 18:02, schrieb Jerome Glisse:
On Tue, Mar 26, 2013 at 12:40 PM, Marek Olšák wrote:
On Tue, Mar 26, 2013 at 3:59 PM, Christian König
wrote:
Am 26.03.2013 15:34, schrieb Marek Olšák:
Speaking of si_pm4_state, I think it's a horrible mechanism for
anything other than con
Am 26.03.2013 20:28, schrieb Marek Olšák:
On Tue, Mar 26, 2013 at 6:44 PM, Christian König
wrote:
Am 26.03.2013 18:02, schrieb Jerome Glisse:
On Tue, Mar 26, 2013 at 12:40 PM, Marek Olšák wrote:
On Tue, Mar 26, 2013 at 3:59 PM, Christian König
wrote:
Am 26.03.2013 15:34, schrieb Marek
Am 27.03.2013 00:30, schrieb Dave Airlie:
On 27 Mar 2013 08:45, "Dave Airlie" wrote:
correctly). But Marek is quite right that this only counts for state
objects
and makes no sense for set_* and draw_* calls (and I'm currently thinking
how to avoid that and can't come up with a proper solution)
Am 27.03.2013 01:43, schrieb Jerome Glisse:
On Tue, Mar 26, 2013 at 6:45 PM, Dave Airlie wrote:
correctly). But Marek is quite right that this only counts for state
objects
and makes no sense for set_* and draw_* calls (and I'm currently thinking
how to avoid that and can't come up with a prope
From: Christian König
v2: reduce key size, don't copy key around to much.
Signed-off-by: Christian König
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 67 +++-
src/gallium/drivers/radeonsi/radeonsi_shader.h | 24 +
src/gallium/drivers/radeonsi/si_st
Am 26.03.2013 18:03, schrieb Michel Dänzer:
On Die, 2013-03-26 at 17:37 +0100, Christian König wrote:
Am 26.03.2013 15:56, schrieb Michel Dänzer:
On Die, 2013-03-26 at 14:51 +0100, Christian König wrote:
From: Christian König
Signed-off-by: Christian König
[...]
diff --git a/src/gallium
Am 27.03.2013 12:02, schrieb Christian König:
Am 26.03.2013 18:03, schrieb Michel Dänzer:
On Die, 2013-03-26 at 17:37 +0100, Christian König wrote:
Am 26.03.2013 15:56, schrieb Michel Dänzer:
On Die, 2013-03-26 at 14:51 +0100, Christian König wrote:
From: Christian König
Signed-off-by
Am 27.03.2013 12:44, schrieb Michel Dänzer:
From: Michel Dänzer
Fixes mplayer -vo vdpau OSD.
Reported-by: Igor Vagulin
Signed-off-by: Michel Dänzer
Reviewed-by: Christian König
Tested-by: Christian König
---
src/gallium/drivers/radeonsi/r600_blit.c | 6 ++
1 file changed, 6
From: Christian König
This works different than on R600, we need to add the start instance manually.
Signed-off-by: Christian König
Reviewed-by: Michel Dänzer
Tested-by: Michel Dänzer
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 13 -
src/gallium/drivers/radeonsi
From: Christian König
This should be used by both SI and R600.
Signed-off-by: Christian König
Reviewed-by: Michel Dänzer
Tested-by: Michel Dänzer
---
src/gallium/drivers/r600/r600_llvm.c | 12
.../drivers/radeon/radeon_setup_tgsi_llvm.c| 12
From: Christian König
Signed-off-by: Christian König
Reviewed-by: Michel Dänzer
Tested-by: Michel Dänzer
---
src/gallium/drivers/radeonsi/radeonsi_pipe.c |2 +-
src/gallium/drivers/radeonsi/radeonsi_shader.c | 35 ++--
src/gallium/drivers/radeonsi
From: Christian König
v2: reduce key size, don't copy key around to much.
v3: remove key size reduction
Signed-off-by: Christian König
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 67 +++-
src/gallium/drivers/radeonsi/radeonsi_shader.h | 24 +
From: Christian König
Signed-off-by: Christian König
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 17 ++---
1 file changed, 6 insertions(+), 11 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
b/src/gallium/drivers/radeonsi/radeonsi_shader.c
Am 27.03.2013 20:37, schrieb Vadim Girlin:
Signed-off-by: Vadim Girlin
---
src/gallium/drivers/r600/r600_shader.c | 19 +++
1 file changed, 15 insertions(+), 4 deletions(-)
diff --git a/src/gallium/drivers/r600/r600_shader.c
b/src/gallium/drivers/r600/r600_shader.c
index 29f
emantic indices should be handled for
input/output arrays.
Yeah, the uncertainly about semantic IDs was one of the reasons I didn't
wanted to do Input/Output arrays in the initial array implementation.
When those changes are the only one then v2 of the patch is:
Reviewed-by
From: Christian König
v2: fix instrinsic name as well
Signed-off-by: Christian König
---
src/gallium/drivers/radeonsi/radeonsi_shader.c | 19 +++
1 file changed, 7 insertions(+), 12 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/radeonsi_shader.c
b/src/gallium
Am 03.04.2013 02:17, schrieb Matt Turner:
On Tue, Apr 2, 2013 at 4:19 PM, Christian König wrote:
diff --git a/configure.ac b/configure.ac
index 81d4a3f..93ec1d2 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1814,6 +1814,7 @@ if test "x$with_gallium_drivers" != x; then
oblem, maybe because I'm building with
--enable-texture-float?
Oh, yes same thing here.
Reviewed-by: Michel Dänzer
Reviewed-by: Christian König
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/lis
Am 03.04.2013 15:19, schrieb Michel Dänzer:
On Mit, 2013-03-27 at 17:57 +0100, Christian König wrote:
the attached patches dynamically adjust the writemask of MIMG
instructions depending on the used components. Additional to that it
also adjust the register class of MIMG instruction so that we
From: Christian König
Separated from UVD patch for clarity.
Signed-off-by: Christian König
---
src/gallium/winsys/radeon/drm/radeon_drm_cs.c | 11 +++
src/gallium/winsys/radeon/drm/radeon_drm_winsys.c | 17 +
src/gallium/winsys/radeon/drm/radeon_winsys.h
From: Christian König
Drawing subtitles didn't increased the dirty area of the surface.
Signed-off-by: Christian König
---
src/gallium/state_trackers/vdpau/output.c |8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/gallium/state_trackers/vdpau/output.c
From: Christian König
Drawing subtitles didn't increased the dirty area of the surface.
v2: don't clear the surface
Signed-off-by: Christian König
---
src/gallium/state_trackers/vdpau/output.c |4
1 file changed, 4 insertions(+)
diff --git a/src/gallium/state_trac
Am 03.04.2013 15:43, schrieb Michel Dänzer:
On Mit, 2013-04-03 at 15:38 +0200, Christian König wrote:
Am 03.04.2013 15:19, schrieb Michel Dänzer:
On Mit, 2013-03-27 at 17:57 +0100, Christian König wrote:
the attached patches dynamically adjust the writemask of MIMG
instructions depending on
Am 10.04.2013 11:46, schrieb Michel Dänzer:
From: Michel Dänzer
21 more little piglits with radeonsi.
Signed-off-by: Michel Dänzer
You figured it out, great! But why the heck is multiplying with
0x4f80 fixing the result?
Cheers,
Christian.
---
v2: Now with lit test.
lib/Target/R6
Am 10.04.2013 12:21, schrieb Michel Dänzer:
On Mit, 2013-04-10 at 12:07 +0200, Christian König wrote:
Am 10.04.2013 11:46, schrieb Michel Dänzer:
From: Michel Dänzer
21 more little piglits with radeonsi.
Signed-off-by: Michel Dänzer
You figured it out, great!
I figured out what? :) This
Hello Ramesh,
currently there is no mesa tree you can fetch, you would need to get the
relevant patches from the maillinglist and apply them manually.
But please be patient, I think I can push the relevant mesa patches to
the master branch today.
Cheers,
Christian.
Am 10.04.2013 06:55, sch
Am 10.04.2013 18:50, schrieb Tom Stellard:
On Wed, Apr 10, 2013 at 05:59:48PM +0200, Michel Dänzer wrote:
[SNIP]
We should start using the updated pattern syntax for all new patterns.
This means replacing register classes with types for the input patterns
and omitting the type in the output pat
From: Christian König
Separated from UVD patch for clarity.
v2: sync with next tree for 3.10
http://cgit.freedesktop.org/~agd5f/linux/log/?h=drm-next-3.10-wip
Signed-off-by: Christian König
---
src/gallium/winsys/radeon/drm/radeon_drm_cs.c | 11 +++
src/gallium/winsys/radeon
Hopefully the last round for this patchset.
Since we now have a stable kernel interface I'm going to commit it this evening
if nobody objects.
Christian.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/lis
Am 11.04.2013 15:54, schrieb Andreas Boll:
2013/4/11 Christian König
From: Christian König
Separated from UVD patch for clarity.
v2: sync with next tree for 3.10
http://cgit.freedesktop.org/~agd5f/linux/log/?h=drm-next-3.10-wip
Signed-off-by: Christian König
---
src/gallium/winsys
From: Christian König
Since we now have UVD support we should enable them by default.
Signed-off-by: Christian König
---
configure.ac |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/configure.ac b/configure.ac
index 55ebb07..aa95406 100644
--- a/configure.ac
+++ b
From: Christian König
Set transfer flag instead of fiddling with the tilling params directly.
Signed-off-by: Christian König
---
src/gallium/drivers/r600/r600_uvd.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/src/gallium/drivers/r600/r600_uvd.c
b/src
Am 17.04.2013 01:13, schrieb Tom Stellard:
From: Tom Stellard
The LLVM C API is considered stable and should never change, so it
is much more desirable to use than the LLVM C++ API, which is constantly in
flux.
Looks good on first glance, but I would separate the initialization and
finding t
Am 18.04.2013 03:07, schrieb Emil Velikov:
On 18/04/13 01:55, Emil Velikov wrote:
On 18/04/13 01:21, Paul Berry wrote:
On 12 April 2013 01:26, Christian König wrote:
From: Christian König
Since we now have UVD support we should enable them by default.
Signed-off-by: Christian König
ood to see you're braver than me, the patch itself is:
Reviewed-by: Christian König
---
configure.ac | 17 +
1 files changed, 1 insertions(+), 16 deletions(-)
diff --git a/configure.ac b/configure.ac
index 70c598e..8266cb7 100644
--- a/configure.ac
+++ b/configure
Hi Vadim,
from your description it seems to be a post processing stage working on
the bytecode of the shaders and additional to that is quite separated
from the rest of the driver.
If that's the case then I don't really see a reason why we shouldn't
merge it, but at least at the beginning it
FER_LOAD_DWORD", SReg_128, SGPR_32
>;
This change looks like it might want to go into a separate patch, apart
from that the patch is:
Reviewed-by: Christian König
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Am 12.12.2013 23:47, schrieb Aaron Watry:
Prevents a potential memory leak found when tracking down something else.
Patch is : Reviewed-by: Christian König
---
src/gallium/state_trackers/vdpau/device.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/state_trackers/vdpau
Don't worry to much about history keeping, anybody who really needs that
should be capable of digging that up anyway.
I would just squash together the changes "Apply si_ file naming scheme
in src/gallium/drive…" and "Fix up file renaming: change file names in
commen…". Also please change the s
Am 13.01.2014 14:13, schrieb Marek Olšák:
From: Marek Olšák
This patch is: Reviewed-by: Christian König
This is the recommended way for streaming vertices. Always use this if you
need to upload vertices every frame.
---
src/gallium/auxiliary/vl/vl_compositor.c | 51
efore every upload.
BTW, I'm not sure if this is the right place for flushing, but it does fix
the bug.
v2 (chk): move the flush to the right place.
Signed-off-by: Christian König
---
src/gallium/state_trackers/vdpau/output.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/stat
MC devs on this.
Christian.
Am 13.01.2014 15:20, schrieb Marek Olšák:
This patch doesn't fix the bug. :(
Marek
On Mon, Jan 13, 2014 at 2:55 PM, Christian König
wrote:
From: Marek Olšák
Bugzilla:
https://bugs.freedesktop.org/show_bug.cgi?id=73191
When VL uploads vertex b
Am 13.01.2014 22:00, schrieb Andreas Hartmetz:
I don't have an fdo account or push rights. Can somebody else push it
for me please? I've added the Reviewed-by: lines so the patches only
need to be pushed now.
You should consider registering an account if you are planning to do
more work on mes
Am 17.01.2014 04:19, schrieb Ilia Mirkin:
It's a bit unreasonable to rely on applications doing the queries and
then obeying their results.
Apart from a minor style comment (see below) the patch looks good to me,
but isn't that a bit superfluous? I mean if the format isn't supported
resource_
I haven't followed the discussion so I can't judge weather or not this
is a good idea.
But as a general note on writing patches: Please limit the subject line
to a sane length! Something between 60 and 80 chars should be ok.
Thanks,
Christian.
Am 17.01.2014 07:37, schrieb Mark Mueller:
This
Am 17.01.2014 09:37, schrieb Ilia Mirkin:
On Fri, Jan 17, 2014 at 3:19 AM, Christian König
wrote:
Am 17.01.2014 04:19, schrieb Ilia Mirkin:
It's a bit unreasonable to rely on applications doing the queries and
then obeying their results.
Apart from a minor style comment (see below
Am 18.01.2014 05:58, schrieb Ilia Mirkin:
NV3x cards don't support NPOT textures. Technically this restriction
could be worked around, but since it also doesn't expose any video
decoding hw, just turn it off entirely.
Signed-off-by: Ilia Mirkin
Cc: 10.0
Actually we had a similar problem for
Am 18.01.2014 11:34, schrieb Ilia Mirkin:
On Sat, Jan 18, 2014 at 5:27 AM, Christian König
wrote:
Am 18.01.2014 05:58, schrieb Ilia Mirkin:
NV3x cards don't support NPOT textures. Technically this restriction
could be worked around, but since it also doesn't expose any video
d
au
I was just about to complain. But it seems like this is the only remark
I can find as well.
Patches are: Reviewed-by: Christian König
+
res = pipe->screen->resource_create(pipe->screen, &res_tmpl);
if (!res) {
ret = VDP_STATUS_RESOURCES;
diff --git a/src/
Not critical, but indeed wrong.
Reviewed-by: Christian König
Am 20.01.2014 16:51, schrieb Brian Paul:
---
src/gallium/state_trackers/vdpau/output.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/vdpau/output.c
b/src/gallium/state_trackers
ssues currently have
priority.
Christian.
Am 23.01.2014 01:38, schrieb Marek Olšák:
Is there any news from the XBMC team? I think we should fix this
sooner rather than later.
Marek
On Mon, Jan 13, 2014 at 3:29 PM, Christian König
wrote:
Yeah, probably because XBMC still (incorrectly) calls the ma
Am 27.03.2014 22:00, schrieb Emil Velikov:
Cc: Christian König
Signed-off-by: Emil Velikov
Reviewed-by: Christian König
---
src/gallium/Automake.inc | 6 ++
src/gallium/targets/r600/omx/Makefile.am | 10 +++---
src/gallium/targets/radeonsi/omx
Am 27.03.2014 22:00, schrieb Emil Velikov:
Otherwise the omx drivers are explicitly linked but never wrapped in
order to use it.
On the other hand I'm not sure if we really need the tracer linked in
here, referencing it was just to make drm_target.c happy.
Christian.
Cc: Christian
Am 28.03.2014 11:32, schrieb Emil Velikov:
On 28/03/14 09:40, Christian König wrote:
Am 27.03.2014 22:00, schrieb Emil Velikov:
Otherwise the omx drivers are explicitly linked but never wrapped in
order to use it.
On the other hand I'm not sure if we really need the tracer linked in
Am 27.03.2014 22:00, schrieb Emil Velikov:
Cc: Christian König
Signed-off-by: Emil Velikov
Reviewed-by: Christian König
---
src/gallium/Automake.inc | 6 ++
src/gallium/targets/r600/omx/Makefile.am | 5 +
src/gallium/targets/radeonsi/omx/Makefile.am | 5
Am 01.04.2014 15:47, schrieb Leo Liu:
From: Leo Liu
This reverts commit 96e8b916a7a39a9ba58e92d1ad77b5501de63ac7.
The patch itself is ok, but we should probably add an explanation why we
want to revert this.
Christian.
Signed-off-by: Leo Liu
---
src/gallium/drivers/radeon/r600_textur
I've applied the original patch because the same thing for reading
textures speeded up the decoding case with UVD quite significantly.
For VCE it reduced the CPU load as well, but I didn't checked if the
encoding time stayed the same (which isn't the case). I think the
problem is that VCE need
_DIRECTLY", the regression can be fixed too.
Going to resend patch for this approach.
Leo
-Original Message-
From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On Behalf Of
Christian König
Sent: Wednesday, April 02, 2014 9:43 AM
To: Marek Olšák; Alex Deucher
Cc: mesa-dev@l
Am 03.04.2014 16:03, schrieb Leo Liu:
From: Leo Liu
Signed-off-by: Leo Liu
---
src/gallium/drivers/radeon/radeon_vce.c| 22 +++---
src/gallium/drivers/radeon/radeon_vce.h| 3 +++
src/gallium/drivers/radeon/radeon_vce_40_2_2.c | 6 --
3 files changed,
501 - 600 of 1819 matches
Mail list logo
