[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V3 3/3] New option -gdb-opts
From: |
Fabien Chouteau |
Subject: |
Re: [Qemu-devel] [PATCH V3 3/3] New option -gdb-opts |
Date: |
Thu, 14 Mar 2013 11:59:39 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130221 Thunderbird/17.0.3 |
On 03/14/2013 09:44 AM, Markus Armbruster wrote:
> Fabien Chouteau <address@hidden> writes:
>
>> We introduce a new command line option. It's a generic option to
>> customize the gdb server:
>>
>> -gdb-opts [attached=on|off]
>>
>> The only parameter for now is "attached".
>>
>> Signed-off-by: Fabien Chouteau <address@hidden>
>
> --gdb-opts complements existing --gdb. You need to use both for full
> control.
>
> I figure you do this because you can't extend --gdb, as its argument is
> in legacy character device syntax, not QemuOpts.
>
That's right, maybe we can do some string manipulations to handle this case.
-gdb tcp::1234,attached=off
find ',attached={on|off}' and remove it from the string.
> We had similar cases before, and solved them differently: create a more
> general option, then make the old one sugar for the new one.
>
> For instance, --monitor and --qmp are sugar for --mon. Desugaring code
> is in monitor_parse().
>
Something like:
-chardev socket,id=gdb1,host=localhost,port=1234,server,nowait,nodelay
-gdb-remote chardev=gdb1,attached=off
You still need two options for full control.
Regards,
--
Fabien Chouteau
[Qemu-devel] [PATCH V3 1/3] Add GDB qAttached support, Fabien Chouteau, 2013/03/12