Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757017Ab0GFMAE (ORCPT ); Tue, 6 Jul 2010 08:00:04 -0400 Received: from rcsinet10.oracle.com ([148.87.113.121]:28966 "EHLO rcsinet10.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754806Ab0GFMAA (ORCPT ); Tue, 6 Jul 2010 08:00:00 -0400 Date: Tue, 6 Jul 2010 04:58:28 -0700 From: Joel Becker To: Tao Ma Cc: Dave Chinner , Linus Torvalds , Linux Kernel , ocfs2-devel@oss.oracle.com, Dave Chinner , Christoph Hellwig , Mark Fasheh Subject: Re: [PATCH 1/2] ocfs2: Zero the tail cluster when extending past i_size v2 Message-ID: <20100706115827.GG13023@mail.oracle.com> Mail-Followup-To: Tao Ma , Dave Chinner , Linus Torvalds , Linux Kernel , ocfs2-devel@oss.oracle.com, Dave Chinner , Christoph Hellwig , Mark Fasheh References: <20100629005403.GC24343@mail.oracle.com> <20100629015615.GZ6590@dastard> <20100629020420.GE24343@mail.oracle.com> <20100629022757.GA6590@dastard> <20100629071817.GA4150@mail.oracle.com> <20100702224912.GC5800@mail.oracle.com> <20100703213219.GB21262@mail.oracle.com> <4C3156D0.5090001@oracle.com> <20100706071741.GF13023@mail.oracle.com> <4C32E152.1090902@oracle.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4C32E152.1090902@oracle.com> 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.0A090202.4C331A96.00FE:SCFMA4539814,ss=1,fgs=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2058 Lines: 53 On Tue, Jul 06, 2010 at 03:54:58PM +0800, Tao Ma wrote: > On 07/06/2010 03:17 PM, Joel Becker wrote: > >>>+ /* Is there a cluster to zero? */ > >>>+ if (!p_cpos) > >>>+ goto out; > >>For unwritten extent, we also need to clear the pages? If yes, the > >>solution doesn't complete if we have 2 unwritten extent, one > >>contains i_size while one passes i_size. Here we only clear the > >>pages for the 1st unwritten extent and leave the 2nd one untouched. > > > > We probably don't need to zero unwritten extents. We cannot > >have an extent past i_size, can we? > we can. AFAICS, ocfs2_change_file_space will allocate unwritten > extents and does't change i_size. Oh, you're right. We need to walk the entire extent range between i_size and pos and figure out what needs CoW. This needs to happen no matter what. > > But you dropped the check for pos_blkno alignment. > >Unconditionally adding the +1 doesn't seem like a good idea. > You can add it as you wish. > I just thought that you add one more extra cluster if pos_blkno > isn't aligned so as to zero blocks in [pos_cpos_start_block, > pos_blkno). > But As I said in the comments, you will soon write pos_blkno(it also > needs to be CoW since it is within this refcounted extent), so if we > can CoW it out now, maybe we have a chance to not call > ocfs2_refcount_cow later. I'd much rather let the write handle its own contiguousness. If we get lucky, that CoW melds with our CoW. If we don't get lucky, isn't it better to have the newly changed area be fully contiguous rather than have the first extent of it not be and then the remaining extents be? Joel -- Life's Little Instruction Book #3 "Watch a sunrise at least once a year." Joel Becker Consulting 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/