diff options
| author | Jason Chen <b02280@freescale.com> | 2011-11-16 17:23:07 +0800 |
|---|---|---|
| committer | Jason Liu <r64343@freescale.com> | 2012-01-09 21:06:53 +0800 |
| commit | da07aab962f6866b0cd85f35053a864651693649 (patch) | |
| tree | b289be3f3d33bd1fcd6bb5f0db1abccbcb0417c4 /drivers/mxc/ipu3 | |
| parent | c5e411a63577f7bc086921a48740d8eb3123272d (diff) | |
ENGR00162354 ipuv3 device: correct kthread operation for split task
make could_finish to volatile.
do_exit in split_task_thread instead of kthread_stop.
Signed-off-by: Jason Chen <b02280@freescale.com>
Diffstat (limited to 'drivers/mxc/ipu3')
| -rw-r--r-- | drivers/mxc/ipu3/ipu_device.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/mxc/ipu3/ipu_device.c b/drivers/mxc/ipu3/ipu_device.c index 0f68441fc83b..9395d8a6661c 100644 --- a/drivers/mxc/ipu3/ipu_device.c +++ b/drivers/mxc/ipu3/ipu_device.c @@ -144,7 +144,7 @@ struct ipu_split_task { struct ipu_task task; struct ipu_task_entry *parent_task; struct task_struct *thread; - bool could_finish; + volatile bool could_finish; wait_queue_head_t waitq; int ret; @@ -944,7 +944,7 @@ static int split_task_thread(void *data) wake_up_interruptible(&t->waitq); - return 0; + do_exit(0); } static int create_split_task( @@ -1176,7 +1176,6 @@ static int queue_split_task(struct ipu_task_entry *t) } else { for (i = 0; i < size; i++) { wait_event_interruptible(sp_task[i].waitq, sp_task[i].could_finish); - kthread_stop(sp_task[i].thread); if (sp_task[i].ret < 0) { ret = sp_task[i].ret; dev_err(t->dev, |