Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753374Ab0ALDCm (ORCPT ); Mon, 11 Jan 2010 22:02:42 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752691Ab0ALDCl (ORCPT ); Mon, 11 Jan 2010 22:02:41 -0500 Received: from bld-mail18.adl2.internode.on.net ([150.101.137.103]:34156 "EHLO mail.internode.on.net" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752610Ab0ALDCl (ORCPT ); Mon, 11 Jan 2010 22:02:41 -0500 Date: Tue, 12 Jan 2010 14:01:34 +1100 From: Dave Chinner To: "Eric W. Biederman" Cc: Tejun Heo , "Rafael J. Wysocki" , =?iso-8859-1?Q?Am=E9rico?= Wang , Miles Lane , LKML , Greg Kroah-Hartman , Jesse Barnes , Len Brown , Pavel Machek , Arjan van de Ven , Peter Zijlstra , Ingo Molnar Subject: Re: 2.6.33-rc3 -- INFO: possible recursive locking -- (s_active){++++.+}, at: [] sysfs_hash_and_remove+0x3d/0x4f Message-ID: <20100112030134.GA17483@discord.disaster> References: <2375c9f91001100058w4bac1cf9s183fc37eafbfde75@mail.gmail.com> <201001101935.31882.rjw@sisk.pl> <4B4BC2D3.7070003@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.18 (2008-05-17) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1359 Lines: 40 On Mon, Jan 11, 2010 at 04:32:31PM -0800, Eric W. Biederman wrote: > Tejun Heo writes: > > > On 01/11/2010 11:26 AM, Eric W. Biederman wrote: > >> From: Eric W. Biederman > >> Date: Sun, 10 Jan 2010 18:13:35 -0800 > >> Subject: [PATCH] sysfs: Add support for lockdep subclasses to s_active > >> > >> We have apparently valid cases where the code for a sysfs attribute > >> removes other sysfs attributes. Without support for subclasses > >> lockdep flags a possible recursive lock problem as it figures > >> the first sysfs attribute could be attempting to remove itself. > >> > >> By adding support for sysfs subclasses we can teach lockdep to > >> distinguish between different types of sysfs attributes and not > >> get confused. > >> > >> Signed-off-by: Eric W. Biederman > > > > Acked-by: Tejun Heo > > Now if I can just get a Tested-by this patch will be all set ;) Hi Eric, Is this the same locking problem that this patch fixes? http://lkml.org/lkml/2010/1/11/26 Cheers, Dave. -- Dave Chinner david@fromorbit.com -- 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/