Skip to content

quincy: mds: inode_t flags may not be protected by the policylock during set_vxattr#57178

Closed
batrick wants to merge 1 commit intoceph:quincyfrom
batrick:wip-65713-quincy
Closed

quincy: mds: inode_t flags may not be protected by the policylock during set_vxattr#57178
batrick wants to merge 1 commit intoceph:quincyfrom
batrick:wip-65713-quincy

Conversation

@batrick
Copy link
Member

@batrick batrick commented Apr 30, 2024

Note: this backport omits testing commits because qa/tasks/cephfs/test_subvolume.py does not exist in quincy.

backport tracker: https://tracker.ceph.com/issues/65713


backport of #56934
parent tracker: https://tracker.ceph.com/issues/65496

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh

@batrick batrick added this to the quincy milestone Apr 30, 2024
@batrick batrick added the cephfs Ceph File System label Apr 30, 2024
@batrick batrick force-pushed the wip-65713-quincy branch from 046f5e5 to bc00318 Compare June 28, 2024 20:50
@lxbsz
Copy link
Member

lxbsz commented Aug 2, 2024

This PR is under test in https://tracker.ceph.com/issues/67315.

@github-actions github-actions bot added the stale label Nov 26, 2024
@batrick batrick removed the stale label Nov 26, 2024
@ceph ceph deleted a comment from github-actions bot Dec 30, 2024
@batrick
Copy link
Member Author

batrick commented Dec 30, 2024

jenkins test make check

@vshankar
Copy link
Contributor

vshankar commented Jan 2, 2025

This PR is under test in https://tracker.ceph.com/issues/69400.

@batrick
Copy link
Member Author

batrick commented Feb 5, 2025

jenkins test make check

The code logic had a serious defect that it would only execute xlock_policylock
once such that a retry would then proceed to executing the vxattr setting.

Fixes: https://tracker.ceph.com/issues/65496
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit 2931860)

Conflicts:
	src/mds/Server.cc: no quiesce in quincy
@vshankar
Copy link
Contributor

Quincy is EOL.

@vshankar vshankar closed this Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cephfs Ceph File System

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants