Summary: | ASTERISK-03838: [patch] Don't hold channel list lock when it's not needed | ||
Reporter: | Paul Cadach (pcadach) | Labels: | |
Date Opened: | 2005-04-01 14:37:53.000-0600 | Date Closed: | 2008-01-15 15:30:00.000-0600 |
Priority: | Trivial | Regression? | No |
Status: | Closed/Complete | Components: | Core/General |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) shortlock.diff | |
Description: | Very short patch which modifies locking scheme on ast_channel_alloc() to not hold lock for channel list until new channel structure is including into the list. ****** ADDITIONAL INFORMATION ****** Disclaimer is on file. | ||
Comments: | By: Kevin P. Fleming (kpfleming) 2005-04-01 15:45:11.000-0600 Committed to CVS, nice catch, thanks! By: Russell Bryant (russell) 2005-04-05 01:47:00 not included in 1.0 since it's more of an optimization than a bug, and I'd have to re-write it to get it in, anyway By: Digium Subversion (svnbot) 2008-01-15 15:30:00.000-0600 Repository: asterisk Revision: 5352 U trunk/channel.c ------------------------------------------------------------------------ r5352 | kpfleming | 2008-01-15 15:29:59 -0600 (Tue, 15 Jan 2008) | 2 lines don't take the channel list lock until we're actually ready to put the new channel into the list in ast_channel_alloc (bug ASTERISK-3838) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=5352 |