qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Aurelio Remonda <aurelio.remonda@tallertechnologies.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: QEMU Developers <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] Memory on stellaris board
Date: Fri, 11 Mar 2016 11:12:18 -0300	[thread overview]
Message-ID: <CANLsssso+rwdLadBCz6YVpDUUPO2L56vfnpsNxHMJMi59ySQjw@mail.gmail.com> (raw)
In-Reply-To: <CAFEAcA8m0OBxu80jjCdT0Tt_qTE=xBYKSYBT7bTBFPitnBH=pA@mail.gmail.com>

I don't quite understand what you mean with non-round-number, are you suggesting
we only accept for example:
64K-128K-256K-512K-1024k(or 1M)-2048K(or 2M)
4096K(or 4M)-8192K(or 8M)-16384K(or 16M)
If that's the case we will never have, for example, the exact default
dc0 value (0x00ff007f)
because you need to divide the given size by 1024 reverting what the
prefix (K or M) have
done and then multiply this value by 1000.

So in the case of the default value 0xff00=65280 will become
0xfaf9=64249 and goes
directly to dc0. Then the sram_size would be just ram_size and not a
value calculated
based on dc0.

Again the problem of this es that you will never get the manual
default dc0 value even if
you run the program without the -m flag.

Thanks


On Wed, Mar 9, 2016 at 10:54 PM, Peter Maydell <peter.maydell@linaro.org> wrote:
> On 10 March 2016 at 07:23, Aurelio Remonda
> <aurelio.remonda@tallertechnologies.com> wrote:
>>
>> El 9 mar. 2016 8:25 PM, "Peter Maydell" <peter.maydell@linaro.org> escribió:
>>> You should reject unworkable ram sizes rather than silently
>>> changing what the user asked for; this matches our preferred
>>> approach where the user asks for more RAM than the board
>>> can support.
>>
>> With unworkable you mean RAM values over dc0 máximum? I make sure that it
>> does not exceed 0xffff before i replace the dc0 value so if it goes over 16M
>> the program report the problem (via error_report()) and exit.
>
> Also if the user asks for a non-round-number ram size:
> don't round ram_size up or down, just fail if the user asked
> for something that's not representable in a dc0 value.
>
> thanks
> -- PMM



-- 
Aurelio Remonda

Taller Technologies Argentina

Software Engineer

San Lorenzo 47, 3rd Floor, Office 5
Córdoba, Argentina
Phone: +54-351-4217888 / 4218211

  reply	other threads:[~2016-03-11 14:12 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-03 13:00 [Qemu-devel] Memory on stellaris board Aurelio Remonda
2016-02-03 13:34 ` Peter Maydell
2016-02-03 13:45   ` Aurelio Remonda
2016-02-05 16:09     ` Aurelio Remonda
2016-02-05 16:23       ` Peter Maydell
2016-02-05 16:55         ` Aurelio Remonda
2016-02-05 17:00           ` Peter Maydell
2016-02-11 12:46             ` Aurelio Remonda
2016-02-11 13:01               ` Peter Maydell
2016-03-07 19:58                 ` Aurelio Remonda
2016-03-07 23:37                   ` Peter Maydell
2016-03-09 20:56                     ` Aurelio Remonda
2016-03-09 23:24                       ` Peter Maydell
2016-03-10  0:23                         ` Aurelio Remonda
2016-03-10  1:54                           ` Peter Maydell
2016-03-11 14:12                             ` Aurelio Remonda [this message]
2016-03-11 22:31                               ` Peter Maydell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CANLsssso+rwdLadBCz6YVpDUUPO2L56vfnpsNxHMJMi59ySQjw@mail.gmail.com \
    --to=aurelio.remonda@tallertechnologies.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).