This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [rfa] Add -m; Was: ARM sim patch: increase default target memory
- From: Andrew Cagney <ac131313 at cygnus dot com>
- To: Nick Clifton <nickc at cambridge dot redhat dot com>
- Cc: Anthony Green <green at redhat dot com>, gdb-patches at sources dot redhat dot com
- Date: Mon, 08 Apr 2002 23:05:24 -0400
- Subject: Re: [rfa] Add -m; Was: ARM sim patch: increase default target memory
- References: <200203171650.g2HGo8714138@louie.sfbay.redhat.com> <3CB07C82.3040902@cygnus.com> <m3g026r0tl.fsf@north-pole.nickc.cambridge.redhat.com>
> Hi Andrew,
>
>
>> The attatched patch (er, hack), adds support for the -m flag from
>> the GDB command line (vis (gdb) target sim -m<size>).
>
>
> Hmm, this looks generally OK to me, except that I would recommend
> using "strtoul" instead of "atoi" to convert the string into a
> number, so that hex values can be specified as well as decimal.
Good point.
> I
> would also suggest that you consider adding support for a megabyte
> postfix so that the user could do:
>
> -m 6M
Good idea, all the sims should do this. Any pointers for the code?
I'll likely bug report it for now.
> to set up a 6 megabyte virtual memory, instead of having to type:
>
> -m 6291456
>
>
>> It also sets the default memory size back to something consistent
>> with the other simulators.
>
>
> This will stop the java tests from running, so maybe you should also
> add the new switch to the java test harness so that they do not fail.
Do you know which file?
Andrew