[Lldb-commits] [lldb] r354181 - Add some unconditional logging on the failure points when attaching
Jason Molenda via lldb-commits
lldb-commits at lists.llvm.org
Fri Feb 15 14:39:30 PST 2019
Author: jmolenda
Date: Fri Feb 15 14:39:30 2019
New Revision: 354181
URL: http://llvm.org/viewvc/llvm-project?rev=354181&view=rev
Log:
Add some unconditional logging on the failure points when attaching
to a process so we'll always get messages in the console logs.
Also make the "is frontboard process" / "is backboard process"
determination lazy, specifically take it out of the
MachProcess::AttachForDebug codepath when we are attaching to a
process, to simplify attaching.
<rdar://problem/47982516>
<rdar://problem/48060134>
Modified:
lldb/trunk/tools/debugserver/source/MacOSX/MachProcess.h
lldb/trunk/tools/debugserver/source/MacOSX/MachProcess.mm
lldb/trunk/tools/debugserver/source/MacOSX/MachTask.mm
Modified: lldb/trunk/tools/debugserver/source/MacOSX/MachProcess.h
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/tools/debugserver/source/MacOSX/MachProcess.h?rev=354181&r1=354180&r2=354181&view=diff
==============================================================================
--- lldb/trunk/tools/debugserver/source/MacOSX/MachProcess.h (original)
+++ lldb/trunk/tools/debugserver/source/MacOSX/MachProcess.h Fri Feb 15 14:39:30 2019
@@ -331,12 +331,11 @@ public:
}
}
- bool ProcessUsingSpringBoard() const {
- return (m_flags & eMachProcessFlagsUsingSBS) != 0;
- }
- bool ProcessUsingBackBoard() const {
- return (m_flags & eMachProcessFlagsUsingBKS) != 0;
- }
+ void CalculateBoardStatus();
+
+ bool ProcessUsingBackBoard();
+
+ bool ProcessUsingFrontBoard();
Genealogy::ThreadActivitySP GetGenealogyInfoForThread(nub_thread_t tid,
bool &timed_out);
@@ -349,9 +348,9 @@ private:
enum {
eMachProcessFlagsNone = 0,
eMachProcessFlagsAttached = (1 << 0),
- eMachProcessFlagsUsingSBS = (1 << 1),
- eMachProcessFlagsUsingBKS = (1 << 2),
- eMachProcessFlagsUsingFBS = (1 << 3)
+ eMachProcessFlagsUsingBKS = (1 << 2), // only read via ProcessUsingBackBoard()
+ eMachProcessFlagsUsingFBS = (1 << 3), // only read via ProcessUsingFrontBoard()
+ eMachProcessFlagsBoardCalculated = (1 << 4)
};
void Clear(bool detaching = false);
void ReplyToAllExceptions();
Modified: lldb/trunk/tools/debugserver/source/MacOSX/MachProcess.mm
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/tools/debugserver/source/MacOSX/MachProcess.mm?rev=354181&r1=354180&r2=354181&view=diff
==============================================================================
--- lldb/trunk/tools/debugserver/source/MacOSX/MachProcess.mm (original)
+++ lldb/trunk/tools/debugserver/source/MacOSX/MachProcess.mm Fri Feb 15 14:39:30 2019
@@ -2463,45 +2463,29 @@ pid_t MachProcess::AttachForDebug(pid_t
const char *err_cstr = err.AsString();
::snprintf(err_str, err_len, "%s",
err_cstr ? err_cstr : "No such process");
+ DNBLogError ("MachProcess::AttachForDebug pid %d does not exist", pid);
return INVALID_NUB_PROCESS;
}
SetState(eStateAttaching);
m_pid = pid;
-// Let ourselves know we are going to be using SBS or BKS if the correct flag
-// bit is set...
-#if defined(WITH_FBS) || defined(WITH_BKS)
- bool found_app_flavor = false;
-#endif
-
-#if defined(WITH_FBS)
- if (!found_app_flavor && IsFBSProcess(pid)) {
- found_app_flavor = true;
- m_flags |= eMachProcessFlagsUsingFBS;
- }
-#elif defined(WITH_BKS)
- if (!found_app_flavor && IsBKSProcess(pid)) {
- found_app_flavor = true;
- m_flags |= eMachProcessFlagsUsingBKS;
- }
-#elif defined(WITH_SPRINGBOARD)
- if (IsSBProcess(pid))
- m_flags |= eMachProcessFlagsUsingSBS;
-#endif
if (!m_task.StartExceptionThread(err)) {
const char *err_cstr = err.AsString();
::snprintf(err_str, err_len, "%s",
err_cstr ? err_cstr : "unable to start the exception thread");
DNBLogThreadedIf(LOG_PROCESS, "error: failed to attach to pid %d", pid);
+ DNBLogError ("MachProcess::AttachForDebug failed to start exception thread: %s", err_str);
m_pid = INVALID_NUB_PROCESS;
return INVALID_NUB_PROCESS;
}
errno = 0;
- if (::ptrace(PT_ATTACHEXC, pid, 0, 0))
+ if (::ptrace(PT_ATTACHEXC, pid, 0, 0)) {
err.SetError(errno);
- else
+ DNBLogError ("MachProcess::AttachForDebug failed to ptrace(PT_ATTACHEXC): %s", err.AsString());
+ } else {
err.Clear();
+ }
if (err.Success()) {
m_flags |= eMachProcessFlagsAttached;
@@ -2513,7 +2497,17 @@ pid_t MachProcess::AttachForDebug(pid_t
return m_pid;
} else {
::snprintf(err_str, err_len, "%s", err.AsString());
- DNBLogThreadedIf(LOG_PROCESS, "error: failed to attach to pid %d", pid);
+ DNBLogError ("MachProcess::AttachForDebug error: failed to attach to pid %d", pid);
+
+ struct kinfo_proc kinfo;
+ int mib[] = {CTL_KERN, KERN_PROC, KERN_PROC_PID, pid};
+ size_t len = sizeof(struct kinfo_proc);
+ if (sysctl(mib, sizeof(mib) / sizeof(mib[0]), &kinfo, &len, NULL, 0) == 0 && len > 0) {
+ if (kinfo.kp_proc.p_flag & P_TRACED) {
+ ::snprintf(err_str, err_len, "%s - process %d is already being debugged", err.AsString(), pid);
+ DNBLogError ("MachProcess::AttachForDebug pid %d is already being debugged", pid);
+ }
+ }
}
}
return INVALID_NUB_PROCESS;
@@ -2945,7 +2939,7 @@ pid_t MachProcess::LaunchForDebug(
const char *app_ext = strstr(path, ".app");
if (app_ext && (app_ext[4] == '\0' || app_ext[4] == '/')) {
std::string app_bundle_path(path, app_ext + strlen(".app"));
- m_flags |= eMachProcessFlagsUsingFBS;
+ m_flags |= (eMachProcessFlagsUsingFBS | eMachProcessFlagsBoardCalculated);
if (BoardServiceLaunchForDebug(app_bundle_path.c_str(), argv, envp,
no_stdio, disable_aslr, event_data,
launch_err) != 0)
@@ -2961,7 +2955,7 @@ pid_t MachProcess::LaunchForDebug(
const char *app_ext = strstr(path, ".app");
if (app_ext && (app_ext[4] == '\0' || app_ext[4] == '/')) {
std::string app_bundle_path(path, app_ext + strlen(".app"));
- m_flags |= eMachProcessFlagsUsingBKS;
+ m_flags |= (eMachProcessFlagsUsingBKS | eMachProcessFlagsBoardCalculated);
if (BoardServiceLaunchForDebug(app_bundle_path.c_str(), argv, envp,
no_stdio, disable_aslr, event_data,
launch_err) != 0)
@@ -3379,7 +3373,6 @@ pid_t MachProcess::SBLaunchForDebug(cons
m_pid = MachProcess::SBForkChildForPTraceDebugging(path, argv, envp, no_stdio,
this, launch_err);
if (m_pid != 0) {
- m_flags |= eMachProcessFlagsUsingSBS;
m_path = path;
size_t i;
char const *arg;
@@ -3734,7 +3727,7 @@ pid_t MachProcess::BoardServiceForkChild
bool success = false;
#ifdef WITH_BKS
- if (m_flags & eMachProcessFlagsUsingBKS) {
+ if (ProcessUsingBackBoard()) {
options =
BKSCreateOptionsDictionary(app_bundle_path, launch_argv, launch_envp,
stdio_path, disable_aslr, event_data);
@@ -3743,7 +3736,7 @@ pid_t MachProcess::BoardServiceForkChild
}
#endif
#ifdef WITH_FBS
- if (m_flags & eMachProcessFlagsUsingFBS) {
+ if (ProcessUsingFrontBoard()) {
options =
FBSCreateOptionsDictionary(app_bundle_path, launch_argv, launch_envp,
stdio_path, disable_aslr, event_data);
@@ -3779,12 +3772,12 @@ bool MachProcess::BoardServiceSendEvent(
// This is an event I cooked up. What you actually do is foreground the system
// app, so:
#ifdef WITH_BKS
- if (m_flags & eMachProcessFlagsUsingBKS) {
+ if (ProcessUsingBackBoard()) {
return_value = BKSCallOpenApplicationFunction(nil, nil, send_err, NULL);
}
#endif
#ifdef WITH_FBS
- if (m_flags & eMachProcessFlagsUsingFBS) {
+ if (ProcessUsingFrontBoard()) {
return_value = FBSCallOpenApplicationFunction(nil, nil, send_err, NULL);
}
#endif
@@ -3808,7 +3801,7 @@ bool MachProcess::BoardServiceSendEvent(
NSMutableDictionary *options = [NSMutableDictionary dictionary];
#ifdef WITH_BKS
- if (m_flags & eMachProcessFlagsUsingBKS) {
+ if (ProcessUsingBackBoard()) {
if (!BKSAddEventDataToOptions(options, event_data, send_err)) {
[pool drain];
return false;
@@ -3820,7 +3813,7 @@ bool MachProcess::BoardServiceSendEvent(
}
#endif
#ifdef WITH_FBS
- if (m_flags & eMachProcessFlagsUsingFBS) {
+ if (ProcessUsingFrontBoard()) {
if (!FBSAddEventDataToOptions(options, event_data, send_err)) {
[pool drain];
return false;
@@ -3916,3 +3909,36 @@ void MachProcess::FBSCleanupAfterAttach(
[pool drain];
}
#endif // WITH_FBS
+
+
+void MachProcess::CalculateBoardStatus()
+{
+ if (m_flags & eMachProcessFlagsBoardCalculated)
+ return;
+ if (m_pid == 0)
+ return;
+
+ bool found_app_flavor = false;
+#if defined(WITH_FBS)
+ if (!found_app_flavor && IsFBSProcess(m_pid)) {
+ found_app_flavor = true;
+ m_flags |= eMachProcessFlagsUsingFBS;
+ }
+#elif defined(WITH_BKS)
+ if (!found_app_flavor && IsBKSProcess(m_pid)) {
+ found_app_flavor = true;
+ m_flags |= eMachProcessFlagsUsingBKS;
+ }
+#endif
+ m_flags |= eMachProcessFlagsBoardCalculated;
+}
+
+bool MachProcess::ProcessUsingBackBoard() {
+ CalculateBoardStatus();
+ return (m_flags & eMachProcessFlagsUsingBKS) != 0;
+}
+
+bool MachProcess::ProcessUsingFrontBoard() {
+ CalculateBoardStatus();
+ return (m_flags & eMachProcessFlagsUsingFBS) != 0;
+}
Modified: lldb/trunk/tools/debugserver/source/MacOSX/MachTask.mm
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/tools/debugserver/source/MacOSX/MachTask.mm?rev=354181&r1=354180&r2=354181&view=diff
==============================================================================
--- lldb/trunk/tools/debugserver/source/MacOSX/MachTask.mm (original)
+++ lldb/trunk/tools/debugserver/source/MacOSX/MachTask.mm Fri Feb 15 14:39:30 2019
@@ -495,8 +495,10 @@ task_t MachTask::TaskPortForProcessID(pi
"pid = %d, &task ) => err = 0x%8.8x (%s)",
task_self, pid, err.Status(),
err.AsString() ? err.AsString() : "success");
- if (err.Fail())
+ if (err.Fail()) {
err.SetErrorString(str);
+ DNBLogError ("MachTask::TaskPortForProcessID task_for_pid failed: %s", str);
+ }
err.LogThreaded(str);
}
More information about the lldb-commits
mailing list