Re: Linux 2.6.9-rc2

[Peter Osterlund]

Gerd Knorr <kraxel@xxxxxxxxxxx> writes:

> On Mon, Sep 13, 2004 at 11:57:44PM +0200, Peter Osterlund wrote:
> > Linus Torvalds <torvalds@xxxxxxxx> writes:
> > 
> > > Gerd Knorr:
> > >   o v4l: bttv driver update
> > 
> > This patch,
> > 
> > Output from dmesg with a working kernel: (-rc2 with the above patch reverted)
> 
> > [ ... ]
> 
> > When running the crashing kernel, the last line in /var/log/messages
> > after the crash is:
> > 
> >     bttv0: pinnacle/mt: id=2 info="PAL+SECAM / stereo" radio=yes
> > 
> > Maybe there is more data that doesn't make it to the disk. I can try
> > again with a serial console if you think that would help.
> 
> That certainly is incomplete, at least the insmod messages should be
> there if mencoder triggeres the crash.  A serial console log would be
> helpful.

I got this with a serial console:

    ...
    bttv0: ioctl 0xc02c5638 (v4l2, VIDIOC_G_FREQUENCY)
    bttv0: ioctl 0xc02c5638 (v4l2, VIDIOC_G_FREQUENCY)
    bttv0: ioctl 0xc0cc5604 (v4l2, VIDIOC_G_FMT)
    bttv0: ioctl 0xc0cc5604 (v4l2, VIDIOC_G_FMT)
    bttv0: ioctl 0xc0cc5604 (v4l2, VIDIOC_G_FMT)
    bttv0: ioctl 0xc0145608 (v4l2, VIDIOC_REQBUFS)
    bttv0: ioctl 0xc0445609 (v4l2, VIDIOC_QUERYBUF)
    bttv0: mmap type=video-cap 0xb791f000+663552
    bttv0: ioctl 0xc044560f (v4l2, VIDIOC_QBUF)
    Unable to handle kernel NULL pointer dereference at virtual a

The computer rebooted while printing the "Unable to handle..."
message. DMA engine gone wild?

> You can also try to reverse only parts of the patch, only the
> changes in bttv-driver.c and/or in bttv-risc.c and see what happens.

I found the change that crashes my computer. This patch is enough to
fix it for me:

diff -puN drivers/media/video/bttv-risc.c~v3_revert drivers/media/video/bttv-risc.c
--- linux/drivers/media/video/bttv-risc.c~v3_revert	2004-09-15 00:18:21.280831360 +0200
+++ linux-petero/drivers/media/video/bttv-risc.c	2004-09-15 00:18:21.283830904 +0200
@@ -153,16 +153,10 @@ bttv_risc_planar(struct bttv *btv, struc
 			chroma = 1;
 			break;
 		case 1:
-			if (!yoffset)
-				chroma = (line & 1) == 0;
-			else
-				chroma = (line & 1) == 1;
+			chroma = !(line & 1);
 			break;
 		case 2:
-			if (!yoffset)
-				chroma = (line & 3) == 0;
-			else
-				chroma = (line & 3) == 2;
+			chroma = !(line & 3);
 			break;
 		default:
 			chroma = 0;

> Which format+size you are capturing with mencoder?

mencoder tv:// -tv driver=v4l2:device=/dev/video0:width=768:height=576:chanlist=europe-west:norm=PAL:channel=$channel -ovc lavc -lavcopts vcodec=mpeg4:vbitrate=$vbitrate -oac mp3lame -lameopts cbr:br=$abitrate -vf pp=lb -endpos $time -o $outfile >$outfile.log 2>&1

-- 
Peter Osterlund - petero2@xxxxxxxxx
http://w1.894.telia.com/~u89404340
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/