Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757915AbYAYSpW (ORCPT ); Fri, 25 Jan 2008 13:45:22 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755638AbYAYSpJ (ORCPT ); Fri, 25 Jan 2008 13:45:09 -0500 Received: from smtp2.linux-foundation.org ([207.189.120.14]:42990 "EHLO smtp2.linux-foundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755582AbYAYSpG (ORCPT ); Fri, 25 Jan 2008 13:45:06 -0500 Date: Fri, 25 Jan 2008 10:44:59 -0800 (PST) From: Linus Torvalds To: Greg KH cc: Andrew Morton , linux-kernel@vger.kernel.org Subject: Re: [GIT PATCH] driver core patches against 2.6.24 In-Reply-To: <20080125071127.GA4860@kroah.com> Message-ID: References: <20080125071127.GA4860@kroah.com> User-Agent: Alpine 1.00 (LFD 882 2007-12-20) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2152 Lines: 58 On Thu, 24 Jan 2008, Greg KH wrote: > > Here are a pretty large number of kobject, documentation, and driver > core patches against your 2.6.24 git tree. I've merged it all, but it causes lots of scary warnings: - from the purely broken ones: ehci_hcd: no version for "struct_module" found: kernel tainted. - to the scary ones: sysfs: duplicate filename 'ehci_hcd' can not be created WARNING: at fs/sysfs/dir.c:424 sysfs_add_one() Pid: 610, comm: insmod Tainted: GF 2.6.24-gb47711bf #28 Call Trace: [] sysfs_add_one+0x54/0xbd [] create_dir+0x4f/0x87 [] sysfs_create_dir+0x35/0x4a [] kobject_get+0x12/0x17 [] kobject_add_internal+0xd9/0x194 [] kobject_add_varg+0x54/0x61 [] __alloc_pages+0x66/0x2ee [] kobject_init+0x42/0x82 [] kobject_init_and_add+0x9a/0xa7 [] __vmalloc_area_node+0x111/0x135 [] mod_sysfs_init+0x6e/0x83 [] sys_init_module+0xa3d/0x1833 [] dput+0x1c/0x10b [] system_call+0x7e/0x83 and the problem seems to be that it does all these checks even for modules that will never be loaded, because I use my own kernel, but with the default Fedora initrd (which is trying to load modules for stuff that I already have built in). Very annoying. Can we please do that *after* the module loading symbol checks have run, so that when you try to load a module that will not load, it won't complain about these silly things? (You can probably trigger this by simply trying to load a module that was compiled for another kernel version - it will fail fine, but in failing it will then generate all these incorrect warnings!) Now it incorrectly taints my kernel, for no good reason. Linus -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/