lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sat, 24 Aug 2013 21:09:39 -0700 From: Greg Kroah-Hartman <gregkh@...uxfoundation.org> To: Fengguang Wu <fengguang.wu@...el.com> Cc: linux-kernel@...r.kernel.org, Samuel Ortiz <sameo@...ux.intel.com>, Ben Dooks <ben@...tec.co.uk>, Wolfram Sang <w.sang@...gutronix.de>, Russell King <rmk+kernel@....linux.org.uk> Subject: Re: [tps_init] BUG: unable to handle kernel paging request at 484970c9 On Sun, Aug 25, 2013 at 08:04:21AM +0800, Fengguang Wu wrote: > Greetings, > > It seems the below patch discloses a bug in tps_init(). > > commit c817a67ecba7c3c2aaa104796d78f160af60920d > Author: Russell King <rmk+kernel@....linux.org.uk> > Date: Thu Jun 27 15:06:14 2013 +0100 > > kobject: delayed kobject release: help find buggy drivers > > Implement debugging for kobject release functions. kobjects are > reference counted, so the drop of the last reference to them is not > predictable. However, the common case is for the last reference to be > the kobject's removal from a subsystem, which results in the release > function being immediately called. > > This can hide subtle bugs, which can occur when another thread holds a > reference to the kobject at the same time that a kobject is removed. > This results in the release method being delayed. > > In order to make these kinds of problems more visible, the following > patch implements a delayed release; this has the effect that the > release function will be out of order with respect to the removal of > the kobject in the same manner that it would be if a reference was > being held. > > This provides us with an easy way to allow driver writers to debug > their drivers and fix otherwise hidden problems. > > Signed-off-by: Russell King <rmk+kernel@....linux.org.uk> > Signed-off-by: Greg Kroah-Hartman <gregkh@...uxfoundation.org> > > > Parent commit not clean. Look out for wrong bisect! > > BUG: kernel boot crashed > > /kernel/i386-randconfig-j1-08220808/7c42721fe0c58a848849b43ff558cf2fb86aa35a/dmesg-kvm-nhm4-6710-20130824125721-3.11.0-rc2-00007-ge56341a-282 > > [ 0.304827] WARNING: CPU: 0 PID: 1 at fs/sysfs/dir.c:530 sysfs_add_one+0x89/0x9b() > [ 0.306316] sysfs: cannot create duplicate filename '/bus/i2c/drivers/tps65010' Same as before, are you unloading and loading modules? We have a fix for modules that cause problems when unloading with the config option above enabled. But that shouldn't be this issue. thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists