PEL: Add PEL repository capping related fields
If not capped, the PEL repository would keep taking up more and more
space as more PELs are added. In preparation for handling this, add
a field to the Repository class to track the total size that all PELs can
take up and a field to track the maximum number of PELs that can be
stored.
The size value is hardcoded to be 20MB (and 100KB in unit test), though
a Repository class constructor allows the number to use to be passed in.
The max number of PELs field is hardcoded to 3000 (100 in test) and this
can also be changed in the constructor.
In the future if different systems or configurations require different
values, then this could be set at configure time. This isn't done now
because I want to only add PEL options to the common phosphor-logging
configure.ac if absolutely necessary as the majority of phosphor-logging
users don't use PELs.
Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I72609a727229f96bd8ac23a630f25fd8b5c73427
diff --git a/extensions/openpower-pels/repository.hpp b/extensions/openpower-pels/repository.hpp
index e55808f..2e2f623 100644
--- a/extensions/openpower-pels/repository.hpp
+++ b/extensions/openpower-pels/repository.hpp
@@ -1,5 +1,6 @@
#pragma once
#include "bcd_time.hpp"
+#include "paths.hpp"
#include "pel.hpp"
#include <algorithm>
@@ -118,7 +119,21 @@
*
* @param[in] basePath - the base filesystem path for the repository
*/
- Repository(const std::filesystem::path& basePath);
+ Repository(const std::filesystem::path& basePath) :
+ Repository(basePath, getPELRepoSize(), getMaxNumPELs())
+ {
+ }
+
+ /**
+ * @brief Constructor that takes the repository size
+ *
+ * @param[in] basePath - the base filesystem path for the repository
+ * @param[in] repoSize - The maximum amount of space to use for PELs,
+ * in bytes
+ * @param[in] maxNumPELs - The maximum number of PELs to allow
+ */
+ Repository(const std::filesystem::path& basePath, size_t repoSize,
+ size_t maxNumPELs);
/**
* @brief Adds a PEL to the repository
@@ -370,6 +385,18 @@
* @brief Subscriptions for deleted PELs.
*/
std::map<std::string, DeleteCallback> _deleteSubscriptions;
+
+ /**
+ * @brief The maximum amount of space that the PELs in the
+ * repository can occupy.
+ */
+ const uint64_t _maxRepoSize;
+
+ /**
+ * @brief The maximum number of PELs to allow in the repo
+ * before pruning.
+ */
+ const size_t _maxNumPELs;
};
} // namespace pels