osd: FileJournal: Add _fdump() that takes Formatter instead of ostream

Signed-off-by: David Zafman <dzafman@redhat.com>
(cherry picked from commit 130eba48495d0e6acd8a472cbf0c47ebbc7a038c)

diff --git a/src/os/FileJournal.cc b/src/os/FileJournal.cc
index 0d34be68da3cb32b1724afd1dc378cb863adc868..8c2635e2f9667f22096735352e4aea31060ceac5 100644 (file)
--- a/src/os/FileJournal.cc
+++ b/src/os/FileJournal.cc
@@ -583,7 +583,15 @@ int FileJournal::simple_dump(ostream& out)
 
 int FileJournal::_dump(ostream& out, bool simple)
 {
-  dout(10) << "dump" << dendl;
+  JSONFormatter f(true);
+  int ret = _fdump(f, simple);
+  f.flush(out);
+  return ret;
+}
+
+int FileJournal::_fdump(Formatter &f, bool simple)
+{
+  dout(10) << "_fdump" << dendl;
 
   assert(fd == -1);
   int err = _open(false, false);
@@ -598,7 +606,6 @@ int FileJournal::_dump(ostream& out, bool simple)
 
   off64_t next_pos = header.start;
 
-  JSONFormatter f(true);
   f.open_object_section("journal");
 
   f.open_object_section("header");
@@ -669,7 +676,6 @@ int FileJournal::_dump(ostream& out, bool simple)
 
   f.close_section();
   f.close_section();
-  f.flush(out);
   dout(10) << "dump finish" << dendl;
 
   close();

diff --git a/src/os/FileJournal.h b/src/os/FileJournal.h
index 964dbe721bc76587fc02164199e9c014da6b5275..71d7f2257b5e467be37f94035998d75259e1e895 100644 (file)
--- a/src/os/FileJournal.h
+++ b/src/os/FileJournal.h
@@ -413,6 +413,7 @@ private:
 
   int dump(ostream& out);
   int simple_dump(ostream& out);
+  int _fdump(Formatter &f, bool simple);
 
   void flush();