From e1e91f6c8597bebca36824d2a139f39953f0f931 Mon Sep 17 00:00:00 2001
From: Ali Mohammad Pur <ali.mpfard@gmail.com>
Date: Sat, 11 Sep 2021 16:48:15 +0430
Subject: [PATCH] Kernel: Deduplicate signpost perf event strings

---
 Kernel/PerformanceEventBuffer.cpp | 4 ++--
 Kernel/PerformanceEventBuffer.h   | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/Kernel/PerformanceEventBuffer.cpp b/Kernel/PerformanceEventBuffer.cpp
index 17380fd3d2..5a08c9facc 100644
--- a/Kernel/PerformanceEventBuffer.cpp
+++ b/Kernel/PerformanceEventBuffer.cpp
@@ -169,7 +169,7 @@ KResult PerformanceEventBuffer::to_json_impl(Serializer& object) const
     {
         auto strings = object.add_array("strings");
         for (auto& it : m_strings) {
-            strings.add(it.view());
+            strings.add(it->view());
         }
     }
 
@@ -309,7 +309,7 @@ KResultOr<FlatPtr> PerformanceEventBuffer::register_string(NonnullOwnPtr<KString
 {
     FlatPtr string_id = m_strings.size();
 
-    if (!m_strings.try_append(move(string)))
+    if (m_strings.try_set(move(string)) == AK::HashSetResult::Failed)
         return ENOBUFS;
 
     return string_id;
diff --git a/Kernel/PerformanceEventBuffer.h b/Kernel/PerformanceEventBuffer.h
index a217f99ad5..3b9a4703a4 100644
--- a/Kernel/PerformanceEventBuffer.h
+++ b/Kernel/PerformanceEventBuffer.h
@@ -133,7 +133,7 @@ private:
     size_t m_count { 0 };
     NonnullOwnPtr<KBuffer> m_buffer;
 
-    NonnullOwnPtrVector<KString> m_strings;
+    HashTable<NonnullOwnPtr<KString>> m_strings;
 };
 
 extern bool g_profiling_all_threads;