Re: em28xx list_add corruption reported by list debug

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Em Sun, 22 Dec 2013 13:06:00 -0200
Mauro Carvalho Chehab <m.chehab@xxxxxxxxxxx> escreveu:

> Em Sun, 22 Dec 2013 00:27:21 +0200
> Antti Palosaari <crope@xxxxxx> escreveu:
> 
> > I ran also this kind of bug. Device was PCTV 290e, which has that video 
> > unused. I have no any analog em28xx webcam to test if that happens here too.
> > 
> > Fortunately I found one video device which does not crash nor dump debug 
> > bug warnings. It is some old gspca webcam. Have to look example how 
> > those videobuf callbacks are implemented there..
> > 
> > regards
> > Antti
> > 
> > 
> > [crope@localhost linux]$ cat /dev/video0
> > cat: /dev/video0: Invalid argument
> > [crope@localhost linux]$ cat /dev/video0
> > cat: /dev/video0: Device or resource busy
> > [crope@localhost linux]$
> > 
> > 
> > joulu 22 00:08:24 localhost.localdomain kernel: em28174 #0: no endpoint 
> > for analog mode and transfer type 0
> 
> It seems that there's something bad on em28174 registration: it should not
> be creating a v4l2 device, if the device is DVB only.
> 
> The thing is that, when this driver was created, all devices were either
> analog only or hybrid. Only very recently, pure DVB devices got added.
> 
> It shouldn't be that hard to split em28xx_init_dev() into a few routines
> that would only register v4l2 if the device has analog support.
> 
> Again, this changeset:
> 	https://patchwork.linuxtv.org/patch/17967/
> 
> Seems to be part of such solution, as it already splits the v4l2
> register logic into a separate function. 

Ok, if I didn't make any mistake, this changeset should do the trick:
	https://patchwork.linuxtv.org/patch/21282/

Please notice that this is compile-tested only.

Regards,
Mauro
-- 

Cheers,
Mauro
--
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




[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux