<div dir="ltr">This just went in. I had to make several adjustments for the OSX build as I had code that had to straddle Host.mm and Host.cpp. I updated the fix to include the Haswell change (that method moved from Host.mm to Host.cpp).<div>
<br></div><div>I used the configure-based OSX build process, which I can get to build but not run tests yet. Let me know if you hit any OSX test breaks due to this.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Wed, Jan 22, 2014 at 3:04 PM, Todd Fiala <span dir="ltr"><<a href="mailto:tfiala@google.com" target="_blank">tfiala@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">(And I'll adjust the comment).</div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jan 22, 2014 at 3:04 PM, Todd Fiala <span dir="ltr"><<a href="mailto:tfiala@google.com" target="_blank">tfiala@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Thanks. It tested fine on Ubuntu, moving over to OS X.</div><div><div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Wed, Jan 22, 2014 at 2:50 PM, Greg Clayton <span dir="ltr"><<a href="mailto:gclayton@apple.com" target="_blank">gclayton@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Looks good except the "// No more thread specific current working directory" comment when about to call __pthread_chdir() should be changed to:<br>
<br>
// Set the working directory on this thread only<br>
<div><div><br>
<br>
On Jan 22, 2014, at 1:45 PM, Todd Fiala <<a href="mailto:tfiala@google.com" target="_blank">tfiala@google.com</a>> wrote:<br>
<br>
> So I just about have this done (the original 3 steps, not switching to fork()/exec()).<br>
><br>
> One thing I hit was an area I noticed yesterday: on Linux (x86_64, Ubuntu 12.04), I'm finding that I need to *not* take defaults for all the signal handlers in lldb-gdbserver; otherwise, the target process never starts operating. Technically the target process is exiting immediately with an exit_state of 127 with signal = 0.<br>
><br>
> For the moment, I have that #ifdef'd in the Apple/Linux/FreeBSD LaunchProcessPosixSpawn method so that only Linux doesn't reset the child signal handlers to defaults. I'm thinking the FreeBSD/Apple BSD lineage might behave more similarly here as my first stake in the sand for a guess, but that would be changing behavior for FreeBSD. (Ed - any thoughts here? I could just as well flip the #if defined(__linux__) to #if !defined(__APPLE__).) Ultimately the root cause of this is not clear to me yet.<br>
><br>
> For the directory handling, I'm having that handled via __pthread* methods on Apple and continuing to do what it did before on everything else. I'll look into the fork()/exec() option as a follow up after I work through some of the more obviously broken bits of lldb-gdbserver.<br>
><br>
> See below for the current state of the patch. I'm about to run tests on Ubuntu and try a build and tests on my MBP.<br>
><br>
> -Todd<br>
><br>
><br>
> On Wed, Jan 22, 2014 at 10:24 AM, Greg Clayton <<a href="mailto:gclayton@apple.com" target="_blank">gclayton@apple.com</a>> wrote:<br>
> You might need to add "#if defined(__APPLE__)" around the calls to __pthread_chdir and __pthread_fchdir. Those might be darwin specific.<br>
><br>
> There is no way to set your current working directory when using posix_spawn, so to work around this, we change directory in a thread specific way by calling __pthread_chdir(working_dir) and reverting it using __pthread_fchdir(-1), so as to not hose over the process that has the lldb shared library loaded by changing the global working directory.<br>
><br>
> So in order to eventually fix being able to obey the launch flag eLaunchFlagDebug and also to be able to set the working directory, non darwin builds might need to change over to using fork()/exec() since after the fork the child process can set its working directory and also call ptrace() to start being debuggable and then calling exec(). When ptrace() has been called to allow the process to be debugged, I believe it stops right at the entry point upon exec. Again, this would be for non-darwin builds since Apple has a posix_spawn flag of POSIX_SPAWN_START_SUSPENDED that can be set to enforce stopping at the entry point and also has the thread specific way to change directory.<br>
><br>
> Greg<br>
><br>
><br>
> On Jan 22, 2014, at 10:00 AM, Todd Fiala <<a href="mailto:tfiala@google.com" target="_blank">tfiala@google.com</a>> wrote:<br>
><br>
> > Correct.<br>
> ><br>
> ><br>
> > On Wed, Jan 22, 2014 at 9:55 AM, Greg Clayton <<a href="mailto:gclayton@apple.com" target="_blank">gclayton@apple.com</a>> wrote:<br>
> > So the fix will be:<br>
> > 1 - Remove LaunchProcessPosixSpawn() from Host.mm<br>
> > 2 - Copy it over into Host.cpp<br>
> > 3 - Make sure LaunchProcessPosixSpawn is compiled for Apple builds as well in Host.cpp<br>
> ><br>
> > Right?<br>
> ><br>
> ><br>
> > On Jan 22, 2014, at 9:54 AM, Todd Fiala <<a href="mailto:tfiala@google.com" target="_blank">tfiala@google.com</a>> wrote:<br>
> ><br>
> > > > It is indeed the right fix.<br>
> > ><br>
> > > Ok - I'll take care of that part. Thanks!<br>
> > ><br>
> > ><br>
> > > On Wed, Jan 22, 2014 at 9:52 AM, Greg Clayton <<a href="mailto:gclayton@apple.com" target="_blank">gclayton@apple.com</a>> wrote:<br>
> > > It is indeed the right fix. Host.mm has a mac specific version which does this:<br>
> > ><br>
> > > sigset_t no_signals;<br>
> > > sigset_t all_signals;<br>
> > > sigemptyset (&no_signals);<br>
> > > sigfillset (&all_signals);<br>
> > > ::posix_spawnattr_setsigmask(&attr, &no_signals);<br>
> > > ::posix_spawnattr_setsigdefault(&attr, &all_signals);<br>
> > ><br>
> > > Compared to the incorrect version you noticed in Host.cpp:<br>
> > ><br>
> > > sigset_t no_signals;<br>
> > > sigset_t all_signals;<br>
> > > sigemptyset (&no_signals);<br>
> > > sigfillset (&all_signals);<br>
> > > ::posix_spawnattr_setsigmask(&attr, &all_signals);<br>
> > > ::posix_spawnattr_setsigdefault(&attr, &no_signals);<br>
> > ><br>
> > > (notice all_signals and no_signals are reversed in the last two function calls in Host.cpp.<br>
> > ><br>
> > > You might compare the "LaunchProcessPosixSpawn" in Host.mm and try and just copy it to Host.cpp and see if it works. I can't find anything darwin specific in the code after a quick glance and we have used the Host.mm LaunchProcessPosixSpawn() for a few years now, so it is definitely tested...<br>
> > ><br>
> > > So the fix would seem to be:<br>
> > > 1 - Remove LaunchProcessPosixSpawn() from Host.mm<br>
> > > 2 - Copy it over into Host.cpp<br>
> > > 3 - Make sure LaunchProcessPosixSpawn is compiled for Apple builds as well in Host.cpp<br>
> > ><br>
> > ><br>
> > > On Jan 21, 2014, at 10:49 PM, Todd Fiala <<a href="mailto:tfiala@google.com" target="_blank">tfiala@google.com</a>> wrote:<br>
> > ><br>
> > > > Hi all,<br>
> > > ><br>
> > > > In source/Host/common/Host.cpp, there is a posix process spawning method called LaunchProcessPosixSpawn. On my Linux system, it is used to start the target process in lldb-gdbserver. It looks like FreeBSD uses it as well. Most of the Platform launchers appear to funnel to it as well (via Host::LaunchProcess ()).<br>
> > > ><br>
> > > > There is a section of code in LaunchProcessPosixSpawn that masks all signals for the child process that is started up:<br>
> > > ><br>
> > > > ::posix_spawnattr_setsigmask(&attr, &all_signals)<br>
> > > ><br>
> > > > When that is set, it seems to be preventing the child from receiving everything except the non-blockable signals. This has the effect of (at the very least) blocking SIGINT (i.e. ^C from the keyboard) and SIGTERM (standard unadorned "kill pid"). This appears to be the cause of a few bugs as I try to get lldb-gdbserver working on Linux. For example:<br>
> > > ><br>
> > > > 1. hitting ^C on an lldb-gdbserver that spawned a debuggee target process would kill the lldb-gdbserver, but not the debuggee target, which continues to run.<br>
> > > ><br>
> > > > 2. sending a SIGTERM (kill {debuggee pid}) while it is getting debugged and running is ignored.<br>
> > > ><br>
> > > > 3. sending a SIGTERM to the debuggee after issue #1 (where lldb-gdbserver is no longer running) is ignored.<br>
> > > ><br>
> > > > 4. killing lldb-gdbserver from an attached lldb that shuts down and kills the remote does indeed kill lldb-gebserver, but does not kill the target process.<br>
> > > ><br>
> > > > (Of course sending a 'kill -9 {debuggee pid}' works fine to kill the target process).<br>
> > > ><br>
> > > ><br>
> > > > I've modified this method locally to not mask any signals:<br>
> > > ><br>
> > > > ::posix_spawnattr_setsigmask (&attr, &no_signals)<br>
> > > ><br>
> > > > This seems to address all the problems I had above for lldb-gdbserver as signals propagate properly, and the target responds correctly to signals sent when the parent dies, etc.<br>
> > > ><br>
> > > > I've also run all the existing tests (on my system, that amounts to 275 tests that really run), and those are all passing.<br>
> > > ><br>
> > > > However, given that so much code flows through here (or at least appears to) that is not directly related to the lldb-gdbserver --- i.e. local linux/FreeBSD debugging --- I'm highly skeptical that this is the right fix. I did try using local debugging with lldb with my change in place, and that seemed to work fine. But I'm thinking that perhaps the signal blocking was intended and that behavior is needed in some cases that (perhaps) are not covered by tests that run on Linux.<br>
> > > ><br>
> > > > Any thoughts on why all the signals were getting masked on process spawning? Does that change look okay as is?<br>
> > > ><br>
> > > > Thanks!<br>
> > > ><br>
> > > > Sincerely,<br>
> > > > Todd Fiala<br>
> > > > _______________________________________________<br>
> > > > lldb-dev mailing list<br>
> > > > <a href="mailto:lldb-dev@cs.uiuc.edu" target="_blank">lldb-dev@cs.uiuc.edu</a><br>
> > > > <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev</a><br>
> > ><br>
> > ><br>
> > ><br>
> > ><br>
> > > --<br>
> > > Todd Fiala | Software Engineer | <a href="mailto:tfiala@google.com" target="_blank">tfiala@google.com</a> | <a href="tel:650-943-3180" value="+16509433180" target="_blank">650-943-3180</a><br>
> > ><br>
> ><br>
> ><br>
> ><br>
> ><br>
> > --<br>
> > Todd Fiala | Software Engineer | <a href="mailto:tfiala@google.com" target="_blank">tfiala@google.com</a> | <a href="tel:650-943-3180" value="+16509433180" target="_blank">650-943-3180</a><br>
> ><br>
><br>
><br>
><br>
><br>
> --<br>
> Todd Fiala | Software Engineer | <a href="mailto:tfiala@google.com" target="_blank">tfiala@google.com</a> | <a href="tel:650-943-3180" value="+16509433180" target="_blank">650-943-3180</a><br>
><br>
</div></div>> <posix_spawn_2014-01-22_01.diff><br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><table cellspacing="0" cellpadding="0" style="color:rgb(136,136,136);font-family:'Times New Roman'"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small">
<td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Todd Fiala |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Software Engineer |</td>
<td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:tfiala@google.com" style="color:rgb(17,85,204)" target="_blank"><span style="background-color:rgb(255,255,204);color:rgb(34,34,34);background-repeat:initial initial">tfiala@google.com</span></a> |</td>
<td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"><font color="#1155cc"> <a>650-943-3180</a></font></td></tr></tbody></table><br></div>
</div>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><table cellspacing="0" cellpadding="0" style="color:rgb(136,136,136);font-family:'Times New Roman'"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small">
<td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Todd Fiala |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Software Engineer |</td>
<td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:tfiala@google.com" style="color:rgb(17,85,204)" target="_blank"><span style="background-color:rgb(255,255,204);color:rgb(34,34,34);background-repeat:initial initial">tfiala@google.com</span></a> |</td>
<td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"><font color="#1155cc"> <a>650-943-3180</a></font></td></tr></tbody></table><br></div>
</div>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><table cellspacing="0" cellpadding="0" style="color:rgb(136,136,136);font-family:'Times New Roman'"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small">
<td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Todd Fiala |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Software Engineer |</td>
<td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:tfiala@google.com" style="color:rgb(17,85,204)" target="_blank"><span style="background-color:rgb(255,255,204);color:rgb(34,34,34);background-repeat:initial initial">tfiala@google.com</span></a> |</td>
<td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"><font color="#1155cc"> <a>650-943-3180</a></font></td></tr></tbody></table><br></div>
</div>