[lldb-dev] GDB RSP qVAttachOrWaitSupported

[Todd Fiala]

Oh right, good point.

All the remote sides I worked with essentially have a default incoming
packet rule that says "if I don't know what this is, respond with $#00", so
no need to add that explicitly if your remote side already does that,
Matthew.

-Todd


On Fri, May 9, 2014 at 9:27 AM, <jingham at apple.com> wrote:

> Note also as a general rule, you never need to add support for a query
> packet unless you support the thing it is querying for, since whoever is
> sending it is supposed to take the "unrecognized packet" error to mean not
> supported.
>
> Jim
>
> On May 9, 2014, at 9:26 AM, jingham at apple.com wrote:
>
> > My bad, there are actually a bunch of vAttach variant packets we added
> to support attaching by process name, and "wait for a process to show up
> and then attach to it" and the like.
> >
> > I added documentation for them.
> >
> > Jim
> >
> > On May 9, 2014, at 8:57 AM, Todd Fiala <todd.fiala at gmail.com> wrote:
> >
> >> Hey Matthew,
> >>
> >> Essentially you'll want to return a string like this:
> >> $OK#9a
> >>
> >> if your gdb remote debug monitor can handle that (undocumented)
> element, or return this:
> >> $#00
> >>
> >> if you do not support the packet.
> >>
> >> I've been digging into the protocol lately since I'm adding tests for
> gdb remote protocol-layer activity, so that I can verify lldb-gdbserver
> (llgs) on Linux is acting identically to debugserver on MacOSX (and to make
> it easier for us to transition from debugserver to llgs).
> >>
> >> I had noted that packet wasn't documented a while back.  When I get to
> testing that one, I'll update the doc text file on it.
> >>
> >>
> >> On Fri, May 9, 2014 at 6:40 AM, Abid, Hafiz <Hafiz_Abid at mentor.com>
> wrote:
> >> I have not read the documentation for this packet. But looking at the
> code,
> >> It seems that you will return OK if your stub supports this packet.
> Otherwise an empty reply can
> >> Indicate that packet is not supported.
> >>
> >> Please look at the following link for more details about query packets.
> >>
> https://sourceware.org/gdb/onlinedocs/gdb/General-Query-Packets.html#General-Query-Packets
> >>
> >> Thanks,
> >> Abid
> >>
> >>> -----Original Message-----
> >>> From: Matthew Gardiner [mailto:mg11 at csr.com]
> >>> Sent: 09 May 2014 13:24
> >>> To: Abid, Hafiz; lldb-dev at cs.uiuc.edu
> >>> Subject: Re: [lldb-dev] GDB RSP qVAttachOrWaitSupported
> >>>
> >>> Abid, Hafiz wrote:
> >>>> RNBRemote::HandlePacket_qVAttachOrWaitSupported has the following
> >>> comments which may give you some idea.
> >>>>     // We support attachOrWait meaning attach if the process exists,
> >>> otherwise wait to attach.
> >>> Ok, thanks.
> >>>
> >>> So this whole entity "VAttachOrWaitSupported" is a concept which the
> stub
> >>> does or not support, correct?
> >>>
> >>> If so, does the stub write back "1", if it does support it, otherwise
> "0"?
> >>>
> >>> Presumably if the stub does support this feature, are there any other
> >>> messages which the stub should then support?
> >>>
> >>> I'm still not really clear what's happening here....
> >>>
> >>> Matt
> >>>
> >>>
> >>>
> >>> Member of the CSR plc group of companies. CSR plc registered in England
> >>> and Wales, registered number 4187346, registered office Churchill
> House,
> >>> Cambridge Business Park, Cowley Road, Cambridge, CB4 0WZ, United
> >>> Kingdom More information can be found at www.csr.com. Keep up to date
> >>> with CSR on our technical blog, www.csr.com/blog, CSR people blog,
> >>> www.csr.com/people, YouTube, www.youtube.com/user/CSRplc, Facebook,
> >>> www.facebook.com/pages/CSR/191038434253534, or follow us on Twitter at
> >>> www.twitter.com/CSR_plc.
> >>> New for 2014, you can now access the wide range of products powered by
> >>> aptX at www.aptx.com.
> >>
> >> _______________________________________________
> >> lldb-dev mailing list
> >> lldb-dev at cs.uiuc.edu
> >> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev
> >>
> >>
> >>
> >> --
> >> -Todd
> >> _______________________________________________
> >> 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
>



-- 
Todd Fiala | Software Engineer | tfiala at google.com | 650-943-3180
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20140509/caf9bc27/attachment.html>