From mboxrd@z Thu Jan  1 00:00:00 1970
From: Tao Ma <tao.ma@oracle.com>
Date: Wed, 14 Jul 2010 08:29:10 +0800
Subject: [Ocfs2-devel] [PATCH v2] ocfs2: Don't duplicate page passes
	i_size during CoW.
In-Reply-To: <20100713183828.GA9468@mail.oracle.com>
References: <20100712091412.GA19583@mail.oracle.com>
	<1278991352-5540-1-git-send-email-tao.ma@oracle.com>
	<20100713081306.GB31479@mail.oracle.com>
	<20100713081538.GC31479@mail.oracle.com>
	<4C3C7771.5000103@oracle.com>
	<20100713183828.GA9468@mail.oracle.com>
Message-ID: <4C3D04D6.4010600@oracle.com>
List-Id: <ocfs2-devel.oss.oracle.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: ocfs2-devel@oss.oracle.com

Joel Becker wrote:
> On Tue, Jul 13, 2010 at 10:25:53PM +0800, Tao Ma wrote:
>   
>> Joel Becker wrote:
>>     
>>> 	Why even worry about that?  We only need up to i_size.  If end
>>> is safe and not page aligned, so is i_size.
>>>       
>> i_size isn't safe in ocfs2_duplicate_clusters_by_page. Check the
>> below function.
>>
>>            if (page_has_buffers(page)) {
>>                        ret = walk_page_buffers(handle, page_buffers(page),
>>                                                from, to, &partial,
>>                                                ocfs2_clear_cow_buffer);
>>                        if (ret) {
>>                                mlog_errno(ret);
>>                                goto unlock;
>>                        }
>>                }
>> So 'to' is limited to 'map_end' and then 'end'. If we set 'end' to
>> i_size, we may not clear all the buffer heads
>>     
>
> 	We don't need to clear all the buffer heads.  This is the same
> as the logic in block_prepare_write().  We only need to clear the blocks
> that encompass i_size.  Unless we did something wrong, the buffers past
> i_size should not be mapped.
> 	More importantly, your 'end' has the same layout.  If end is not
> page aligned, it will only clear some of the blocks in its page.  Which
> is fine.  There's no difference between end and i_size - they don't have
> to fall on page boundaries.
>   
oh, I see.
block_read_full_page will only map and read the blocks within i_size, cool.
Then end = i_size should work fine.
I will test it and then send a final one(hope so ;))
Thanks for pointing it out.

Regards,
Tao