[asterisk-dev] [Code Review] chan_dahdi: create and destroy channels at run-time

[Tzafrir Cohen]

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

(Updated Nov. 30, 2011, 1:08 p.m.)


Review request for Asterisk Developers.


Changes
-------

Fixed most of rmudgett's comments. Also:

* Two separate command-line parameters.
* A global (static) has_pseudo.


Summary
-------

This code adds chan_dahdi the command 'dahdi create channels <range>' (where <range> is a single <n>-<m>) and updates 'dahdi destroy channel' with a similar 'dahdi destroy channels'.

'dahdi create channels' acts as a filter for process_dahdi() and such in DAHDI: it re-parses the whole configuration, but only attempts to create channels in the range.

Right not this patch disables the initial probing at module load time (workaround: run 'dahdi create channels 1-10000' at startup through cli.conf). This is for debugging, mainly and should be removed later on.

This changeg is intended to provide a hook for a script running from udev once a span has been assigned ("registered") / unassigned ("unregistered") for its channels. The udev hook configures the span's channels with dahdi_cfg -S, and can then ask Asterisk to create ethe channels.

See: https://gitorious.org/~tzafrir/asterisk-tools/tzafrirs-dahdi-tools/commits/pinned-spans

This means that a separate DAHDI init script (running before the Asterisk one) would no longer be required. There is no longer a need to run a single command after all DAHDI devices are up.


Diffs (updated)
-----

  /trunk/channels/chan_dahdi.c 346465 

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


Testing
-------

Works well with analog devices. Seems to work with ISDN ones as well. Not fully complete, though.


Thanks,

Tzafrir

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20111130/e14230e9/attachment.htm>