diff options
author | Jason Chen <b02280@freescale.com> | 2010-05-18 14:58:01 +0800 |
---|---|---|
committer | Jason Chen <b02280@freescale.com> | 2010-05-18 14:58:01 +0800 |
commit | 9fda539a123fcd0b8692614cda0c317230ef251a (patch) | |
tree | 1404454463591d1a863d1cf27d910ce3122805be | |
parent | e30d2d898598f8f00ed291ddd6f216452a0bc6a4 (diff) |
ENGR00123558 v4l2 output: add own workqueue for v4l output
During v4l2 output playback, there is performance issue after below
commit:
UBUNTU: SAUCE: (upstream) netdev/fec: fix performance impact from
mdio poll operation
Use common workqueue cause this issue, so create an own workqueue for
v4l2 output.
Signed-off-by: Jason Chen <b02280@freescale.com>
-rw-r--r-- | drivers/media/video/mxc/output/mxc_v4l2_output.c | 18 | ||||
-rw-r--r-- | drivers/media/video/mxc/output/mxc_v4l2_output.h | 1 |
2 files changed, 15 insertions, 4 deletions
diff --git a/drivers/media/video/mxc/output/mxc_v4l2_output.c b/drivers/media/video/mxc/output/mxc_v4l2_output.c index b783baae8040..3cc2e93aa75f 100644 --- a/drivers/media/video/mxc/output/mxc_v4l2_output.c +++ b/drivers/media/video/mxc/output/mxc_v4l2_output.c @@ -522,9 +522,8 @@ static void mxc_v4l2out_timer_handler(unsigned long arg) } /* set next buffer ready */ - if (schedule_work(&vout->timer_work) == 0) { - dev_err(&vout->video_dev->dev, - "timer work already in queue\n"); + if (queue_work(vout->v4l_wq, &vout->timer_work) == 0) { + dev_err(&vout->video_dev->dev, "work was in queue already!\n "); vout->state = STATE_STREAM_PAUSED; } @@ -1581,7 +1580,7 @@ static int mxc_v4l2out_streamon(vout_data * vout) 1, vout->v4l2_bufs[vout->ipu_buf[1]].m.offset); ipu_select_buffer(vout->display_ch, IPU_INPUT_BUFFER, 0); ipu_select_buffer(vout->display_ch, IPU_INPUT_BUFFER, 1); - schedule_work(&vout->timer_work); + queue_work(vout->v4l_wq, &vout->timer_work); } } else { ipu_select_buffer(vout->post_proc_ch, IPU_INPUT_BUFFER, 0); @@ -1982,6 +1981,14 @@ static int mxc_v4l2out_open(struct file *file) vout->state = STATE_STREAM_OFF; vout->rotate = IPU_ROTATE_NONE; + vout->v4l_wq = create_singlethread_workqueue("v4l2q"); + if (!vout->v4l_wq) { + dev_dbg(&dev->dev, + "Could not create work queue\n"); + err = -ENOMEM; + goto oops; + } + INIT_WORK(&vout->timer_work, timer_work_func); } @@ -2022,6 +2029,9 @@ static int mxc_v4l2out_close(struct file *file) /* capture off */ wake_up_interruptible(&vout->v4l_bufq); + + flush_workqueue(vout->v4l_wq); + destroy_workqueue(vout->v4l_wq); } return 0; diff --git a/drivers/media/video/mxc/output/mxc_v4l2_output.h b/drivers/media/video/mxc/output/mxc_v4l2_output.h index 7a868f377018..91cf3b488219 100644 --- a/drivers/media/video/mxc/output/mxc_v4l2_output.h +++ b/drivers/media/video/mxc/output/mxc_v4l2_output.h @@ -79,6 +79,7 @@ typedef struct _vout_data { struct semaphore param_lock; struct timer_list output_timer; + struct workqueue_struct *v4l_wq; struct work_struct timer_work; unsigned long start_jiffies; u32 frame_count; |