labath added inline comments.

================
Comment at: 
lldb/trunk/packages/Python/lldbsuite/test/macosx/queues/TestQueues.py:132
             self.fail("Failed to stop at breakpoint 1.")
+        self.await_token(exe)
 
----------------
Will this actually help? It could be I'm missing something about how macos 
queues work, but it seems to me that at this point the process is already 
stopped, and so it has either managed to create the token files, or it hasn't. 
Waiting isn't going to make things better.

The way I'd synchronize these things is via barriers. I'd put one 
`pseudo_barrier_wait` in place of the `touch` calls, and then another just 
before the call to `stopper`. That way you guarantee that the queues have been 
initialized even before you hit the `stopper` breakpoint.

Note that using barriers will require porting this code to (obj)c++ (they use 
atomics), but that doesn't seem like it would alter the purpose of this test.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56208/new/

https://reviews.llvm.org/D56208



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to