From 8833e12e06b2041a93ee5774553f568e9f3b14c8 Mon Sep 17 00:00:00 2001 From: Sage Weil Date: Mon, 22 Sep 2008 14:37:18 -0700 Subject: [PATCH] journal: fix off-by-one bug in committed_thru callbacks --- src/os/FileJournal.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/os/FileJournal.cc b/src/os/FileJournal.cc index 3ddc660bb791d..f680997d69042 100644 --- a/src/os/FileJournal.cc +++ b/src/os/FileJournal.cc @@ -585,7 +585,7 @@ void FileJournal::committed_thru(__u64 seq) print_header(); // committed but writing - while (!writing_seq.empty() && writing_seq.front() < seq) { + while (!writing_seq.empty() && writing_seq.front() <= seq) { dout(15) << " finishing committed but writing|waiting seq " << writing_seq.front() << dendl; finisher->queue(writing_fin.front()); writing_seq.pop_front(); @@ -593,7 +593,7 @@ void FileJournal::committed_thru(__u64 seq) } // committed but unjournaled items - while (!writeq.empty() && writeq.front().seq < seq) { + while (!writeq.empty() && writeq.front().seq <= seq) { dout(15) << " dropping committed but unwritten seq " << writeq.front().seq << " len " << writeq.front().bl.length() << " (" << writeq.front().fin << ")" -- 2.39.5