On 02/07/2014 04:43 PM, Brian Paul wrote: > We don't need to allocate all the state related to GL_ARB_debug_output > until some aspect of that extension is actually needed. > > The sizeof(gl_debug_state) is huge (~285KB on 64-bit systems), not even > counting the 54(!) hash tables and lists that it contains. This change > reduces the size of gl_context alone from 431KB bytes to 145KB bytes on > 64-bit systems and from 277KB bytes to 78KB bytes on 32-bit systems. > --- > src/mesa/drivers/dri/common/dri_util.c | 6 +- > src/mesa/main/enable.c | 33 +++- > src/mesa/main/errors.c | 331 > +++++++++++++++++++------------- > src/mesa/main/errors.h | 3 + > src/mesa/main/get.c | 21 ++ > src/mesa/main/get_hash_params.py | 6 +- > src/mesa/main/getstring.c | 16 +- > src/mesa/main/mtypes.h | 2 +- > src/mesa/state_tracker/st_manager.c | 9 +- > 9 files changed, 274 insertions(+), 153 deletions(-)
This looks okay to me. Thanks for doing this! All three patches are: Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>
signature.asc
Description: OpenPGP digital signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev