[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