Project

General

Profile

Bug #55284

kclient: filesystem sync will stuck for around 5 seconds sometimes

Added by Xiubo Li 3 months ago. Updated 2 months ago.

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

0%

Source:
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
kcephfs
Crash signature (v1):
Crash signature (v2):

Description

We have fixed the fsync(fd) case in the following commits:

6316d4197437 ("ceph: put the requests/sessions when it fails to alloc memory")
716e68d445fa ("ceph: fix off by one bugs in unsafe_request_wait()")
2e31f19343da ("ceph: flush the mdlog before waiting on unsafe reqs")
65628853caba ("ceph: flush mdlog before umounting")
4c0090260622 ("ceph: make iterate_sessions a global symbol")
90803e2b16ec ("ceph: make ceph_create_session_msg a global symbol")

But when syncing the whole filesystem we still could hit this bug. We need to send the mdlog flush request to the MDSes when waiting the unsafe requests to finish.


Related issues

Related to CephFS - Feature #55283: qa: add fsync/sync stuck waiting for unsafe request test Resolved
Related to Linux kernel client - Bug #55411: BUG: Dentry XXXXXX still in use (1) [unmount of ceph ceph] Resolved

History

#1 Updated by Xiubo Li 3 months ago

  • Related to Feature #55283: qa: add fsync/sync stuck waiting for unsafe request test added

#2 Updated by Xiubo Li 3 months ago

  • Status changed from New to In Progress
  • Assignee set to Xiubo Li

#3 Updated by Xiubo Li 3 months ago

  • Status changed from In Progress to Fix Under Review

#4 Updated by Xiubo Li 2 months ago

  • Related to Bug #55409: client: incorrect operator precedence in Client.cc added

#5 Updated by Xiubo Li 2 months ago

  • Status changed from Fix Under Review to Resolved

#6 Updated by Xiubo Li 2 months ago

  • Related to deleted (Bug #55409: client: incorrect operator precedence in Client.cc)

#7 Updated by Xiubo Li 2 months ago

  • Related to Bug #55411: BUG: Dentry XXXXXX still in use (1) [unmount of ceph ceph] added

Also available in: Atom PDF