Hello, On Monday, November 07, 2011 2:46 PM Mauro Carvalho Chehab wrote: > Em 25-10-2011 06:11, Pawel Osciak escreveu: > > On Tue, Oct 25, 2011 at 00:59, Marek Szyprowski > > <m.szyprowski@xxxxxxxxxxx> wrote: > >> Queued list was not reset on REQBUFS(0) call. This caused enqueuing > >> a freed buffer to the driver. > >> > >> Reported-by: Angela Wan <angela.j.wan@xxxxxxxxx> > >> Signed-off-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx> > >> Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx> > >> --- > >> drivers/media/video/videobuf2-core.c | 1 + > >> 1 files changed, 1 insertions(+), 0 deletions(-) > >> > >> diff --git a/drivers/media/video/videobuf2-core.c > >> b/drivers/media/video/videobuf2-core.c > >> index 3015e60..5722b81 100644 > >> --- a/drivers/media/video/videobuf2-core.c > >> +++ b/drivers/media/video/videobuf2-core.c > >> @@ -254,6 +254,7 @@ static void __vb2_queue_free(struct vb2_queue *q) > >> > >> q->num_buffers = 0; > >> q->memory = 0; > >> + INIT_LIST_HEAD(&q->queued_list); > >> } > >> > >> /** > >> -- > >> 1.7.1 > >> > >> > >> > > > > Acked-by: Pawel Osciak <pawel@xxxxxxxxxx> > > > > This patch doesn't apply anymore. Is it still needed? If so, please rebase. Yes, it is needed. This patch, together with other fixes (all rebased) is waiting in the pull request I've sent on 4.11.2011 - "[GIT PULL] More Samsung patches for v3.2 (updated)" thread. Do you want me to resend all of them? Best regards -- Marek Szyprowski Samsung Poland R&D Center -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
