[Lldb-commits] [lldb] r152438 - /lldb/trunk/tools/debugserver/source/MacOSX/MachVMMemory.cpp

Jim Ingham jingham at apple.com
Fri Mar 9 13:09:43 PST 2012


Author: jingham
Date: Fri Mar  9 15:09:42 2012
New Revision: 152438

URL: http://llvm.org/viewvc/llvm-project?rev=152438&view=rev
Log:
Make debugserver quieter about memory read errors.
<rdar://problem/10681220>

Modified:
    lldb/trunk/tools/debugserver/source/MacOSX/MachVMMemory.cpp

Modified: lldb/trunk/tools/debugserver/source/MacOSX/MachVMMemory.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/tools/debugserver/source/MacOSX/MachVMMemory.cpp?rev=152438&r1=152437&r2=152438&view=diff
==============================================================================
--- lldb/trunk/tools/debugserver/source/MacOSX/MachVMMemory.cpp (original)
+++ lldb/trunk/tools/debugserver/source/MacOSX/MachVMMemory.cpp Fri Mar  9 15:09:42 2012
@@ -104,9 +104,7 @@
         vm_offset_t vm_memory = NULL;
         m_err = ::mach_vm_read (task, curr_addr, curr_size, &vm_memory, &curr_bytes_read);
         
-        // We end up being asked to read memory at 0x0 a lot without that being a real error, so that ends up just
-        // causing a lot of useless log spam.  Only complain on failing reads if the address is not 0x0.
-        if (DNBLogCheckLogBit(LOG_MEMORY) || (m_err.Fail() && curr_addr != 0))
+        if (DNBLogCheckLogBit(LOG_MEMORY))
             m_err.LogThreaded("::mach_vm_read ( task = 0x%4.4x, addr = 0x%8.8llx, size = %llu, data => %8.8p, dataCnt => %i )", task, (uint64_t)curr_addr, (uint64_t)curr_size, vm_memory, curr_bytes_read);
 
         if (m_err.Success())





More information about the lldb-commits mailing list