KLapshin added a comment.

In http://reviews.llvm.org/D12968#261762, @labath wrote:

> I have committed http://reviews.llvm.org/D13056 with the event hijacking 
> portion from your patch. I think your use case should be working now. I am 
> planning to return to this later, as I believe there are still some edge 
> cases lurking here.


Yes, of course - hijacked listener setup solves (or hides) issue.

Problem on my side (if no hijacking listener used and without your patch for 
race condition) is - halt listener often just got timeout (10 secs), then 
target process killed without crash. Or may work without timeout - this is 
really threads conflict issue - m_listener in process object unusable because 
operates with invalid data - I even saw strings in log belongs to RSP module - 
just random memory areas, memory corrupted. If you set hijacked listener - no 
problem because it looks has non-shared m_events collection.

Again - this is remote debugging issue only. Also I checked on Linux host 
debugging - process killed just fine.

I will try pure public lldb build with your patch and will see then, thanks.


Repository:
  rL LLVM

http://reviews.llvm.org/D12968



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

Reply via email to