From mboxrd@z Thu Jan 1 00:00:00 1970 Received: with ECARTIS (v1.0.0; list linux-mips); Wed, 29 Jun 2011 19:24:53 +0200 (CEST) Received: from bedivere.hansenpartnership.com ([66.63.167.143]:42052 "EHLO bedivere.hansenpartnership.com" rhost-flags-OK-OK-OK-OK) by eddie.linux-mips.org with ESMTP id S1491139Ab1F2RYq (ORCPT ); Wed, 29 Jun 2011 19:24:46 +0200 Received: from localhost (localhost [127.0.0.1]) by bedivere.hansenpartnership.com (Postfix) with ESMTP id 454518EE094; Wed, 29 Jun 2011 10:24:40 -0700 (PDT) Received: from bedivere.hansenpartnership.com ([127.0.0.1]) by localhost (bedivere.hansenpartnership.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id fSz0A2jakJte; Wed, 29 Jun 2011 10:24:40 -0700 (PDT) Received: from [192.168.2.107] (dagonet.hansenpartnership.com [76.243.235.53]) by bedivere.hansenpartnership.com (Postfix) with ESMTPSA id B02D98EE065; Wed, 29 Jun 2011 10:24:38 -0700 (PDT) Subject: Re: [PATCH 00/12] Fix various section mismatches and build errors. From: James Bottomley To: Ralf Baechle Cc: Greg KH , David Miller , akpm@linux-foundation.org, alan@linux.intel.com, bcasavan@sgi.com, airlied@linux.ie, grundler@parisc-linux.org, perex@perex.cz, rpurdie@rpsys.net, klassert@mathematik.tu-chemnitz.de, tj@kernel.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-mips@linux-mips.org, linux-scsi@vger.kernel.org, linux-serial@vger.kernel.org, netdev@vger.kernel.org In-Reply-To: <20110629161921.GA25833@linux-mips.org> References: <17dd5038b15d7135791aadbe80464a13c80758d3.1309182742.git.ralf@linux-mips.org> <20110627.221257.1290251511587162468.davem@davemloft.net> <20110629130711.GA15649@linux-mips.org> <1309355899.2551.4.camel@mulgrave> <20110629151424.GD18023@kroah.com> <20110629161921.GA25833@linux-mips.org> Content-Type: text/plain; charset="UTF-8" Date: Wed, 29 Jun 2011 12:24:36 -0500 Message-ID: <1309368276.13937.2.camel@mulgrave> Mime-Version: 1.0 X-Mailer: Evolution 2.32.2 (2.32.2-1.fc14) Content-Transfer-Encoding: 7bit X-archive-position: 30542 X-ecartis-version: Ecartis v1.0.0 Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org X-original-sender: James.Bottomley@HansenPartnership.com Precedence: bulk X-list: linux-mips Return-Path: X-Keywords: X-UID: 24144 On Wed, 2011-06-29 at 17:19 +0100, Ralf Baechle wrote: > On Wed, Jun 29, 2011 at 08:14:24AM -0700, Greg KH wrote: > > > > > On Wed, Jun 29, 2011 at 08:58:19AM -0500, James Bottomley wrote: > > > I think we should simply concentrate on __init and __exit; that's where > > > most of the discard value lies and stop expending huge efforts on the > > > __devX stuff which adds huge complexity for no real gain. > > > > I have long felt that those __devX markings should just go away as they > > cause nothing but problems and have no real gain as you point out. > > The suggestion to do that has been floated around before but seems to > have missed sufficient thrust. I'm all for it; the manual tagging with > __devX has not been very efficient on developer time. I just want to see > meaningful warnings again over all that noise the current mechanisn may > produce. For me, just go ahead and fix the actual problems: so _init sections and _exit sections that are used from the main body, just strip the annotations, don't try to change them for _devX ones. Thanks, James