This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH v2 3/7] Implement breakpoint_from_kind for supported architectures in GDBServer.
- From: Yao Qi <qiyaoltc at gmail dot com>
- To: Antoine Tremblay <antoine dot tremblay at ericsson dot com>
- Cc: <gdb-patches at sourceware dot org>
- Date: Thu, 15 Oct 2015 10:19:25 +0100
- Subject: Re: [PATCH v2 3/7] Implement breakpoint_from_kind for supported architectures in GDBServer.
- Authentication-results: sourceware.org; auth=none
- References: <1444063455-31558-1-git-send-email-antoine dot tremblay at ericsson dot com> <1444063455-31558-4-git-send-email-antoine dot tremblay at ericsson dot com>
Antoine Tremblay <antoine.tremblay@ericsson.com> writes:
There is no changelog entry.
> +/* Implementation of linux_target_ops method "breakpoint_from_kind". */
> +
> +static const unsigned char *
> +aarch64_breakpoint_from_kind (int *kind)
> +{
> + return (const unsigned char *) &aarch64_breakpoint;
Indentation looks odd, and do we really need the cast?
Note that this function is correct because we restrict the usage of Z0
packet. Z0 packet is only used with non-extended protocol and inferior
is 64bit. See aarch64_supports_z_point_type. Once we remove the
restriction, we need to update this function to return different
breakpoint instructions (aarch64, arm, thumb, and thumb2) according to
*KIND and other information.
Otherwise, patch is OK to me.
--
Yao (éå)