Project

General

Profile

Actions

Bug #44963

closed

fix MClientCaps::FLAG_SYNC in check_caps

Added by Jeff Layton about 4 years ago. Updated almost 4 years ago.

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

0%

Source:
Tags:
Backport:
octopus,nautilus
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
libcephfs
Labels (FS):
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

Sidharth noticed that we were probably issuing more FLAG_SYNC cap messages than is really required. What we're mostly interested in expediting is dirty cap flushes, so we really only need to set FLAG_SYNC when flushing auth caps that are dirty.


Related issues 2 (0 open2 closed)

Copied to CephFS - Backport #45477: octopus: fix MClientCaps::FLAG_SYNC in check_capsResolvedNathan CutlerActions
Copied to CephFS - Backport #45478: nautilus: fix MClientCaps::FLAG_SYNC in check_capsResolvedWei-Chung ChengActions
Actions #1

Updated by Jeff Layton about 4 years ago

  • Status changed from New to Resolved
  • Pull request ID set to 44963

PR merged for Octopus.

Actions #2

Updated by Patrick Donnelly about 4 years ago

  • Status changed from Resolved to Pending Backport
  • Target version set to v16.0.0
  • Backport set to octopus,nautilus
  • Pull request ID changed from 44963 to 34455
Actions #3

Updated by Nathan Cutler about 4 years ago

  • Copied to Backport #45477: octopus: fix MClientCaps::FLAG_SYNC in check_caps added
Actions #4

Updated by Nathan Cutler about 4 years ago

  • Copied to Backport #45478: nautilus: fix MClientCaps::FLAG_SYNC in check_caps added
Actions #5

Updated by Nathan Cutler almost 4 years ago

  • Status changed from Pending Backport to Resolved

While running with --resolve-parent, the script "backport-create-issue" noticed that all backports of this issue are in status "Resolved" or "Rejected".

Actions

Also available in: Atom PDF