Date: Mon, 14 Dec 2009 11:19:16 +0900
From: Paul Mundt <lethal@linux-sh.org>
To: Emese Revfy <re.emese@gmail.com>
Cc: Matthew Wilcox <matthew@wil.cx>, linux-kernel@vger.kernel.org,
       torvalds@linux-foundation.org, viro@zeniv.linux.org.uk,
       akpm@linux-foundation.org
Subject: Re: [PATCH 0/1] Constify struct address_space_operations for 2.6.32-git-053fe57ac v2
Message-ID: <20091214021916.GB12196@linux-sh.org>
References: <cover.1260662994.git.re.emese@gmail.com> <20091214003836.GD7812@parisc-linux.org> <4B2595E7.701@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <4B2595E7.701@gmail.com>
User-Agent: Mutt/1.5.13 (2006-08-11)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1349
Lines: 31

On Mon, Dec 14, 2009 at 02:33:27AM +0100, Emese Revfy wrote:
> Matthew Wilcox wrote:
> > On Mon, Dec 14, 2009 at 12:59:08AM +0100, re.emese@gmail.com wrote:
> >> The following patch series attempts to constify several structures
> >> that hold function pointers. This is only the initial batch, there
> >> are about over 150 candidate structures, some of which can be
> >> constified as well, I plan to submit them in the future.
> > 
> > What a complete waste of time.  Until you respond to Al's:
> 
> I did: http://lkml.org/lkml/2009/12/5/140
> 
> For even more discussion see: http://lkml.org/lkml/2009/12/6/111
> 
Since you seem to have both the interest and abundance of spare time
for working on this, have you considered just doing this in sparse? Al
mentioned it here:

	http://lkml.org/lkml/2009/12/8/511

which you don't seem to have replied to.

Until such a consensus is reached one way or the other, please refrain
from sending hundreds of patches -- one or two are sufficient for showing
what you want to do until folks are on board with it, as is the typical
nature of mechanical changes.
--
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/