[lldb-dev] Process::SyncIOHandler(), everyone please try out the following patch

[Greg Clayton]

Try this one out and let me know if anything improves?

-------------- next part --------------
A non-text attachment was scrubbed...
Name: io2.patch
Type: application/octet-stream
Size: 26886 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20150409/d5e1197c/attachment.obj>
-------------- next part --------------



> On Apr 9, 2015, at 11:23 AM, Zachary Turner <zturner at google.com> wrote:
> 
> If you need a VM or something let me know I can proabbly make something that you can remote into and is already properly set up and ready to build.
> 
> On Thu, Apr 9, 2015 at 11:02 AM Greg Clayton <gclayton at apple.com> wrote:
> Thanks everyone for trying this out. I will modify this patch and try again soon.
> 
> > On Apr 9, 2015, at 7:56 AM, Pavel Labath <labath at google.com> wrote:
> >
> > The situation is even worse on Linux, i'm afraid. Whereas I was
> > getting no wrong prompts before (at least when debugging locally), now
> > the prompt appears in the wrong place in about 10% of the cases.
> >
> > pl
> >
> > On 9 April 2015 at 15:19, Ed Maste <emaste at freebsd.org> wrote:
> >> On 8 April 2015 at 16:35, Greg Clayton <gclayton at apple.com> wrote:
> >>>
> >>> This one removes the m_iohandler_sync completely and coordinates async IO with the IOHandlers. Let me know if this works any better for freebsd, linux or windows.
> >>
> >> Problem persists with this version on FreeBSD, unfortunately:
> >>
> >> ...
> >>   165
> >>   166          (void)setlocale(LC_ALL, "");
> >> (lldb) step
> >> (lldb) Process 77191 stopped
> >> * thread #1: tid = 100853, 0x00000000004023fa ls`main(argc=1,
> >> argv=0x00007fffffffe588) + 42 at ls.c:166, stop reason = step in
> >>    frame #0: 0x00000000004023fa ls`main(argc=1,
> >> argv=0x00007fffffffe588) + 42 at ls.c:166
> >>   163          char *bp = tcapbuf;
> >>   164  #endif
> >>   165
> >> -> 166          (void)setlocale(LC_ALL, "");
> >>   167
> >>   168          /* Terminal defaults to -Cq, non-terminal defaults to -1. */
> >>   169          if (isatty(STDOUT_FILENO)) {
> >> step
> >> ...
> >> _______________________________________________
> >> lldb-dev mailing list
> >> lldb-dev at cs.uiuc.edu
> >> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev
> 
> 
> _______________________________________________
> lldb-dev mailing list
> lldb-dev at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev