Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751747Ab2K2Env (ORCPT ); Wed, 28 Nov 2012 23:43:51 -0500 Received: from aserp1040.oracle.com ([141.146.126.69]:20378 "EHLO aserp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750899Ab2K2Ent (ORCPT ); Wed, 28 Nov 2012 23:43:49 -0500 Message-ID: <50B6E7CB.1040504@oracle.com> Date: Thu, 29 Nov 2012 12:42:51 +0800 From: Jeff Liu User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:16.0) Gecko/20121028 Thunderbird/16.0.2 MIME-Version: 1.0 To: Jim Meyering CC: Hugh Dickins , Jaegeuk Hanse , Andrew Morton , "Theodore Ts'o" , Zheng Liu , Paul Eggert , Christoph Hellwig , Josef Bacik , Andi Kleen , Andreas Dilger , Dave Chinner , Marco Stornelli , Chris Mason , Sunil Mushran , linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org Subject: Re: [PATCH] tmpfs: support SEEK_DATA and SEEK_HOLE (reprise) References: <20121129012933.GA9112@kernel> <87lidlxcw9.fsf@rho.meyering.net> In-Reply-To: <87lidlxcw9.fsf@rho.meyering.net> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-Source-IP: ucsinet22.oracle.com [156.151.31.94] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2286 Lines: 53 On 11/29/2012 12:15 PM, Jim Meyering wrote: > Hugh Dickins wrote: >> On Thu, 29 Nov 2012, Jaegeuk Hanse wrote: > ... >>> But this time in which scenario will use it? >> >> I was not very convinced by the grep argument from Jim and Paul: >> that seemed to be grep holding on to a no-arbitrary-limits dogma, >> at the expense of its users, causing an absurd line-length issue, >> which use of SEEK_DATA happens to avoid in some cases. >> >> The cp of sparse files from Jeff and Dave was more convincing; >> but I still didn't see why little old tmpfs needed to be ahead >> of the pack. >> >> But at LinuxCon/Plumbers in San Diego in August, a more convincing >> case was made: I was hoping you would not ask, because I did not take >> notes, and cannot pass on the details - was it rpm building on tmpfs? >> I was convinced enough to promise support on tmpfs when support on >> ext4 goes in. > > Re the cp-vs-sparse-file case, the current FIEMAP-based code in GNU > cp is ugly and complicated enough that until recently it harbored a > hard-to-reproduce data-corrupting bug[*]. Now that SEEK_DATA/SEEK_HOLE > support work will work also for tmpfs and ext4, we can plan to remove > the FIEMAP-based code in favor of a simpler SEEK_DATA/SEEK_HOLE-based > implementation. How do we teach du(1) to aware of the real disk footprint with Btrfs clone or OCFS2 reflinked files if we remove the FIEMAP-based code? How about if we still keep it there, and introduce SEEK_DATA/SEEK_HOLE code to the extent-scan module which is dedicated to deal with sparse files? Thanks, -Jeff > > With the rise of virtualization, copying sparse images efficiently > (probably searching, too) is becoming more and more important. > > So, yes, GNU cp will soon use this feature. > > [*] https://plus.google.com/u/0/114228401647637059102/posts/FDV3JEaYsKD > -- > To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- 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/