Project

General

Profile

Actions

Bug #62058

open

mds: inode snaplock only acquired for open in create codepath

Added by Patrick Donnelly 10 months ago. Updated 9 months ago.

Status:
Pending Backport
Priority:
Normal
Category:
Correctness/Safety
Target version:
% Done:

0%

Source:
Development
Tags:
backport_processed
Backport:
reef,quincy,pacific
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
MDS
Labels (FS):
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

https://github.com/ceph/ceph/blob/236f8b632fbddcfe9dcdb484561c0fede717fd2f/src/mds/Server.cc#L4612-L4615

It doesn't make sense that

open("file", O_CREAT|...)

will acquire a readlock on the file's snaplock when the file already exists, but:

open("file", ...)

will not.


Related issues 3 (2 open1 closed)

Copied to CephFS - Backport #62516: quincy: mds: inode snaplock only acquired for open in create codepathIn ProgressPatrick DonnellyActions
Copied to CephFS - Backport #62517: pacific: mds: inode snaplock only acquired for open in create codepathResolvedPatrick DonnellyActions
Copied to CephFS - Backport #62518: reef: mds: inode snaplock only acquired for open in create codepathIn ProgressPatrick DonnellyActions
Actions #1

Updated by Patrick Donnelly 10 months ago

  • Status changed from In Progress to Fix Under Review
  • Pull request ID set to 52520
Actions #2

Updated by Venky Shankar 9 months ago

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

Updated by Backport Bot 9 months ago

  • Copied to Backport #62516: quincy: mds: inode snaplock only acquired for open in create codepath added
Actions #4

Updated by Backport Bot 9 months ago

  • Copied to Backport #62517: pacific: mds: inode snaplock only acquired for open in create codepath added
Actions #5

Updated by Backport Bot 9 months ago

  • Copied to Backport #62518: reef: mds: inode snaplock only acquired for open in create codepath added
Actions #6

Updated by Backport Bot 9 months ago

  • Tags set to backport_processed
Actions

Also available in: Atom PDF