[asterisk-commits] kpfleming: trunk r96245 - /trunk/channels/chan_alsa.c
SVN commits to the Asterisk project
asterisk-commits at lists.digium.com
Thu Jan 3 12:10:14 CST 2008
Author: kpfleming
Date: Thu Jan 3 12:10:13 2008
New Revision: 96245
URL: http://svn.digium.com/view/asterisk?view=rev&rev=96245
Log:
coding guidelines cleanup
remove background thread and all sound generation mechanisms, as the built-in indications can handle everything that is needed
Modified:
trunk/channels/chan_alsa.c
Modified: trunk/channels/chan_alsa.c
URL: http://svn.digium.com/view/asterisk/trunk/channels/chan_alsa.c?view=diff&rev=96245&r1=96244&r2=96245
==============================================================================
--- trunk/channels/chan_alsa.c (original)
+++ trunk/channels/chan_alsa.c Thu Jan 3 12:10:13 2008
@@ -56,15 +56,6 @@
#include "asterisk/abstract_jb.h"
#include "asterisk/musiconhold.h"
-#include "busy.h"
-#include "ringtone.h"
-#include "ring10.h"
-#include "answer.h"
-
-#ifdef ALSA_MONITOR
-#include "alsa-monitor.h"
-#endif
-
/*! Global jitterbuffer configuration - by default, jb is disabled */
static struct ast_jb_conf default_jbconf = {
.flags = 0,
@@ -102,10 +93,6 @@
static char indevname[50] = ALSA_INDEV;
static char outdevname[50] = ALSA_OUTDEV;
-#if 0
-static struct timeval lasttime;
-#endif
-
static int silencesuppression = 0;
static int silencethreshold = 1000;
@@ -120,28 +107,6 @@
static char mohinterpret[MAX_MUSICCLASS];
static int hookstate = 0;
-
-static short silence[FRAME_SIZE] = { 0, };
-
-struct sound {
- int ind;
- short *data;
- int datalen;
- int samplen;
- int silencelen;
- int repeat;
-};
-
-static struct sound sounds[] = {
- {AST_CONTROL_RINGING, ringtone, sizeof(ringtone) / 2, 16000, 32000, 1},
- {AST_CONTROL_BUSY, busy, sizeof(busy) / 2, 4000, 4000, 1},
- {AST_CONTROL_CONGESTION, busy, sizeof(busy) / 2, 2000, 2000, 1},
- {AST_CONTROL_RING, ring10, sizeof(ring10) / 2, 16000, 32000, 1},
- {AST_CONTROL_ANSWER, answer, sizeof(answer) / 2, 2200, 0, 0},
-};
-
-/* Sound command pipe */
-static int sndcmd[2];
static struct chan_alsa_pvt {
/* We only have one ALSA structure -- near sighted perhaps, but it
@@ -149,9 +114,6 @@
struct ast_channel *owner;
char exten[AST_MAX_EXTENSION];
char context[AST_MAX_CONTEXT];
-#if 0
- snd_pcm_t *card;
-#endif
snd_pcm_t *icard, *ocard;
} alsa;
@@ -160,8 +122,6 @@
with 160 sample frames, and a buffer size of 3, we have a 60ms buffer,
usually plenty. */
-pthread_t sthread;
-
#define MAX_BUFFER_SIZE 100
/* File descriptors for sound device */
@@ -170,13 +130,6 @@
static int autoanswer = 1;
-static int cursound = -1;
-static int sampsent = 0;
-static int silencelen = 0;
-static int offset = 0;
-static int nosound = 0;
-
-/* ZZ */
static struct ast_channel *alsa_request(const char *type, int format, void *data, int *cause);
static int alsa_digit(struct ast_channel *c, char digit, unsigned int duration);
static int alsa_text(struct ast_channel *c, const char *text);
@@ -204,136 +157,6 @@
.fixup = alsa_fixup,
};
-static int send_sound(void)
-{
- short myframe[FRAME_SIZE];
- int total = FRAME_SIZE;
- short *frame = NULL;
- int amt = 0, res, myoff;
- snd_pcm_state_t state;
-
- if (cursound == -1)
- return 0;
-
- res = total;
- if (sampsent < sounds[cursound].samplen) {
- myoff = 0;
- while (total) {
- amt = total;
- if (amt > (sounds[cursound].datalen - offset))
- amt = sounds[cursound].datalen - offset;
- memcpy(myframe + myoff, sounds[cursound].data + offset, amt * 2);
- total -= amt;
- offset += amt;
- sampsent += amt;
- myoff += amt;
- if (offset >= sounds[cursound].datalen)
- offset = 0;
- }
- /* Set it up for silence */
- if (sampsent >= sounds[cursound].samplen)
- silencelen = sounds[cursound].silencelen;
- frame = myframe;
- } else {
- if (silencelen > 0) {
- frame = silence;
- silencelen -= res;
- } else {
- if (sounds[cursound].repeat) {
- /* Start over */
- sampsent = 0;
- offset = 0;
- } else {
- cursound = -1;
- nosound = 0;
- }
- return 0;
- }
- }
-
- if (res == 0 || !frame)
- return 0;
-
-#ifdef ALSA_MONITOR
- alsa_monitor_write((char *) frame, res * 2);
-#endif
- state = snd_pcm_state(alsa.ocard);
- if (state == SND_PCM_STATE_XRUN)
- snd_pcm_prepare(alsa.ocard);
- res = snd_pcm_writei(alsa.ocard, frame, res);
- if (res > 0)
- return 0;
- return 0;
-}
-
-static void *sound_thread(void *unused)
-{
- fd_set rfds;
- fd_set wfds;
- int max, res;
-
- for (;;) {
- FD_ZERO(&rfds);
- FD_ZERO(&wfds);
- max = sndcmd[0];
- FD_SET(sndcmd[0], &rfds);
- if (cursound > -1) {
- FD_SET(writedev, &wfds);
- if (writedev > max)
- max = writedev;
- }
-#ifdef ALSA_MONITOR
- if (!alsa.owner) {
- FD_SET(readdev, &rfds);
- if (readdev > max)
- max = readdev;
- }
-#endif
- res = ast_select(max + 1, &rfds, &wfds, NULL, NULL);
- if (res < 1) {
- ast_log(LOG_WARNING, "select failed: %s\n", strerror(errno));
- continue;
- }
-#ifdef ALSA_MONITOR
- if (FD_ISSET(readdev, &rfds)) {
- /* Keep the pipe going with read audio */
- snd_pcm_state_t state;
- short buf[FRAME_SIZE];
- int r;
-
- state = snd_pcm_state(alsa.ocard);
- if (state == SND_PCM_STATE_XRUN) {
- snd_pcm_prepare(alsa.ocard);
- }
- r = snd_pcm_readi(alsa.icard, buf, FRAME_SIZE);
- if (r == -EPIPE) {
-#if DEBUG
- ast_log(LOG_ERROR, "XRUN read\n");
-#endif
- snd_pcm_prepare(alsa.icard);
- } else if (r == -ESTRPIPE) {
- ast_log(LOG_ERROR, "-ESTRPIPE\n");
- snd_pcm_prepare(alsa.icard);
- } else if (r < 0) {
- ast_log(LOG_ERROR, "Read error: %s\n", snd_strerror(r));
- } else
- alsa_monitor_read((char *) buf, r * 2);
- }
-#endif
- if (FD_ISSET(sndcmd[0], &rfds)) {
- read(sndcmd[0], &cursound, sizeof(cursound));
- silencelen = 0;
- offset = 0;
- sampsent = 0;
- }
- if (FD_ISSET(writedev, &wfds))
- if (send_sound())
- ast_log(LOG_WARNING, "Failed to write sound\n");
- }
- /* Never reached */
- return NULL;
-}
-
static snd_pcm_t *alsa_card_init(char *dev, snd_pcm_stream_t stream)
{
int err;
@@ -343,14 +166,8 @@
snd_pcm_sw_params_t *swparams = NULL;
struct pollfd pfd;
snd_pcm_uframes_t period_size = PERIOD_FRAMES * 4;
- /* int period_bytes = 0; */
snd_pcm_uframes_t buffer_size = 0;
-
unsigned int rate = DESIRED_RATE;
-#if 0
- unsigned int per_min = 1;
-#endif
- /* unsigned int per_max = 8; */
snd_pcm_uframes_t start_threshold, stop_threshold;
err = snd_pcm_open(&handle, dev, stream, O_NONBLOCK);
@@ -398,17 +215,6 @@
ast_debug(1, "Buffer size is set to %d frames\n", err);
}
-#if 0
- direction = 0;
- err = snd_pcm_hw_params_set_periods_min(handle, hwparams, &per_min, &direction);
- if (err < 0)
- ast_log(LOG_ERROR, "periods_min: %s\n", snd_strerror(err));
-
- err = snd_pcm_hw_params_set_periods_max(handle, hwparams, &per_max, 0);
- if (err < 0)
- ast_log(LOG_ERROR, "periods_max: %s\n", snd_strerror(err));
-#endif
-
err = snd_pcm_hw_params(handle, hwparams);
if (err < 0)
ast_log(LOG_ERROR, "Couldn't set the new hw params: %s\n", snd_strerror(err));
@@ -417,7 +223,6 @@
memset(swparams, 0, snd_pcm_sw_params_sizeof());
snd_pcm_sw_params_current(handle, swparams);
-#if 1
if (stream == SND_PCM_STREAM_PLAYBACK)
start_threshold = period_size;
else
@@ -426,9 +231,7 @@
err = snd_pcm_sw_params_set_start_threshold(handle, swparams, start_threshold);
if (err < 0)
ast_log(LOG_ERROR, "start threshold: %s\n", snd_strerror(err));
-#endif
-
-#if 1
+
if (stream == SND_PCM_STREAM_PLAYBACK)
stop_threshold = buffer_size;
else
@@ -437,18 +240,7 @@
err = snd_pcm_sw_params_set_stop_threshold(handle, swparams, stop_threshold);
if (err < 0)
ast_log(LOG_ERROR, "stop threshold: %s\n", snd_strerror(err));
-#endif
-#if 0
- err = snd_pcm_sw_params_set_xfer_align(handle, swparams, PERIOD_FRAMES);
- if (err < 0)
- ast_log(LOG_ERROR, "Unable to set xfer alignment: %s\n", snd_strerror(err));
-#endif
-
-#if 0
- err = snd_pcm_sw_params_set_silence_threshold(handle, swparams, silencethreshold);
- if (err < 0)
- ast_log(LOG_ERROR, "Unable to set silence threshold: %s\n", snd_strerror(err));
-#endif
+
err = snd_pcm_sw_params(handle, swparams);
if (err < 0)
ast_log(LOG_ERROR, "sw_params: %s\n", snd_strerror(err));
@@ -477,7 +269,7 @@
alsa.ocard = alsa_card_init(outdevname, SND_PCM_STREAM_PLAYBACK);
if (!alsa.icard || !alsa.ocard) {
- ast_log(LOG_ERROR, "Problem opening alsa I/O devices\n");
+ ast_log(LOG_ERROR, "Problem opening ALSA I/O devices\n");
return -1;
}
@@ -490,6 +282,7 @@
ast_verbose(" << Console Received digit %c of duration %u ms >> \n",
digit, duration);
ast_mutex_unlock(&alsalock);
+
return 0;
}
@@ -498,6 +291,7 @@
ast_mutex_lock(&alsalock);
ast_verbose(" << Console Received text %s >> \n", text);
ast_mutex_unlock(&alsalock);
+
return 0;
}
@@ -512,8 +306,8 @@
static int alsa_call(struct ast_channel *c, char *dest, int timeout)
{
- int res = 3;
struct ast_frame f = { AST_FRAME_CONTROL };
+
ast_mutex_lock(&alsalock);
ast_verbose(" << Call placed to '%s' on console >> \n", dest);
if (autoanswer) {
@@ -531,56 +325,39 @@
f.subclass = AST_CONTROL_RINGING;
ast_queue_frame(alsa.owner, &f);
ast_channel_unlock(alsa.owner);
- }
- write(sndcmd[1], &res, sizeof(res));
+ ast_indicate(alsa.owner, AST_CONTROL_RINGING);
+ }
}
snd_pcm_prepare(alsa.icard);
snd_pcm_start(alsa.icard);
ast_mutex_unlock(&alsalock);
+
return 0;
}
-static void answer_sound(void)
-{
- int res;
- nosound = 1;
- res = 4;
- write(sndcmd[1], &res, sizeof(res));
-
-}
-
static int alsa_answer(struct ast_channel *c)
{
ast_mutex_lock(&alsalock);
ast_verbose(" << Console call has been answered >> \n");
- answer_sound();
ast_setstate(c, AST_STATE_UP);
- cursound = -1;
snd_pcm_prepare(alsa.icard);
snd_pcm_start(alsa.icard);
ast_mutex_unlock(&alsalock);
+
return 0;
}
static int alsa_hangup(struct ast_channel *c)
{
- int res;
- ast_mutex_lock(&alsalock);
- cursound = -1;
+ ast_mutex_lock(&alsalock);
c->tech_pvt = NULL;
alsa.owner = NULL;
ast_verbose(" << Hangup on console >> \n");
ast_module_unref(ast_module_info->self);
- if (hookstate) {
- hookstate = 0;
- if (!autoanswer) {
- /* Congestion noise */
- res = 2;
- write(sndcmd[1], &res, sizeof(res));
- }
- }
+ hookstate = 0;
snd_pcm_drop(alsa.icard);
ast_mutex_unlock(&alsalock);
+
return 0;
}
@@ -594,18 +371,7 @@
/* size_t frames = 0; */
snd_pcm_state_t state;
- /* Immediately return if no sound is enabled */
- if (nosound)
- return 0;
-
- ast_mutex_lock(&alsalock);
- /* Stop any currently playing sound */
- if (cursound != -1) {
- snd_pcm_drop(alsa.ocard);
- snd_pcm_prepare(alsa.ocard);
- cursound = -1;
- }
-
+ ast_mutex_lock(&alsalock);
/* We have to digest the frame in 160-byte portions */
if (f->datalen > sizeof(sizbuf) - sizpos) {
@@ -615,9 +381,6 @@
memcpy(sizbuf + sizpos, f->data, f->datalen);
len += f->datalen;
pos = 0;
-#ifdef ALSA_MONITOR
- alsa_monitor_write(sizbuf, len);
-#endif
state = snd_pcm_state(alsa.ocard);
if (state == SND_PCM_STATE_XRUN)
snd_pcm_prepare(alsa.ocard);
@@ -643,9 +406,8 @@
}
}
ast_mutex_unlock(&alsalock);
- if (res > 0)
- res = 0;
- return res;
+
+ return res >= 0 ? 0 : res;
}
@@ -661,7 +423,6 @@
int off = 0;
ast_mutex_lock(&alsalock);
- /* Acknowledge any pending cmd */
f.frametype = AST_FRAME_NULL;
f.subclass = 0;
f.samples = 0;
@@ -715,21 +476,21 @@
f.offset = AST_FRIENDLY_OFFSET;
f.src = "Console";
f.mallocd = 0;
-#ifdef ALSA_MONITOR
- alsa_monitor_read((char *) buf, FRAME_SIZE * 2);
-#endif
-
- }
- ast_mutex_unlock(&alsalock);
+
+ }
+ ast_mutex_unlock(&alsalock);
+
return &f;
}
static int alsa_fixup(struct ast_channel *oldchan, struct ast_channel *newchan)
{
struct chan_alsa_pvt *p = newchan->tech_pvt;
+
ast_mutex_lock(&alsalock);
p->owner = newchan;
ast_mutex_unlock(&alsalock);
+
return 0;
}
@@ -740,36 +501,28 @@
ast_mutex_lock(&alsalock);
switch (cond) {
- case AST_CONTROL_BUSY:
- res = 1;
- break;
- case AST_CONTROL_CONGESTION:
- res = 2;
- break;
- case AST_CONTROL_RINGING:
- case AST_CONTROL_PROGRESS:
- break;
- case -1:
- res = -1;
- break;
- case AST_CONTROL_VIDUPDATE:
- res = -1;
- break;
- case AST_CONTROL_HOLD:
- ast_verbose(" << Console Has Been Placed on Hold >> \n");
- ast_moh_start(chan, data, mohinterpret);
- break;
- case AST_CONTROL_UNHOLD:
- ast_verbose(" << Console Has Been Retrieved from Hold >> \n");
- ast_moh_stop(chan);
- break;
- default:
- ast_log(LOG_WARNING, "Don't know how to display condition %d on %s\n", cond, chan->name);
- res = -1;
- }
-
- if (res > -1)
- write(sndcmd[1], &res, sizeof(res));
+ case AST_CONTROL_BUSY:
+ case AST_CONTROL_CONGESTION:
+ case AST_CONTROL_RINGING:
+ case -1:
+ res = -1; /* Ask for inband indications */
+ break;
+ case AST_CONTROL_PROGRESS:
+ case AST_CONTROL_PROCEEDING:
+ case AST_CONTROL_VIDUPDATE:
+ break;
+ case AST_CONTROL_HOLD:
+ ast_verbose(" << Console Has Been Placed on Hold >> \n");
+ ast_moh_start(chan, data, mohinterpret);
+ break;
+ case AST_CONTROL_UNHOLD:
+ ast_verbose(" << Console Has Been Retrieved from Hold >> \n");
+ ast_moh_stop(chan);
+ break;
+ default:
+ ast_log(LOG_WARNING, "Don't know how to display condition %d on %s\n", cond, chan->name);
+ res = -1;
+ }
ast_mutex_unlock(&alsalock);
@@ -814,8 +567,7 @@
int oldformat = format;
struct ast_channel *tmp = NULL;
- format &= AST_FORMAT_SLINEAR;
- if (!format) {
+ if (!(format &= AST_FORMAT_SLINEAR)) {
ast_log(LOG_NOTICE, "Asked to get a channel of format '%d'\n", oldformat);
return NULL;
}
@@ -825,8 +577,9 @@
if (alsa.owner) {
ast_log(LOG_NOTICE, "Already have a call on the ALSA channel\n");
*cause = AST_CAUSE_BUSY;
- } else if (!(tmp = alsa_new(&alsa, AST_STATE_DOWN)))
+ } else if (!(tmp = alsa_new(&alsa, AST_STATE_DOWN))) {
ast_log(LOG_WARNING, "Unable to create new ALSA channel\n");
+ }
ast_mutex_unlock(&alsalock);
@@ -848,6 +601,7 @@
default:
return NULL;
}
+
return NULL;
}
@@ -870,6 +624,7 @@
if ((a->argc != 2) && (a->argc != 3))
return CLI_SHOWUSAGE;
+
ast_mutex_lock(&alsalock);
if (a->argc == 2) {
ast_cli(a->fd, "Auto answer is %s.\n", autoanswer ? "on" : "off");
@@ -882,6 +637,7 @@
res = CLI_SHOWUSAGE;
}
ast_mutex_unlock(&alsalock);
+
return res;
}
@@ -900,7 +656,6 @@
case CLI_GENERATE:
return NULL;
}
-
if (a->argc != 2)
return CLI_SHOWUSAGE;
@@ -912,14 +667,13 @@
res = CLI_FAILURE;
} else {
hookstate = 1;
- cursound = -1;
grab_owner();
if (alsa.owner) {
struct ast_frame f = { AST_FRAME_CONTROL, AST_CONTROL_ANSWER };
+
ast_queue_frame(alsa.owner, &f);
ast_channel_unlock(alsa.owner);
}
- answer_sound();
}
snd_pcm_prepare(alsa.icard);
@@ -952,16 +706,17 @@
ast_mutex_lock(&alsalock);
if (!alsa.owner) {
- ast_cli(a->fd, "No one is calling us\n");
+ ast_cli(a->fd, "No channel active\n");
res = CLI_FAILURE;
} else {
struct ast_frame f = { AST_FRAME_TEXT, 0 };
char text2send[256] = "";
- text2send[0] = '\0';
+
while (tmparg < a->argc) {
strncat(text2send, a->argv[tmparg++], sizeof(text2send) - strlen(text2send) - 1);
strncat(text2send, " ", sizeof(text2send) - strlen(text2send) - 1);
}
+
text2send[strlen(text2send) - 1] = '\n';
f.data = text2send;
f.datalen = strlen(text2send) + 1;
@@ -1001,12 +756,10 @@
if (a->argc != 2)
return CLI_SHOWUSAGE;
- cursound = -1;
-
ast_mutex_lock(&alsalock);
if (!alsa.owner && !hookstate) {
- ast_cli(a->fd, "No call to hangup up\n");
+ ast_cli(a->fd, "No call to hangup\n");
res = CLI_FAILURE;
} else {
hookstate = 0;
@@ -1047,13 +800,11 @@
if (alsa.owner) {
if (a->argc == 3) {
- d = a->argv[2];
if (alsa.owner) {
- struct ast_frame f = { AST_FRAME_DTMF };
- while (*d) {
- f.subclass = *d;
+ for (d = a->argv[2]; *d; d++) {
+ struct ast_frame f = { .frametype = AST_FRAME_DTMF, .subclass = *d };
+
ast_queue_frame(alsa.owner, &f);
- d++;
}
}
} else {
@@ -1065,6 +816,7 @@
myc = context;
if (a->argc == 3) {
char *stringp = NULL;
+
ast_copy_string(tmp, a->argv[2], sizeof(tmp));
stringp = tmp;
strsep(&stringp, "@");
@@ -1137,11 +889,6 @@
}
ast_config_destroy(cfg);
- if (pipe(sndcmd)) {
- ast_log(LOG_ERROR, "Unable to create pipe\n");
- return AST_MODULE_LOAD_FAILURE;
- }
-
if (soundcard_init() < 0) {
ast_verb(2, "No sound card detected -- console channel will be unavailable\n");
ast_verb(2, "Turn off ALSA support by adding 'noload=chan_alsa.so' in /etc/asterisk/modules.conf\n");
@@ -1155,11 +902,6 @@
ast_cli_register_multiple(cli_alsa, sizeof(cli_alsa) / sizeof(struct ast_cli_entry));
- ast_pthread_create_background(&sthread, NULL, sound_thread, NULL);
-#ifdef ALSA_MONITOR
- if (alsa_monitor_start())
- ast_log(LOG_ERROR, "Problem starting Monitoring\n");
-#endif
return AST_MODULE_LOAD_SUCCESS;
}
@@ -1172,14 +914,11 @@
snd_pcm_close(alsa.icard);
if (alsa.ocard)
snd_pcm_close(alsa.ocard);
- if (sndcmd[0] > 0) {
- close(sndcmd[0]);
- close(sndcmd[1]);
- }
if (alsa.owner)
ast_softhangup(alsa.owner, AST_SOFTHANGUP_APPUNLOAD);
if (alsa.owner)
return -1;
+
return 0;
}
More information about the asterisk-commits
mailing list