[lldb-dev] GDB RSP qVAttachOrWaitSupported

[jingham at apple.com]

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
>