[asterisk-dev] [Code Review] Threadpool support: Taskprocessor changes and tests

[Mark Michelson]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2200/
-----------------------------------------------------------

(Updated Nov. 19, 2012, 4:39 p.m.)


Review request for Asterisk Developers.


Changes
-------

Trailing whitespace removed.


Summary
-------

For the new SIP channel driver, we would really like to use a thread pool to handle the tasks. In order to do this correctly, I am designing a general-purpose thread pool that can be used for SIP or for anything else in Asterisk if desired. I am basing the design of the thread pool off of the thread pool used in Asterisk SCF.

The first set of changes is to essentially gut main/taskprocessor.c. Taskprocessors in Asterisk are currently limited to managing a single thread of execution. The set of changes provided here introduces a listener model. When a taskprocessor's state changes (e.g. a task is added, the queue becomes empty) the listener is notified. The listener may then take whatever action it desires. If the listener wants to execute all tasks in a single thread, it can. If it would rather allow for multiple threads to execute threads, it can facilitate that instead.

To prove the concept, a default taskprocessor listener was created that behaves exactly like taskprocessors always have. Unit tests are included in this patch set that show that the default taskprocessor listener behaves as expected. A unit test has also been added to ensure that listener callbacks are called when expected and that the data provided in the callbacks is accurate.

In an effort to not cause far-reaching changes, I have kept the API for taskprocessors backwards-compatible. Honestly, I'm not a big fan of current method of shutting down a taskprocessor, but it works nonetheless.


This addresses bug ASTERISK-20691.
    https://issues.asterisk.org/jira/browse/ASTERISK-20691


Diffs (updated)
-----

  /trunk/include/asterisk/taskprocessor.h 376499 
  /trunk/main/astobj2.c 376499 
  /trunk/main/taskprocessor.c 376499 
  /trunk/tests/test_taskprocessor.c PRE-CREATION 

Diff: https://reviewboard.asterisk.org/r/2200/diff


Testing
-------

In addition to the unit tests provided, I also ran calls to be sure that mechanisms in Asterisk that use taskprocessors still functioned as expected. I verified that the event system and that the PBX function as expected.


Thanks,

Mark

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20121119/12c634a3/attachment-0001.htm>