From mboxrd@z Thu Jan 1 00:00:00 1970 From: Theodore Tso Subject: Re: [PATCH 1/9] extents for ext4 Date: Thu, 10 Aug 2006 13:17:55 -0400 Message-ID: <20060810171755.GA19238@thunk.org> References: <1155172827.3161.80.camel@localhost.localdomain> <20060809233940.50162afb.akpm@osdl.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: linux-fsdevel@vger.kernel.org, ext2-devel@lists.sourceforge.net, cmm@us.ibm.com, linux-kernel@vger.kernel.org Return-path: To: Andrew Morton Content-Disposition: inline In-Reply-To: <20060809233940.50162afb.akpm@osdl.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: ext2-devel-bounces@lists.sourceforge.net Errors-To: ext2-devel-bounces@lists.sourceforge.net List-Id: linux-fsdevel.vger.kernel.org On Wed, Aug 09, 2006 at 11:39:40PM -0700, Andrew Morton wrote: > - replace all brelse() calls with put_bh(). Because brelse() is > old-fashioned, has a weird name and neelessly permits a NULL arg. > > In fact it would be beter to convert JBD and ext3 to put_bh before > copying it all over. Wouldn't it be better to preserve in the source code history the brelse->put_bh conversion? We can pour a huge number of changes in ext4 before we submit, but I would have thought it would be easier for everyone to see what is going on if we submit with just the minimal changes, and then have patches that address concerns like this one at a time. - Ted ------------------------------------------------------------------------- Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642