Project

General

Profile

Actions

Bug #14397

closed

disable filestore_xfs_extsize by default

Added by Ken Dreyer over 8 years ago. Updated about 8 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
% Done:

0%

Source:
other
Tags:
Backport:
hammer
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

Description of problem:
filestore_xfs_extsize defaults to "true" in Hammer. This option is designed to reduce fragmentation on the OSDs.

Tests have found that disabling filestore_xfs_extsize in Hammer improves large sequential write performance by about 20% (Ben Turner's cluster), and in some other tests by lesser amounts. This brings us closer to the large sequential write performance of Firefly.

Steps to Reproduce:
  1. Leave filestore_xfs_extsize unset (currently defaults to "true"),
  2. Run large sequential writes tests via CBT, note performance numbers,
  3. Set filestore_xfs_extsize to "false",
  4. Re-run large sequential writes in CBT, note performance numbers.

Actual results:
The default filestore_xfs_extsize setting results in a large sequential write performance degradation.

Expected results:
The default filestore_xfs_extsize setting should not result in a large sequential write performance degradation.

The proposed fix is to set filestore_xfs_extsize back to "false" in src/common/config_opts.h.

Additional info:
This change comes with a trade-off, because it introduces fragmentation on the OSDs. To address this, we should introduce documentation which explains the fragmentation cost and suggest that users who have a large sequential read use-cases (object storage/CDN) toggle the value to reduce the fragmentation impact over time.


Related issues 1 (0 open1 closed)

Copied to Ceph - Backport #14467: hammer: disable filestore_xfs_extsize by defaultResolvedLoïc DacharyActions
Actions #1

Updated by Ken Dreyer over 8 years ago

  • Backport set to hammer
Actions #2

Updated by Ken Dreyer over 8 years ago

  • Status changed from New to Fix Under Review
Actions #3

Updated by Sage Weil over 8 years ago

  • Status changed from Fix Under Review to Pending Backport
Actions #4

Updated by Loïc Dachary over 8 years ago

  • Copied to Backport #14467: hammer: disable filestore_xfs_extsize by default added
Actions #5

Updated by Nathan Cutler about 8 years ago

  • Status changed from Pending Backport to Resolved
Actions

Also available in: Atom PDF