* WARNING: at drivers/media/v4l2-core/videobuf2-core.c:2065 vb2_queue_init+0x74/0x142()
@ 2013-05-08 20:11 Borislav Petkov
2013-05-10 12:06 ` Hans Verkuil
0 siblings, 1 reply; 3+ messages in thread
From: Borislav Petkov @ 2013-05-08 20:11 UTC (permalink / raw)
To: linux-media
This one looks legit: bw-qcam.c doesn't set q->timestamp_type.
[ 146.989016] Colour QuickCam for Video4Linux v0.06
[ 147.713065] ------------[ cut here ]------------
[ 147.928854] WARNING: at drivers/media/v4l2-core/videobuf2-core.c:2065 vb2_queue_init+0x74/0x142()
[ 148.364433] CPU: 0 PID: 1 Comm: swapper/0 Not tainted 3.9.0-12947-g0f99ebe5052a #1
[ 148.799135] Hardware name: Bochs Bochs, BIOS Bochs 01/01/2007
[ 149.017598] ffffffff8239cab3 ffff88007b789d48 ffffffff81d81cb0 ffff88007b789d88
[ 149.465524] ffffffff810943a8 ffff88007b789d88 0000000000000000 ffff880079a7a700
[ 149.909985] ffff880079a7a068 ffff880079a7a608 ffff8800798d1c00 ffff88007b789d98
[ 150.345653] Call Trace:
[ 150.550444] [<ffffffff81d81cb0>] dump_stack+0x19/0x1b
[ 150.756860] [<ffffffff810943a8>] warn_slowpath_common+0x62/0x7b
[ 150.962012] [<ffffffff8109448c>] warn_slowpath_null+0x1a/0x1e
[ 151.160574] [<ffffffff818e914a>] vb2_queue_init+0x74/0x142
[ 151.354795] [<ffffffff818ff072>] bwqcam_attach+0x1e0/0x54a
[ 151.543644] [<ffffffff815cdeb1>] parport_register_driver+0x2e/0x6d
[ 151.727651] [<ffffffff82a8ddf5>] ? cqcam_init+0x20/0x20
[ 151.906958] [<ffffffff82a8de05>] init_bw_qcams+0x10/0x12
[ 152.084031] [<ffffffff82a3cdb9>] do_one_initcall+0x7b/0x116
[ 152.262088] [<ffffffff82a3cfb4>] kernel_init_freeable+0x160/0x1f2
[ 152.441466] [<ffffffff82a3c73a>] ? do_early_param+0x8c/0x8c
[ 152.619998] [<ffffffff81d6874b>] ? rest_init+0xdf/0xdf
[ 152.797458] [<ffffffff81d68759>] kernel_init+0xe/0xdb
[ 152.970650] [<ffffffff81d9663c>] ret_from_fork+0x7c/0xb0
[ 153.140434] [<ffffffff81d6874b>] ? rest_init+0xdf/0xdf
[ 153.305520] ---[ end trace a72f2983de4c60b5 ]---
[ 154.459479] No Quickcam found on port parport0
[ 154.613448] Quickcam detection counter: 0
--
Regards/Gruss,
Boris.
Sent from a fat crate under my desk. Formatting is fine.
--
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: WARNING: at drivers/media/v4l2-core/videobuf2-core.c:2065 vb2_queue_init+0x74/0x142()
2013-05-08 20:11 WARNING: at drivers/media/v4l2-core/videobuf2-core.c:2065 vb2_queue_init+0x74/0x142() Borislav Petkov
@ 2013-05-10 12:06 ` Hans Verkuil
2013-05-10 14:33 ` Borislav Petkov
0 siblings, 1 reply; 3+ messages in thread
From: Hans Verkuil @ 2013-05-10 12:06 UTC (permalink / raw)
To: Borislav Petkov; +Cc: linux-media
On Wed May 8 2013 22:11:18 Borislav Petkov wrote:
> This one looks legit: bw-qcam.c doesn't set q->timestamp_type.
>
> [ 146.989016] Colour QuickCam for Video4Linux v0.06
> [ 147.713065] ------------[ cut here ]------------
> [ 147.928854] WARNING: at drivers/media/v4l2-core/videobuf2-core.c:2065 vb2_queue_init+0x74/0x142()
> [ 148.364433] CPU: 0 PID: 1 Comm: swapper/0 Not tainted 3.9.0-12947-g0f99ebe5052a #1
> [ 148.799135] Hardware name: Bochs Bochs, BIOS Bochs 01/01/2007
> [ 149.017598] ffffffff8239cab3 ffff88007b789d48 ffffffff81d81cb0 ffff88007b789d88
> [ 149.465524] ffffffff810943a8 ffff88007b789d88 0000000000000000 ffff880079a7a700
> [ 149.909985] ffff880079a7a068 ffff880079a7a608 ffff8800798d1c00 ffff88007b789d98
> [ 150.345653] Call Trace:
> [ 150.550444] [<ffffffff81d81cb0>] dump_stack+0x19/0x1b
> [ 150.756860] [<ffffffff810943a8>] warn_slowpath_common+0x62/0x7b
> [ 150.962012] [<ffffffff8109448c>] warn_slowpath_null+0x1a/0x1e
> [ 151.160574] [<ffffffff818e914a>] vb2_queue_init+0x74/0x142
> [ 151.354795] [<ffffffff818ff072>] bwqcam_attach+0x1e0/0x54a
> [ 151.543644] [<ffffffff815cdeb1>] parport_register_driver+0x2e/0x6d
> [ 151.727651] [<ffffffff82a8ddf5>] ? cqcam_init+0x20/0x20
> [ 151.906958] [<ffffffff82a8de05>] init_bw_qcams+0x10/0x12
> [ 152.084031] [<ffffffff82a3cdb9>] do_one_initcall+0x7b/0x116
> [ 152.262088] [<ffffffff82a3cfb4>] kernel_init_freeable+0x160/0x1f2
> [ 152.441466] [<ffffffff82a3c73a>] ? do_early_param+0x8c/0x8c
> [ 152.619998] [<ffffffff81d6874b>] ? rest_init+0xdf/0xdf
> [ 152.797458] [<ffffffff81d68759>] kernel_init+0xe/0xdb
> [ 152.970650] [<ffffffff81d9663c>] ret_from_fork+0x7c/0xb0
> [ 153.140434] [<ffffffff81d6874b>] ? rest_init+0xdf/0xdf
> [ 153.305520] ---[ end trace a72f2983de4c60b5 ]---
> [ 154.459479] No Quickcam found on port parport0
> [ 154.613448] Quickcam detection counter: 0
>
>
Hi Borislav!
Thanks for the report.
Can you try this patch? This should fix it.
Regards,
Hans
diff --git a/drivers/media/parport/bw-qcam.c b/drivers/media/parport/bw-qcam.c
index 06231b8..d12bd33 100644
--- a/drivers/media/parport/bw-qcam.c
+++ b/drivers/media/parport/bw-qcam.c
@@ -687,6 +687,7 @@ static int buffer_finish(struct vb2_buffer *vb)
parport_release(qcam->pdev);
mutex_unlock(&qcam->lock);
+ v4l2_get_timestamp(&vb->v4l2_buf.timestamp);
if (len != size)
vb->state = VB2_BUF_STATE_ERROR;
vb2_set_plane_payload(vb, 0, len);
@@ -964,6 +965,7 @@ static struct qcam *qcam_init(struct parport *port)
q->drv_priv = qcam;
q->ops = &qcam_video_qops;
q->mem_ops = &vb2_vmalloc_memops;
+ q->timestamp_type = V4L2_BUF_FLAG_TIMESTAMP_MONOTONIC;
err = vb2_queue_init(q);
if (err < 0) {
v4l2_err(v4l2_dev, "couldn't init vb2_queue for %s.\n", port->name);
--
1.7.10.4
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: WARNING: at drivers/media/v4l2-core/videobuf2-core.c:2065 vb2_queue_init+0x74/0x142()
2013-05-10 12:06 ` Hans Verkuil
@ 2013-05-10 14:33 ` Borislav Petkov
0 siblings, 0 replies; 3+ messages in thread
From: Borislav Petkov @ 2013-05-10 14:33 UTC (permalink / raw)
To: Hans Verkuil; +Cc: linux-media
On Fri, May 10, 2013 at 02:06:50PM +0200, Hans Verkuil wrote:
> Can you try this patch? This should fix it.
Yep, it does.
Tested-by: Borislav Petkov <bp@suse.de>
> diff --git a/drivers/media/parport/bw-qcam.c b/drivers/media/parport/bw-qcam.c
> index 06231b8..d12bd33 100644
> --- a/drivers/media/parport/bw-qcam.c
> +++ b/drivers/media/parport/bw-qcam.c
> @@ -687,6 +687,7 @@ static int buffer_finish(struct vb2_buffer *vb)
>
> parport_release(qcam->pdev);
> mutex_unlock(&qcam->lock);
> + v4l2_get_timestamp(&vb->v4l2_buf.timestamp);
> if (len != size)
> vb->state = VB2_BUF_STATE_ERROR;
> vb2_set_plane_payload(vb, 0, len);
> @@ -964,6 +965,7 @@ static struct qcam *qcam_init(struct parport *port)
> q->drv_priv = qcam;
> q->ops = &qcam_video_qops;
> q->mem_ops = &vb2_vmalloc_memops;
> + q->timestamp_type = V4L2_BUF_FLAG_TIMESTAMP_MONOTONIC;
However, just FYI: I do trigger the warning in a guest and not on the
real hardware. So I can't really confirm whether _MONOTONIC is the
proper timestamp type or not. But it looks like you know what you're
doing. :-)
Thanks.
--
Regards/Gruss,
Boris.
Sent from a fat crate under my desk. Formatting is fine.
--
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-05-10 14:26 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-05-08 20:11 WARNING: at drivers/media/v4l2-core/videobuf2-core.c:2065 vb2_queue_init+0x74/0x142() Borislav Petkov
2013-05-10 12:06 ` Hans Verkuil
2013-05-10 14:33 ` Borislav Petkov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox