Re: [Mesa-dev] gallium-bind-sampler-states branch

2013-09-16 Thread Chia-I Wu
On Tue, Sep 17, 2013 at 12:09 AM, Brian Paul bri...@vmware.com wrote: On 09/15/2013 09:31 AM, Chia-I Wu wrote: On Sun, Sep 15, 2013 at 12:24 AM, Brian Paul bri...@vmware.com wrote: On 09/12/2013 09:06 PM, Chia-I Wu wrote: Hi Brian, On Fri, Sep 13, 2013 at 8:46 AM, Brian Paul bri

Re: [Mesa-dev] gallium-bind-sampler-states branch

2013-09-15 Thread Chia-I Wu
On Sun, Sep 15, 2013 at 12:24 AM, Brian Paul bri...@vmware.com wrote: On 09/12/2013 09:06 PM, Chia-I Wu wrote: Hi Brian, On Fri, Sep 13, 2013 at 8:46 AM, Brian Paul bri...@vmware.com wrote: I just pushed a gallium-bind-sampler-states branch to my git repo at git://people.freedesktop.org

Re: [Mesa-dev] [PATCH] i965/hsw: approximate DDX with a uniform value across a subspan

2013-09-12 Thread Chia-I Wu
On Thu, Sep 12, 2013 at 2:06 PM, Chris Forbes chr...@ijw.co.nz wrote: Can we make this approximation conditional on an image-quality control in driconf [or somewhere else]? Sure. What would be the default behavior? On Thu, Sep 12, 2013 at 5:00 PM, Chia-I Wu olva...@gmail.com wrote: From

Re: [Mesa-dev] [PATCH] i965/hsw: approximate DDX with a uniform value across a subspan

2013-09-12 Thread Chia-I Wu
anything equivalent -- but I might just be being blind. CC'ing Ian -- any opinion? Is there any conformance issue here? -- Chris On Thu, Sep 12, 2013 at 8:41 PM, Chia-I Wu olva...@gmail.com wrote: On Thu, Sep 12, 2013 at 2:06 PM, Chris Forbes chr...@ijw.co.nz wrote: Can we make

Re: [Mesa-dev] gallium-bind-sampler-states branch

2013-09-12 Thread Chia-I Wu
Hi Brian, On Fri, Sep 13, 2013 at 8:46 AM, Brian Paul bri...@vmware.com wrote: I just pushed a gallium-bind-sampler-states branch to my git repo at git://people.freedesktop.org/~brianp/mesa It replaces the four pipe_context::bind_fragment/vertex/geometry/compute_sampler_states() functions

[Mesa-dev] [PATCH] i965/hsw: compute DDX in a subspan based only on top row

2013-09-12 Thread Chia-I Wu
From: Chia-I Wu o...@lunarg.com Consider only the top-left and top-right pixels to approximate DDX in a 2x2 subspan, unless the application or the user requests a more accurate approximation. This results in a less accurate approximation. However, it improves the performance of Xonotic

Re: [Mesa-dev] [PATCH 01/21] ilo: Fix out-of-tree build.

2013-09-11 Thread Chia-I Wu
On Thu, Sep 12, 2013 at 6:32 AM, Johannes Obermayr johannesoberm...@gmx.de wrote: --- src/gallium/drivers/ilo/Makefile.am | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/ilo/Makefile.am b/src/gallium/drivers/ilo/Makefile.am index 10b3da3..33f2045

[Mesa-dev] [PATCH] i965/hsw: approximate DDX with a uniform value across a subspan

2013-09-11 Thread Chia-I Wu
From: Chia-I Wu o...@lunarg.com Replicate the gradient of the top-left pixel to the other three pixels in the subspan, as how DDY is implemented. Before, different graidents were used for pixels in the top row and pixels in the bottom row. This change results in a less accurate approximation

Re: [Mesa-dev] [PATCH] i965/gen7: always lower textureGrad() on gen7

2013-09-11 Thread Chia-I Wu
On Tue, Sep 10, 2013 at 2:01 PM, Chia-I Wu olva...@gmail.com wrote: On Tue, Sep 10, 2013 at 4:05 AM, Ian Romanick i...@freedesktop.org wrote: On 09/05/2013 03:35 AM, Chia-I Wu wrote: sample_d is slower than the lowered version on gen7. For gen7, this improves Xonotic benchmark with Ultimate

Re: [Mesa-dev] [PATCH] i965/gen7: always lower textureGrad() on gen7

2013-09-11 Thread Chia-I Wu
On Tue, Sep 10, 2013 at 1:37 PM, Chia-I Wu olva...@gmail.com wrote: On Tue, Sep 10, 2013 at 4:01 AM, Ian Romanick i...@freedesktop.org wrote: On 09/06/2013 05:05 AM, Chia-I Wu wrote: On Thu, Sep 5, 2013 at 9:57 PM, Chia-I Wu olva...@gmail.com wrote: On Thu, Sep 5, 2013 at 5:12 PM, Chris Forbes

Re: [Mesa-dev] [PATCH] i965/gen7: always lower textureGrad() on gen7

2013-09-10 Thread Chia-I Wu
On Tue, Sep 10, 2013 at 4:05 AM, Ian Romanick i...@freedesktop.org wrote: On 09/05/2013 03:35 AM, Chia-I Wu wrote: sample_d is slower than the lowered version on gen7. For gen7, this improves Xonotic benchmark with Ultimate effects by as much as 25%: before the change

Re: [Mesa-dev] [PATCH] i965/gen7: always lower textureGrad() on gen7

2013-09-09 Thread Chia-I Wu
On Tue, Sep 10, 2013 at 3:48 AM, Ian Romanick i...@freedesktop.org wrote: On 09/05/2013 08:57 AM, Chia-I Wu wrote: On Thu, Sep 5, 2013 at 5:12 PM, Chris Forbes chr...@ijw.co.nz wrote: A possible explanation for the perf change is that Xonotic uses anisotropic filtering at this quality level

Re: [Mesa-dev] [PATCH] i965/gen7: always lower textureGrad() on gen7

2013-09-09 Thread Chia-I Wu
On Tue, Sep 10, 2013 at 4:01 AM, Ian Romanick i...@freedesktop.org wrote: On 09/06/2013 05:05 AM, Chia-I Wu wrote: On Thu, Sep 5, 2013 at 9:57 PM, Chia-I Wu olva...@gmail.com wrote: On Thu, Sep 5, 2013 at 5:12 PM, Chris Forbes chr...@ijw.co.nz wrote: A possible explanation for the perf change

Re: [Mesa-dev] [PATCH] i965/gen7: always lower textureGrad() on gen7

2013-09-06 Thread Chia-I Wu
(though if your hw has blazing fast sqrt it won't matter...). Nice. Roland Am 05.09.2013 10:35, schrieb Chia-I Wu: sample_d is slower than the lowered version on gen7. For gen7, this improves Xonotic benchmark with Ultimate effects by as much as 25%: before the change

Re: [Mesa-dev] [PATCH] i965/gen7: always lower textureGrad() on gen7

2013-09-06 Thread Chia-I Wu
On Thu, Sep 5, 2013 at 9:57 PM, Chia-I Wu olva...@gmail.com wrote: On Thu, Sep 5, 2013 at 5:12 PM, Chris Forbes chr...@ijw.co.nz wrote: A possible explanation for the perf change is that Xonotic uses anisotropic filtering at this quality level. Lowering to txl defeats it. I had a look

[Mesa-dev] [PATCH] i965/gen7: always lower textureGrad() on gen7

2013-09-05 Thread Chia-I Wu
after the change. Thus textureGrad() is lowered unconditionally for now. Due to this and that I haven't tried it on Haswell, this is still RFC. No piglit regressions. Signed-off-by: Chia-I Wu olva...@gmail.com --- .../dri/i965/brw_lower_texture_gradients.cpp | 54 ++ 1

Re: [Mesa-dev] [PATCH] i965/gen7: always lower textureGrad() on gen7

2013-09-05 Thread Chia-I Wu
allow things like anisotropic filtering to be honored. It would be worth doing an image quality comparison before and after the change. Yeah, that is worth doing. I will do that. -- Chris On Thu, Sep 5, 2013 at 8:35 PM, Chia-I Wu olva...@gmail.com wrote: sample_d is slower than the lowered

[Mesa-dev] [PATCH] glx: make the interval of LIBGL_SHOW_FPS adjustable

2013-08-27 Thread Chia-I Wu
LIBGL_SHOW_FPS=1 makes GLX print FPS every second while other values do nothing. Extend it so that LIBGL_SHOW_FPS=N will print the FPS every N seconds. --- src/glx/dri2_glx.c | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/src/glx/dri2_glx.c b/src/glx/dri2_glx.c

Re: [Mesa-dev] [PATCH 1/4] ilo: implement new float comparison instructions

2013-08-14 Thread Chia-I Wu
On Wed, Aug 14, 2013 at 1:04 AM, srol...@vmware.com wrote: From: Roland Scheidegger srol...@vmware.com untested. Looks good to me. --- src/gallium/drivers/ilo/shader/toy_tgsi.c | 20 1 file changed, 12 insertions(+), 8 deletions(-) diff --git

Re: [Mesa-dev] Removing egl_glx?

2013-08-14 Thread Chia-I Wu
On Thu, Aug 15, 2013 at 10:03 AM, Kenneth Graunke kenn...@whitecape.org wrote: On 08/08/2013 03:13 PM, Chad Versace wrote: [snip] By the way, I talked to krh today, and he suggested that we delete egl_glx rather than allow it to bitrot. I'm in favor, but I don't know who uses that. GLX

Re: [Mesa-dev] [PATCH] gallium/dri-targets: hide all symbols except for __driDriverExtensions

2013-08-14 Thread Chia-I Wu
On Sat, Aug 10, 2013 at 2:56 AM, Marek Olšák mar...@gmail.com wrote: Most importantly, this hides all LLVM symbols. They shouldn't clash with a different LLVM version used by apps (at least in theory). $ nm -g --defined-only radeonsi_dri.so 01148f30 D __driDriverExtensions I am not familiar

Re: [Mesa-dev] [PATCH] gallium/dri-targets: hide all symbols except for __driDriverExtensions

2013-08-14 Thread Chia-I Wu
On Thu, Aug 15, 2013 at 1:26 PM, Chia-I Wu olva...@gmail.com wrote: On Sat, Aug 10, 2013 at 2:56 AM, Marek Olšák mar...@gmail.com wrote: Most importantly, this hides all LLVM symbols. They shouldn't clash with a different LLVM version used by apps (at least in theory). $ nm -g --defined-only

Re: [Mesa-dev] RFC/intel: Separate batch buffers from dynamic state

2013-07-16 Thread Chia-I Wu
On Tue, May 7, 2013 at 4:24 AM, Eric Anholt e...@anholt.net wrote: Paul Berry stereotype...@gmail.com writes: Currently the i965 driver uses a single buffer object to hold both batch buffer commands and dynamic state data structures (which are pointed to by batch buffer commands). We use a

[Mesa-dev] [PATCH] glsl/build: build builtin_compiler with VISIBILITY_CFLAGS

2013-07-10 Thread Chia-I Wu
libglslcore.la and libglcpp.la that are built with builtin_compiler are also linked to by drivers not using libdricore. Since there is no public symbol in them, it is better to mark all symbols hidden. Signed-off-by: Chia-I Wu olva...@gmail.com --- src/glsl/builtin_compiler/Makefile.am | 3

Re: [Mesa-dev] [PATCH] Fix typo in ETIMEOUT (should be ETIMEDOUT)

2013-06-26 Thread Chia-I Wu
On Wed, Jun 26, 2013 at 8:58 PM, Jean-Sébastien Pédron jean-sebastien.ped...@dumbbell.fr wrote: Hello, The subject should be explicit :) Applied, thanks. -- Jean-Sébastien Pédron ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org

[Mesa-dev] [PATCH] st/mesa: fix temp texture bindings in st_CopyPixels()

2013-06-13 Thread Chia-I Wu
The temporary texture should have either PIPE_BIND_RENDER_TARGET or PIPE_BIND_DEPTH_STENCIL set in addition to PIPE_BIND_SAMPLER_VIEW. Signed-off-by: Chia-I Wu olva...@gmail.com --- src/mesa/state_tracker/st_cb_drawpixels.c | 30 + 1 file changed, 13 insertions

[Mesa-dev] [PATCH 2/2] ilo: fix textureSize() for single-layered array textures

2013-06-07 Thread Chia-I Wu
We returned 0 instead of 1 for the number of layers when the array texutre is single-layered. This fixes it on GEN7+. Signed-off-by: Chia-I Wu olva...@gmail.com --- src/gallium/drivers/ilo/ilo_gpe_gen7.c | 20 ++-- 1 file changed, 18 insertions(+), 2 deletions(-) diff --git

[Mesa-dev] [PATCH 1/2] util: add util_resource_is_array_texture()

2013-06-07 Thread Chia-I Wu
Checking if array_size is greater than 1 is not enough for single-layered array textures. Signed-off-by: Chia-I Wu olva...@gmail.com --- src/gallium/auxiliary/util/u_resource.h | 20 +++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/util

Re: [Mesa-dev] [PATCH] u_vbuf: fix index buffer leak

2013-06-07 Thread Chia-I Wu
:58 AM, Marek Olšák mar...@gmail.com wrote: Reviewed-by: Marek Olšák mar...@gmail.com Marek On Fri, Jun 7, 2013 at 6:25 AM, Chia-I Wu olva...@gmail.com wrote: Signed-off-by: Chia-I Wu olva...@gmail.com --- src/gallium/auxiliary/util/u_vbuf.c |3 +++ 1 file changed, 3 insertions

[Mesa-dev] [PATCH] u_vbuf: fix index buffer leak

2013-06-06 Thread Chia-I Wu
Signed-off-by: Chia-I Wu olva...@gmail.com --- src/gallium/auxiliary/util/u_vbuf.c |3 +++ 1 file changed, 3 insertions(+) diff --git a/src/gallium/auxiliary/util/u_vbuf.c b/src/gallium/auxiliary/util/u_vbuf.c index 244b04d..5936f74 100644 --- a/src/gallium/auxiliary/util/u_vbuf.c +++ b

Re: [Mesa-dev] [PATCH] ilo: Ensure array 'shaders' in fully initialized.

2013-05-31 Thread Chia-I Wu
On Fri, May 31, 2013 at 2:59 PM, Vinson Lee v...@freedesktop.org wrote: Fixes Uninitialized pointer read defect reported by Coverity. This looks like a false alarm, as shaders are not read when num_shaders is zero. Does the report give more details? Signed-off-by: Vinson Lee

Re: [Mesa-dev] [PATCH] ilo: Ensure array 'shaders' in fully initialized.

2013-05-31 Thread Chia-I Wu
On Fri, May 31, 2013 at 3:48 PM, Vinson Lee v...@freedesktop.org wrote: On Fri, May 31, 2013 at 12:35 AM, Chia-I Wu olva...@gmail.com wrote: On Fri, May 31, 2013 at 2:59 PM, Vinson Lee v...@freedesktop.org wrote: Fixes Uninitialized pointer read defect reported by Coverity. This looks like

[Mesa-dev] [PATCH] tgsi: add buffer texture to tgsi_util_get_texture_coord_dim()

2013-05-23 Thread Chia-I Wu
TGSI_TEXTURE_BUFFER is one-dimensional. Assert that exec_tex() is never called with TGSI_TEXTURE_BUFFER. Signed-off-by: Chia-I Wu olva...@gmail.com --- src/gallium/auxiliary/tgsi/tgsi_exec.c |1 + src/gallium/auxiliary/tgsi/tgsi_util.c |2 ++ 2 files changed, 3 insertions(+) diff --git

Re: [Mesa-dev] [PATCH] ilo: Initialize need_flush in draw_vbo.

2013-05-23 Thread Chia-I Wu
On Thu, May 23, 2013 at 2:24 PM, Vinson Lee v...@freedesktop.org wrote: need_flush was uninitialized if hw3d-new_batch was true. Fixes Uninitialized scalar variable defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org Applied, thanks. Now I wonder why gcc did not give

Re: [Mesa-dev] [PATCH 5/5] i965 gen7: use SURFACE_STATE fields to select render level/layer

2013-05-22 Thread Chia-I Wu
On Thu, May 23, 2013 at 11:00 AM, Jordan Justen jljus...@gmail.com wrote: On Wed, May 22, 2013 at 3:56 PM, Eric Anholt e...@anholt.net wrote: Jordan Justen jordan.l.jus...@intel.com writes: - surf[0] = BRW_SURFACE_2D BRW_SURFACE_TYPE_SHIFT | + switch (gl_target) { + case

Re: [Mesa-dev] [PATCH 5/5] i965 gen7: add support for layered color renderbuffers

2013-05-20 Thread Chia-I Wu
On Tue, May 21, 2013 at 5:08 AM, Paul Berry stereotype...@gmail.com wrote: On 17 May 2013 21:44, Chia-I Wu olva...@gmail.com wrote: On Sat, May 18, 2013 at 10:11 AM, Jordan Justen jordan.l.jus...@intel.com wrote: Rather than pointing the surface_state directly at a single sub-image

Re: [Mesa-dev] [PATCH 5/5] i965 gen7: add support for layered color renderbuffers

2013-05-17 Thread Chia-I Wu
On Sat, May 18, 2013 at 10:11 AM, Jordan Justen jordan.l.jus...@intel.com wrote: Rather than pointing the surface_state directly at a single sub-image of the texture for rendering, we now point the surface_state at the top level of the texture, and configure the surface_state as needed based

Re: [Mesa-dev] [PATCH] ilo: Initialize read_back in transfer_map_sys.

2013-05-10 Thread Chia-I Wu
On Fri, May 10, 2013 at 1:42 PM, Vinson Lee v...@freedesktop.org wrote: Fixes Uninitialized scalar variable defect reported by Coverity. Committed, thanks. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/ilo/ilo_resource.c | 2 +- 1 file changed, 1 insertion(+), 1

Re: [Mesa-dev] [PATCH] ilo: Add missing break statement in aos_tex TGSI_OPCODE_TEX2 case.

2013-05-07 Thread Chia-I Wu
On Mon, May 6, 2013 at 3:51 AM, Vinson Lee v...@freedesktop.org wrote: Fixes Missing break in switch defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org Applied. Thanks. --- src/gallium/drivers/ilo/shader/toy_tgsi.c | 1 + 1 file changed, 1 insertion(+) diff --git

Re: [Mesa-dev] [PATCH 4/4] tgsi: fix operand type of TGSI_OPCODE_NOT

2013-05-07 Thread Chia-I Wu
On Mon, May 6, 2013 at 6:45 PM, Roland Scheidegger srol...@vmware.com wrote: Am 05.05.2013 18:34, schrieb Chia-I Wu: It should be TGSI_TYPE_UNSIGNED, not TGSI_TYPE_FLOAT. Fixed also gallivm not_emit_cpu() to use uint build context. Signed-off-by: Chia-I Wu olva...@gmail.com --- src

Re: [Mesa-dev] non x11/xlib based EGL and software only renderer

2013-05-07 Thread Chia-I Wu
On Tue, May 7, 2013 at 1:28 PM, Divick Kishore divick.kish...@gmail.com wrote: Hi, is there a possibility in mesa to have egl backend based on complete offscreen buffers and complete s/w only gles renderer? If yes, then could someone please guide me how to build it? You may try $

Re: [Mesa-dev] [PATCH] egl/android: Fix error condition for EGL_ANDROID_image_native_buffer

2013-05-07 Thread Chia-I Wu
On Tue, May 7, 2013 at 3:49 PM, Pohjolainen, Topi topi.pohjolai...@intel.com wrote: On Mon, May 06, 2013 at 02:23:52PM -0700, Chad Versace wrote: Emit EGL_BAD_CONTEXT if the user passes a context to eglCreateImageKHR(type=EGL_ANDROID_image_native_buffer). From the

Re: [Mesa-dev] non x11/xlib based EGL and software only renderer

2013-05-07 Thread Chia-I Wu
On Tue, May 7, 2013 at 5:40 PM, Divick Kishore divick.kish...@gmail.com wrote: Hi Chia, $ ./configure --disable-dri --enable-gallium-egl --with-egl-platforms=null \ --with-gallium-drivers=swrast It will give you an EGL/GLES driver that uses a software renderer and

[Mesa-dev] [PATCH 0/2] tgsi/exec: clean up exec_tex()

2013-05-07 Thread Chia-I Wu
Hi, This series adds a util function to get the dimension of texture coordinates given a texture target. The function allows exec_tex() in tgsi_exec.c to be greatly simplified. There is a subtle difference in how TXP works on array texture. That is, layer is now also projected. You can find

[Mesa-dev] [PATCH 1/2] tgsi: add tgsi_util_get_texture_coord_dim()

2013-05-07 Thread Chia-I Wu
(that is, the Z channel). Signed-off-by: Chia-I Wu olva...@gmail.com --- src/gallium/auxiliary/tgsi/tgsi_util.c | 91 src/gallium/auxiliary/tgsi/tgsi_util.h |3 ++ 2 files changed, 94 insertions(+) diff --git a/src/gallium/auxiliary/tgsi/tgsi_util.c b/src/gallium

[Mesa-dev] [PATCH 2/2] tgsi: clean up exec_tex()

2013-05-07 Thread Chia-I Wu
not sure which behavior is correct or preferred. Signed-off-by: Chia-I Wu olva...@gmail.com --- src/gallium/auxiliary/tgsi/tgsi_exec.c | 220 1 file changed, 52 insertions(+), 168 deletions(-) diff --git a/src/gallium/auxiliary/tgsi/tgsi_exec.c b/src/gallium

Re: [Mesa-dev] [PATCH] egl/android: Fix error condition for EGL_ANDROID_image_native_buffer

2013-05-07 Thread Chia-I Wu
On Wed, May 8, 2013 at 12:15 AM, Chad Versace chad.vers...@linux.intel.com wrote: On 05/07/2013 01:19 AM, Chia-I Wu wrote: On Tue, May 7, 2013 at 3:49 PM, Pohjolainen, Topi topi.pohjolai...@intel.com wrote: On Mon, May 06, 2013 at 02:23:52PM -0700, Chad Versace wrote: Emit EGL_BAD_CONTEXT

Re: [Mesa-dev] non x11/xlib based EGL and software only renderer

2013-05-07 Thread Chia-I Wu
On Wed, May 8, 2013 at 2:34 AM, Divick Kishore divick.kish...@gmail.com wrote: Hi Chia, I haven't tried that for a while, but it should not have X11 dependencies. You probably need to disable other stuffs such as --disable-glx and etc. It might still require X11 at compile time, because

[Mesa-dev] [PATCH 0/4] tgsi: clean up opcode type inference

2013-05-05 Thread Chia-I Wu
Hi, The first three patches of this series move the code around such that tgsi_opcode_infer_src_type() and tgsi_opcode_infer_dst_type() both call a helper function. The idea is that most opcodes should expect the same data type for their src and dst operands so most code can be shared. No

[Mesa-dev] [PATCH 1/4] tgsi: reorder opcodes in opcode type inference

2013-05-05 Thread Chia-I Wu
Reorder opcodes by their assigned numbers. This makes it easier to see the differences between tgsi_opcode_infer_src_type() and tgsi_opcode_infer_dst_type(). Signed-off-by: Chia-I Wu olva...@gmail.com --- src/gallium/auxiliary/tgsi/tgsi_info.c | 48 1 file

[Mesa-dev] [PATCH 2/4] tgsi: refactor tgsi_opcode_infer_dst_type()

2013-05-05 Thread Chia-I Wu
tgsi_opcode_infer_type(). Signed-off-by: Chia-I Wu olva...@gmail.com --- src/gallium/auxiliary/tgsi/tgsi_info.c | 60 +++- 1 file changed, 35 insertions(+), 25 deletions(-) diff --git a/src/gallium/auxiliary/tgsi/tgsi_info.c b/src/gallium/auxiliary/tgsi/tgsi_info.c

[Mesa-dev] [PATCH 3/4] tgsi: refactor tgsi_opcode_infer_src_type()

2013-05-05 Thread Chia-I Wu
Call tgsi_opcode_infer_type() from tgsi_opcode_infer_src_type(). Signed-off-by: Chia-I Wu olva...@gmail.com --- src/gallium/auxiliary/tgsi/tgsi_info.c | 44 +++- 1 file changed, 9 insertions(+), 35 deletions(-) diff --git a/src/gallium/auxiliary/tgsi/tgsi_info.c b

[Mesa-dev] [PATCH 4/4] tgsi: fix operand type of TGSI_OPCODE_NOT

2013-05-05 Thread Chia-I Wu
It should be TGSI_TYPE_UNSIGNED, not TGSI_TYPE_FLOAT. Fixed also gallivm not_emit_cpu() to use uint build context. Signed-off-by: Chia-I Wu olva...@gmail.com --- src/gallium/auxiliary/gallivm/lp_bld_tgsi_action.c |2 +- src/gallium/auxiliary/tgsi/tgsi_info.c |1 + 2 files

[Mesa-dev] [PATCH 0/2] android: add ilo

2013-05-03 Thread Chia-I Wu
Hi, This patch series allows ilo to be selected as the GPU driver on Android-IA. The second patch fixes a linking error, which should be common to other Gallium-based drivers. ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org

[Mesa-dev] [PATCH 1/2] android: add ilo to the build system

2013-05-03 Thread Chia-I Wu
It can be selected with BOARD_GPU_DRIVERS := ilo Signed-off-by: Chia-I Wu olva...@gmail.com --- Android.mk|4 +-- src/egl/main/Android.mk |6 + src/gallium/Android.mk|5 src/gallium/drivers/ilo

[Mesa-dev] [PATCH 2/2] android: libsync is needed on Android 4.2+ for any driver

2013-05-03 Thread Chia-I Wu
Add libsync not only for MESA_BUILD_CLASSIC, but also for MESA_BUILD_GALLIUM. Signed-off-by: Chia-I Wu olva...@gmail.com --- src/egl/main/Android.mk |8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/src/egl/main/Android.mk b/src/egl/main/Android.mk index b2cbe05

[Mesa-dev] [PATCHv2] gallium: fix type of flags in pipe_context::flush()

2013-05-03 Thread Chia-I Wu
-by: Chia-I Wu olva...@gmail.com --- src/gallium/drivers/freedreno/freedreno_context.c |2 +- src/gallium/drivers/galahad/glhd_context.c|2 +- src/gallium/drivers/i915/i915_batch.h |2 +- src/gallium/drivers/i915/i915_flush.c |4 ++-- src/gallium/drivers

Re: [Mesa-dev] [PATCH] gallium: fix type of flags in pipe_context::flush()

2013-05-03 Thread Chia-I Wu
On Fri, May 3, 2013 at 4:00 PM, Jose Fonseca jfons...@vmware.com wrote: - Original Message - It should be unsigned, not enum pipe_flush_flags. Fixed a build error: src/gallium/state_trackers/egl/android/native_android.cpp:426:29: error: invalid conversion from 'int' to

[Mesa-dev] [PATCH 0/6] util/prim: clean up

2013-05-02 Thread Chia-I Wu
Hi list, This patch series attemps to clean up u_prim.h, with an exception that a new function to get the tessellated (as opposed to decomposed) primitive count is added by the last patch. I need that function for ilo to update PIPE_QUERY_PRIMITIVES_GENERATED.

[Mesa-dev] [PATCH 1/7] util/prim: fix primitive trimming for triangles with adjacency

2013-05-02 Thread Chia-I Wu
Fix for PIPE_PRIM_TRIANGLES_ADJACENCY and PIPE_PRIM_TRIANGLE_STRIP_ADJACENCY. Signed-off-by: Chia-I Wu olva...@gmail.com --- src/gallium/auxiliary/util/u_prim.h |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/auxiliary/util/u_prim.h b/src/gallium/auxiliary

[Mesa-dev] [PATCH 2/7] util/prim: clean up and add comments

2013-05-02 Thread Chia-I Wu
Move together (or add) functions to decompose/reduce/assemble a primitive, give them consistent names, and document them. Add u_prim_vertex_count() so that the vertex count information can be used elsewhere. u_assembled_primitive() will be removed in a folow-on commit. Signed-off-by: Chia-I Wu

[Mesa-dev] [PATCH 3/7] draw: use u_assembled_prim() instead of u_assembled_primitive()

2013-05-02 Thread Chia-I Wu
The latter function is also removed as a result of the change. Signed-off-by: Chia-I Wu olva...@gmail.com --- src/gallium/auxiliary/draw/draw_prim_assembler.c |4 ++-- src/gallium/auxiliary/draw/draw_pt_fetch_shade_pipeline.c |2 +- src/gallium/auxiliary/util/u_prim.h

[Mesa-dev] [PATCH 4/7] util/prim: fix the name of the include guard

2013-05-02 Thread Chia-I Wu
It should be U_PRIM_H, not U_BLIT_H. Signed-off-by: Chia-I Wu olva...@gmail.com --- src/gallium/auxiliary/util/u_prim.h |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/auxiliary/util/u_prim.h b/src/gallium/auxiliary/util/u_prim.h index b9c0c15..c6a0708

[Mesa-dev] [PATCH 5/7] util/prim: use vertex count info in u_validate_pipe_prim()

2013-05-02 Thread Chia-I Wu
As a side effect, primitives with adjacency are now correctly validated. Signed-off-by: Chia-I Wu olva...@gmail.com --- src/gallium/auxiliary/util/u_prim.h | 34 ++ 1 file changed, 2 insertions(+), 32 deletions(-) diff --git a/src/gallium/auxiliary/util

[Mesa-dev] [PATCH 6/7] util/prim: assorted fixes for u_decomposed_prims_for_vertices()

2013-05-02 Thread Chia-I Wu
Switch to '=' for comparisons, and it becomes obvious that the comparison for PIPE_PRIM_QUAD_STRIP was wrong. Add minimum vertex count check for PIPE_PRIM_LINE_LOOP. Return 1 for PIPE_PRIM_POLYGON with 3 vertices. Signed-off-by: Chia-I Wu olva...@gmail.com --- src/gallium/auxiliary/util

[Mesa-dev] [PATCH 7/7] util/prim: add u_reduced_prims_for_vertices()

2013-05-02 Thread Chia-I Wu
The function returns the number of reduced/tessellated primitives for the given vertex count. Signed-off-by: Chia-I Wu olva...@gmail.com --- src/gallium/auxiliary/util/u_prim.h | 20 1 file changed, 20 insertions(+) diff --git a/src/gallium/auxiliary/util/u_prim.h b/src

[Mesa-dev] [PATCH] gallium: fix type of flags in pipe_context::flush()

2013-05-02 Thread Chia-I Wu
It should be unsigned, not enum pipe_flush_flags. Fixed a build error: src/gallium/state_trackers/egl/android/native_android.cpp:426:29: error: invalid conversion from 'int' to 'pipe_flush_flags' [-fpermissive] Signed-off-by: Chia-I Wu olva...@gmail.com --- src/gallium/include/pipe

Re: [Mesa-dev] renaming i965g

2013-04-27 Thread Chia-I Wu
Hi Chris, On Fri, Apr 26, 2013 at 5:42 PM, Chris Wilson ch...@chris-wilson.co.uk wrote: On Fri, Apr 26, 2013 at 04:26:05PM +0800, Chia-I Wu wrote: I just pushed the driver to master, under the name ilo. The driver is still new and has many bugs or known issues, but I will continue improving

Re: [Mesa-dev] renaming i965g

2013-04-26 Thread Chia-I Wu
On Sat, Apr 20, 2013 at 1:06 AM, Chia-I Wu olva...@gmail.com wrote: On Thu, Apr 18, 2013 at 2:29 PM, Chia-I Wu olva...@gmail.com wrote: Hi list, Per the discussion, i965g is confusing and misleading. Instead of preventing the confusions via --with-gallium-drivers=i965g-unofficial

Re: [Mesa-dev] renaming i965g

2013-04-19 Thread Chia-I Wu
On Thu, Apr 18, 2013 at 2:29 PM, Chia-I Wu olva...@gmail.com wrote: Hi list, Per the discussion, i965g is confusing and misleading. Instead of preventing the confusions via --with-gallium-drivers=i965g-unofficial or --with-gallium-drivers=experimental-i965g, which kind of makes i965g

Re: [Mesa-dev] new i965g pipe driver for Intel GEN6 (and later)

2013-04-18 Thread Chia-I Wu
On Wed, Apr 17, 2013 at 10:26 PM, Brian Paul bri...@vmware.com wrote: On 04/17/2013 02:31 AM, Jose Fonseca wrote: [snipped] How hard would it be to make GLSL IR an alternative IR for pipe drivers? On the driver side, it seems I only need to add PIPE_SHADER_IR_GLSL for the preferred IR and

[Mesa-dev] renaming i965g

2013-04-18 Thread Chia-I Wu
Hi list, Per the discussion, i965g is confusing and misleading. Instead of preventing the confusions via --with-gallium-drivers=i965g-unofficial or --with-gallium-drivers=experimental-i965g, which kind of makes i965g a second-class citizen, I think it is better to rename it. Then whenever

Re: [Mesa-dev] renaming i965g

2013-04-18 Thread Chia-I Wu
On Thu, Apr 18, 2013 at 11:36 PM, Eric Anholt e...@anholt.net wrote: Chia-I Wu olva...@gmail.com writes: Hi list, Per the discussion, i965g is confusing and misleading. Instead of preventing the confusions via --with-gallium-drivers=i965g-unofficial or --with-gallium-drivers

Re: [Mesa-dev] new i965g pipe driver for Intel GEN6 (and later)

2013-04-17 Thread Chia-I Wu
On Wed, Apr 17, 2013 at 5:25 AM, Dave Airlie airl...@gmail.com wrote: Those are just ideas. I'm open to discussion. The driver is disabled by default and needs to be enabled via --with-gallium-drivers=i965. I think a warning + maybe something like

Re: [Mesa-dev] new i965g pipe driver for Intel GEN6 (and later)

2013-04-17 Thread Chia-I Wu
On Wed, Apr 17, 2013 at 8:09 AM, Marek Olšák mar...@gmail.com wrote: On Tue, Apr 16, 2013 at 9:58 PM, Chia-I Wu olva...@gmail.com wrote: On Wed, Apr 17, 2013 at 12:58 AM, Matt Turner matts...@gmail.com wrote: I think everything Marek said was correct. If you could extend Gallium to consume

Re: [Mesa-dev] new i965g pipe driver for Intel GEN6 (and later)

2013-04-16 Thread Chia-I Wu
Hi list, On Thu, Dec 13, 2012 at 6:41 AM, Chia-I Wu olva...@gmail.com wrote: Hi list, I've been working on i965g, a new pipe driver for Intel GEN6 (and later), for a while now. I would like to know if there is any interest in it and if it can be merged upstream. The code is currently

Re: [Mesa-dev] new i965g pipe driver for Intel GEN6 (and later)

2013-04-16 Thread Chia-I Wu
Hi Matt, On Wed, Apr 17, 2013 at 12:58 AM, Matt Turner matts...@gmail.com wrote: On Tue, Apr 16, 2013 at 9:45 AM, Chia-I Wu olva...@gmail.com wrote: If there is no objection, I'd like to merge it in a day or two. My only objection is over adding a driver that is explicitly a toy

Re: [Mesa-dev] new i965g pipe driver for Intel GEN6 (and later)

2013-04-16 Thread Chia-I Wu
Hi Ken, On Wed, Apr 17, 2013 at 1:18 AM, Kenneth Graunke kenn...@whitecape.orgwrote: On 04/16/2013 09:58 AM, Matt Turner wrote: On Tue, Apr 16, 2013 at 9:45 AM, Chia-I Wu olva...@gmail.com wrote: If there is no objection, I'd like to merge it in a day or two. My only objection is over

[Mesa-dev] [PATCH] i965: fix SURFACE_STATE dumping

2013-04-10 Thread Chia-I Wu
Wrong fields were used when dumping width and height. --- src/mesa/drivers/dri/i965/brw_state_dump.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_state_dump.c b/src/mesa/drivers/dri/i965/brw_state_dump.c index 9ea3fac..6fc8837

[Mesa-dev] [PATCH] i965/gen7: fix 3DSTATE_LINE_STIPPLE_PATTERN

2013-04-10 Thread Chia-I Wu
The inverse repeat count should taks up bits 31:15 and is in U1.16. Demos from mesa/demos seem to render correctly with this change, bu piglit linestipple test still fails. --- src/mesa/drivers/dri/i965/brw_misc_state.c | 17 ++--- 1 file changed, 14 insertions(+), 3 deletions(-)

[Mesa-dev] [PATCH] i965/gen7: fix (huge) entry count for BRW_SURFACE_BUFFER

2013-04-10 Thread Chia-I Wu
Unlike GEN6, the bits of entry count are distributed like this width = (entry_count 0x007f); /* bits [6:0] */ height = (entry_count 0x001fff80) 7; /* bits [20:7] */ depth = (entry_count 0x7fe0) 21; /* bits [30:21] */ The maximum entry count is still limited to 2^27.

Re: [Mesa-dev] [PATCH 05/11] gallium/auxiliary: Build libgallium shared.

2013-01-11 Thread Chia-I Wu
On Sat, Jan 12, 2013 at 10:23 AM, Johannes Obermayr johannesoberm...@gmx.de wrote: diff --git a/src/gallium/auxiliary/Makefile.am b/src/gallium/auxiliary/Makefile.am index a4eee47..21b3c6b 100644 --- a/src/gallium/auxiliary/Makefile.am +++ b/src/gallium/auxiliary/Makefile.am @@ -3,13 +3,17

Re: [Mesa-dev] [PATCH] Revert configure.ac: Disable compiler optimizations when --enable-debug is set

2013-01-09 Thread Chia-I Wu
On Wed, Jan 9, 2013 at 5:32 PM, Michel Dänzer mic...@daenzer.net wrote: On Die, 2013-01-08 at 09:31 -0800, Brian Paul wrote: On 01/08/2013 08:38 AM, Marek Olšák wrote: Those who do not want optimizations should set the CFLAGS and CXXFLAGS environment variables. In my opinion,

Re: [Mesa-dev] Mesa master branch: forced update

2012-12-14 Thread Chia-I Wu
Hi Kristian, On Wed, Jul 11, 2012 at 7:15 AM, Kristian Høgsberg k...@bitplanet.net wrote: [snip] Chia-I hasn't been active for a long time and the suddenly pushes a forced update of the repo, I don't think anything was compromised or any history lost. The freedesktop.org account has been

[Mesa-dev] new i965g pipe driver for Intel GEN6 (and later)

2012-12-12 Thread Chia-I Wu
Hi list, I've been working on i965g, a new pipe driver for Intel GEN6 (and later), for a while now. I would like to know if there is any interest in it and if it can be merged upstream. The code is currently available here https://github.com/olvaffe/mesa/tree/i965g The project was started

Re: [Mesa-dev] new i965g pipe driver for Intel GEN6 (and later)

2012-12-12 Thread Chia-I Wu
. Roland Am 12.12.2012 23:41, schrieb Chia-I Wu: Hi list, I've been working on i965g, a new pipe driver for Intel GEN6 (and later), for a while now. I would like to know if there is any interest in it and if it can be merged upstream. The code is currently available here https://github.com

Re: [Mesa-dev] new i965g pipe driver for Intel GEN6 (and later)

2012-12-12 Thread Chia-I Wu
not done any profiling or optimization so far. But this is definitely one of the things that needs to be fixed. Marek On Wed, Dec 12, 2012 at 11:41 PM, Chia-I Wu olva...@gmail.com wrote: Hi list, I've been working on i965g, a new pipe driver for Intel GEN6 (and later), for a while now. I would

Re: [Mesa-dev] new i965g pipe driver for Intel GEN6 (and later)

2012-12-12 Thread Chia-I Wu
+ only? I'm just curious as I thought that despite some differences they still could share quite some code. Roland Am 12.12.2012 23:41, schrieb Chia-I Wu: Hi list, I've been working on i965g, a new pipe driver for Intel GEN6 (and later), for a while now. I would like to know if there is any

Re: [Mesa-dev] Mesa master branch: forced update

2012-07-11 Thread Chia-I Wu
On Wed, Jul 11, 2012 at 7:15 AM, Kristian Høgsberg k...@bitplanet.net wrote: On Tue, Jul 10, 2012 at 4:54 PM, Kristian Høgsberg k...@bitplanet.net wrote: On Tue, Jul 10, 2012 at 4:24 PM, Ferry Huberts maili...@hupie.com wrote: On 10-07-12 22:13, Kenneth Graunke wrote: On 07/10/2012 12:50 PM,

Re: [Mesa-dev] [PATCH] glapi: Fix incorrect enum value.

2012-02-28 Thread Chia-I Wu
On Tue, Feb 28, 2012 at 1:55 AM, Ian Romanick i...@freedesktop.org wrote: On 02/23/2012 11:52 AM, Chad Versace wrote: On 02/23/2012 12:52 AM, Ian Romanick wrote: On 02/22/2012 04:06 PM, Chad Versace wrote: On 02/22/2012 02:22 PM, Ian Romanick wrote: On 02/22/2012 02:17 PM, Paul Berry

Re: [Mesa-dev] [PATCH] Always build shared glapi

2012-01-11 Thread Chia-I Wu
work. I tried searching on gmail as to why this was added, but I turned up nothing. Maybe Chia-I Wu knows. Shared glapi adds libglapi.so. Weird things could happen when libGL.so and libglapi.so are from different versions of Mesa. But it is more a problem for distros to deal with. If we all

Re: [Mesa-dev] Drooping multiple driver support in EGL?

2011-12-23 Thread Chia-I Wu
On Sat, Dec 24, 2011 at 12:37 AM, Ian Romanick i...@freedesktop.org wrote: On 12/23/2011 07:20 AM, Jose Fonseca wrote: - Original Message - Hi list, Multiple driver support in EGL is hard to get right, if not impossible. On Linux desktop, we almost always want to use egl_dri2.  It

[Mesa-dev] Drooping multiple driver support in EGL?

2011-12-22 Thread Chia-I Wu
Hi list, Multiple driver support in EGL is hard to get right, if not impossible. On Linux desktop, we almost always want to use egl_dri2. It allows EGL to loads DRI2 drivers, thus allowing it to share DRI2 drivers with libGL. In one case where the app wants to use OpenVG, libEGL needs to load

Re: [Mesa-dev] [PATCH 01/20] st-api: Clean-up OpenGL profile handling

2011-12-21 Thread Chia-I Wu
: NULL); Olv should probably double-check this, but LGTM. Reviewed-by: Brian Paul bri...@vmware.com It looks good to me Reviewed-by: Chia-I Wu o...@lunarg.com ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org

Re: [Mesa-dev] [PATCH 02/20] st-api: Have context_create explain why creation failed

2011-12-21 Thread Chia-I Wu
,                                               struct st_context_iface *stsharei); Maybe error should come after stsharei? Anyway, Reviewed-by: Chia-I Wu o...@lunarg.com    /** diff --git a/src/gallium/state_trackers/dri/common/dri_context.c b/src/gallium/state_trackers/dri/common/dri_context.c index e9e0049..3e5a040 100644

Re: [Mesa-dev] [PATCH 2/2] egl: update headers

2011-12-20 Thread Chia-I Wu
On Tue, Dec 20, 2011 at 5:28 AM, Ian Romanick i...@freedesktop.org wrote: On 12/19/2011 02:05 AM, Chia-I Wu wrote: From: Chia-I Wuo...@lunarg.com Update to revision 15052. EGL_MESA_drm_image is now official.  But apparently we have our own extension to it and we need this in eglmesaext.h

Re: [Mesa-dev] [PATCH 1/2] st/egl: Add support for EGL_NOK_swap_region

2011-12-14 Thread Chia-I Wu
On Thu, Dec 15, 2011 at 4:06 AM, Fredrik Höglund fred...@kde.org wrote: Backends indicate that they support this extension by returning EGL_TRUE when native_display::get_param() is called with NATIVE_PARAM_PRESENT_REGION and NATIVE_PARAM_PRESERVE_BUFFER. native_present_control is extended to

Re: [Mesa-dev] [PATCH 1/3] egl: add EGL_NV_post_sub_buffer

2011-12-14 Thread Chia-I Wu
On Thu, Dec 15, 2011 at 5:21 AM, Ian Romanick i...@freedesktop.org wrote: On 12/14/2011 12:24 PM, Fredrik Höglund wrote: v2: Handle EGL_POST_SUB_BUFFER_SUPPORTED_NV in     _eglParseSurfaceAttribList() Signed-off-by: Fredrik Höglundfred...@kde.org ---  include/EGL/eglext.h      |    9

Re: [Mesa-dev] [PATCH 1/2] st/egl: Add support for EGL_NOK_swap_region

2011-12-13 Thread Chia-I Wu
On Fri, Dec 9, 2011 at 11:36 PM, Fredrik Höglund fred...@kde.org wrote: Backends indicate that they support this extension by returning EGL_TRUE when native_display::get_param() is called with NATIVE_PARAM_PRESENT_REGION. native_present_control is extended to include the region that should

Re: [Mesa-dev] [PATCH 1/3] egl: add EGL_NV_post_sub_buffer

2011-12-13 Thread Chia-I Wu
On Sun, Dec 11, 2011 at 1:56 AM, Fredrik Höglund fred...@kde.org wrote: Signed-off-by: Fredrik Höglund fred...@kde.org ---  include/EGL/eglext.h      |    9 +  src/egl/main/eglapi.c     |   24  src/egl/main/eglapi.h     |    8  

<    1   2   3   4   5   6   7   >