Index: lldb/trunk/include/lldb/Host/common/NativeRegisterContext.h =================================================================== --- lldb/trunk/include/lldb/Host/common/NativeRegisterContext.h +++ lldb/trunk/include/lldb/Host/common/NativeRegisterContext.h @@ -99,6 +99,15 @@ virtual Error ClearAllHardwareWatchpoints (); + virtual Error + IsWatchpointHit (uint8_t wp_index); + + virtual Error + IsWatchpointVacant (uint32_t wp_index); + + virtual lldb::addr_t + GetWatchpointAddress (uint32_t wp_index); + virtual bool HardwareSingleStep (bool enable); Index: lldb/trunk/source/Host/common/NativeRegisterContext.cpp =================================================================== --- lldb/trunk/source/Host/common/NativeRegisterContext.cpp +++ lldb/trunk/source/Host/common/NativeRegisterContext.cpp @@ -303,6 +303,24 @@ return Error ("not implemented"); } +Error +NativeRegisterContext::IsWatchpointHit (uint8_t wp_index) +{ + return Error ("not implemented"); +} + +Error +NativeRegisterContext::IsWatchpointVacant (uint32_t wp_index) +{ + return Error ("not implemented"); +} + +lldb::addr_t +NativeRegisterContext::GetWatchpointAddress (uint32_t wp_index) +{ + return LLDB_INVALID_ADDRESS; +} + bool NativeRegisterContext::HardwareSingleStep (bool enable) { Index: lldb/trunk/source/Plugins/Process/Linux/NativeRegisterContextLinux_x86_64.h =================================================================== --- lldb/trunk/source/Plugins/Process/Linux/NativeRegisterContextLinux_x86_64.h +++ lldb/trunk/source/Plugins/Process/Linux/NativeRegisterContextLinux_x86_64.h @@ -46,10 +46,10 @@ WriteAllRegisterValues (const lldb::DataBufferSP &data_sp) override; Error - IsWatchpointHit(uint8_t wp_index); + IsWatchpointHit(uint8_t wp_index) override; Error - IsWatchpointVacant(uint32_t wp_index); + IsWatchpointVacant(uint32_t wp_index) override; bool ClearHardwareWatchpoint(uint32_t wp_index) override; @@ -66,7 +66,7 @@ uint32_t watch_flags) override; lldb::addr_t - GetWatchpointAddress(uint32_t wp_index); + GetWatchpointAddress(uint32_t wp_index) override; uint32_t NumSupportedHardwareWatchpoints() override; Index: lldb/trunk/source/Plugins/Process/Linux/NativeThreadLinux.cpp =================================================================== --- lldb/trunk/source/Plugins/Process/Linux/NativeThreadLinux.cpp +++ lldb/trunk/source/Plugins/Process/Linux/NativeThreadLinux.cpp @@ -382,36 +382,51 @@ void NativeThreadLinux::SetStoppedByWatchpoint () { + Log *log (GetLogIfAllCategoriesSet (LIBLLDB_LOG_THREAD)); + lldb::pid_t pid = LLDB_INVALID_PROCESS_ID; + if (log) + { + NativeProcessProtocolSP process_sp = m_process_wp.lock (); + if (process_sp) + pid = process_sp->GetID (); + } + const StateType new_state = StateType::eStateStopped; MaybeLogStateChange (new_state); m_state = new_state; - m_stop_info.reason = StopReason::eStopReasonWatchpoint; - m_stop_info.details.signal.signo = SIGTRAP; - - NativeRegisterContextLinux_x86_64 *reg_ctx = - reinterpret_cast (GetRegisterContext().get()); - const uint32_t num_hw_watchpoints = - reg_ctx->NumSupportedHardwareWatchpoints(); + NativeRegisterContextSP reg_ctx = GetRegisterContext (); + const uint32_t num_hw_watchpoints = reg_ctx->NumSupportedHardwareWatchpoints (); m_stop_description.clear (); for (uint32_t wp_index = 0; wp_index < num_hw_watchpoints; ++wp_index) - if (reg_ctx->IsWatchpointHit(wp_index).Success()) + { + if (reg_ctx->IsWatchpointHit (wp_index).Success()) { + if (log) + log->Printf ("NativeThreadLinux:%s (pid=%" PRIu64 ", tid=%" PRIu64 ") watchpoint found with idx: %u", + __FUNCTION__, pid, GetID (), wp_index); + std::ostringstream ostr; - ostr << reg_ctx->GetWatchpointAddress(wp_index) << " " << wp_index; + ostr << reg_ctx->GetWatchpointAddress (wp_index) << " " << wp_index; m_stop_description = ostr.str(); + + m_stop_info.reason = StopReason::eStopReasonWatchpoint; + m_stop_info.details.signal.signo = SIGTRAP; return; } - Log *log (GetLogIfAllCategoriesSet (LIBLLDB_LOG_THREAD)); - if (log) - { - NativeProcessProtocolSP m_process_sp = m_process_wp.lock (); - lldb::pid_t pid = m_process_sp ? m_process_sp->GetID () : LLDB_INVALID_PROCESS_ID; - log->Printf ("NativeThreadLinux: thread (pid=%" PRIu64 ", tid=%" PRIu64 ") " - "stopped by a watchpoint, but failed to find it", - pid, GetID ()); } + + // The process reported a watchpoint was hit, but we haven't found the + // watchpoint. Assume that a stopped by trace is reported as a hardware + // watchpoint what happens on some linux kernels (e.g.: android-arm64 + // platfrom-21). + + if (log) + log->Printf ("NativeThreadLinux:%s (pid=%" PRIu64 ", tid=%" PRIu64 ") none of the watchpoint was hit.", + __FUNCTION__, pid, GetID ()); + + SetStoppedByTrace (); } bool