sr-dev November 2012

sr-dev@lists.kamailio.org

28 participants
122 discussions

presence module makes excessive cleanups when subs_db_mode=3
by Andrew Pogrebennyk 09 Nov '12

09 Nov '12

Hi, when presence module is running with subs_db_mode=3 it makes an excessive number of SQL select queries, litelly dozens per second: > 25232 Query select presentity_uri,callid,to_tag,from_tag,event from active_watchers where updated=64 AND event<>'presence.winfo' > 25233 Query select presentity_uri,callid,to_tag,from_tag,event from active_watchers where updated=116 AND event<>'presence.winfo' > 25233 Query select presentity_uri,callid,to_tag,from_tag,event from active_watchers where updated=116 AND event='presence.winfo' > 25232 Query select presentity_uri,callid,to_tag,from_tag,event from active_watchers where updated=64 AND event='presence.winfo' > 25231 Query select presentity_uri,callid,to_tag,from_tag,event from active_watchers where updated=14 AND event<>'presence.winfo' > 25231 Query select presentity_uri,callid,to_tag,from_tag,event from active_watchers where updated=14 AND event='presence.winfo' > 25233 Query select presentity_uri,callid,to_tag,from_tag,event from active_watchers where updated=118 AND event<>'presence.winfo' > 25232 Query select presentity_uri,callid,to_tag,from_tag,event from active_watchers where updated=66 AND event<>'presence.winfo' > 25233 Query select presentity_uri,callid,to_tag,from_tag,event from active_watchers where updated=118 AND event='presence.winfo' > 25232 Query select presentity_uri,callid,to_tag,from_tag,event from active_watchers where updated=66 AND event='presence.winfo' > 25231 Query select presentity_uri,callid,to_tag,from_tag,event from active_watchers where updated=16 AND event<>'presence.winfo' > 25231 Query select presentity_uri,callid,to_tag,from_tag,event from active_watchers where updated=16 AND event='presence.winfo' Steps to reproduce: registers two subscribers in jitsi and add them to contact lists of each other. The kamailio version is 3.3.2. I'm not doing anything special configuration-wise: loadmodule "presence.so" modparam("presence", "db_url", "mysql://kamailio:snbF93@localhost/kamailio") # in 3.3 the fallback2db change to subs_db_mode modparam("presence", "subs_db_mode", 3) modparam("presence", "notifier_processes", 3) loadmodule "presence_xml.so" modparam("presence_xml", "db_url", "mysql://kamailio:snbF93@localhost/kamailio") modparam("presence_xml", "force_active", 0) modparam("presence_xml", "integrated_xcap_server", 1) # retry-after 5 minutes modparam("presence_xml", "xcapauth_userdel_reason", "probation;retry-after=300") Q: it is a bug of a feature? :) Thanks. Andrew

2 2

reg event(rfc 3680) support to Kamailio server
by Ram Anji 09 Nov '12

09 Nov '12

Hi, I want to add Reg event support to my kamailio server which is having RLS support. I am added following lines to support reg event package modparam("pua_reginfo", "default_domain", "kamailio.org") modparam("pua_reginfo", "outbound_proxy", "sip:kamailio.org:5060") modparam("pua_reginfo", "server_address", "sip:kamailio.org:5060") But my client is getting 489 bad event for reg event subscribe request. with the *Allow-Events:* message-summary, presence.winfo, presence Please suggest me the required changes to add reg event support to the server. Regards, Raman

1 0

git:master: pkg/kamailio/(centos|fedora): Updated rel in .spec file
by Peter Dunkley 09 Nov '12

09 Nov '12

Module: sip-router Branch: master Commit: 6d2015a050f9d0bee80058e46732dd8da577c4d8 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=6d2015a… Author: Peter Dunkley <peter.dunkley(a)crocodile-rcs.com> Committer: Peter Dunkley <peter.dunkley(a)crocodile-rcs.com> Date: Fri Nov 9 10:43:33 2012 +0000 pkg/kamailio/(centos|fedora): Updated rel in .spec file --- pkg/kamailio/fedora/16/kamailio.spec | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/pkg/kamailio/fedora/16/kamailio.spec b/pkg/kamailio/fedora/16/kamailio.spec index 4529ee5..c365c06 100644 --- a/pkg/kamailio/fedora/16/kamailio.spec +++ b/pkg/kamailio/fedora/16/kamailio.spec @@ -1,6 +1,6 @@ %define name kamailio %define ver 3.4.0 -%define rel dev4%{dist} +%define rel dev5%{dist} @@ -990,6 +990,8 @@ fi %changelog +* Fri Nov 9 2012 Peter Dunkley <peter(a)dunkley.me.uk> + - Updated rel to dev5 * Tue Oct 30 2012 Peter Dunkley <peter(a)dunkley.me.uk> - Added xhttp_pi module to RPM builds * Fri Oct 20 2012 Peter Dunkley <peter(a)dunkley.me.uk>

1 0

git:master: modules_k/xcap_server: fixed length of pres-content AUID
by Jason Penton 09 Nov '12

09 Nov '12

Module: sip-router Branch: master Commit: f25202a6d1a8713524985f83304230556c036da7 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=f25202a… Author: Jason Penton <jason.penton(a)smilecoms.com> Committer: Jason Penton <jason.penton(a)smilecoms.com> Date: Fri Nov 9 12:28:19 2012 +0200 modules_k/xcap_server: fixed length of pres-content AUID --- modules_k/xcap_server/xcap_misc.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/modules_k/xcap_server/xcap_misc.c b/modules_k/xcap_server/xcap_misc.c index d817c1c..540e190 100644 --- a/modules_k/xcap_server/xcap_misc.c +++ b/modules_k/xcap_server/xcap_misc.c @@ -56,7 +56,7 @@ xcaps_auid_list_t xcaps_auid_list[] = { '/', XCAP_CAPS }, { { "org.openmobilealliance.user-profile", 35}, '/', USER_PROFILE }, - { { "org.openmobilealliance.pres-content", 15}, + { { "org.openmobilealliance.pres-content", 35}, '/', PRES_CONTENT }, { { "org.openmobilealliance.search", 29}, '?', SEARCH },

1 0

git:master: core: tcp - proper handling of '\n-' sequence for detecting end of MSRP frame
by Daniel-Constantin Mierla 09 Nov '12

09 Nov '12

Module: sip-router Branch: master Commit: b255c406ba09ff229361a3bc14cc1c9fd076b52e URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=b255c40… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Fri Nov 9 11:03:46 2012 +0100 core: tcp - proper handling of '\n-' sequence for detecting end of MSRP frame - in some cases it could go in wrong reading MSRP state - reported by Gavin Llewellyn --- tcp_read.c | 12 ++++++++---- 1 files changed, 8 insertions(+), 4 deletions(-) diff --git a/tcp_read.c b/tcp_read.c index 1c80c19..b969e6c 100644 --- a/tcp_read.c +++ b/tcp_read.c @@ -934,14 +934,16 @@ int tcp_read_headers(struct tcp_connection *c, int* read_flags) if(r->pos - r->start < 10) { LM_ERR("weird situation when reading MSRP frame" " - continue reading\n"); + /* *p=='\n' */ + r->state=H_MSRP_BODY_LF; p++; - r->state=H_MSRP_BODY; break; } if(*(p-1)!='\r') { /* not ending in '\r\n' - not end-line */ + /* *p=='\n' */ + r->state=H_MSRP_BODY_LF; p++; - r->state=H_MSRP_BODY; break; } /* locate transaction id in first line @@ -956,16 +958,18 @@ int tcp_read_headers(struct tcp_connection *c, int* read_flags) p - 1 /*\r*/ - 1 /* '+'|'#'|'$' */ - mtransid.len, mtransid.len)!=0) { /* no match on session id - not end-line */ + /* *p=='\n' */ + r->state=H_MSRP_BODY_LF; p++; - r->state=H_MSRP_BODY; break; } if(memcmp(p - 1 /*\r*/ - 1 /* '+'|'#'|'$' */ - mtransid.len - 7 /* 7 x '-' */ - 1 /* '\n' */, "\n-------", 8)!=0) { /* no match on "\n-------" - not end-line */ + /* *p=='\n' */ + r->state=H_MSRP_BODY_LF; p++; - r->state=H_MSRP_BODY; break; } r->state=H_MSRP_FINISH;

1 0

git:3.3: dialog(k): use proper scan string for optional headers parameter in rpc commands
by Daniel-Constantin Mierla 09 Nov '12

09 Nov '12

Module: sip-router Branch: 3.3 Commit: 78d4b145c57e5e93b70f7441ec2c7e63a2f9f2c0 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=78d4b14… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Fri Nov 9 08:41:55 2012 +0100 dialog(k): use proper scan string for optional headers parameter in rpc commands - credits to Kristofer Signer for report and troubleshooting (cherry picked from commit d5120187eb41e709c840b3cfbb0dc1803f61b036) --- modules_k/dialog/dialog.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/modules_k/dialog/dialog.c b/modules_k/dialog/dialog.c index 3a404d1..e25890d 100644 --- a/modules_k/dialog/dialog.c +++ b/modules_k/dialog/dialog.c @@ -1585,7 +1585,7 @@ static void rpc_end_dlg_entry_id(rpc_t *rpc, void *c) { dlg_cell_t * dlg = NULL; str rpc_extra_hdrs = {NULL,0}; - if (rpc->scan(c, "ddS", &h_entry, &h_id, &rpc_extra_hdrs) < 2) return; + if (rpc->scan(c, "dd*S", &h_entry, &h_id, &rpc_extra_hdrs) < 2) return; dlg = dlg_lookup(h_entry, h_id); if(dlg){ @@ -1622,7 +1622,7 @@ static void rpc_dlg_bridge(rpc_t *rpc, void *c) { str to = {NULL,0}; str op = {NULL,0}; - if (rpc->scan(c, "SSS", &from, &to, &op) < 2) return; + if (rpc->scan(c, "SS*S", &from, &to, &op) < 2) return; dlg_bridge(&from, &to, &op); }

1 0

git:3.3: Makefile.defs: arm compiler flags update
by Daniel-Constantin Mierla 09 Nov '12

09 Nov '12

Module: sip-router Branch: 3.3 Commit: f860ed61a19a1119ff6612fa6b3ebdfa6dc2f0bd URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=f860ed6… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Fri Nov 9 08:07:42 2012 +0100 Makefile.defs: arm compiler flags update - patch by Matthias Klose <doko(a)debian.org>, http://bugs.debian.org/cgi-bin/bugreport.cgi?msg=5;bug=690388 (cherry picked from commit e916a4680d37f481167770fff27a710088cb5d3d) --- Makefile.defs | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/Makefile.defs b/Makefile.defs index 8957769..a1854a3 100644 --- a/Makefile.defs +++ b/Makefile.defs @@ -1186,7 +1186,7 @@ ifeq ($(ARCH), arm) ifeq ($(CC_NAME), gcc) C_DEFS+=-DCC_GCC_LIKE_ASM #common stuff - CFLAGS=-O9 -funroll-loops -fsigned-char $(PROFILE) + CFLAGS=-marm -march=armv5t -O9 -funroll-loops -fsigned-char $(PROFILE) #if gcc 4.5+ or 4.2+ ifeq (,$(strip $(filter-out 4.2+ 4.5+,$(CC_SHORTVER)))) CFLAGS+= -ftree-vectorize -fno-strict-overflow

1 0

git:master: Makefile.defs: version set to 3.4.0-dev5
by Daniel-Constantin Mierla 09 Nov '12

09 Nov '12

Module: sip-router Branch: master Commit: ddac5a16da60b0bbb6f58aa9e4fd26db3015c10d URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=ddac5a1… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Fri Nov 9 08:18:06 2012 +0100 Makefile.defs: version set to 3.4.0-dev5 --- Makefile.defs | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/Makefile.defs b/Makefile.defs index 04064fc..c3d5e54 100644 --- a/Makefile.defs +++ b/Makefile.defs @@ -162,7 +162,7 @@ INSTALL_FLAVOUR=$(FLAVOUR) VERSION = 3 PATCHLEVEL = 4 SUBLEVEL = 0 -EXTRAVERSION = -dev4 +EXTRAVERSION = -dev5 # memory debugger switcher # 0 - off (release mode)

1 0

git:master: dialog(k): use proper scan string for optional headers parameter in rpc commands
by Daniel-Constantin Mierla 09 Nov '12

09 Nov '12

Module: sip-router Branch: master Commit: d5120187eb41e709c840b3cfbb0dc1803f61b036 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=d512018… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Fri Nov 9 08:41:55 2012 +0100 dialog(k): use proper scan string for optional headers parameter in rpc commands - credits to Kristofer Signer for report and troubleshooting --- modules_k/dialog/dialog.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/modules_k/dialog/dialog.c b/modules_k/dialog/dialog.c index 4d7af53..a22576e 100644 --- a/modules_k/dialog/dialog.c +++ b/modules_k/dialog/dialog.c @@ -1587,7 +1587,7 @@ static void rpc_end_dlg_entry_id(rpc_t *rpc, void *c) { dlg_cell_t * dlg = NULL; str rpc_extra_hdrs = {NULL,0}; - if (rpc->scan(c, "ddS", &h_entry, &h_id, &rpc_extra_hdrs) < 2) return; + if (rpc->scan(c, "dd*S", &h_entry, &h_id, &rpc_extra_hdrs) < 2) return; dlg = dlg_lookup(h_entry, h_id); if(dlg){ @@ -1624,7 +1624,7 @@ static void rpc_dlg_bridge(rpc_t *rpc, void *c) { str to = {NULL,0}; str op = {NULL,0}; - if (rpc->scan(c, "SSS", &from, &to, &op) < 2) return; + if (rpc->scan(c, "SS*S", &from, &to, &op) < 2) return; dlg_bridge(&from, &to, &op); }

1 0

git:master: Makefile.defs: arm compiler flags update
by Daniel-Constantin Mierla 09 Nov '12

09 Nov '12

Module: sip-router Branch: master Commit: e916a4680d37f481167770fff27a710088cb5d3d URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=e916a46… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Fri Nov 9 08:07:42 2012 +0100 Makefile.defs: arm compiler flags update - patch by Matthias Klose <doko(a)debian.org>, http://bugs.debian.org/cgi-bin/bugreport.cgi?msg=5;bug=690388 --- Makefile.defs | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/Makefile.defs b/Makefile.defs index 039ca33..04064fc 100644 --- a/Makefile.defs +++ b/Makefile.defs @@ -1212,7 +1212,7 @@ ifeq ($(ARCH), arm) ifeq ($(CC_NAME), gcc) C_DEFS+=-DCC_GCC_LIKE_ASM #common stuff - CFLAGS=-O9 -funroll-loops -fsigned-char $(PROFILE) + CFLAGS=-marm -march=armv5t -O9 -funroll-loops -fsigned-char $(PROFILE) #if gcc 4.5+ or 4.2+ ifeq (,$(strip $(filter-out 4.2+ 4.5+,$(CC_SHORTVER)))) CFLAGS+= -ftree-vectorize -fno-strict-overflow

1 0

← Newer
1
...
7
8
9
10
11
12
13
Older →

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev November 2012