From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1762774AbYD3PgS (ORCPT ); Wed, 30 Apr 2008 11:36:18 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1756575AbYD3PgG (ORCPT ); Wed, 30 Apr 2008 11:36:06 -0400 Received: from smtp1.linux-foundation.org ([140.211.169.13]:41130 "EHLO smtp1.linux-foundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752293AbYD3PgF (ORCPT ); Wed, 30 Apr 2008 11:36:05 -0400 Date: Wed, 30 Apr 2008 08:34:35 -0700 From: Andrew Morton To: "Michael Krufky" Cc: "Mike Galbraith" , sfr@canb.auug.org.au, video4linux-list@redhat.com, "Sam Ravnborg" , linux-kernel@vger.kernel.org, mchehab@infradead.org, linux-dvb-maintainer@linuxtv.org, "Ingo Molnar" , torvalds@linux-foundation.org, "David Miller" Subject: Re: [v4l-dvb-maintainer] [patch, -git] drivers/media build fix for modular builds Message-Id: <20080430083435.4302f82a.akpm@linux-foundation.org> In-Reply-To: <37219a840804300739t6bade9c0o2eeaeb0ef42dffee@mail.gmail.com> References: <20080429185009.716c3284@gaivota> <20080430074807.GA11224@elte.hu> <20080430.005212.46815036.davem@davemloft.net> <20080430081345.GA16519@elte.hu> <1209544030.30990.2.camel@marge.simson.net> <1209544951.15609.3.camel@marge.simson.net> <1209545784.31419.2.camel@marge.simson.net> <1209547231.4898.2.camel@marge.simson.net> <20080430044009.6c5bbc8d.akpm@linux-foundation.org> <1209560592.5265.2.camel@marge.simson.net> <37219a840804300739t6bade9c0o2eeaeb0ef42dffee@mail.gmail.com> X-Mailer: Sylpheed 2.4.8 (GTK+ 2.12.5; x86_64-redhat-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 30 Apr 2008 10:39:31 -0400 "Michael Krufky" wrote: > On Wed, Apr 30, 2008 at 9:03 AM, Mike Galbraith wrote: > > > > On Wed, 2008-04-30 at 04:40 -0700, Andrew Morton wrote: > > > On Wed, 30 Apr 2008 11:20:31 +0200 Mike Galbraith wrote: > > > > > > > [ 16.299803] kernel BUG at kernel/module.c:784! > > > > [ 16.307170] invalid opcode: 0000 [1] SMP > > > > [ 16.314110] CPU 3 > > > > [ 16.318911] Modules linked in: tuner(+) snd_hda_intel(+) saa7134(+) compat_ioctl32 firewire_ohci videodev snd_pcm firewire_core v4l1_compat v4l2_common snd_timer videobuf_dma_sg videobuf_core crc_itu_t snd_page_alloc snd_hwdep ir_kbd_i2c ohci1394 snd ir_common usb_storage rtc_cmos ieee1394 tveeprom rtc_core i2c_i801 button sr_mod intel_agp serio_raw e1000e iTCO_wdt i2c_core iTCO_vendor_support rtc_lib cdrom sg soundcore usbhid hid ff_memless ehci_hcd uhci_hcd sd_mod usbcore edd ext3 mbcache jbd fan ahci libata scsi_mod dock thermal processor > > > > [ 16.386303] Pid: 2239, comm: modprobe Not tainted 2.6.26-smp #1 > > > > [ 16.395339] RIP: 0010:[] [] __symbol_put+0x22/0x31 > > > > > > Does this make it feel better? > > > > Well, it doesn't go BUG() any more, but it now goes oops. > > > > [ 13.443404] saa7133[0]: i2c eeprom 00: be 16 0d 00 54 20 1c 00 43 43 a9 1c 55 d2 b2 92 > > [ 13.456281] saa7133[0]: i2c eeprom 10: 00 ff 86 0f ff 20 ff 00 01 50 32 79 01 3c ca 50 > > [ 13.469575] saa7133[0]: i2c eeprom 20: 01 40 01 02 02 03 01 00 06 ff 00 29 02 51 96 2b > > [ 13.483794] saa7133[0]: i2c eeprom 30: a7 58 7a 1f 03 8e 84 5e da 7a 04 b3 05 87 b2 3c > > [ 13.498518] saa7133[0]: i2c eeprom 40: ff 28 00 c0 96 10 03 00 c0 1c fd 79 44 9f c2 8f > > [ 13.507664] saa7133[0]: i2c eeprom 50: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff > > [ 13.507670] saa7133[0]: i2c eeprom 60: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff > > [ 13.507676] saa7133[0]: i2c eeprom 70: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff > > [ 13.507680] saa7133[0]: i2c eeprom 80: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff > > [ 13.507684] saa7133[0]: i2c eeprom 90: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff > > [ 13.507688] saa7133[0]: i2c eeprom a0: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff > > [ 13.507692] saa7133[0]: i2c eeprom b0: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff > > [ 13.507696] saa7133[0]: i2c eeprom c0: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff > > [ 13.507700] saa7133[0]: i2c eeprom d0: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff > > [ 13.507703] saa7133[0]: i2c eeprom e0: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff > > [ 13.507707] saa7133[0]: i2c eeprom f0: ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff ff > > [ 13.666587] TUNER: Unable to find symbol tda829x_probe() > > [ 13.674638] tuner' 1-004b: chip found @ 0x96 (saa7133[0]) > > [ 13.691175] DVB: Unable to find symbol tda9887_attach() > > [ 13.698968] BUG: unable to handle kernel NULL pointer dereference at 0000000000000000 > > [ 13.709509] IP: [] strlcpy+0x11/0x36 > > [ 13.711135] PGD be167067 PUD be140067 PMD 0 > > [ 13.711137] Oops: 0000 [1] SMP > > [ 13.711139] CPU 2 > > [ 13.711140] Modules linked in: tuner(+) snd_hda_intel(+) saa7134(+) compat_ioctl32 snd_pcm videodev firewire_ohci v4l1_compat firewire_core v4l2_common snd_timer videobuf_dma_sg videobuf_core crc_itu_t snd_page_alloc snd_hwdep ir_kbd_i2c rtc_cmos snd rtc_core ohci1394 ir_common usb_storage usbhid tveeprom ieee1394 sr_mod e1000e i2c_i801 rtc_lib cdrom i2c_core hid iTCO_wdt button intel_agp serio_raw ff_memless soundcore iTCO_vendor_support sg ehci_hcd uhci_hcd sd_mod usbcore edd ext3 mbcache jbd fan ahci libata scsi_mod dock thermal processor > > [ 13.711158] Pid: 2175, comm: modprobe Not tainted 2.6.26-smp #3 > > [ 13.711160] RIP: 0010:[] [] strlcpy+0x11/0x36 > > [ 13.711163] RSP: 0018:ffff8100bcc45b68 EFLAGS: 00010286 > > [ 13.711164] RAX: 0000000000000000 RBX: 0000000000000000 RCX: ffffffffffffffff > > [ 13.711165] RDX: 0000000000000014 RSI: 0000000000000000 RDI: 0000000000000000 > > [ 13.711167] RBP: ffff8100bcc45b68 R08: ffff8100bf0b7404 R09: 0000000000000000 > > [ 13.711169] R10: ffff8100bcc45908 R11: ffff81003784ab70 R12: ffff8100bf0b6400 > > [ 13.711170] R13: ffff8100bf0b7400 R14: 000000000000004a R15: 000000000000000e > > [ 13.711172] FS: 00007fcb8bad66f0(0000) GS:ffff8100bf806780(0000) knlGS:0000000000000000 > > [ 13.711174] CS: 0010 DS: 0000 ES: 0000 CR0: 000000008005003b > > [ 13.711175] CR2: 0000000000000000 CR3: 00000000be142000 CR4: 00000000000006e0 > > [ 13.711177] DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000 > > [ 13.711178] DR3: 0000000000000000 DR6: 00000000ffff0ff0 DR7: 0000000000000400 > > [ 13.711180] Process modprobe (pid: 2175, threadinfo ffff8100bcc44000, task ffff8100bd112c40) > > [ 13.711181] Stack: ffff8100bcc45c08 ffffffffa02d43a3 0000000000000096 ffff8100bd486490 > > [ 13.711185] 0000000000000000 ffffffffa02d681c ffff8100bcc45bd8 ffffffff80253b08 > > [ 13.711187] ffff8100bcc45bf8 ffff8100bf0b7400 ffff8100bf0b6400 ffffffffa02d8e80 > > [ 13.711190] Call Trace: > > [ 13.711196] [] :tuner:set_type+0x682/0x9ed I worry about this case: case 0x42: case 0x43: case 0x4a: case 0x4b: /* If chip is not tda8290, don't register. since it can be tda9887*/ if (tuner_symbol_probe(tda829x_probe, t->i2c->adapter, t->i2c->addr) == 0) { tuner_dbg("tda829x detected\n"); } else { /* Default is being tda9887 */ t->type = TUNER_TDA9887; t->mode_mask = T_RADIO | T_ANALOG_TV | T_DIGITAL_TV; t->mode = T_STANDBY; goto register_client; } break; did we really want to go ahead and perform registration if tuner_symbol_probe() failed?