This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFC/TileGX 1/2] fix gdbserver build failure
- From: Joel Brobecker <brobecker at adacore dot com>
- To: Jiong Wang <jiwang at tilera dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Tue, 17 Sep 2013 05:30:42 -0700
- Subject: Re: [RFC/TileGX 1/2] fix gdbserver build failure
- Authentication-results: sourceware.org; auth=none
- References: <50F9148F dot 3010602 at tilera dot com> <5237C847 dot 3040908 at tilera dot com>
> TileGX gdbserver build failed on native board.
>
> "srv_tgtobjâ in gdbserver/configure.src not updated.
>
>
> gdbserver/ChangeLog:
>
> * configure.srv (srv_tgtobj): Remove linux-osdata.o, add linux-tile-low.o.
>
> please review.
I am trying to understand the circumstances that lead to the failure,
and it's been made harder due to the lack of information. After some
investigation, my guess is that you're getting some duplicate symbols
because linux-osdata.o" is already in srv_tgtobj. Is that correct?
If yes, this part of the patch is OK, obvious even (once explained!).
I don't understand, however, what difference adding linux-tile-low.o
makes in terms of build failure. Again, I have been trying to figure
out what lead to the failure, and searched our recent past, but could
not find anything. I know the change makes complete sense, but if not
logically tied to the build failure you are mentioning, I think we
should, if nothing else, document that fact for future archival, and
also commit this part of the change separately.
> diff --git a/gdb/gdbserver/configure.srv b/gdb/gdbserver/configure.srv
> index 6eb5a8a..f65dd10 100644
> --- a/gdb/gdbserver/configure.srv
> +++ b/gdb/gdbserver/configure.srv
> @@ -340,7 +340,7 @@ case "${target}" in
> ;;
> tilegx-*-linux*) srv_regobj=reg-tilegx.o
> srv_regobj="${srv_regobj} reg-tilegx32.o"
> - srv_tgtobj="$srv_linux_obj linux-osdata.o"
> + srv_tgtobj="$srv_linux_obj linux-tile-low.o"
> srv_linux_regsets=yes
> srv_linux_thread_db=yes
> ;;
--
Joel