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

Signed-off-by: David Zafman <dzafman@redhat.com>

diff --git a/src/os/FileJournal.cc b/src/os/FileJournal.cc
index 136de61d0ff031154e92a037c90b62c5a737529b..4ba468f7e13f5c0169eb256dc59417fee2c53f66 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 d565252f4efdeed2a2464017c3e2bd9ab446b510..9e07b4046d97dc958b07becdeb572d5ae400e2aa 100644 (file)
--- a/src/os/FileJournal.h
+++ b/src/os/FileJournal.h
@@ -415,6 +415,7 @@ private:
 
   int dump(ostream& out);
   int simple_dump(ostream& out);
+  int _fdump(Formatter &f, bool simple);
 
   void flush();