[Lldb-commits] [PATCH] D98822: [lldb] [Process] Watch for fork/vfork notifications
Michał Górny via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Tue Apr 13 03:04:55 PDT 2021
mgorny added a comment.
In D98822#2685246 <https://reviews.llvm.org/D98822#2685246>, @labath wrote:
> I've reverted this (121cff78a8032a73aa4fb820625dc1ecae8e3997 <https://reviews.llvm.org/rG121cff78a8032a73aa4fb820625dc1ecae8e3997>) because it made the linux bot super-flaky. I think it's likely that this affects _only_ linux (and the BSD code is fine -- though we don't have a bot to verify that), but I didn't want to experiment with partial reverts while the bots are wonky. However, it may be interesting the re-land the linux support in a separate commit, once we figure out the problem -- my money is on the nondeterminism of thread creation.
How about recommitting without the `clone()` change, i.e. assuming that `clone` always means thread, and seeing if that causes trouble still?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D98822/new/
https://reviews.llvm.org/D98822
More information about the lldb-commits
mailing list