From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from services.gcu-squad.org (zone0.gcu-squad.org [212.85.147.21]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 81B7AB6F01 for ; Wed, 11 Aug 2010 00:30:20 +1000 (EST) Date: Tue, 10 Aug 2010 15:43:37 +0200 From: Jean Delvare To: Greg KH Subject: Re: [PATCH 0/6] Remove owner field from sysfs attribute structure Message-ID: <20100810154337.006a0df9@hyperion.delvare> In-Reply-To: <20100802233128.GA21286@kroah.com> References: <1280380166-29196-1-git-send-email-guenter.roeck@ericsson.com> <20100802233128.GA21286@kroah.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Cc: Mark Brown , Jani Nikula , Linus Walleij , James Smart , Eric Biederman , Benjamin Thery , Greg Kroah-Hartman , linux-kernel@vger.kernel.org, Chris Wright , linuxppc-dev@ozlabs.org, "James E.J. Bottomley" , Richard Purdie , Nick Cheng , Dmitry Torokhov , Tejun Heo , Paul Mackerras , Liam Girdwood , linux-scsi@vger.kernel.org, Alex Iannicelli , Guenter Roeck List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Mon, 2 Aug 2010 16:31:28 -0700, Greg KH wrote: > On Wed, Jul 28, 2010 at 11:16:35PM -0700, Eric Biederman wrote: > > On Wed, Jul 28, 2010 at 10:09 PM, Guenter Roeck > > wrote: > > > The following comment is found in include/linux/sysfs.h: > > > > > > =C2=A0 /* FIXME > > > =C2=A0 =C2=A0* The *owner field is no longer used. > > > =C2=A0 =C2=A0* x86 tree has been cleaned up. The owner > > > =C2=A0 =C2=A0* attribute is still left for other arches. > > > =C2=A0 =C2=A0*/ > > > > > > As it turns out, the *owner field is (again?) initialized in several = modules, > > > suggesting that such initialization may be creeping back into the cod= e. > > > > > > This patch set removes the above comment, the *owner field, and each = instance > > > in the code where it was found to be initialized. > > > > > > Compiled with x86 allmodconfig as well as with all alpha, arm, mips, = powerpc, > > > and sparc defconfig builds. > >=20 > > This seems reasonable to me. Can we get this in linux-next? >=20 > It will show up in linux-next tomorrow. Related bug? https://bugzilla.kernel.org/show_bug.cgi?id=3D16544 --=20 Jean Delvare