Re: [patch 41/48] ocfs2: Initialize the cluster were writing to ina non-sparse extend

From: Joel Becker
Date: Fri Sep 04 2009 - 17:03:21 EST


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 <sunil.mushran@xxxxxxxxxx>
>
> 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@xxxxxxxxxx
Phone: (650) 506-8127
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/