Re: [PATCH 2/2] Support single step by arch or target

Eli Zaretskii
2015-09-01T14:40:38Z

> From: Yao Qi <qiyaoltc@gmail.com>
> Date: Tue, 1 Sep 2015 09:41:55 +0100
>
> gdb:
>
> 2015-09-01 Yao Qi <yao.qi@linaro.org>
>
> * aarch64-linux-nat.c (aarch64_linux_can_do_single_step): New
> function.
> (_initialize_aarch64_linux_nat): Install it to to_can_do_single_step.
> * arm-linux-tdep.c (arm_linux_software_single_step): Return 0
> if target_can_do_single_step returns 1.
> * remote.c (struct vCont_action_support) <s, S>: New fields.
> (PACKET_vContSupported): New enum.
> (remote_protocol_features): New element for vContSupported.
> (remote_query_supported): Append "vContSupported+".
> (remote_vcont_probe): Remove support_s and support_S, use
> rs->supports_vCont.s and rs->supports_vCont.S instead. Disable
> vCont packet if c and C actions are not supported.
> (remote_can_do_single_step): New function.
> (init_remote_ops): Install it to to_can_do_single_step.
> (_initialize_remote): Call add_packet_config_cmd.
> * target.h (struct target_ops) <to_can_do_single_step>: New field.
> (target_can_do_single_step): New macro.
> * target-delegates.c: Re-generated.
>
> gdb/gdbserver:
>
> 2015-09-01 Yao Qi <yao.qi@linaro.org>
>
> * server.c (vCont_supported): New global variable.
> (handle_query): Set vCont_supported to 1 if "vContSupported+"
> matches. Append ";vContSupported+" to own_buf.
> (handle_v_requests): Append ";s;S" to own_buf if target supports
> hardware single step or vCont_supported is false.
> (capture_main): Set vCont_supported to zero.
>
> gdb/doc:
>
> 2015-09-01 Yao Qi <yao.qi@linaro.org>
>
> * gdb.texinfo (General Query Packets): Add vContSupported to
> tables of 'gdbfeatures' and 'stub features' supported in the
> qSupported packet, as well as to the list containing stub
> feature details.

OK for the documentation part.

Thanks.