FGCOM/dahdi-linux
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

wcte12xp: Use interruptible waits to decrease impact on load average.

Browse Source 
The wcte12xp does all the checking for alarm in a user space workqueue.
Most of this time is spent sleeping waiting for reads from the framer to
complete.  Tasks in uninterruptible sleeps are added to running tasks
for the purposes of calculating load average. This change makes the
sleeps interruptible so as to not affect the load average as much.

For example, the following command will load and configure the driver and
then print the load average every 10 seconds.

]# modprobe wcte12xp && dahdi_cfg && ((x=12)); while [[ $x -gt 0 ]]; do cat
/proc/loadavg; sleep 10; let x=$x-1; done

With this change:
0.29 0.10 0.02 1/101 29945
0.24 0.10 0.02 1/101 29967
0.20 0.09 0.02 1/101 30019
0.17 0.09 0.02 1/101 30041
0.15 0.09 0.02 1/101 30062
0.12 0.08 0.02 1/101 30085
0.10 0.08 0.02 1/101 30107
0.09 0.08 0.02 1/101 30129
0.07 0.08 0.02 1/101 30151
0.14 0.09 0.02 1/101 30173
0.12 0.09 0.02 1/101 30195
0.10 0.08 0.02 1/101 30217
(and I've seen it get down to 0.0)

Before this change:
0.57 0.22 0.07 1/101 31920
0.48 0.21 0.07 1/101 31942
0.48 0.22 0.07 1/101 31964
0.48 0.23 0.08 1/101 31986
0.41 0.22 0.07 1/101 32008
0.42 0.23 0.08 1/101 32030
0.43 0.24 0.08 1/101 32054
0.45 0.25 0.09 1/101 32076
0.45 0.25 0.09 1/101 32098
0.46 0.26 0.10 1/101 32120
0.47 0.27 0.10 1/101 32172
0.39 0.26 0.10 1/101 32194

(closes issue #18142)
Reported by: foxfire
Tested by: foxfire

Signed-off-by: Shaun Ruffell <sruffell@digium.com>

git-svn-id: http://svn.asterisk.org/svn/dahdi/linux/trunk@9512 a0bf4364-ded3-4de4-8d8a-66a801d63aff
This commit is contained in:
Shaun Ruffell 2010-12-08 01:56:56 +00:00
parent f7925643de
commit 51bff0d802
2 changed files with 21 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
drivers/dahdi/wcte12xp/base.c
View File

@ -578,7 +578,7 @@ static int t1_getreg(struct t1 *wc, int addr)
cmd->data = 0x00; cmd->data = 0x00;
cmd->flags = __CMD_RD; cmd->flags = __CMD_RD;
submit_cmd(wc, cmd); submit_cmd(wc, cmd);
ret = wait_for_completion_timeout(&cmd->complete, HZ*10); ret = wait_for_completion_interruptible_timeout(&cmd->complete, HZ*10);
if (unlikely(!ret)) { if (unlikely(!ret)) {
spin_lock_irqsave(&wc->cmd_list_lock, flags); spin_lock_irqsave(&wc->cmd_list_lock, flags);
if (!list_empty(&cmd->node)) { if (!list_empty(&cmd->node)) {
@ -586,12 +586,15 @@ static int t1_getreg(struct t1 *wc, int addr)
* can go ahead and free it right away. */ * can go ahead and free it right away. */
list_del_init(&cmd->node); list_del_init(&cmd->node);
spin_unlock_irqrestore(&wc->cmd_list_lock, flags); spin_unlock_irqrestore(&wc->cmd_list_lock, flags);
if (printk_ratelimit()) {
dev_warn(&wc->vb.pdev->dev,
"Timeout in %s\n", __func__);
}
free_cmd(wc, cmd); free_cmd(wc, cmd);
return -EIO; if (-ERESTARTSYS != ret) {
if (printk_ratelimit()) {
dev_warn(&wc->vb.pdev->dev,
"Timeout in %s\n", __func__);
}
ret = -EIO;
}
return ret;
} else { } else {
/* Looks like this command was removed from the list by /* Looks like this command was removed from the list by
* someone else already. Let's wait for them to complete * someone else already. Let's wait for them to complete
@ -636,17 +639,20 @@ static inline int t1_getpins(struct t1 *wc, int inisr)
cmd->data = 0x00; cmd->data = 0x00;
cmd->flags = __CMD_PINS; cmd->flags = __CMD_PINS;
submit_cmd(wc, cmd); submit_cmd(wc, cmd);
ret = wait_for_completion_timeout(&cmd->complete, HZ*2); ret = wait_for_completion_interruptible_timeout(&cmd->complete, HZ*2);
if (unlikely(!ret)) { if (unlikely(!ret)) {
spin_lock_irqsave(&wc->cmd_list_lock, flags); spin_lock_irqsave(&wc->cmd_list_lock, flags);
list_del_init(&cmd->node); list_del_init(&cmd->node);
spin_unlock_irqrestore(&wc->cmd_list_lock, flags); spin_unlock_irqrestore(&wc->cmd_list_lock, flags);
if (printk_ratelimit()) {
dev_warn(&wc->vb.pdev->dev,
"Timeout in %s\n", __func__);
}
free_cmd(wc, cmd); free_cmd(wc, cmd);
return -EIO; if (-ERESTARTSYS != ret) {
if (printk_ratelimit()) {
dev_warn(&wc->vb.pdev->dev,
"Timeout in %s\n", __func__);
}
ret = -EIO;
}
return ret;
} }
ret = cmd->data; ret = cmd->data;
free_cmd(wc, cmd); free_cmd(wc, cmd);

5
include/dahdi/kernel.h
View File

@ -1271,12 +1271,13 @@ static inline void list_replace(struct list_head *old, struct list_head *new)
#if LINUX_VERSION_CODE < KERNEL_VERSION(2, 6, 11) #if LINUX_VERSION_CODE < KERNEL_VERSION(2, 6, 11)
#if !defined(HAVE_WAIT_FOR_COMPLETION_TIMEOUT) #if !defined(HAVE_WAIT_FOR_COMPLETION_TIMEOUT)
static inline unsigned long static inline unsigned long
wait_for_completion_timeout(struct completion *x, unsigned long timeout) wait_for_completion_interruptible_timeout(struct completion *x,
unsigned long timeout)
{ {
/* There is a race condition here. If x->done is reset to 0 /* There is a race condition here. If x->done is reset to 0
* before the call to wait_for_completion after this thread wakes. * before the call to wait_for_completion after this thread wakes.
*/ */
timeout = wait_event_timeout(x->wait, x->done, timeout); timeout = wait_event_interruptible_timeout(x->wait, x->done, timeout);
if (timeout) if (timeout)
wait_for_completion(x); wait_for_completion(x);