Re: [media/dvb_usb_af9005] BUG: unable to handle kernel paging request (WAS: [media/em28xx] BUG: unable to handle kernel)

[Antti Palosaari]

Moikka!

On 09/19/2014 10:55 PM, Luca Olivetti wrote:
> El 19/09/14 21:22, Frank Schäfer ha escrit:
>
> > > So symbol_request() returns pointers.!= NULL
> > >
> > > A closer look at the definition of symbol_request() shows, that it does
> > > nothing if CONFIG_MODULES is disabled (it just returns its argument).
> > >
> > >
> > > One possibility to fix this bug would be to embrace these three lines with
> > >
> > > #ifdef CONFIG_DVB_USB_AF9005_REMOTE
> > > ...
> > > #endif
> > Luca, what do you think ?
> >
> > This seems to be an ancient bug, which is known at least since 5 1/2 years:
> > https://lkml.org/lkml/2009/2/4/350
>
> Well, it's been a while so I don't remember the details, but I think the
> same now as then ;-)
> The idea behind CONFIG_DVB_USB_AF9005_REMOTE was to provide an
> alternative implementation (based on lirc, at the time it wasn't in the
> kernel), since this adapter doesn't decode the IR pulses by itself.
> In theory you could leave it undefined but still provide an
> implementation in a different module. Just adding
>
> #ifdef CONFIG_DVB_USB_AF9005_REMOTE
>
> would nuke the (futile?) effort.
>
> Now, since the problem seems to be with CONFIG_MODULES disabled, maybe
> you could combine both conditions
>
> #if defined(CONFIG_MODULE) || defined(CONFIG_DVB_USB_AF9005_REMOTE)

Proper fix is to remove whole home made IR decider and use common IR 
decoders in kernel nowadays... And yes, that is very old driver and at 
the time Luca made it there was no IR decoders nor whole remote 
controller framework.

regards
Antti

--
http://palosaari.fi/
--
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/