Message ID | 527EBEA4.1070202@xs4all.nl (mailing list archive) |
---|---|
State | Accepted, archived |
Delegated to: | Hans Verkuil |
Headers |
Received: from mail.tu-berlin.de ([130.149.7.33]) by www.linuxtv.org with esmtp (Exim 4.72) (envelope-from <linux-media-owner@vger.kernel.org>) id 1VfHWr-0007ie-Nu; Sun, 10 Nov 2013 00:01:21 +0100 X-tubIT-Incoming-IP: 209.132.180.67 Received: from vger.kernel.org ([209.132.180.67]) by mail.tu-berlin.de (exim-4.72/mailfrontend-5) with esmtp id 1VfHWp-0001oZ-8h; Sun, 10 Nov 2013 00:01:21 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758319Ab3KIXA7 (ORCPT <rfc822;mkrufky@linuxtv.org> + 1 other); Sat, 9 Nov 2013 18:00:59 -0500 Received: from smtp-vbr7.xs4all.nl ([194.109.24.27]:1746 "EHLO smtp-vbr7.xs4all.nl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758289Ab3KIXA7 (ORCPT <rfc822; linux-media@vger.kernel.org>); Sat, 9 Nov 2013 18:00:59 -0500 Received: from tschai.lan (209.80-203-20.nextgentel.com [80.203.20.209] (may be forged)) (authenticated bits=0) by smtp-vbr7.xs4all.nl (8.13.8/8.13.8) with ESMTP id rA9N0sLX039316; Sun, 10 Nov 2013 00:00:55 +0100 (CET) (envelope-from hverkuil@xs4all.nl) Received: from [127.0.0.1] (localhost [127.0.0.1]) by tschai.lan (Postfix) with ESMTPSA id 84A892A1F7C; Sun, 10 Nov 2013 00:00:52 +0100 (CET) Message-ID: <527EBEA4.1070202@xs4all.nl> Date: Sun, 10 Nov 2013 00:00:52 +0100 From: Hans Verkuil <hverkuil@xs4all.nl> User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20131103 Icedove/17.0.10 MIME-Version: 1.0 To: =?ISO-8859-1?Q?Lorenz_R=F6hrl?= <sheepshit@gmx.de> CC: linux-media@vger.kernel.org Subject: Re: BUG: Freeze upon loading bttv module References: <527E606A.40101@gmx.de> In-Reply-To: <527E606A.40101@gmx.de> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8bit X-Virus-Scanned: by XS4ALL Virus Scanner Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: <linux-media.vger.kernel.org> X-Mailing-List: linux-media@vger.kernel.org X-PMX-Version: 6.0.0.2142326, Antispam-Engine: 2.7.2.2107409, Antispam-Data: 2013.11.9.225415 X-PMX-Spam: Gauge=IIIIIIII, Probability=8%, Report=' HTML_00_01 0.05, HTML_00_10 0.05, MSGID_ADDED_BY_MTA 0.05, BODYTEXTP_SIZE_3000_LESS 0, BODY_SIZE_2000_2999 0, BODY_SIZE_5000_LESS 0, BODY_SIZE_7000_LESS 0, ECARD_WORD 0, LINK_TO_IMAGE 0, URI_ENDS_IN_HTML 0, __ANY_URI 0, __BOUNCE_CHALLENGE_SUBJ 0, __BOUNCE_NDR_SUBJ_EXEMPT 0, __CP_MEDIA_BODY 0, __CP_URI_IN_BODY 0, __CT 0, __CTE 0, __CT_TEXT_PLAIN 0, __FORWARDED_MSG 0, __HAS_FROM 0, __HAS_MSGID 0, __HAS_X_MAILING_LIST 0, __IN_REP_TO 0, __MIME_TEXT_ONLY 0, __MIME_VERSION 0, __MOZILLA_MSGID 0, __MOZILLA_USER_AGENT 0, __SANE_MSGID 0, __STOCK_PHRASE_7 0, __SUBJ_ALPHA_END 0, __SUBJ_ALPHA_NEGATE 0, __TO_MALFORMED_2 0, __URI_NO_WWW 0, __URI_NS , __USER_AGENT 0' |
Commit Message
Hans Verkuil
Nov. 9, 2013, 11 p.m. UTC
Hi Lorenz, On 11/09/2013 05:18 PM, Lorenz Röhrl wrote: > Hi, > > i'm having problems loading the bttv-module for my bt878 based DVB-T > card: my system just freezes. Magic-Syskeys also won't work then. > With kernel 3.9.0 this worked just fine. Versions 3.10, 3.11 and 3.12 > won't work. > > Last messages on screen with 3.12 upon booting/loading the module is: > http://abload.de/img/bttv_freezeqxdn2.png > > With kernel 3.9 i get an additional line on module loading and the > device works fine: > [ 1.895037] bttv: 0: add subdevice "dvb0" > > I traced the problem, it dies somewhere in v4l2_ctrl_handler_setup on > line 4169 > https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/drivers/media/pci/bt8xx/bttv-driver.c#n4169 Can you try this patch? I'm not 100% but I think this might be the cause of the problem. Regards, Hans > > lspci output from kernel 3.9: > [...] > 04:01.0 Multimedia video controller: Brooktree Corporation Bt878 Video > Capture (rev 11) > Subsystem: Twinhan Technology Co. Ltd VisionPlus DVB card > Flags: bus master, medium devsel, latency 32, IRQ 16 > Memory at f0401000 (32-bit, prefetchable) [size=4K] > Capabilities: [44] Vital Product Data > Capabilities: [4c] Power Management version 2 > Kernel driver in use: bttv > Kernel modules: bttv > > 04:01.1 Multimedia controller: Brooktree Corporation Bt878 Audio Capture > (rev 11) > Subsystem: Twinhan Technology Co. Ltd VisionPlus DVB Card > Flags: bus master, medium devsel, latency 32, IRQ 16 > Memory at f0400000 (32-bit, prefetchable) [size=4K] > Capabilities: [44] Vital Product Data > Capabilities: [4c] Power Management version 2 > Kernel driver in use: bt878 > Kernel modules: bt878 > > > > Please CC me as i'm not subscribed to the list. > > Thanks! > > - Lorenz > -- > To unsubscribe from this list: send the line "unsubscribe linux-media" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Comments
Hi Hans, On 11/10/2013 12:00 AM, Hans Verkuil wrote: > > Can you try this patch? I'm not 100% but I think this might be the cause of > the problem. > > diff --git a/drivers/media/pci/bt8xx/bttv-driver.c b/drivers/media/pci/bt8xx/bttv-driver.c > index c6532de..4f0aaa5 100644 > --- a/drivers/media/pci/bt8xx/bttv-driver.c > +++ b/drivers/media/pci/bt8xx/bttv-driver.c > @@ -4182,7 +4182,8 @@ static int bttv_probe(struct pci_dev *dev, const struct pci_device_id *pci_id) > } > btv->std = V4L2_STD_PAL; > init_irqreg(btv); > - v4l2_ctrl_handler_setup(hdl); > + if (!bttv_tvcards[btv->c.type].no_video) > + v4l2_ctrl_handler_setup(hdl); > if (hdl->error) { > result = hdl->error; > goto fail2; > > I tried the patch and indeed it's working :) No freeze on loading the module and the dvb-device is also working. lolo@hurra ~ % ls /dev/dvb/adapter0 demux0 dvr0 frontend0 lolo@hurra ~ % dmesg |grep bttv [ 0.871060] bttv: driver version 0.9.19 loaded [ 0.872005] bttv: using 8 buffers with 2080k (520 pages) each for capture [ 0.873137] bttv: Bt8xx card found (0) [ 0.874186] bttv: 0: Bt878 (rev 17) at 0000:04:01.0, irq: 16, latency: 32, mmio: 0xf0401000 [ 0.875156] bttv: 0: detected: Twinhan VisionPlus DVB [card=113], PCI subsystem ID is 1822:0001 [ 0.876138] bttv: 0: using: Twinhan DST + clones [card=113,autodetected] [ 0.884082] bttv: 0: tuner absent [ 0.894011] bttv: 0: add subdevice "dvb0" [ 0.901398] DVB: registering new adapter (bttv0) Will this patch be included upstream? When will it appear in official kernel sources? Thanks a lot! - Lorenz -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Lorenz, On 11/10/2013 09:35 AM, Lorenz Röhrl wrote: > Hi Hans, > > > On 11/10/2013 12:00 AM, Hans Verkuil wrote: >> >> Can you try this patch? I'm not 100% but I think this might be the cause of >> the problem. >> >> diff --git a/drivers/media/pci/bt8xx/bttv-driver.c b/drivers/media/pci/bt8xx/bttv-driver.c >> index c6532de..4f0aaa5 100644 >> --- a/drivers/media/pci/bt8xx/bttv-driver.c >> +++ b/drivers/media/pci/bt8xx/bttv-driver.c >> @@ -4182,7 +4182,8 @@ static int bttv_probe(struct pci_dev *dev, const struct pci_device_id *pci_id) >> } >> btv->std = V4L2_STD_PAL; >> init_irqreg(btv); >> - v4l2_ctrl_handler_setup(hdl); >> + if (!bttv_tvcards[btv->c.type].no_video) >> + v4l2_ctrl_handler_setup(hdl); >> if (hdl->error) { >> result = hdl->error; >> goto fail2; >> >> > > I tried the patch and indeed it's working :) > No freeze on loading the module and the dvb-device is also working. Thanks for testing this! Good news that it fixed the problem. > > lolo@hurra ~ % ls /dev/dvb/adapter0 > demux0 dvr0 frontend0 > > lolo@hurra ~ % dmesg |grep bttv > [ 0.871060] bttv: driver version 0.9.19 loaded > [ 0.872005] bttv: using 8 buffers with 2080k (520 pages) each for capture > [ 0.873137] bttv: Bt8xx card found (0) > [ 0.874186] bttv: 0: Bt878 (rev 17) at 0000:04:01.0, irq: 16, > latency: 32, mmio: 0xf0401000 > [ 0.875156] bttv: 0: detected: Twinhan VisionPlus DVB [card=113], PCI > subsystem ID is 1822:0001 > [ 0.876138] bttv: 0: using: Twinhan DST + clones [card=113,autodetected] > [ 0.884082] bttv: 0: tuner absent > [ 0.894011] bttv: 0: add subdevice "dvb0" > [ 0.901398] DVB: registering new adapter (bttv0) > > > Will this patch be included upstream? When will it appear in official > kernel sources? I'll make a pull request for this tomorrow for 3.13 with a CC to the stable kernel mailinglist. It will probably take a few weeks before it appears in the mainline kernel and in the older, stable, kernels. I would expect this to be fixed by the end of the year. Regards, Hans -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/media/pci/bt8xx/bttv-driver.c b/drivers/media/pci/bt8xx/bttv-driver.c index c6532de..4f0aaa5 100644 --- a/drivers/media/pci/bt8xx/bttv-driver.c +++ b/drivers/media/pci/bt8xx/bttv-driver.c @@ -4182,7 +4182,8 @@ static int bttv_probe(struct pci_dev *dev, const struct pci_device_id *pci_id) } btv->std = V4L2_STD_PAL; init_irqreg(btv); - v4l2_ctrl_handler_setup(hdl); + if (!bttv_tvcards[btv->c.type].no_video) + v4l2_ctrl_handler_setup(hdl); if (hdl->error) { result = hdl->error; goto fail2;