This is the mail archive of the
mailing list for the GDB project.
Re: [PATCH 5/7] Add support for software single step on ARM aarch32-linux in GDBServer.
- From: Yao Qi <qiyaoltc at gmail dot com>
- To: Antoine Tremblay <antoine dot tremblay at ericsson dot com>
- Cc: Yao Qi <qiyaoltc at gmail dot com>, <gdb-patches at sourceware dot org>
- Date: Tue, 15 Sep 2015 08:22:03 +0100
- Subject: Re: [PATCH 5/7] Add support for software single step on ARM aarch32-linux in GDBServer.
- Authentication-results: sourceware.org; auth=none
- References: <1441973603-15247-1-git-send-email-antoine dot tremblay at ericsson dot com> <1441973603-15247-6-git-send-email-antoine dot tremblay at ericsson dot com> <8637yh5kpz dot fsf at gmail dot com> <55F6C071 dot 1040104 at ericsson dot com> <86twqx3rty dot fsf at gmail dot com> <55F703AF dot 5010707 at ericsson dot com>
Antoine Tremblay <email@example.com> writes:
> Well my goal is to have feature parity between GDB and GDBServer as
> much as possible, so yes I would like to support BE8 in GDBServer.
This feature parity should be addressed separately, not by this patch
> Since BE8 support entails endianness awareness I can't split them out
> logically based on the BE8 feature, that would be writing the
> breakpoint handling code code without endianness support and then
> rewriting it completely with a patch labeled BE8 that would teach
> endianness to these features. This would be a major overhead with no
> value imho.
No, the value is we can make progress and address one specific problem
in one time. Patches are easy to review and discuss in this way.
I don't think BE8 GDBserver is fully supported, and I don't know anyone
running GDBserver on BE8.
> Maybe I misunderstood what you meant ?
> I think it's better to include endianness awareness in GDBServer from
> patch1 and not redo the work. In that case BE8 support becomes only 1
> variable in a function and thus does not need it's own patch.
From reviewer's point of view, I'd like each patch series is
self-contained and independent. Your patch series is about supporting
software breakpoint, software single step and tracepoint in ARM linux
GDBserver. Why do we need to worry about the different endianness of
GDBserver and inferior in your patch series? Your code may be right,
and we may need them sooner or later, but we can't do all of them in one