Re: Question about 2 gp8psk patches I noticed, and possible bug.

2016-11-12 Thread VDR User
https://patchwork.linuxtv.org/patch/38040/raw/) Question-about-2-gp8psk-patches-I-noticed-and-possible-bug..patch Attach bug is fixed, tuning works, module unloads without crashing. Everything seems ok! I think the gp8psk issues are resolved with the above 4 patches. As you know, there are some

Re: Question about 2 gp8psk patches I noticed, and possible bug.

2016-11-12 Thread Mauro Carvalho Chehab
Em Fri, 11 Nov 2016 20:45:31 -0800 VDR User escreveu: > > Sorry, forgot to add one file to the patch. > > > > The right fix is this one. > > This patch seems to fix the unload crash but unfortunately now all I > get is "frontend 0/0 timed out while tuning". I don't see

Re: Question about 2 gp8psk patches I noticed, and possible bug.

2016-11-11 Thread VDR User
>>> Sorry, forgot to add one file to the patch. >>> >>> The right fix is this one. >> >> This patch seems to fix the unload crash but unfortunately now all I >> get is "frontend 0/0 timed out while tuning". > > Forgot to mention that I didn't see the gp8psk-fe entry in menuconfig > customize

Re: Question about 2 gp8psk patches I noticed, and possible bug.

2016-11-11 Thread VDR User
>> Sorry, forgot to add one file to the patch. >> >> The right fix is this one. > > This patch seems to fix the unload crash but unfortunately now all I > get is "frontend 0/0 timed out while tuning". Forgot to mention that I didn't see the gp8psk-fe entry in menuconfig customize frontends even

Re: Question about 2 gp8psk patches I noticed, and possible bug.

2016-11-11 Thread VDR User
> Sorry, forgot to add one file to the patch. > > The right fix is this one. This patch seems to fix the unload crash but unfortunately now all I get is "frontend 0/0 timed out while tuning". > [PATCH v2] [media] gp8psk: Fix DVB frontend attach > > The DVB binding schema at the DVB core

Re: Question about 2 gp8psk patches I noticed, and possible bug.

2016-11-11 Thread Mauro Carvalho Chehab
Em Fri, 11 Nov 2016 19:53:53 -0200 Mauro Carvalho Chehab escreveu: > Em Fri, 11 Nov 2016 07:33:59 -0800 > VDR User escreveu: > > > > Hmm... dvb_attach() assumes that the symbol is exported. Please try > > > this patch. If it fixes the bug, I'll likely

Re: Question about 2 gp8psk patches I noticed, and possible bug.

2016-11-11 Thread Mauro Carvalho Chehab
Em Fri, 11 Nov 2016 07:33:59 -0800 VDR User escreveu: > > Hmm... dvb_attach() assumes that the symbol is exported. Please try > > this patch. If it fixes the bug, I'll likely do something else, to > > avoid the need of EXPORT_SYMBOL. > > > > > > [PATCH] [media] gp8psk: Fix

Re: Question about 2 gp8psk patches I noticed, and possible bug.

2016-11-11 Thread VDR User
> Hmm... dvb_attach() assumes that the symbol is exported. Please try > this patch. If it fixes the bug, I'll likely do something else, to > avoid the need of EXPORT_SYMBOL. > > > [PATCH] [media] gp8psk: Fix DVB frontend attach > > it should be calling module_get() at attach, as otherwise >

Re: Question about 2 gp8psk patches I noticed, and possible bug.

2016-11-11 Thread Mauro Carvalho Chehab
Em Thu, 10 Nov 2016 07:01:44 -0800 VDR User escreveu: > > commit 0c979a12309af49894bb1dc60e747c3cd53fa888 > > Author: Mauro Carvalho Chehab > > Date: Wed Nov 9 15:33:17 2016 -0200 > > > > [media] gp8psk: Fix DVB frontend attach > > > > it

Re: Question about 2 gp8psk patches I noticed, and possible bug.

2016-11-10 Thread VDR User
> commit 0c979a12309af49894bb1dc60e747c3cd53fa888 > Author: Mauro Carvalho Chehab > Date: Wed Nov 9 15:33:17 2016 -0200 > > [media] gp8psk: Fix DVB frontend attach > > it should be calling module_get() at attach, as otherwise > module_put() will crash. > >

Re: Question about 2 gp8psk patches I noticed, and possible bug.

2016-11-10 Thread Mauro Carvalho Chehab
Em Wed, 9 Nov 2016 17:03:52 -0800 VDR User escreveu: > >> (gdb) l *module_put+0x67 > >> 0xc10a4b87 is in module_put (kernel/module.c:1108). > >> 1103int ret; > >> 1104 > >> 1105if (module) { > >> 1106preempt_disable(); > >> 1107

Re: Question about 2 gp8psk patches I noticed, and possible bug.

2016-11-09 Thread VDR User
>> (gdb) l *module_put+0x67 >> 0xc10a4b87 is in module_put (kernel/module.c:1108). >> 1103int ret; >> 1104 >> 1105if (module) { >> 1106preempt_disable(); >> 1107ret = atomic_dec_if_positive(>refcnt); >> 1108

Re: Question about 2 gp8psk patches I noticed, and possible bug.

2016-11-09 Thread Mauro Carvalho Chehab
Em Wed, 9 Nov 2016 07:49:43 -0800 VDR User escreveu: > $ gdb /usr/src/linux/vmlinux > GNU gdb (Debian 7.11.1-2) 7.11.1 > ... > Reading symbols from /usr/src/linux/vmlinux...done. > (gdb) l *module_put+0x67 > 0xc10a4b87 is in module_put (kernel/module.c:1108). > 1103

Re: Question about 2 gp8psk patches I noticed, and possible bug.

2016-11-09 Thread VDR User
$ gdb /usr/src/linux/vmlinux GNU gdb (Debian 7.11.1-2) 7.11.1 ... Reading symbols from /usr/src/linux/vmlinux...done. (gdb) l *module_put+0x67 0xc10a4b87 is in module_put (kernel/module.c:1108). 1103int ret; 1104 1105if (module) { 1106preempt_disable();

Re: Question about 2 gp8psk patches I noticed, and possible bug.

2016-11-09 Thread VDR User
Hi Mauro, Here are the results after testing the latest patch: [33922.643770] usbcore: deregistering interface driver dvb_usb_gp8psk [33922.643789] gp8psk: unregistering fe0 [33922.643865] gp8psk: detaching fe0 [33922.643868] [ cut here ] [33922.643875] WARNING: CPU: 1

Re: Question about 2 gp8psk patches I noticed, and possible bug.

2016-11-09 Thread Mauro Carvalho Chehab
Em Tue, 8 Nov 2016 22:00:41 -0800 VDR User escreveu: > Hi Mauro, > > Unfortunately the patch doesn't seem to have solved the problem. I do > have the kernel recompiled with debug enabled though per your irc msg. > dmesg gives me: > > [ 70.741073] usbcore: deregistering

Re: Question about 2 gp8psk patches I noticed, and possible bug.

2016-11-08 Thread VDR User
Hi Mauro, Unfortunately the patch doesn't seem to have solved the problem. I do have the kernel recompiled with debug enabled though per your irc msg. dmesg gives me: [ 70.741073] usbcore: deregistering interface driver dvb_usb_gp8psk [ 70.741165] [ cut here ] [

Re: Question about 2 gp8psk patches I noticed, and possible bug.

2016-11-08 Thread Mauro Carvalho Chehab
Em Sat, 5 Nov 2016 19:24:58 -0700 VDR User escreveu: > I have > several different Genpix devices that use the gp8psk driver and in all > cases the following happens when I unload it: > > [494896.234616] usbcore: deregistering interface driver dvb_usb_gp8psk > [494896.234712]