diff options
author | Joshua Colp <jcolp@digium.com> | 2015-03-10 23:09:49 +0000 |
---|---|---|
committer | Joshua Colp <jcolp@digium.com> | 2015-03-10 23:09:49 +0000 |
commit | 2889f074a0a2ce83481fd89a0039bf85a96b6f84 (patch) | |
tree | d44ea4fb3181229c9e63a99e269c843c2903aa8a /main/stasis_channels.c | |
parent | 15d266bf850ee62cc7708587cde6fd53a3c35be6 (diff) |
core: Don't create snapshots with locks.
Snapshots are immutable and are never changed. Allocating them
with a lock is wasteful.
Review: https://reviewboard.asterisk.org/r/4469/
........
Merged revisions 432742 from http://svn.asterisk.org/svn/asterisk/branches/13
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@432743 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'main/stasis_channels.c')
-rw-r--r-- | main/stasis_channels.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/main/stasis_channels.c b/main/stasis_channels.c index 6b3f76592..85abe58bc 100644 --- a/main/stasis_channels.c +++ b/main/stasis_channels.c @@ -209,7 +209,8 @@ struct ast_channel_snapshot *ast_channel_snapshot_create(struct ast_channel *cha return NULL; } - snapshot = ao2_alloc(sizeof(*snapshot), channel_snapshot_dtor); + snapshot = ao2_alloc_options(sizeof(*snapshot), channel_snapshot_dtor, + AO2_ALLOC_OPT_LOCK_NOLOCK); if (!snapshot || ast_string_field_init(snapshot, 1024)) { ao2_cleanup(snapshot); return NULL; @@ -617,7 +618,8 @@ void ast_multi_channel_blob_add_channel(struct ast_multi_channel_blob *obj, cons return; } - role_snapshot = ao2_alloc(sizeof(*role_snapshot) + role_len, channel_role_snapshot_dtor); + role_snapshot = ao2_alloc_options(sizeof(*role_snapshot) + role_len, channel_role_snapshot_dtor, + AO2_ALLOC_OPT_LOCK_NOLOCK); if (!role_snapshot) { return; } |