[Lldb-commits] [PATCH] D145624: [lldb] Make MemoryCache::Read more resilient
Alex Langford via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Wed Mar 8 15:57:38 PST 2023
bulbazord created this revision.
bulbazord added reviewers: JDevlieghere, mib, clayborg, jasonmolenda, jingham, labath.
Herald added a project: All.
bulbazord requested review of this revision.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.
MemoryCache::Read is not resilient to partial reads when reading memory
chunks less than or equal in size to L2 cache lines. There have been
attempts in the past to fix this but nothing really solved the root of
the issue.
I first created a test exercising MemoryCache's implementation and
documenting how I believe MemoryCache::Read should behave. I then
rewrote the implementation of MemoryCache::Read as needed to make sure
that the different scenarios behaved correctly.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D145624
Files:
lldb/source/Target/Memory.cpp
lldb/unittests/Target/CMakeLists.txt
lldb/unittests/Target/MemoryTest.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D145624.503548.patch
Type: text/x-patch
Size: 19788 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20230308/8a4adbbc/attachment-0001.bin>
More information about the lldb-commits
mailing list