[Asterisk-code-review] cdr.c: Defer getting ao2 global obj ref() until needed. (asterisk[15])

Richard Mudgett asteriskteam at digium.com
Thu Oct 12 19:12:49 CDT 2017


Hello Jenkins2, Corey Farrell, 

I'd like you to reexamine a change. Please visit

    https://gerrit.asterisk.org/6778

to look at the new patch set (#2).

Change subject: cdr.c: Defer getting ao2_global_obj_ref() until needed.
......................................................................

cdr.c: Defer getting ao2_global_obj_ref() until needed.

The CDR performance gets worse the further it gets behind in processing
stasis messages.  One of the reasons is we were getting the global config
even if we didn't need it.

* Most uses of the global config were only needed on off nominal code
paths so it makes sense to not get it until absolutely needed.

ASTERISK-27335

Change-Id: I00c63b7ec233e5bfffd5d976f05568613d3c2365
---
M main/cdr.c
1 file changed, 40 insertions(+), 41 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/78/6778/2
-- 
To view, visit https://gerrit.asterisk.org/6778
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 15
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I00c63b7ec233e5bfffd5d976f05568613d3c2365
Gerrit-Change-Number: 6778
Gerrit-PatchSet: 2
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20171012/b3108fc9/attachment.html>


More information about the asterisk-code-review mailing list