| To: | Artem Bityutskiy <dedekind1@xxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH v7 00/47] xfs: add reverse mapping support |
| From: | "Darrick J. Wong" <darrick.wong@xxxxxxxxxx> |
| Date: | Fri, 5 Aug 2016 00:22:26 -0700 |
| Cc: | Dave Chinner <david@xxxxxxxxxxxxx>, Mark Fasheh <mfasheh@xxxxxxx>, linux-fsdevel@xxxxxxxxxxxxxxx, vishal.l.verma@xxxxxxxxx, bfoster@xxxxxxxxxx, xfs@xxxxxxxxxxx |
| Delivered-to: | xfs@xxxxxxxxxxx |
| In-reply-to: | <1470380474.2311.71.camel@xxxxxxxxx> |
| References: | <146907695530.25461.3225785294902719773.stgit@xxxxxxxxxxxxxxxx> <20160803194536.GJ5316@xxxxxxxxxxxxx> <20160803205520.GQ8590@xxxxxxxxxxxxxxxx> <20160804005843.GJ8593@xxxxxxxxxxxxxxxx> <20160804021852.GK5316@xxxxxxxxxxxxx> <20160804154845.GV8590@xxxxxxxxxxxxxxxx> <20160804235015.GC16044@dastard> <1470380474.2311.71.camel@xxxxxxxxx> |
| User-agent: | Mutt/1.5.24 (2015-08-30) |
On Fri, Aug 05, 2016 at 10:01:14AM +0300, Artem Bityutskiy wrote: > On Fri, 2016-08-05 at 09:50 +1000, Dave Chinner wrote: > > I'd much prefer that fiemap gives exact information about shared > > extents. FIEMAP is a diagnostic tool and as such we need it to > > accurately reflect the exact extent map of the inode being queried > > so we aren't mislead about the layout of the file during trouble > > shooting. > > Hi Dave, you are right, and here is a side note: we were using FIEMAP > for optimizing image deployment in production, so it is a diagnostic > tool and more. Will send an xfstest exercising this shortly, since AFAICT there isn't already one. --D |
| Previous by Date: | [PATCH] generic: test accurate shared extent reporting, Darrick J. Wong |
|---|---|
| Next by Date: | Re: [PATCH] generic: test accurate shared extent reporting, Eryu Guan |
| Previous by Thread: | Re: [PATCH v7 00/47] xfs: add reverse mapping support, Artem Bityutskiy |
| Next by Thread: | Re: [PATCH v7 00/47] xfs: add reverse mapping support, Dave Chinner |
| Indexes: | [Date] [Thread] [Top] [All Lists] |