Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933788AbZIDVDL (ORCPT ); Fri, 4 Sep 2009 17:03:11 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S933748AbZIDVDK (ORCPT ); Fri, 4 Sep 2009 17:03:10 -0400 Received: from acsinet11.oracle.com ([141.146.126.233]:54927 "EHLO acsinet11.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933788AbZIDVDJ (ORCPT ); Fri, 4 Sep 2009 17:03:09 -0400 Date: Fri, 4 Sep 2009 14:00:41 -0700 From: Joel Becker To: Greg KH Cc: linux-kernel@vger.kernel.org, stable@kernel.org, stable-review@kernel.org, torvalds@linux-foundation.org, akpm@linux-foundation.org, alan@lxorguk.ukuu.org.uk, Sunil Mushran Subject: Re: [patch 41/48] ocfs2: Initialize the cluster were writing to in a non-sparse extend Message-ID: <20090904210040.GA945@mail.oracle.com> Mail-Followup-To: Greg KH , linux-kernel@vger.kernel.org, stable@kernel.org, stable-review@kernel.org, torvalds@linux-foundation.org, akpm@linux-foundation.org, alan@lxorguk.ukuu.org.uk, Sunil Mushran References: <20090904200712.724048145@mini.kroah.org> <20090904200856.335297402@mini.kroah.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20090904200856.335297402@mini.kroah.org> X-Burt-Line: Trees are cool. X-Red-Smith: Ninety feet between bases is perhaps as close as man has ever come to perfection. User-Agent: Mutt/1.5.20 (2009-06-14) X-Source-IP: acsmt353.oracle.com [141.146.40.153] X-Auth-Type: Internal IP X-CT-RefId: str=0001.0A090209.4AA1802A.0175:SCFMA4539814,ss=1,fgs=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1377 Lines: 41 On Fri, Sep 04, 2009 at 01:07:53PM -0700, Greg KH wrote: > 2.6.27-stable review patch. If anyone has any objections, please let us know. > > ------------------ > From: Sunil Mushran > > commit e7432675f8ca868a4af365759a8d4c3779a3d922 upstream. > > In a non-sparse extend, we correctly allocate (and zero) the clusters between > the old_i_size and pos, but we don't zero the portions of the cluster we're > writing to outside of pos<->len. > > It handles clustersize > pagesize and blocksize < pagesize. > > [Cleaned up by Joel Becker.] Greg, et al, Yesterday we found a bug in this patch. The patch still fixes the original problem, but we introduced an error in certain page_mkwrite() corner cases. We have one more validation test and then I'm sending the fix to Linus. So we really want both this commit and the followup fix in -stable. You'll be CC'd when I send Linus the pull request. Joel -- "Copy from one, it's plagiarism; copy from two, it's research." - Wilson Mizner Joel Becker Principal Software Developer Oracle E-mail: joel.becker@oracle.com Phone: (650) 506-8127 -- 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/