RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-26 20:07:36 +00:00
Code Issues Activity Actions

Kernel: Convert klog() => AK::Format in PurgeablePageRanges

Browse source
This commit is contained in:
Andreas Kling 2021-03-12 11:38:43 +01:00
parent 72cccfddbf
commit 3985468e83
1 changed files with 30 additions and 27 deletions
Download patch file Download diff file Expand all files Collapse all files

41
Kernel/VM/PurgeablePageRanges.cpp
View file

@ -33,23 +33,26 @@
#include <Kernel/VM/PhysicalPage.h> #include <Kernel/VM/PhysicalPage.h>
#include <Kernel/VM/PurgeablePageRanges.h> #include <Kernel/VM/PurgeablePageRanges.h>
namespace Kernel { namespace AK {
template<>
#if VOLATILE_PAGE_RANGES_DEBUG struct Formatter<Kernel::VolatilePageRange> : Formatter<String> {
inline LogStream& operator<<(const LogStream& stream, const VolatilePageRange& range) void format(FormatBuilder& builder, const Kernel::VolatilePageRange& value)
{ {
stream << "{" << range.base << " (" << range.count << ") purged: " << range.was_purged << "}"; return Formatter<String>::format(builder, String::formatted("{{{} ({}) purged: {}}}", value.base, value.count, value.was_purged));
return const_cast<LogStream&>(stream);
} }
};
}
namespace Kernel {
static void dump_volatile_page_ranges(const Vector<VolatilePageRange>& ranges) static void dump_volatile_page_ranges(const Vector<VolatilePageRange>& ranges)
{ {
if constexpr (VOLATILE_PAGE_RANGES_DEBUG) {
for (size_t i = 0; i < ranges.size(); i++) { for (size_t i = 0; i < ranges.size(); i++) {
const auto& range = ranges[i]; dbgln("[{}] {}", i, ranges[i]);
klog() << " [" << i << "] " << range; }
} }
} }
#endif
void VolatilePageRanges::add_unchecked(const VolatilePageRange& range) void VolatilePageRanges::add_unchecked(const VolatilePageRange& range)
{ {
@ -66,15 +69,15 @@ bool VolatilePageRanges::add(const VolatilePageRange& range)
return false; return false;
add_range.was_purged = range.was_purged; add_range.was_purged = range.was_purged;
#if VOLATILE_PAGE_RANGES_DEBUG if constexpr (VOLATILE_PAGE_RANGES_DEBUG) {
klog() << "ADD " << range << " (total range: " << m_total_range << ") -->"; dbgln("ADD {} (total range: {}) -->", range, m_total_range);
dump_volatile_page_ranges(m_ranges); dump_volatile_page_ranges(m_ranges);
ScopeGuard debug_guard([&]() { ScopeGuard debug_guard([&]() {
klog() << "After adding " << range << " (total range: " << m_total_range << ")"; dbgln("After adding {} (total range: {})", range, m_total_range);
dump_volatile_page_ranges(m_ranges); dump_volatile_page_ranges(m_ranges);
klog() << "<-- ADD " << range << " (total range: " << m_total_range << ")"; dbgln("<-- ADD {} (total range: {})", range, m_total_range);
}); });
#endif }
size_t nearby_index = 0; size_t nearby_index = 0;
auto* existing_range = binary_search( auto* existing_range = binary_search(
@ -140,15 +143,15 @@ bool VolatilePageRanges::remove(const VolatilePageRange& range, bool& was_purged
if (remove_range.is_empty()) if (remove_range.is_empty())
return false; return false;
#if VOLATILE_PAGE_RANGES_DEBUG if constexpr (VOLATILE_PAGE_RANGES_DEBUG) {
klog() << "REMOVE " << range << " (total range: " << m_total_range << ") -->"; dbgln("REMOVE {} (total range: {}) -->", range, m_total_range);
dump_volatile_page_ranges(m_ranges); dump_volatile_page_ranges(m_ranges);
ScopeGuard debug_guard([&]() { ScopeGuard debug_guard([&]() {
klog() << "After removing " << range << " (total range: " << m_total_range << ")"; dbgln("After removing {} (total range: {})", range, m_total_range);
dump_volatile_page_ranges(m_ranges); dump_volatile_page_ranges(m_ranges);
klog() << "<-- REMOVE " << range << " (total range: " << m_total_range << ") was_purged: " << was_purged; dbgln("<-- REMOVE {} (total range: {}) was_purged: {}", range, m_total_range, was_purged);
}); });
#endif }
size_t nearby_index = 0; size_t nearby_index = 0;
auto* existing_range = binary_search( auto* existing_range = binary_search(