From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?utf-8?B?SsO2cm4=?= Engel Subject: Re: [RFC][PATCH] Implement SEEK_HOLE/SEEK_DATA Date: Thu, 29 Nov 2007 04:07:59 +0100 Message-ID: <20071129030758.GA11841@lazybastard.org> References: <20071128200206.GB3977@dhcp243-37.rdu.redhat.com> <1196290614.18231.17.camel@entropy> <20071128233959.GF4913@webber.adilger.int> <20071128234842.GA8664@lazybastard.org> <1196294785.18231.31.camel@entropy> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: =?utf-8?B?SsO2cm4=?= Engel , Andreas Dilger , Josef Bacik , linux-fsdevel@vger.kernel.org, Andreas Dilger To: Nicholas Miell Return-path: Received: from lazybastard.de ([212.112.238.170]:48178 "EHLO longford.lazybastard.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756896AbXK2DNQ (ORCPT ); Wed, 28 Nov 2007 22:13:16 -0500 Content-Disposition: inline In-Reply-To: <1196294785.18231.31.camel@entropy> Sender: linux-fsdevel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On Wed, 28 November 2007 16:06:25 -0800, Nicholas Miell wrote: >=20 > I'd have to reread the original proposal, but I remember FIEMAP as be= ing > a generalized way of getting information about a files extents. I thi= nk > the original proposal only dealt with mapping file offsets to physica= l > extents, but IIRC the interface was flexible enough to implement a > "where are the holes" request. >=20 > Regardless, SEEK_HOLE/SEEK_DATA being a better suited interface for t= he > needs of logfs doesn't make it the best interface for that need. I'm not sure we are looking for the best interface. A significant improvement without significant drawbacks would be enough for me. Having to provide "physical extends" complete with device offsets would be a significant drawback. Even if such an interface may have greater advantages under ideal circumstances. J=C3=B6rn --=20 He who knows others is wise. He who knows himself is enlightened. -- Lao Tsu - 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