Commit 58c8c226 authored Sep 13, 2017 by Maheshwar Ajja

msm: vidc: Check for VB2_BUF_STATE_ACTIVE state



Video usecases with same buffer queued multiple times
are failed because driver is sending vb2_buffer_done()
on the same vb2 buffer multiple times without checking
the vb2 buffer state. Resolve the issue by sending
vb2_buffer_done() for the buffers with vb2 state is
VB2_BUF_STATE_ACTIVE only.

CRs-Fixed: 2101026
Change-Id: Iab61c03c12469860b751da0daa54834e044ef445
Signed-off-by: Maheshwar Ajja <majja@codeaurora.org>

parent f4d2bc58

drivers/media/platform/msm/vidc/msm_vidc_common.c

+2 −0

Original line number	Diff line number	Diff line
		@@ -2272,6 +2272,8 @@ struct vb2_buffer *msm_comm_get_vb_using_vidc_buffer(
		mutex_lock(&inst->bufq[port].lock);
		found = false;
		list_for_each_entry(vb, &q->queued_list, queued_entry) {
		if (vb->state != VB2_BUF_STATE_ACTIVE)
		continue;
		if (msm_comm_compare_vb2_planes(inst, mbuf, vb)) {
		found = true;
		break;