From: Patrick Donnelly <pdonnell@redhat.com>
Date: Wed, 26 Jun 2024 13:49:17 +0000 (-0400)
Subject: include/ceph_fs: correct ceph_mds_cap_item field name
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=88267276ad077f85902213c28aeb9c37bbbb17f9;p=ceph.git

include/ceph_fs: correct ceph_mds_cap_item field name

Originally, the last_sent sequence from the MDS was sent by the client during
bulk cap release but it was shortly after changed to the last_issue which is
the sequence number that the cap was originally "issued" by the MDS rank (which
may be updated after import of caps).

Fixes: 6208f57f487ac170df24a9018f1cc87a5ac8b4b3
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit 655cddb7c9f32c9dd9cddf40ac17f385d539c8f9)
---

diff --git a/src/client/MetaSession.cc b/src/client/MetaSession.cc
index b5160a84331b..3baa833851fd 100644
--- a/src/client/MetaSession.cc
+++ b/src/client/MetaSession.cc
@@ -56,7 +56,7 @@ void MetaSession::enqueue_cap_release(inodeno_t ino, uint64_t cap_id, ceph_seq_t
   ceph_mds_cap_item i;
   i.ino = ino;
   i.cap_id = cap_id;
-  i.seq = iseq;
+  i.issue_seq = iseq;
   i.migrate_seq = mseq;
   release->caps.push_back(i);
 }
diff --git a/src/include/ceph_fs.h b/src/include/ceph_fs.h
index 57eb18b0d3e5..0ab4e46ed681 100644
--- a/src/include/ceph_fs.h
+++ b/src/include/ceph_fs.h
@@ -1057,7 +1057,7 @@ struct ceph_mds_cap_release {
 struct ceph_mds_cap_item {
 	__le64 ino;
 	__le64 cap_id;
-	__le32 migrate_seq, seq;
+	__le32 migrate_seq, issue_seq;
 } __attribute__ ((packed));
 
 #define CEPH_MDS_LEASE_REVOKE           1  /*    mds  -> client */
diff --git a/src/mds/Locker.cc b/src/mds/Locker.cc
index b142848f164d..25c45b04825c 100644
--- a/src/mds/Locker.cc
+++ b/src/mds/Locker.cc
@@ -4218,7 +4218,7 @@ void Locker::handle_client_cap_release(const cref_t<MClientCapRelease> &m)
   Session *session = mds->get_session(m);
 
   for (const auto &cap : m->caps) {
-    _do_cap_release(client, inodeno_t((uint64_t)cap.ino) , cap.cap_id, cap.migrate_seq, cap.seq);
+    _do_cap_release(client, inodeno_t((uint64_t)cap.ino) , cap.cap_id, cap.migrate_seq, cap.issue_seq);
   }
 
   if (session) {