[lttng-dev] [PATCH 0/6] cleanup and change default mm type

[Mathieu Desnoyers]

* Lai Jiangshan (laijs at cn.fujitsu.com) wrote:
> Patch append here are the Makefile for memory_backend test.
> 
> Lai Jiangshan (6):
>   change default mm
>   disable warning when -DNDEBUG(disable assert())
>   cleanup unneed declare
>   cleanup duplicated code
>   rculfhash,test: add max_hash_buckets_size argument
>   rculfhash,test: add memory_backend argument
> 
>  rculfhash-internal.h   |   22 +++++++++++++++++++
>  rculfhash-mm-chunk.c   |   16 +++----------
>  rculfhash-mm-mmap.c    |   31 ++++++++++++--------------
>  rculfhash-mm-order.c   |   16 ++-----------
>  rculfhash.c            |   23 ++++++++++++-------
>  tests/test_urcu_hash.c |   55 +++++++++++++++++++++++++++++++++++++++++++----
>  urcu/rculfhash.h       |    2 +-
>  7 files changed, 108 insertions(+), 57 deletions(-)


Hi Lai,

Note sure if I should pull the patch below here, in patch #0 ?

> 
> diff --git a/Makefile.am b/Makefile.am
> index 91a9fd1..fdaf929 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -3,7 +3,7 @@ INCLUDES = -I$(top_builddir)/urcu
>  #Add the -version-info directly here since we are only building
>  # library that use the version-info
>  AM_LDFLAGS=-lpthread -version-info $(URCU_LIBRARY_VERSION)
> -AM_CFLAGS=-Wall
> +AM_CFLAGS=-Wall -O2 -DNDEBUG
>  
>  SUBDIRS = . tests
>  
> diff --git a/tests/Makefile.am b/tests/Makefile.am
> index 05a8024..385318d 100644
> --- a/tests/Makefile.am
> +++ b/tests/Makefile.am
> @@ -176,7 +176,7 @@ test_urcu_wfs_dynlink_CFLAGS = -DDYNAMIC_LINK_TEST $(AM_CFLAGS)
>  test_urcu_wfs_dynlink_LDADD = $(URCU_COMMON_LIB)
>  
>  test_urcu_hash_SOURCES = test_urcu_hash.c $(COMPAT)
> -test_urcu_hash_CFLAGS = -DRCU_QSBR $(AM_CFLAGS)
> +test_urcu_hash_CFLAGS = -DRCU_QSBR -O2 $(AM_CFLAGS)
>  test_urcu_hash_LDADD = $(URCU_QSBR_LIB) $(URCU_CDS_LIB)
>  
>  urcutorture.c: api.h
> 
> -- 
> 1.7.4.4
> 
> 
> _______________________________________________
> lttng-dev mailing list
> lttng-dev at lists.lttng.org
> http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
> 

-- 
Mathieu Desnoyers
Operating System Efficiency R&D Consultant
EfficiOS Inc.
http://www.efficios.com