Project

General

Profile

Bug #17610

FileStore: fiemap cannot be totally retrieved in xfs when the number of extents > 1364

Added by Yao Ning 8 months ago. Updated 5 months ago.

Status:
Resolved
Priority:
Urgent
Assignee:
Category:
OSD
Target version:
-
Start date:
10/19/2016
Due date:
10/21/2016
% Done:

100%

Estimated time:
10.00 h
Source:
Community (user)
Tags:
Backport:
jewel,hammer
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Release:
Needs Doc:
No

Description

Retrieve Fiemap from XFS filesystem:

0: [0..7]: 22108440..22108447
1: [8..15]: hole
2: [16..23]: 22108456..22108463
3: [24..31]: hole
4: [32..39]: 22108472..22108479
5: [40..47]: hole
6: [48..55]: 22108488..22108495
7: [56..63]: hole
...
...
3991: [31928..31935]: hole
3992: [31936..31943]: 22057896..22057903
3993: [31944..31951]: hole
3994: [31952..31959]: 22057960..22057967
3995: [31960..31967]: hole
3996: [31968..31975]: 22058008..22058015
3997: [31976..31983]: hole
3998: [31984..31991]: 22058040..22058047

Actually, 2000 extents should be retrived here.

However, the result is:
total extents retrieved: 1364


Related issues

Copied to Ceph - Backport #17877: jewel: FileStore: fiemap cannot be totally retrieved in xfs when the number of extents > 1364 Resolved
Copied to Ceph - Backport #17878: hammer: FileStore: fiemap cannot be totally retrieved in xfs when the number of extents > 1364 Resolved

Associated revisions

Revision d1e12072 (diff)
Added by Ning Yao 7 months ago

FileStore:: fix fiemap issue in xfs when #extents > 1364

Fixes: #17610
Backport: jewel, hammer
Signed-off-by: Ning Yao <>
(cherry picked from commit 1a1c126d80b427d5230347fbc71a4edea5c0b6c8)

Conflicts:
src/os/FileStore.cc
in hammer, there is no _do_seek_hole_data() function so remove it
in hammer, the logic is in FileStore::fiemap not in _do_fiemap()
so port the logic to the else branch in FileStore::fiemap

History

#1 Updated by Sage Weil 8 months ago

  • Status changed from New to Pending Backport
  • Backport changed from hammer to jewel,hammer

#2 Updated by Nathan Cutler 8 months ago

  • Copied to Backport #17877: jewel: FileStore: fiemap cannot be totally retrieved in xfs when the number of extents > 1364 added

#3 Updated by Nathan Cutler 8 months ago

  • Copied to Backport #17878: hammer: FileStore: fiemap cannot be totally retrieved in xfs when the number of extents > 1364 added

#4 Updated by Yao Ning 7 months ago

How shall we do in this case? @Sage Weil

the non-trivial backport in hammer may need to use the new commit, like this PR: https://github.com/ceph/ceph/pull/11615 with a seperate QA testing on branch hammer?

#5 Updated by Nathan Cutler 5 months ago

  • Target version deleted (v0.94.10)

#6 Updated by Nathan Cutler 5 months ago

  • Status changed from Pending Backport to Resolved

Also available in: Atom PDF