-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Reviewed-by: Chad Versace c...@chad-versace.us
On 08/16/2011 11:28 PM, Chia-I Wu wrote:
Add EGL_ANDROID_image_native_buffer and EGL_ANDROID_swap_rectangle.
There is no spec for them though.
---
include/EGL/eglext.h | 14
- --
Chad Versace
c...@chad-versace.us
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
iQIcBAEBAgAGBQJOTAJ4AAoJEAIvNt057x8ilcAP/0wvxg/hUFAE/96bCsSDFjsB
5pl7KdyGpV7UyS71gpTSa/+P79zXRFqLb7+eXNTkApIyVn4G6XhDmhzaU0oLVlGT
Y
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
The version tag in the copyright header should be removed. But, either way,
this is
Reviewed-by: Chad Versace c...@chad-versace.us
On 08/16/2011 11:28 PM, Chia-I Wu wrote:
This builds the static library libmesa_egl from core EGL.
---
Android.mk
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Overall this looks good, but some of it I don't follow. I trust that you're
doing
the right thing for those parts.
Acked-by: Chad Versace c...@chad-versace.us
On 08/16/2011 11:28 PM, Chia-I Wu wrote:
This builds the static library libmesa_st_mesa
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Reviewed-by: Chad Versace c...@chad-versace.us
On 08/16/2011 11:28 PM, Chia-I Wu wrote:
This builds the shared library libglapi from shared glapi.
---
Android.mk |1 +
src/mapi/Android.mk | 61
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 08/16/2011 11:28 PM, Chia-I Wu wrote:
This builds the static library libmesa_glsl and executable glsl_compiler
from glsl. glsl_compiler is only installed for engineering build.
---
Android.mk |1 +
src/glsl/Android.gen.mk |
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Acked-by: Chad Versace c...@chad-versace.us
On 08/16/2011 11:28 PM, Chia-I Wu wrote:
libGLES_mesa with swrast should link in these libraries
libmesa_egl
libmesa_st_egl
libmesa_st_mesa
libmesa_glsl
libmesa_glsl_utils
to fold Android support into
master and make progress towards a stable Mesa on Android.
I've commented, acked, or rb'd all the patches that are within my
domain. I've avoided commenting on patches specific to Gallium.
- --
Chad Versace
c...@chad-versace.us
-BEGIN PGP SIGNATURE-
Version
On 08/17/2011 08:06 PM, Chia-I Wu wrote:
On Thu, Aug 18, 2011 at 12:28 AM, Chad Versace c...@chad-versace.us wrote:
On 08/17/2011 08:31 AM, Ian Romanick wrote:
On 08/16/2011 11:28 PM, Chia-I Wu wrote:
Define SIZE_MAX to UINT32_MAX for it. --- src/glsl/ralloc.c |4 1
files changed, 4
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 08/17/2011 08:24 PM, Chia-I Wu wrote:
On Thu, Aug 18, 2011 at 2:03 AM, Chad Versace c...@chad-versace.us wrote:
I've commented below on TOP/Android.mk and TOP/Android.common.mk.
The Gallium makefiles are outside of my domain, so I have
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Great. Then the glsl makefile is
Reviewed-by: Chad Versace c...@chad-versace.us
On 08/17/2011 08:26 PM, Chia-I Wu wrote:
On Thu, Aug 18, 2011 at 4:06 AM, Chad Versace c...@chad-versace.us wrote:
On 08/16/2011 11:28 PM, Chia-I Wu wrote:
This builds
Thanks for doing this, Chia-I. I like this approach.
Chad Versace
c...@chad-versace.us
On 08/18/2011 03:33 AM, Jose Fonseca wrote:
I think this is great. Thanks for stepping up and doing this, Olv!
Concerning the implementation, my only suggestion is that it might be useful
to have
On 08/17/2011 08:37 PM, Chia-I Wu wrote:
On Thu, Aug 18, 2011 at 4:14 AM, Chad Versace c...@chad-versace.us wrote:
On 08/16/2011 11:28 PM, Chia-I Wu wrote:
Hi list,
This patch series adds Android Gingerbread support to Mesa. To begin
with, it only supports software rendering
Reviewed-by: Kenneth Graunke kenn...@whitecape.org
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/glsl/Makefile |8
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/glsl/Makefile b/src/glsl/Makefile
index c20a6c9..00b7b91 100644
--- a/src/glsl/Makefile
+++ b/src
Declare _mesa_meta_begin()/end() in meta.h so that drivers can write
custom meta-ops (such as HiZ resolves for i965).
This necessitates moving the the META_* macros into meta.h. To prevent
naming collisions, this commit renames each macro to be MESA_META_*.
Signed-off-by: Chad Versace c...@chad
location is
sufficiently unrelated to the DRI2 HiZ handshake as to be uninformative.
In experimenting with HiZ, Anholt encountered this assertion failure and
was unable to understand its cause.
CC: Eric Anholt e...@anholt.net
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Reviewed-by: Chad Versace c...@chad-versace.us
On 08/18/2011 07:37 PM, Chia-I Wu wrote:
Define log2f(v) to be logf(v) / M_LN2 and ffs to __builtin_ffs.
---
src/mesa/main/imports.h |8 +++-
1 files changed, 7 insertions(+), 1 deletions
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Reviewed-by: Chad Versace c...@chad-versace.us
On 08/18/2011 07:37 PM, Chia-I Wu wrote:
libGLES_mesa with swrast should link in these libraries
libmesa_egl
libmesa_egl_gallium
libmesa_st_egl
libmesa_st_mesa
libmesa_glsl
.
Looks good to me. Thanks olv.
- --
Chad Versace
c...@chad-versace.us
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
iQIcBAEBAgAGBQJOTp9zAAoJEAIvNt057x8iYB0P/07r/fxAjTe8pOWhqP6lZS4M
fMDRtRSwSrHDRISWO+bGjcbvZ45SqsbAA
On 08/19/2011 10:35 AM, Eric Anholt wrote:
On Thu, 18 Aug 2011 14:02:46 -0700, Chad Versace c...@chad-versace.us wrote:
When intel_verify_dri2_has_hiz() discovers that DRI2 (that is, the DDX
driver) cannot provide a separate stencil buffer, but
intel_context::hw_must_use_separate_stencil
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 08/19/2011 03:18 PM, Kenneth Graunke wrote:
On 08/19/2011 10:44 AM, Chad Versace wrote:
On 08/19/2011 10:35 AM, Eric Anholt wrote:
On Thu, 18 Aug 2011 14:02:46 -0700, Chad Versace c...@chad-versace.us
wrote:
When intel_verify_dri2_has_hiz
Versace c...@chad-versace.us
- --
Chad Versace
c...@chad-versace.us
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
iQIcBAEBAgAGBQJOUoNYAAoJEAIvNt057x8icuQQAKHW2IF6BRv8RsOmsK4TSzuQ
vwi8AhYLo1eIUTVTApFT
In preparation for porting i965 to Android, factor its source lists into
a shared makefile. This prevents duplication of source lists, and hence
prevents the Android from breaking as often.
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/i965/Makefile | 132
In preparation for porting i915 to Android, factor its source lists into
a shared makefile. This prevents duplication of source lists, and hence
prevents the Android from breaking as often.
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/i915/Makefile | 49
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
The first two lines of the diff,
+ MESA_TOP := $(TOP)
+
ignore them. They slipped in by accident.
- --
Chad Versace
c...@chad-versace.us
On 08/23/2011 05:04 PM, Chad Versace wrote:
In preparation for porting i965 to Android, factor its
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
I see why it's necessary, and can think of no cleaner way of doing it. So this
is
Reviewed-by: Chad Versace c...@chad-versace.us
On 08/23/2011 08:10 PM, Chia-I Wu wrote:
Add rgba_masks to dri2_add_config. When it is non-NULL, the DRI config
+++
src/egl/main/Android.mk |3 ++
3 files changed, 63 insertions(+), 1 deletions(-)
create mode 100644 src/egl/drivers/dri2/Android.mk
Reviewed-by: Chad Versace c...@chad-versace.us
- --
Chad Versace
c...@chad-versace.us
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.11
*/
+#include i915_drm.h
+/* for radeon */
+#include radeon_drm.h
These includes should be moved to the top of the file.
- --
Chad Versace
c...@chad-versace.us
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
On 08/24/2011 06:20 PM, Chia-I Wu wrote:
On Thu, Aug 25, 2011 at 8:58 AM, Chad Versace c...@chad-versace.us wrote:
Comments below.
On 08/23/2011 08:10 PM, Chia-I Wu wrote:
Add platform_android.c that supports _EGL_PLAFORM_ANDROID. It works
with drm_gralloc, where back buffers of windows
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Simple enough.
Reviewed-by: Chad Versace c...@chad-versace.us
On 08/25/2011 09:43 AM, Kenneth Graunke wrote:
Otherwise we continue and hit the Illegal formal parameter mode
assertion.
Fixes negative compile test texelFetchOffset.frag in piglit
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
We weren't catching this? Embarrassing.
Reviewed-by: Chad Versace c...@chad-versace.us
We may also want to emit an error when a variable is declared as 'in'
a VS and as 'out' in a FS. But I'm not sure that's actually illegal.
On 08/25/2011 08:26 AM
[CC'ing krh because I learned much of this from him, so may have some
insight to share with us.]
On 08/25/2011 08:14 PM, Chia-I Wu wrote:
On Fri, Aug 26, 2011 at 4:09 AM, Chad Versace c...@chad-versace.us wrote:
On 08/24/2011 06:20 PM, Chia-I Wu wrote:
On Thu, Aug 25, 2011 at 8:58 AM, Chad
Any comments from the Intel guys? I plan on pushing this Friday if
no one objects.
--
Chad Versace
c...@chad-versace.us
On 08/23/2011 05:04 PM, Chad Versace wrote:
In preparation for porting i965 to Android, factor its source lists into
a shared makefile. This prevents duplication of source
On 08/25/2011 11:19 PM, Chia-I Wu wrote:
On Fri, Aug 26, 2011 at 2:05 PM, Chia-I Wu olva...@gmail.com wrote:
On Fri, Aug 26, 2011 at 12:55 PM, Chad Versace c...@chad-versace.us wrote:
[CC'ing krh because I learned much of this from him, so may have some
insight to share with us.]
On 08/25
This series builds libmesa_dricore.so and i965_dri.so for Android. Along the
way, a few cleanups are necessary.
This just gets the driver to build. I have not tested it yet.
Chad Versace (8):
android: Remove -fvisibility=hidden from CFLAGS so that we can build
a shared dricore
mesa
If you attempt to link together libmesa_dricore.so from object files built with
visibility=hidden, then the library is useless. It exposes no symbols.
Signed-off-by: Chad Versace c...@chad-versace.us
---
Android.common.mk |1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git
In src/mesa/Android.mk, it is non-trivial to determine which variables are
imported by `include sources.mak`. So document them.
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/Android.mk |5 +
1 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/src/mesa
libmesa_dricore is analogous to the libdricore built by the Autoconf
build.
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/Android.mk | 35 +++
1 files changed, 35 insertions(+), 0 deletions(-)
diff --git a/src/mesa/Android.mk b/src/mesa
-off: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/Makefile.defines| 23 ++-
src/mesa/drivers/dri/common/Makefile.sources | 18 ++
2 files changed, 28 insertions(+), 13 deletions(-)
create mode 100644 src/mesa/drivers/dri/common
Stop building driverfuncs.c twice for each DRI driver. driverfuncs.c is
included in both libmesa.a and in
src/mesa/drivers/dri/Makefiles.defines:COMMON_SOURCES, so remove it from
COMMON_SOURCES.
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/Makefile.defines |1
Add $(TOP)/src/mesa to include paths and replace each occurence of
#include ../glsl/*.h
with
#include glsl/*.h
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/i965/Makefile |1 +
src/mesa/drivers/dri/i965/brw_clip.c |2 +-
src
libmesa_dri_common is a static library that contains the sources in
src/mesa/drivers/dri/common. Each DRI driver should link to it.
Signed-off-by: Chad Versace c...@chad-versace.us
---
Android.mk |1 +
src/mesa/drivers/dri/common/Android.mk | 49
This just gets the driver to build. I have not yet tested if the driver
actually works.
Signed-off-by: Chad Versace c...@chad-versace.us
---
Android.mk |3 +-
src/mesa/drivers/dri/Android.mk | 61 ++
src/mesa/drivers/dri/i965
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 08/26/2011 10:56 PM, Chia-I Wu wrote:
On Sat, Aug 27, 2011 at 9:53 AM, Chad Versace c...@chad-versace.us wrote:
In order that the autoconf and Android build can share the same source
lists, move the lists from
src/mesa/drivers/dri
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 08/26/2011 10:51 PM, Chia-I Wu wrote:
On Sat, Aug 27, 2011 at 9:54 AM, Chad Versace c...@chad-versace.us wrote:
This just gets the driver to build. I have not yet tested if the driver
actually works.
Signed-off-by: Chad Versace c...@chad
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Spurious whitespace is fixed in v2.
On 08/27/2011 12:19 AM, Ian Romanick wrote:
On 08/26/2011 06:54 PM, Chad Versace wrote:
Add $(TOP)/src/mesa to include paths and replace each occurence of
#include ../glsl/*.h
with
#include glsl/*.h
Chad Versace (3):
make: Remove duplicate defintion of COMMON_SOURCES in Radeon
makefiles
make: Remove duplicate occurence of driverfuncs.c
make: Factor out source lists from drivers/dri/common into
Makefile.sources
src/mesa/drivers/dri/Makefile.defines| 21
Remove defintion of COMMON_SOURCES from {r300,r660}/Makefile. The
defintion is a duplicate of that found in
src/mesa/drivers/dri/Makefile.defines.
CC: Ian Romanick i...@freedesktop.org
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/r300/Makefile |9 -
src
commit message. For the autoconf build, it was incorrect to say
that driverfuncs.o was built twice.
CC: Ian Romanick i...@freedesktop.org
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/Makefile.defines |1 -
src/mesa/drivers/dri/swrast/Makefile |1 -
2 files
- [olv] Remove MESA_TOP from definition of *_INCLUDES so that
Makefile.sources can be self-contained.
CC: Chia-I Wu o...@lunarg.com
Signed-off: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/Makefile.defines| 20 +++-
src/mesa/drivers/dri/common
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
v2 of this series has been posted.
- --
Chad Versace
c...@chad-versace.us
On 08/26/2011 06:53 PM, Chad Versace wrote:
This series builds libmesa_dricore.so and i965_dri.so for Android. Along the
way, a few cleanups are necessary.
This just
Changes noted in each patch.
Chad Versace (5):
mesa: Build libmesa_dricore.a for Android
make: Document imported variables
dri: Build libmesa_dri_common for Android
i965: Fix Android build by removing relative includes
i965: Build i965_dri.so for Android
Android.mk
...@freedesktop.org
CC: Chia-I Wu o...@lunarg.com
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/Android.mk | 29 +
1 files changed, 29 insertions(+), 0 deletions(-)
diff --git a/src/mesa/Android.mk b/src/mesa/Android.mk
index 67808d4..5f0f4af 100644
--- a/src
In src/mesa/Android.mk, it is non-trivial to determine which variables are
imported by `include sources.mak`. So document them.
v2
- [olv] Add X86_sources to list of documented variables.
CC: Chia-I Wu o...@lunarg.com
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/Android.mk
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 08/29/2011 08:46 PM, Chia-I Wu wrote:
On Tue, Aug 30, 2011 at 2:43 AM, Chad Versace c...@chad-versace.us wrote:
In order that the Autoconf and Android build can share the same source
lists, move the lists from
src/mesa/drivers/dri
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 08/29/2011 08:59 PM, Chia-I Wu wrote:
On Tue, Aug 30, 2011 at 2:50 AM, Chad Versace c...@chad-versace.us wrote:
Compile tested only.
v2
- [olv] Update comments to include i965 in list of classic drivers.
CC: Ian Romanick i
))
return NULL;
Either way,
Reviewed-by: Chad Versace c...@chad-versace.us
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
iQIcBAEBAgAGBQJOXWfjAAoJEAIvNt057x8ikBUP/33hYl97ZxN/ftSLbPwzwqHW
g28pshV4q2CUe5y8hWngKOjXq
) that the last fixup is equivalent to:
dri2_conf-base.SurfaceType = 0;
If it is, then the code should just set SurfaceType = 0.
Other than that, the patch looks good.
Reviewed-by: Chad Versace c...@chad-versace.us
}
return (count != 0);
-BEGIN PGP SIGNATURE-
Version
eglQuerySurface(EGL_RENDER_BUFFER) will do the right thing.
Reviewed-by: Chad Versace c...@chad-versace.us
- --
Chad Versace
c...@chad-versace.us
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
deletions(-)
Thanks for fixing the mistakes in i965/Android.mk. I was unaware of TARGET_CC.
I tested the i915 build, both with Android and Autoconf. So, for the series:
Reviewed-by: Chad Versace c...@chad-versace.us
--
Chad Versace
c...@chad-versace.us
it to be:
_meas_meta_begin:
line 465) ActiveShader := gl_shader_state::ActiveProgram
_mesa_meta_end:
line 741) gl_shader_state::ActiveProgram := ActiveShader
I discovered this bug by inspection.
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/common/meta.c |2 +-
1
.
Chad Versace (18):
intel: Add HiZ operations to intel_context.vtbl
i965: Initialize intel_context.vtbl after calling intelInitContext
i965: Add stub functions for HiZ operations
i965: Set HiZ operations in brw vtbl
intel: Add HiZ control state to intel_region
intel: Add 'mode' param
Add the following functions:
resolve_depthbuffer
resolve_hizbuffer
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/intel/intel_context.h | 11 +++
1 files changed, 11 insertions(+), 0 deletions(-)
diff --git a/src/mesa/drivers/dri/intel
The intel_context.gen field is set by intelInitContext. So, by calling
intelInitContext before initializing the vtable, we can can construct
different vtables for different gens.
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/i965/brw_context.c |3 ++-
1 files
Add stub functions for the following:
brw_hiz_resolve_depthbuffer
brw_hiz_resolve_hizbuffer
These will later be used to implement:
intel.vtbl.resolve_depthbuffer
intel.vtbl.resolve_hizbuffer
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/i965
If Gen = 6, then set the following:
vtbl.resolve_depthbuffer
vtbl.resolve_hizbuffer.
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/i965/brw_vtbl.c |9 +
1 files changed, 9 insertions(+), 0 deletions(-)
diff --git a/src/mesa/drivers/dri/i965
by intel_renderbuffer::region::hiz::region.
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/intel/intel_regions.c |2 ++
src/mesa/drivers/dri/intel/intel_regions.h | 21 +
2 files changed, 23 insertions(+), 0 deletions(-)
diff --git a/src/mesa/drivers/dri
-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/intel/intel_mipmap_tree.c | 11 +++
src/mesa/drivers/dri/intel/intel_mipmap_tree.h |1 +
src/mesa/drivers/dri/intel/intel_regions.c |6 --
src/mesa/drivers/dri/intel/intel_regions.h |8 ++--
src
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/intel/intel_regions.c | 16
1 files changed, 16 insertions(+), 0 deletions(-)
diff --git a/src/mesa/drivers/dri/intel/intel_regions.c
b/src/mesa/drivers/dri/intel/intel_regions.c
index 0a5c2c1..304de8b
This eliminates the need for intel_fbo.h to include intel_regions.h on the
next commit. And this function was too large to be inline, anyway.
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/intel/intel_fbo.c | 12
src/mesa/drivers/dri/intel/intel_fbo.h
This function allocates the HiZ region and initializes the HiZ control
state for a depthbuffer.
It is wise to define the logic for HiZ region allocation in a single
location, since when MSAA arrives that logic will become more complicated.
Signed-off-by: Chad Versace c...@chad-versace.us
And replace it with intel_renderbuffer::region::hiz::region.
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/i965/brw_misc_state.c | 27 +++--
src/mesa/drivers/dri/intel/intel_context.c | 14 --
src/mesa/drivers/dri/intel/intel_fbo.c
struct brw_context::hiz contains state needed to perform HiZ meta-ops and
indicates if a HiZ operation is currently in progress.
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/i965/brw_context.h | 35 +++
src/mesa/drivers/dri/i965
.
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/i965/brw_hiz.c | 266 ++-
1 files changed, 262 insertions(+), 4 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_hiz.c
b/src/mesa/drivers/dri/i965/brw_hiz.c
index 4e8baa0..8815a26
Before this commit, brw_set_prim() used intel_context.reduced_primitive
and BRW_NEW_REDUCED_PRIMITIVE regardless of generation. This was
confusing, because those bits are not actually used on Gen = 6.
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/i965/brw_draw.c
For example, GL_TRIANLES is converted to _3DPRIM_TRILIST.
The conversion is necessary because HiZ and MSAA resolve operations emit
a 3DPRIM_RECTLIST, which cannot be conveyed by GLenum.
As a consequence, brw_gs_prog_key.primitive is also converted.
Signed-off-by: Chad Versace c...@chad
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/i965/brw_draw.c |3 ++
src/mesa/drivers/dri/i965/gen6_clip_state.c | 17 +++
src/mesa/drivers/dri/i965/gen6_depthstencil.c | 22 +-
src/mesa/drivers/dri/i965/gen6_sf_state.c
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/i965/brw_misc_state.c |6 ++
1 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_misc_state.c
b/src/mesa/drivers/dri/i965/brw_misc_state.c
index 5adf2d7..1121f61 100644
In brw_try_draw_prims(), perform a HiZ resolve on the attached depthbuffer
and perform a depth resolve on each enabled depth texture, if needed.
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/i965/brw_draw.c | 43 ++
1 files changed
From: Kenneth Graunke kenn...@whitecape.org
Signed-off-by: Chad Versace c...@chad-versace.us
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/mesa/drivers/dri/i965/gen7_clip_state.c | 17 ++
src/mesa/drivers/dri/i965/gen7_misc_state.c | 33
And replace it with intel_renderbuffer::region::hiz::region.
v2: In intel_process_dri2_buffer*(), don't call intel_get_renderbuffer() to
get the depthbuffer. The variable 'rb' already points to it.
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/i965
.
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/i965/brw_misc_state.c | 27 +++--
src/mesa/drivers/dri/intel/intel_context.c | 10 +--
src/mesa/drivers/dri/intel/intel_fbo.c | 36 ++-
src/mesa/drivers/dri/intel/intel_fbo.h
.
--
Chad Versace
c...@chad-versace.us
On 09/23/2011 05:37 PM, Chad Versace wrote:
By default, HiZ is still disabled. This series causes no Piglit regressions
when HiZ is disabled.
When HiZ *is* enabled, it is now stable enough to run gnome-shell, nexuiz, and
openarena without any artifacts
.
CC: Paul Berry stereotype...@gmail.com
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/main/version.c | 55 +-
1 files changed, 53 insertions(+), 2 deletions(-)
diff --git a/src/mesa/main/version.c b/src/mesa/main/version.c
index 80fa0c2
MESA_GL_VERSION_OVERRIDE.
v2
---
- [brianp] Change variable name to MESA_GL_VERSION_OVERRIDE.
- [brianp] Document in envvars.html.
- [idr, kwg] Remove overrides for GLES1 and GLES2.
Reviewed-by: Brian Paul bri...@vmware.com
Signed-off-by: Chad Versace c...@chad-versace.us
---
docs/envvars.html
On 09/26/2011 01:57 PM, Brian Paul wrote:
On 09/26/2011 02:50 PM, Chad Versace wrote:
[idr and kwg, I see your point. I've removed the overrides for GLES1 and
GLES2.]
It is necessary to manually set the GL version to 3.0 in order to run
Piglit tests using glGetUniform*().
This patch allows
to execute that op, given that the fundamental object we're tracking
is the depthbuffer's (or texture's) region. So, with that in mind, I can't
think of any other data we may need.
--
Chad Versace
c...@chad-versace.us
___
mesa-dev mailing list
mesa-dev
)) {
+brw-intel.reduced_primitive = brw_get_reduced_prim(hw_prim);
brw-state.dirty.brw |= BRW_NEW_REDUCED_PRIMITIVE;
}
}
-
- return prim_to_hw_prim[mode];
}
--
Chad Versace
c...@chad-versace.us
___
mesa-dev mailing list
if the meta code just set up an empty fragment
shader with its VS, none of the hiz.op checks here would be needed. I
think that would be cleaner.
I agree. I'll test that idea and, if it works, this commit can be deleted.
--
Chad Versace
c...@chad-versace.us
is enabled and the depthbuffer has hiz, then at the point of
emitting a primitive we know that a depth resolve must be needed.
--
Chad Versace
c...@chad-versace.us
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman
is
equivalent to the old. This isn't evident at first glance, and I
first thought that the new was perhaps just a good, stable approximation
of the old.
Reviewed-by: Chad Versace c...@chad-versace.us
--
Chad Versace
c...@chad-versace.us
___
mesa-dev mailing
On 09/27/2011 09:29 AM, Ian Romanick wrote:
On 09/26/2011 01:50 PM, Chad Versace wrote:
[idr and kwg, I see your point. I've removed the overrides for GLES1 and
GLES2.]
It is necessary to manually set the GL version to 3.0 in order to run
Piglit tests using glGetUniform*().
This patch allows
] Remove overrides for GLES1 and GLES2.
Reviewed-by: Brian Paulbri...@vmware.com
Signed-off-by: Chad Versacec...@chad-versace.us
Don't suppose you want to make the GLSL version override more generic as well?
Dave.
Actually, I do. How about MESA_GLSL_VERSION_OVERIDE?
--
Chad Versace
c...@chad
. Since the former
supercedes the latter, this patch disables the latter.
CC: Dave Airlie airl...@gmail.com
Signed-off-by: Chad Versace c...@chad-versace.us
---
docs/envvars.html |4
src/mesa/drivers/dri/intel/intel_extensions.c |3 ++-
src/mesa/drivers/dri
It was replaced by _mesa_override_glsl_version().
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/dri/intel/intel_extensions.c | 16
1 files changed, 0 insertions(+), 16 deletions(-)
diff --git a/src/mesa/drivers/dri/intel/intel_extensions.c
b/src/mesa
than rint(). When
`rint(n) != n`, rint() takes the extra trouble to set FE_INEXACT, but
nearbyint() just does the rounding and nothing else.
Either way, the code is correct.
Reviewed-by: Chad Versace c...@chad-versace.us
--
Chad Versace
c...@chad-versace.us
For pathes 1-6:
Reviewed-by: Chad Versace c...@chad-versace.us
I have a minor comment on patch 7.
--
Chad Versace
c...@chad-versace.us
On 09/27/2011 03:08 PM, Eric Anholt wrote:
For hardware drivers, we only have ir_to_mesa called for the purposes
of potential swrast fallbacks (basically
This patch fixes many Piglit tests [failing due to assert(region-map_refcount
== 0)]
on SNB when HiZ is enabled, and causes no regressions.
Tested-by: Chad Versace c...@chad-versace.us
On 09/27/2011 12:27 PM, Eric Anholt wrote:
From: Brian Paulbri...@vmware.com
Now that we can zero-copy
Remove NeedValidate and ValidateTnlModule.
Signed-off-by: Chad Versace c...@chad-versace.us
---
src/mesa/drivers/common/driverfuncs.c |2 --
src/mesa/main/dd.h| 19 ---
2 files changed, 0 insertions(+), 21 deletions(-)
diff --git a/src/mesa/drivers
?
When I cross-compile or do other pkg-config hocus pocus, I invoke configure as
./configure ... PKG_CONFIG_PATH=/my-libdrm/pkgconfig
- --
Chad Versace
c...@chad-versace.us
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http
201 - 300 of 2646 matches
Mail list logo
