The patch number 9596 was added via Andy Walls <[EMAIL PROTECTED]>
to http://linuxtv.org/hg/v4l-dvb master development tree.
Kernel patches in this development tree may be modified to be backward
compatible with older kernels. Compatibility modifications will be
removed before inclusion into the mainstream Kernel
If anyone has any objections, please let us know by sending a message to:
[EMAIL PROTECTED]
------
From: Andy Walls <[EMAIL PROTECTED]>
cx18: Further changes to improve mailbox protocol integrity & performnce
All waits for cx18 mailbox API commands are now uninterruptable. Added
code to collect mailbox ack statistics. Tweaked timeouts based on collected
stats and video vertical frame and field rates.
Priority: normal
Signed-off-by: Andy Walls <[EMAIL PROTECTED]>
---
linux/drivers/media/video/cx18/cx18-driver.c | 2
linux/drivers/media/video/cx18/cx18-driver.h | 7 ++
linux/drivers/media/video/cx18/cx18-io.c | 4 +
linux/drivers/media/video/cx18/cx18-mailbox.c | 47 +++++++++++-------
4 files changed, 42 insertions(+), 18 deletions(-)
diff -r 928e80134880 -r 98e1056b0daf
linux/drivers/media/video/cx18/cx18-driver.c
--- a/linux/drivers/media/video/cx18/cx18-driver.c Fri Nov 07 21:57:46
2008 -0500
+++ b/linux/drivers/media/video/cx18/cx18-driver.c Sat Nov 08 12:19:37
2008 -0500
@@ -187,7 +187,7 @@ MODULE_VERSION(CX18_VERSION);
/* Generic utility functions */
int cx18_msleep_timeout(unsigned int msecs, int intr)
{
- int timeout = msecs_to_jiffies(msecs);
+ long int timeout = msecs_to_jiffies(msecs);
int sig;
do {
diff -r 928e80134880 -r 98e1056b0daf
linux/drivers/media/video/cx18/cx18-driver.h
--- a/linux/drivers/media/video/cx18/cx18-driver.h Fri Nov 07 21:57:46
2008 -0500
+++ b/linux/drivers/media/video/cx18/cx18-driver.h Sat Nov 08 12:19:37
2008 -0500
@@ -360,6 +360,12 @@ struct cx18_mmio_stats {
atomic_t retried_read[CX18_MAX_MMIO_RD_RETRIES+1];
};
+#define CX18_MAX_MB_ACK_DELAY 100
+
+struct cx18_mbox_stats {
+ atomic_t mb_ack_delay[CX18_MAX_MB_ACK_DELAY+1];
+};
+
/* Struct to hold info about cx18 cards */
struct cx18 {
int num; /* board number, -1 during init! */
@@ -452,6 +458,7 @@ struct cx18 {
/* Statistics */
struct cx18_mmio_stats mmio_stats;
+ struct cx18_mbox_stats mbox_stats;
/* v4l2 and User settings */
diff -r 928e80134880 -r 98e1056b0daf linux/drivers/media/video/cx18/cx18-io.c
--- a/linux/drivers/media/video/cx18/cx18-io.c Fri Nov 07 21:57:46 2008 -0500
+++ b/linux/drivers/media/video/cx18/cx18-io.c Sat Nov 08 12:19:37 2008 -0500
@@ -37,6 +37,10 @@ void cx18_log_statistics(struct cx18 *cx
for (i = 0; i <= CX18_MAX_MMIO_RD_RETRIES; i++)
CX18_DEBUG_INFO("retried_read[%d] = %d\n", i,
atomic_read(&cx->mmio_stats.retried_read[i]));
+ for (i = 0; i <= CX18_MAX_MB_ACK_DELAY; i++)
+ if (atomic_read(&cx->mbox_stats.mb_ack_delay[i]))
+ CX18_DEBUG_INFO("mb_ack_delay[%d] = %d\n", i,
+ atomic_read(&cx->mbox_stats.mb_ack_delay[i]));
return;
}
diff -r 928e80134880 -r 98e1056b0daf
linux/drivers/media/video/cx18/cx18-mailbox.c
--- a/linux/drivers/media/video/cx18/cx18-mailbox.c Fri Nov 07 21:57:46
2008 -0500
+++ b/linux/drivers/media/video/cx18/cx18-mailbox.c Sat Nov 08 12:19:37
2008 -0500
@@ -118,6 +118,12 @@ long cx18_mb_ack(struct cx18 *cx, const
return 0;
}
+static void cx18_api_log_ack_delay(struct cx18 *cx, int msecs)
+{
+ if (msecs > CX18_MAX_MB_ACK_DELAY)
+ msecs = CX18_MAX_MB_ACK_DELAY;
+ atomic_inc(&cx->mbox_stats.mb_ack_delay[msecs]);
+}
static int cx18_api_call(struct cx18 *cx, u32 cmd, int args, u32 data[])
{
@@ -127,8 +133,7 @@ static int cx18_api_call(struct cx18 *cx
u32 __iomem *xpu_state;
wait_queue_head_t *waitq;
struct mutex *mb_lock;
- int timeout = 100;
- long unsigned int j, ret;
+ long int timeout, ret;
int i;
if (info == NULL) {
@@ -176,18 +181,18 @@ static int cx18_api_call(struct cx18 *cx
*/
state = cx18_readl(cx, xpu_state);
req = cx18_readl(cx, &mb->request);
- j = msecs_to_jiffies(timeout);
+ timeout = msecs_to_jiffies(20); /* 1 field at 50 Hz vertical refresh */
ret = wait_event_timeout(*waitq,
(ack = cx18_readl(cx, &mb->ack)) == req,
- j);
+ timeout);
if (req != ack) {
/* waited long enough, make the mbox "not busy" from our end */
cx18_writel(cx, req, &mb->ack);
CX18_ERR("mbox was found stuck busy when setting up for %s; "
"clearing busy and trying to proceed\n", info->name);
- } else if (ret != j)
+ } else if (ret != timeout)
CX18_DEBUG_API("waited %u usecs for busy mbox to be acked\n",
- jiffies_to_usecs(j-ret));
+ jiffies_to_usecs(timeout-ret));
/* Build the outgoing mailbox */
req = ((req & 0xfffffffe) == 0xfffffffe) ? 1 : req + 1;
@@ -199,24 +204,28 @@ static int cx18_api_call(struct cx18 *cx
cx18_writel(cx, req, &mb->request);
cx18_writel(cx, req - 1, &mb->ack); /* ensure ack & req are distinct */
- /* Notify the XPU and wait for it to send an Ack back */
- if (info->flags & API_FAST)
- timeout /= 2;
- j = msecs_to_jiffies(timeout);
+ /*
+ * Notify the XPU and wait for it to send an Ack back
+ * 21 ms = ~ 0.5 frames at a frame rate of 24 fps
+ * 42 ms = ~ 1 frame at a frame rate of 24 fps
+ */
+ timeout = msecs_to_jiffies((info->flags & API_FAST) ? 21 : 42);
CX18_DEBUG_HI_IRQ("sending interrupt SW1: %x to send %s\n",
irq, info->name);
cx18_write_reg_expect(cx, irq, SW1_INT_SET, irq, irq);
- ret = wait_event_interruptible_timeout(
+ ret = wait_event_timeout(
*waitq,
cx18_readl(cx, &mb->ack) == cx18_readl(cx, &mb->request),
- j);
+ timeout);
if (ret == 0) {
/* Timed out */
mutex_unlock(mb_lock);
- CX18_ERR("sending %s timed out waiting for RPU "
- "acknowledgement\n", info->name);
+ i = jiffies_to_msecs(timeout);
+ cx18_api_log_ack_delay(cx, i);
+ CX18_WARN("sending %s timed out waiting %d msecs for RPU "
+ "acknowledgement\n", info->name, i);
return -EINVAL;
} else if (ret < 0) {
/* Interrupted */
@@ -224,9 +233,13 @@ static int cx18_api_call(struct cx18 *cx
CX18_WARN("sending %s was interrupted waiting for RPU"
"acknowledgement\n", info->name);
return -EINTR;
- } else if (ret != j)
- CX18_DEBUG_HI_API("waited %u usecs for %s to be acked\n",
- jiffies_to_usecs(j-ret), info->name);
+ }
+
+ i = jiffies_to_msecs(timeout-ret);
+ cx18_api_log_ack_delay(cx, i);
+ if (ret != timeout)
+ CX18_DEBUG_HI_API("waited %u msecs for %s to be acked\n",
+ i, info->name);
/* Collect data returned by the XPU */
for (i = 0; i < MAX_MB_ARGUMENTS; i++)
---
Patch is available at:
http://linuxtv.org/hg/v4l-dvb/rev/98e1056b0daf076de1c92f8472fe2f9ceda2534a
_______________________________________________
linuxtv-commits mailing list
[email protected]
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits
