On Wed, 11 Mar 2009 04:55:19 +0100, Patch from Jean Delvare wrote: > The patch number 10935 was added via Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> > to http://linuxtv.org/hg/v4l-dvb master development tree. > > Kernel patches in this development tree may be modified to be backward > compatible with older kernels. Compatibility modifications will be > removed before inclusion into the mainstream Kernel > > If anyone has any objections, please let us know by sending a message to: > Linux Media Mailing List <linux-media@xxxxxxxxxxxxxxx> Ugh, you committed the wrong patches :( I have sent updated patches with much cleaner code since then: http://www.spinics.net/lists/linux-media/msg02646.html http://www.spinics.net/lists/linux-media/msg02647.html http://www.spinics.net/lists/linux-media/msg02648.html http://www.spinics.net/lists/linux-media/msg02649.html > ------ > > From: Jean Delvare <khali@xxxxxxxxxxxx> > cx88: Prevent general protection fault on rmmod > > > When unloading the cx8800 driver I sometimes get a general protection > fault. Analysis revealed a race in cx88_ir_stop(). It can be solved by > using a delayed work instead of a timer for infrared input polling. > > This fixes kernel.org bug #12802. > > Signed-off-by: Jean Delvare <khali@xxxxxxxxxxxx> > Signed-off-by: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> > > > --- > > linux/drivers/media/video/cx88/cx88-input.c | 27 +++++++++++++++----- > 1 file changed, 21 insertions(+), 6 deletions(-) > > diff -r 60b0989f6c7a -r 46412997b3c0 linux/drivers/media/video/cx88/cx88-input.c > --- a/linux/drivers/media/video/cx88/cx88-input.c Tue Mar 10 19:28:33 2009 -0700 > +++ b/linux/drivers/media/video/cx88/cx88-input.c Thu Mar 05 09:38:24 2009 +0000 > @@ -49,8 +49,12 @@ struct cx88_IR { > > /* poll external decoder */ > int polling; > +#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,20) > struct work_struct work; > struct timer_list timer; > +#else > + struct delayed_work work; > +#endif > u32 gpio_addr; > u32 last_gpio; > u32 mask_keycode; > (...) -- Jean Delvare -- 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
