Skip to contentSkip to conversation

WARNING: at drivers/media/v4l2-core/videobuf2-core.c:2065 vb2_queue_init+0x74/0x142()

DISCUSSION Borislav Petkov 2 participants 3 messages May 8, 2013 – May 10, 2013 Media (V4L/DVB)

lore
Share & tools
lore

Borislav Petkov

author
#0 lore raw

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
Signature · 5 lines
Regards/Gruss,
    Boris.

Sent from a fat crate under my desk. Formatting is fine.
--
#1 lore raw
Quoted 21 lines · 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
Quoted 10 lines
[  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

drivers/media/parport/bw-qcam.c +2 -0
index 06231b8..d12bd33 100644
@@ -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);
Signature · 1 line
1.7.10.4
author Tested-by
#2 lore raw
Quoted 1 line · Hans Verkuil

Can you try this patch? This should fix it.

Yep, it does.

Tested-by Borislav Petkov <[email protected]>
Quoted 13 lines
drivers/media/parport/bw-qcam.c +1 -0
index 06231b8..d12bd33 100644
@@ -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);

drivers/media/parport/bw-qcam.c static struct qcam *qcam_init(struct parport *port) · line 968

Quoted 4 lines
drivers/media/parport/bw-qcam.c +1 -0
@@ -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.

Signature · 5 lines
Regards/Gruss,
    Boris.

Sent from a fat crate under my desk. Formatting is fine.
--