From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754677AbbGNKLn (ORCPT ); Tue, 14 Jul 2015 06:11:43 -0400 Received: from mailout4.samsung.com ([203.254.224.34]:56498 "EHLO mailout4.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752084AbbGNKLl (ORCPT ); Tue, 14 Jul 2015 06:11:41 -0400 X-AuditID: cbfee61b-f79416d0000014c0-16-55a4e05c577e From: Chao Yu To: "'Jaegeuk Kim'" Cc: linux-kernel@vger.kernel.org, linux-f2fs-devel@lists.sourceforge.net, Fan Li References: <00b001d0b964$e403d930$ac0b8b90$@samsung.com> In-reply-to: Subject: RE: [f2fs-dev] [PATCH RESEND] f2fs: maintain extent cache in separated file Date: Tue, 14 Jul 2015 18:10:50 +0800 Message-id: <00f201d0be1d$78aa7370$69ff5a50$@samsung.com> MIME-version: 1.0 Content-type: text/plain; charset=us-ascii Content-transfer-encoding: 7bit X-Mailer: Microsoft Outlook 14.0 Thread-index: AQFcVYuTjew6dy5SO9kixJEUqF7+fQGkFQV1nrX6vFA= Content-language: zh-cn X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrJLMWRmVeSWpSXmKPExsVy+t9jAd2YB0tCDb7fZbf4vLWR3eLJ+lnM FpcWuVtc3jWHzYHFY9OqTjaP3Qs+M3n0bVnF6PF5k1wASxSXTUpqTmZZapG+XQJXxplV81gL bvFV7L5p28A4iaeLkZNDQsBE4u3+LiYIW0ziwr31bCC2kMAiRolb/eZdjFxA9itGif4Z8xlB EmwCKhLLO/6DNYgIqEn07psCZjML5Eo8On+PEaI5R2LCzXfsIDangK3El7bfYDXCAmESb2ee YgaxWQRUJe59mQ1WzytgKbFk31coW1Dix+R7LBAztSTW7zwONV9eYvOat8wQhypI7Dj7mhHi BiuJ9n/dzBA14hIbj9ximcAoNAvJqFlIRs1CMmoWkpYFjCyrGEVTC5ILipPSc430ihNzi0vz 0vWS83M3MYIj4Jn0DsZVDRaHGAU4GJV4eCdYLgkVYk0sK67MPcQowcGsJMJrsREoxJuSWFmV WpQfX1Sak1p8iFGag0VJnPdkvk+okEB6YklqdmpqQWoRTJaJg1OqgTHsXsn7F8EerA/fty8w UbWdGmYpd4EpYJedduKZ0988NqyNfRc4gXdXqt23tm2Jkiz5Z8N3tgrM9Nzf4rnI5u0XE3Xm vCV3aq2iLjDX9Vcor5iWUuNu8fbv1ecZPSq/ay+UrvaMynx3UutbIjfLvL0+t65X7Pm0UuJT 6+My9ysOtScu+F1T9ldiKc5INNRiLipOBAAdp7aHfAIAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Jaegeuk, > -----Original Message----- > From: Chao Yu [mailto:yuchaochina@hotmail.com] > Sent: Thursday, July 09, 2015 12:16 AM > To: 'Jaegeuk Kim' > Cc: linux-kernel@vger.kernel.org; linux-f2fs-devel@lists.sourceforge.net > Subject: Re: [f2fs-dev] [PATCH RESEND] f2fs: maintain extent cache in separated file > > >From 8956df8fa1a669e8e476456e4afe0eccd8174684 Mon Sep 17 00:00:00 2001 > From: Chao Yu > Date: Wed, 8 Jul 2015 20:49:05 +0800 > Subject: [PATCH] f2fs: maintain extent cache in separated file > > This patch moves extent cache related code from data.c into extent_cache.c > since extent cache is independent feature, and its codes are not relate to > others in data.c, it's better for us to maintain them in separated place. > > There is no functionality change, but several small coding style fixes > including: > * rename __drop_largest_extent to f2fs_drop_largest_extent for exporting; > * rename misspelled word 'untill' to 'until'; > * remove unneeded 'return' in the end of f2fs_destroy_extent_tree(). > I think commit 0b84ceec9891 ("f2fs: don't try to split extents shorter than F2FS_MIN_EXTENT_LEN") is conflict with commit 392a1c419916 ("f2fs: maintain extent cache in separated file"), and it's not well resolved in dev-test. in commit 392a1c419916 ("f2fs: maintain extent cache in separated file"): - if (dei.len > F2FS_MIN_EXTENT_LEN) { + if (dei.len > 1) { So how about merging this patch (f2fs: maintain extent cache in separated file) first, and then merge Fan's patch? because one line modification in extent_cache.c for fixing conflict seems much easier. BTW, I have already fixed the building error in this patch, you can directly use the last sent patch in this thread showed in below link. :) http://sourceforge.net/p/linux-f2fs/mailman/message/34277570/ Thanks,