From mboxrd@z Thu Jan 1 00:00:00 1970 Date: Fri, 10 Oct 2008 09:10:30 -0400 From: Theodore Tso Subject: Re: [patch 4/8] mm: write_cache_pages type overflow fix Message-ID: <20081010131030.GB16353@mit.edu> References: <20081009155039.139856823@suse.de> <20081009174822.516911376@suse.de> <20081009082336.GB6637@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20081009082336.GB6637@infradead.org> Sender: owner-linux-mm@kvack.org Return-Path: To: Christoph Hellwig Cc: npiggin@suse.de, Andrew Morton , Mikulas Patocka , linux-mm@kvack.org, linux-fsdevel@vger.kernel.org List-ID: On Thu, Oct 09, 2008 at 04:23:36AM -0400, Christoph Hellwig wrote: > On Fri, Oct 10, 2008 at 02:50:43AM +1100, npiggin@suse.de wrote: > > In the range_cont case, range_start is set to index << PAGE_CACHE_SHIFT, but > > index is a pgoff_t and range_start is loff_t, so we can get truncation of the > > value on 32-bit platforms. Fix this by adding the standard loff_t cast. > > > > This is a data interity bug (depending on how range_cont is used). > > Aneesh has a patch to kill the range_cont flag, which is queued up for > 2.6.28. Which tree is this queued up in? It's not in ext4 or the mm tree... - Ted -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org