From: Patrick Donnelly <pdonnell@redhat.com>
Date: Thu, 6 Sep 2018 15:52:25 +0000 (-0700)
Subject: log: use smaller buffer for ConcreteEntry
X-Git-Tag: v14.0.1~260^2
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=9c105090928bcd3955c68d26b787dc324c9d42d6;p=ceph.git

log: use smaller buffer for ConcreteEntry

This brings down the static size of the memory used by the logging infrastructure:

    If we used 1024, we'd have 1088*10000 = 10880000 = 10MB in use by the ring
    buffer and 2*1088*100 = 2*108800 = 2*106KB for the m_new and m_flush
    vectors.

In my testing, 1024 covers most log entries.

Note, I've kept 4096 for the StackStringStream via MutableEntry as these are
already allocated on the heap and cached in a thread local vector. Generally
there should only be about a dozen of these allocated so it's worth keeping a
larger buffer.

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
---

diff --git a/src/log/Entry.h b/src/log/Entry.h
index 076512cbe0c01..2753b1942b59d 100644
--- a/src/log/Entry.h
+++ b/src/log/Entry.h
@@ -107,7 +107,7 @@ public:
   }
 
 private:
-  boost::container::small_vector<char, 4096> str;
+  boost::container::small_vector<char, 1024> str;
 };
 
 }