diff options
| author | Ezequiel Garcia <[email protected]> | 2018-07-25 13:15:12 -0400 |
|---|---|---|
| committer | Mauro Carvalho Chehab <[email protected]> | 2018-07-27 12:59:13 -0400 |
| commit | 9db3bbf58be59ab496ccd1c7528f4b0e8835c5b1 (patch) | |
| tree | 8bb35f66cf211c928f9668e127eab11f54348b1f /tools/perf/scripts/python/bin/stackcollapse-record | |
| parent | 39dd23dc9d4c512954ec511d22d93ea854cf0265 (diff) | |
media: v4l2-mem2mem: Fix missing v4l2_m2m_try_run call
Commit 34dbb848d5e4 ("media: mem2mem: Remove excessive try_run call")
removed a redundant call to v4l2_m2m_try_run but instead introduced
a bug. Consider the following case:
1) Context A schedules, queues and runs job A.
2) While the m2m device is running, context B schedules
and queues job B. Job B cannot run, because it has to
wait for job A.
3) Job A completes, calls v4l2_m2m_job_finish, and tries
to queue a job for context A, but since the context is
empty it won't do anything.
In this scenario, queued job B will never run. Fix this by calling
v4l2_m2m_try_run from v4l2_m2m_try_schedule.
While here, add more documentation to these functions.
Fixes: 34dbb848d5e4 ("media: mem2mem: Remove excessive try_run call")
Signed-off-by: Ezequiel Garcia <[email protected]>
Signed-off-by: Hans Verkuil <[email protected]>
[[email protected]: split >80 cols line]
Signed-off-by: Mauro Carvalho Chehab <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/bin/stackcollapse-record')
0 files changed, 0 insertions, 0 deletions