Skip to content

Commit

Permalink
mds: don't warn when inodes are not allocated out sequentially
Browse files Browse the repository at this point in the history
In the future, we may be allocating from different inode number sets
in the same session. Drop this check.

Signed-off-by: Jeff Layton <[email protected]>
  • Loading branch information
jtlayton committed Jan 11, 2020
1 parent 5679c3d commit 2e498df
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 9 deletions.
5 changes: 0 additions & 5 deletions src/mds/SessionMap.h
Original file line number Diff line number Diff line change
Expand Up @@ -172,11 +172,6 @@ class Session : public RefCountedObject {
return session_cache_liveness.get();
}

inodeno_t next_ino() const {
if (info.prealloc_inos.empty())
return 0;
return info.prealloc_inos.range_start();
}
inodeno_t take_ino(inodeno_t ino = 0) {
ceph_assert(!info.prealloc_inos.empty());

Expand Down
4 changes: 0 additions & 4 deletions src/mds/journal.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1496,11 +1496,7 @@ void EMetaBlob::replay(MDSRank *mds, LogSegment *logseg, MDSlaveUpdate *slaveup)
dout(20) << " (session prealloc " << session->info.prealloc_inos << ")" << dendl;
if (used_preallocated_ino) {
if (!session->info.prealloc_inos.empty()) {
inodeno_t next = session->next_ino();
inodeno_t i = session->take_ino(used_preallocated_ino);
if (next != i)
mds->clog->warn() << " replayed op " << client_reqs << " used ino " << i
<< " but session next is " << next;
ceph_assert(i == used_preallocated_ino);
session->info.used_inos.clear();
}
Expand Down

0 comments on commit 2e498df

Please sign in to comment.