This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [Patch] Microblaze: Port of Linux gdbserver
- From: Pedro Alves <palves at redhat dot com>
- To: Ajit Kumar Agarwal <ajit dot kumar dot agarwal at xilinx dot com>, Michael Eager <eager at eagercon dot com>, Joel Brobecker <brobecker at adacore dot com>
- Cc: "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>, Vinod Kathail <vinodk at xilinx dot com>, Vidhumouli Hunsigida <vidhum at xilinx dot com>, Nagaraju Mekala <nmekala at xilinx dot com>
- Date: Thu, 18 Dec 2014 17:39:49 +0000
- Subject: Re: [Patch] Microblaze: Port of Linux gdbserver
- Authentication-results: sourceware.org; auth=none
- References: <25de23b98e054fd291ea232d10f2800c at BN1BFFO11FD018 dot protection dot gbl> <5436B7D0 dot 9060004 at eagercon dot com> <d0a23ff7601a4dbcab91d5b680bc2e13 at BY2FFO11FD017 dot protection dot gbl> <543E7630 dot 5060001 at redhat dot com> <99de529c620a4bb0a118a9fc39ae6ad0 at BY2FFO11FD008 dot protection dot gbl> <548F2240 dot 7020700 at redhat dot com> <b1a42502cd1843449e5668728141f5ed at BY2FFO11FD059 dot protection dot gbl> <5492BA45 dot 8090205 at redhat dot com> <8cc311a2f0ac4925b15f90a7b7853db8 at BN1AFFO11FD021 dot protection dot gbl>
On 12/18/2014 04:53 PM, Ajit Kumar Agarwal wrote:
>> 1. push_dummy_code is not implemented for Micro blaze port due to this there are 350+ failures.
>
>>> Eh, no inferior function call support. Are you planning on implementing this?
>
> Currently in the gdb microblaze-tdep.c, the following code is there for push_dummy_code.
>
> static CORE_ADDR
> microblaze_push_dummy_code (struct gdbarch *gdbarch, CORE_ADDR sp,
> CORE_ADDR funcaddr,
> struct value **args, int nargs,
> struct type *value_type,
> CORE_ADDR *real_pc, CORE_ADDR *bp_addr,
> struct regcache *regcache)
> {
> error (_("push_dummy_code not implemented"));
> return sp;
> }
> This causes the failures.
Yes, this are the hooks for supporting calling inferior functions
from gdb: "print foo()", etc.
The question was whether you were planning on replacing the
errors (this and microblaze_push_dummy_call's) with a real
implementation. :-)
Thanks,
Pedro Alves