Hi,
@saghul , @aerringer is indeed right - I wasn;t paying too much attached to the
inners of the timer routing. But starting with 2.1, there is no such concept as
timer process. The timer routines are executed in any (ballanced) opensips
process (worker). So using static vars between the
Branch: refs/heads/master
Home: https://github.com/OpenSIPS/opensips
Commit: 76034ad60dfd991232ac3053c07f7a72de234e43
https://github.com/OpenSIPS/opensips/commit/76034ad60dfd991232ac3053c07f7a72de234e43
Author: Razvan Crainea
Date: 2016-01-13 (Wed, 13 Jan
Closed #753 via 76034ad60dfd991232ac3053c07f7a72de234e43.
---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/issues/753#event-513520575___
Devel mailing list
Devel@lists.opensips.org
Branch: refs/heads/2.1
Home: https://github.com/OpenSIPS/opensips
Commit: 5d32b84dadb5879dd47cae644f9b049fe746bf55
https://github.com/OpenSIPS/opensips/commit/5d32b84dadb5879dd47cae644f9b049fe746bf55
Author: Razvan Crainea
Date: 2016-01-13 (Wed, 13 Jan
Hey @bogdan-iancu, thanks for chiming in :-)
> that may simplify your patch a lot - the nat_helper timer routing is
> registered with "TIMER_FLAG_DELAY_ON_DELAY" - this guarantees that (even
> executed in various processes) the handler will not be run in parallel (all
> its executions will be
Branch: refs/heads/2.1
Home: https://github.com/OpenSIPS/opensips
Commit: 5f486bb9c13c369e0ff331517e236e08edabe038
https://github.com/OpenSIPS/opensips/commit/5f486bb9c13c369e0ff331517e236e08edabe038
Author: Bogdan-Andrei Iancu
Date: 2016-01-13 (Wed, 13
It may be related to #723 , The fix will be shortly uploaded on GIT
---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/issues/732#issuecomment-171262503___
Devel mailing list
Devel@lists.opensips.org
Branch: refs/heads/master
Home: https://github.com/OpenSIPS/opensips
Commit: ea6e1fa7fed29efed6057894a8cc4054308160b3
https://github.com/OpenSIPS/opensips/commit/ea6e1fa7fed29efed6057894a8cc4054308160b3
Author: Bogdan-Andrei Iancu
Date: 2016-01-13 (Wed,
Branch: refs/heads/2.1
Home: https://github.com/OpenSIPS/opensips
Commit: f267746b9170a7c21cd18e42420f0ba2bff6137e
https://github.com/OpenSIPS/opensips/commit/f267746b9170a7c21cd18e42420f0ba2bff6137e
Author: Bogdan-Andrei Iancu
Date: 2016-01-13 (Wed, 13
Branch: refs/heads/1.11
Home: https://github.com/OpenSIPS/opensips
Commit: 0dfb46df96d594c9b592d64913a44ab1cccd80e3
https://github.com/OpenSIPS/opensips/commit/0dfb46df96d594c9b592d64913a44ab1cccd80e3
Author: Bogdan-Andrei Iancu
Date: 2016-01-13 (Wed, 13
Branch: refs/heads/master
Home: https://github.com/OpenSIPS/opensips
Commit: 7d090c33e73901b97f12ed6d826b38c1173e61de
https://github.com/OpenSIPS/opensips/commit/7d090c33e73901b97f12ed6d826b38c1173e61de
Author: Bogdan-Andrei Iancu
Date: 2016-01-13 (Wed,
@saghul , I push a fix based on what we discussed above. This will obsolete the
original PR.
@aerringer many thanks for spotting the problem and coming up with the solution
- I think it was a good discussion and outcome !
---
Reply to this email directly or view it on GitHub:
Closed #751.
---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/pull/751#event-513611708___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel
The fix is available on GIT repo. @46labs please update if the fix holds for you
---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/issues/723#issuecomment-171277652___
Devel mailing list
Devel@lists.opensips.org
The fix is available on GIT - please update and let me know if it solves your
problem
---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/issues/732#issuecomment-171277854___
Devel mailing list
Thanks. I will test and update you guys later today.
---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/issues/732#issuecomment-171283133___
Devel mailing list
Devel@lists.opensips.org
Branch: refs/heads/master
Home: https://github.com/OpenSIPS/opensips
Commit: aa5d4831accb4da4e4dca06826c48fbfaafb2d4d
https://github.com/OpenSIPS/opensips/commit/aa5d4831accb4da4e4dca06826c48fbfaafb2d4d
Author: Bogdan-Andrei Iancu
Date: 2016-01-13 (Wed,
Branch: refs/heads/2.1
Home: https://github.com/OpenSIPS/opensips
Commit: 4395575d7f251e14fdc7b0dd92b7f5b329515b4e
https://github.com/OpenSIPS/opensips/commit/4395575d7f251e14fdc7b0dd92b7f5b329515b4e
Author: Bogdan-Andrei Iancu
Date: 2016-01-13 (Wed, 13
@achalkov you already have the patch for fixing #723 - can you confirm that it
is fixing this crash too ?
---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/issues/749#issuecomment-171261627___
Devel mailing
Branch: refs/heads/master
Home: https://github.com/OpenSIPS/opensips
Commit: eef55c9995ac6cab772623efcc284073b5eee360
https://github.com/OpenSIPS/opensips/commit/eef55c9995ac6cab772623efcc284073b5eee360
Author: Liviu Chircu
Date: 2016-01-13 (Wed, 13 Jan
This exists on latest master 055f4b1
Scenario: 2 PUBLISHES with the same E-Tag enter
https://github.com/OpenSIPS/opensips/blob/master/modules/presence/presentity.c#L546
as a result of a retransmission.
The first publish finds the presentity, increments last turn to say 5 for the
sake of
> It continues processing, but it never calls next_turn_phtable before
> exiting.
It technically *does* call next_turn_phtable, but on a *new* presentity in the
phtable. It does not increment the current_turn on the same presentity that it
updated last_turn on. This causes a mismatch on
Branch: refs/heads/2.1
Home: https://github.com/OpenSIPS/opensips
Commit: 4c775f2b63c4e7def59654cd70f30934d8e08e10
https://github.com/OpenSIPS/opensips/commit/4c775f2b63c4e7def59654cd70f30934d8e08e10
Author: ionutrazvanionita
Date: 2016-01-13 (Wed,
Branch: refs/heads/master
Home: https://github.com/OpenSIPS/opensips
Commit: 055f4b132a06f4fdc2c19d6cbae9f0c94114cc24
https://github.com/OpenSIPS/opensips/commit/055f4b132a06f4fdc2c19d6cbae9f0c94114cc24
Author: ionutrazvanionita
Date: 2016-01-13
Branch: refs/heads/1.11
Home: https://github.com/OpenSIPS/opensips
Commit: a533e7279147d6cc5f8ec2aac6260e8b3a4e0cdf
https://github.com/OpenSIPS/opensips/commit/a533e7279147d6cc5f8ec2aac6260e8b3a4e0cdf
Author: ionutrazvanionita
Date: 2016-01-13 (Wed,
i have updated opensips, now we need a couple of days to look at its behavior
---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/issues/749#issuecomment-171322700___
Devel mailing list
Devel@lists.opensips.org
26 matches
Mail list logo