Commit a42743c2 authored by Kanigeri, Hari's avatar Kanigeri, Hari Committed by Hari Kanigeri

OMAP: mailbox: send message in process context

Schedule the Tasklet to send only when mailbox fifo is full and there are
pending messages in kfifo, else send the message directly in the Process
context. This would avoid needless scheduling of Tasklet for every message
transfer
Signed-off-by: default avatarHari Kanigeri <h-kanigeri2@ti.com>
Acked-by: default avatarHiroshi Doyu <hiroshi.doyu@nokia.com>
parent ab66ac30
...@@ -92,20 +92,25 @@ int omap_mbox_msg_send(struct omap_mbox *mbox, mbox_msg_t msg) ...@@ -92,20 +92,25 @@ int omap_mbox_msg_send(struct omap_mbox *mbox, mbox_msg_t msg)
struct omap_mbox_queue *mq = mbox->txq; struct omap_mbox_queue *mq = mbox->txq;
int ret = 0, len; int ret = 0, len;
spin_lock(&mq->lock); spin_lock_bh(&mq->lock);
if (kfifo_avail(&mq->fifo) < sizeof(msg)) { if (kfifo_avail(&mq->fifo) < sizeof(msg)) {
ret = -ENOMEM; ret = -ENOMEM;
goto out; goto out;
} }
if (kfifo_is_empty(&mq->fifo) && !__mbox_poll_for_space(mbox)) {
mbox_fifo_write(mbox, msg);
goto out;
}
len = kfifo_in(&mq->fifo, (unsigned char *)&msg, sizeof(msg)); len = kfifo_in(&mq->fifo, (unsigned char *)&msg, sizeof(msg));
WARN_ON(len != sizeof(msg)); WARN_ON(len != sizeof(msg));
tasklet_schedule(&mbox->txq->tasklet); tasklet_schedule(&mbox->txq->tasklet);
out: out:
spin_unlock(&mq->lock); spin_unlock_bh(&mq->lock);
return ret; return ret;
} }
EXPORT_SYMBOL(omap_mbox_msg_send); EXPORT_SYMBOL(omap_mbox_msg_send);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment