sr-dev February 2009

sr-dev@lists.kamailio.org

13 participants
478 discussions

git:master: config: tcp_async alias for tcp_buf_write
by Jan Janak 27 Feb '09

27 Feb '09

Module: sip-router Branch: master Commit: 69515cab5eefb3b272617fc46565d58e8b392619 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=69515ca… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Thu Feb 26 23:13:59 2009 +0000 config: tcp_async alias for tcp_buf_write - tcp_buf_write was not the best choosen name, so tcp_async was added as an alias for it --- NEWS | 3 ++- cfg.lex | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS b/NEWS index 6318d2a..f3cbfb3 100644 --- a/NEWS +++ b/NEWS @@ -273,9 +273,10 @@ new config variables: will be cached inside the process calling tcp_send (performance increase for sending over tcp at the cost of slightly slower connection closing and extra FDs kept open) - tcp_buf_write = yes | no (default no) - if enabled all the tcp writes that + tcp_async = yes | no (default no) - if enabled all the tcp writes that would block / wait for connect to finish, will be queued and attempted latter (see also tcp_conn_wq_max and tcp_wq_max). + tcp_buf_write = synonim for tcp_async tcp_conn_wq_max = bytes (default 32 K) - maximum bytes queued for write allowed per connection. Attempting to queue more bytes would result in an error and in the connection being closed (too slow). If diff --git a/cfg.lex b/cfg.lex index 0c4ac17..e36a625 100644 --- a/cfg.lex +++ b/cfg.lex @@ -301,7 +301,7 @@ TCP_MAX_CONNECTIONS "tcp_max_connections" TCP_SOURCE_IPV4 "tcp_source_ipv4" TCP_SOURCE_IPV6 "tcp_source_ipv6" TCP_OPT_FD_CACHE "tcp_fd_cache" -TCP_OPT_BUF_WRITE "tcp_buf_write" +TCP_OPT_BUF_WRITE "tcp_buf_write"|"tcp_async" TCP_OPT_CONN_WQ_MAX "tcp_conn_wq_max" TCP_OPT_WQ_MAX "tcp_wq_max" TCP_OPT_DEFER_ACCEPT "tcp_defer_accept"

1 0

git:master: cfg_size macro renamed to cfg_sizeof to avoid a conflict with one of the
by Jan Janak 27 Feb '09

27 Feb '09

Module: sip-router Branch: master Commit: b7cfc0a8139fee47d42597113dc5bd25753cbff5 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=b7cfc0a… Author: Jan Janak <jan(a)iptel.org> Committer: Jan Janak <jan(a)iptel.org> Date: Thu Feb 26 11:39:37 2009 +0000 cfg_size macro renamed to cfg_sizeof to avoid a conflict with one of the functions in libconfuse, this library is used by one of the kamailio modules. Acked by Miklos Tirpak. --- cfg/cfg.h | 2 +- doc/cfg.txt | 2 +- main.c | 2 +- modules/tm/tm.c | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/cfg/cfg.h b/cfg/cfg.h index 21be1aa..f2bd85b 100644 --- a/cfg/cfg.h +++ b/cfg/cfg.h @@ -77,7 +77,7 @@ typedef struct _cfg_def { int cfg_declare(char *group_name, cfg_def_t *def, void *values, int def_size, void **handler); -#define cfg_size(gname) \ +#define cfg_sizeof(gname) \ sizeof(struct cfg_group_##gname) #define cfg_get(gname, handle, var) \ diff --git a/doc/cfg.txt b/doc/cfg.txt index 730f1bf..ddfbec6 100644 --- a/doc/cfg.txt +++ b/doc/cfg.txt @@ -147,7 +147,7 @@ Each row consists of the following items: static int mod_init(void) { - if (cfg_declare("foo", cfg_def, &default_cfg, cfg_size(foo), + if (cfg_declare("foo", cfg_def, &default_cfg, cfg_sizeof(foo), &cfg_handle) ) { /* error */ diff --git a/main.c b/main.c index 81d6149..750694f 100644 --- a/main.c +++ b/main.c @@ -1988,7 +1988,7 @@ try_again: goto error; } /* declare the core cfg before the module configs */ - if (cfg_declare("core", core_cfg_def, &default_core_cfg, cfg_size(core), + if (cfg_declare("core", core_cfg_def, &default_core_cfg, cfg_sizeof(core), &core_cfg) ) { LOG(L_CRIT, "could not declare the core configuration\n"); diff --git a/modules/tm/tm.c b/modules/tm/tm.c index ef8c9af..9a4fea1 100644 --- a/modules/tm/tm.c +++ b/modules/tm/tm.c @@ -707,7 +707,7 @@ static int mod_init(void) #endif /* declare the configuration */ - if (cfg_declare("tm", tm_cfg_def, &default_tm_cfg, cfg_size(tm), + if (cfg_declare("tm", tm_cfg_def, &default_tm_cfg, cfg_sizeof(tm), &tm_cfg)) { LOG(L_ERR, "ERROR: mod_init: failed to declare the configuration\n"); return -1;

1 0

Annotated git configuration file
by Jan Janak 27 Feb '09

27 Feb '09

Hello, There is a new page in the wiki which contains my version of the configuration file for git: http://sip-router.org/wiki/git/gitconfig Some of the options were not so easy to figure out, so I decided to share the config file to make your git learning curve less steep than mine was (and is :-). The file is well commented. Save the configuration file in ~/.gitconfig and set your name and email address to your real name and real email address. After that you get your copy of the sip-router repository using: mkdir my_repo cd my_repo git init git fetch sr You have pulled the whole history when the last command finished. "sr" is a shorthand for the sip-router git repository. As the next step you can check out the master branch, this is where the most recent source code is (it is an equivalent of the trunk in svn or HEAD in cvs): git co -b master sr/master "co" is an alias for checkout, this alias is defined in the configuration file. So now you have your very own copy of the sip-router repository, but that's not all, you can also pull the whole kamailio repository into it: g fetch km "km" is a shorthand for the git mirror of the kamailio svn repository. This command will produce a lot of output (don't be scared) and after it finishes you will have the full history of both projects--sip-router and kamailio--in your local git repository. You can, for example, check out kamailio svn trunk using: git co -b kam_trunk km/trunk And now you have the sources from kamailio trunk in your working directory. You can, of course do the same for SER from CVS, to get the latest sources of ser: git fetch cvs "cvs" is a shorthand for the git import of the ser cvs repository. Now you can switch to cvs trunk using: git co -b cvs_trunk cvs/cvs-head And now you have the latest ser sources in your working directory. And the last repository you can fetch from is the opensips git import: git fetch os git co -b opensips_trunk os/trunk And you have the latest sources from opensips trunk. To display all branches from all repositories run: git branch -a The most important branches are: sr/master - Latest version of sip-router core cvs/cvs-head - Latest version of ser from cvs km/trunk - Kamailio svn trunk km/1.4 - Kamailio branch with release 1.4 os/trunk - Opensips svn trunk Jan.

2 3

git:master: Merge branch 'master' of ssh://janakj@git.sip-router.org/ sip-router
by Jan Janak 26 Feb '09

26 Feb '09

Module: sip-router Branch: master Commit: 90bb3224cc75db8dbaab3c2c21c54dddad544b79 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=90bb322… Author: Jan Janak <jan(a)iptel.org> Committer: Jan Janak <jan(a)iptel.org> Date: Mon Feb 23 22:58:35 2009 +0100 Merge branch 'master' of ssh://janakj@git.sip-router.org/sip-router * 'master' of ssh://janakj@git.sip-router.org/sip-router: tcp: fix compilation problem on solaris (FIONREAD) core: fix bad level name in new LOG() t_check_status() checks also the blind UACs if t_pick_branch() Documenting t_lookup_cancel() script function. Removing set_t() from t_lookup_cancel() function, and introducing updated udp_mtu handling code - fixes SER-433 documenting t_is_expired() function t_is_expired() script function is introduced. * logging API updated (see doc/logging-api.txt for details) - AS support disabled by default. When building the route set of ACKs for local UACs, only the reply is now In case the AVP is a regexp, an allocation is required to build an AVP Currently, SER matches E2E ACKs only if there is an equality between From HF The calculate_routeset_length() produces an invalid result in the case 'memapp' and 'append_mem_block' are now both only used in source "Route :" prefix (and separator) is used some more time across the script: udp_mtu fallback script config & commands core: forward: tcp fallback for big udp packets ---

2 3

git:master: Merge branch 'master' of ssh://janakj@git.sip-router.org/ sip-router
by Jan Janak 26 Feb '09

26 Feb '09

Module: sip-router Branch: master Commit: 665489993dcb8b55970505d07cb7b46e383401ca URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=6654899… Author: Jan Janak <jan(a)iptel.org> Committer: Jan Janak <jan(a)iptel.org> Date: Thu Feb 26 12:46:27 2009 +0100 Merge branch 'master' of ssh://janakj@git.sip-router.org/sip-router * 'master' of ssh://janakj@git.sip-router.org/sip-router: makefiles: fix auto lib re-building script parsing: string switch support script engine: string switch optimization and fixup script engine: string switch execution rvalues: fix rval_new( empty string ) core: sint2str check for space before adding the sign ---

1 0

git:master: Rename cfg_size to cfg_sizeof
by Jan Janak 26 Feb '09

26 Feb '09

Module: sip-router Branch: master Commit: a5ecca6386388a8bf465c1c0cc623db18b120936 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=a5ecca6… Author: Jan Janak <jan(a)iptel.org> Committer: Jan Janak <jan(a)iptel.org> Date: Thu Feb 26 12:52:17 2009 +0100 Rename cfg_size to cfg_sizeof One of kamailio modules uses libconfuse library which happens to contain a function with the same name, and this causes a conflict. We can rename cfg_size to cfg_sizeof to solve the problem. Signed-off-by: Jan Janak <jan(a)iptel.org> Acked-by: Miklos Tirpak <miklos(a)iptel.org> --- cfg/cfg.h | 2 +- doc/cfg.txt | 2 +- main.c | 2 +- modules/tm/tm.c | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/cfg/cfg.h b/cfg/cfg.h index 21be1aa..f2bd85b 100644 --- a/cfg/cfg.h +++ b/cfg/cfg.h @@ -77,7 +77,7 @@ typedef struct _cfg_def { int cfg_declare(char *group_name, cfg_def_t *def, void *values, int def_size, void **handler); -#define cfg_size(gname) \ +#define cfg_sizeof(gname) \ sizeof(struct cfg_group_##gname) #define cfg_get(gname, handle, var) \ diff --git a/doc/cfg.txt b/doc/cfg.txt index 730f1bf..ddfbec6 100644 --- a/doc/cfg.txt +++ b/doc/cfg.txt @@ -147,7 +147,7 @@ Each row consists of the following items: static int mod_init(void) { - if (cfg_declare("foo", cfg_def, &default_cfg, cfg_size(foo), + if (cfg_declare("foo", cfg_def, &default_cfg, cfg_sizeof(foo), &cfg_handle) ) { /* error */ diff --git a/main.c b/main.c index 453f578..b3f567f 100644 --- a/main.c +++ b/main.c @@ -1998,7 +1998,7 @@ try_again: goto error; } /* declare the core cfg before the module configs */ - if (cfg_declare("core", core_cfg_def, &default_core_cfg, cfg_size(core), + if (cfg_declare("core", core_cfg_def, &default_core_cfg, cfg_sizeof(core), &core_cfg) ) { LOG(L_CRIT, "could not declare the core configuration\n"); diff --git a/modules/tm/tm.c b/modules/tm/tm.c index ef8c9af..9a4fea1 100644 --- a/modules/tm/tm.c +++ b/modules/tm/tm.c @@ -707,7 +707,7 @@ static int mod_init(void) #endif /* declare the configuration */ - if (cfg_declare("tm", tm_cfg_def, &default_tm_cfg, cfg_size(tm), + if (cfg_declare("tm", tm_cfg_def, &default_tm_cfg, cfg_sizeof(tm), &tm_cfg)) { LOG(L_ERR, "ERROR: mod_init: failed to declare the configuration\n"); return -1;

1 0

Conversion of kamailio modules
by Jan Janak 25 Feb '09

25 Feb '09

Hello, I started converting kamailio modules yesterday, in the first step I'm trying to see if I can make them compile and load. I haven't tried to verify if they really work. Currently I can compile the following kamailio modules: acc auth auth_db auth_diameter auth_radius alias_db benchmark db_unixodbc carrierroute cfgutils db_berkeley db_flatstore db_mysql db_oracle db_postgres db_text db_unixodbc diversion exec h350 maxfwd misc_radius mi_xmlrpc pdt peering permissions pike regex rr sanity siputils speeddial sqlops statistics uri_db userblacklist xcap_client xlog There is 93 modules in kamailio repository, that means I can currently compile about 40% of them on the sip-router core. Most of the modules above required some manual changes, but not exteremely complicated. The biggest issue is extra stuff in kamalio core which is missing (or present on other modules) in the sip-router core. I created a new shared library called libkcore and whenever I find that something is missing in sip-router, I take it from kamailio and put it in the library. It's not that nice because there will be duplicated code, but I think we should first try to make everything compile and then we can think about how to organize the code better. Jan.

3 4

md5 wrappers
by Jan Janak 25 Feb '09

25 Feb '09

Hello, I was wondering what is the purpose of wrappers like this one: /*! * \brief Small wrapper around MD5Update * * Small wrapper around MD5Update, because everybody uses this on 'str' types * \param context MD5 context * \param input input block * \param inputLen length of input block * \note please not use this in new code * \todo review and fix all wrong usage */ static inline void MD5Update (MD5_CTX *context, char *input, unsigned int inputLen) { return U_MD5Update(context, (unsigned char *)input, inputLen); } And this is U_MD5Update: void U_MD5Update (MD5_CTX *context, unsigned char *input, unsigned int inputLen) Jan.

2 1

git:master: makefiles: fix auto lib re-building
by Andrei Pelinescu-Onciul 24 Feb '09

24 Feb '09

Module: sip-router Branch: master Commit: 19c546113aa78e454ec0f995550fde51828fce81 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=19c5461… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Tue Feb 24 21:04:21 2009 +0100 makefiles: fix auto lib re-building - libs used by modules where rebuilt too often (more then once during a make all) as a result of different DEFS for different modules (because of the different module interfaces and the new -DMOD_NAME). --- Makefile.rules | 22 +++++++++++++++++----- 1 files changed, 17 insertions(+), 5 deletions(-) diff --git a/Makefile.rules b/Makefile.rules index ea2190a..b03342f 100644 --- a/Makefile.rules +++ b/Makefile.rules @@ -27,12 +27,20 @@ # (if not rebuild everything) ifeq (,$(filter $(nodep_targets),$(MAKECMDGOALS))) -include makecfg.lst -ifneq ($(strip $(DEFS)), $(strip $(CFG_DEFS))) +# if trying to build a lib automatically and the lib is already compiled, +# don't rebuild it if the only differences in DEFS or INCLUDES are covered +# by LIB_NOREBUILD_DEFS/LIB_NOREBUILD_INCLUDES +LIB_NOREBUILD_DEFS+= -DMOD_NAME% -D%_MOD_INTERFACE -DMOD_INTERFACE_% -DSR_% + +# don't rebuild if the differences are covered by NOREBUILD_DEFS or +# NOREBUILD_INCLUDES +ifneq ($(filter-out $(NOREBUILD_DEFS),$(strip $(DEFS))), $(strip $(CFG_DEFS))) #$(warning different defs: <$(strip $(DEFS))> != ) #$(warning : <$(strip $(CFG_DEFS))>) $(shell rm -f makecfg.lst) endif -ifneq ($(strip $(INCLUDES)), $(strip $(CFG_INCLUDES))) +ifneq ($(filter-out $(NOREBUILD_INCLUDES), $(strip $(INCLUDES))),\ + $(strip $(CFG_INCLUDES))) $(shell rm -f makecfg.lst) endif endif @@ -95,7 +103,9 @@ ALL_LIBS+=$(foreach l, $(SER_LIBS), -L$(dir $l) -l$(notdir $l)) $(NAME): librpath.lst $(SER_LIBS_DEPS) $(SER_LIBS_DEPS): FORCE - $(MAKE) -wC $(dir $@) compile_for_install=$(lib_compile_for_install) + @$(MAKE) -wC $(dir $@) compile_for_install=$(lib_compile_for_install) \ + NOREBUILD_DEFS="$(NOREBUILD_DEFS) $(LIB_NOREBUILD_DEFS)" \ + NOREBUILD_INCLUDES="$(NOREBUILD_INCLUDES) $(LIB_NOREBUILD_INCLUDES)" .PHONY: FORCE FORCE: @@ -134,8 +144,10 @@ librpath.lst: $(ALLDEP) @echo LIB_RPATH_LST:=$(SER_RPATH_LST) >librpath.lst makecfg.lst: - @echo CFG_DEFS:=$(subst ',\', $(subst ",\", $(strip $(DEFS)))) >>$@ - @echo CFG_INCLUDES:=$(subst ',\', $(subst ",\", $(strip $(INCLUDES)))) >>$@ + @echo CFG_DEFS:=$(subst ',\', $(subst ",\", \ + $(filter-out $(NOREBUILD_DEFS), $(strip $(DEFS))))) >>$@ + @echo CFG_INCLUDES:=$(subst ',\', $(subst ",\", \ + $(filter-out $(NOREBUILD_INCLUDES), $(strip $(INCLUDES))))) >>$@ .PHONY: all all: $(NAME) modules

1 0

git:master: Merge commit 'origin/andrei/switch'
by Andrei Pelinescu-Onciul 24 Feb '09

24 Feb '09

Module: sip-router Branch: master Commit: 0c150104190692ce367cebd366f26b9ed25da6ee URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=0c15010… Author: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Committer: Andrei Pelinescu-Onciul <andrei(a)iptel.org> Date: Fri Feb 20 18:20:20 2009 +0100 Merge commit 'origin/andrei/switch' * commit 'origin/andrei/switch': script parsing: string switch support script engine: string switch optimization and fixup script engine: string switch execution rvalues: fix rval_new( empty string ) core: sint2str check for space before adding the sign ---

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev February 2009