All of lore.kernel.org
 help / color / mirror / Atom feed
From: Avi Kivity <avi@redhat.com>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [RFC128 3/2] Adjust system and pci address spaces to full 64-bit
Date: Mon, 17 Oct 2011 12:38:04 +0200	[thread overview]
Message-ID: <4E9C058C.6060307@redhat.com> (raw)
In-Reply-To: <20111017053348.GC30114@truffala.fritz.box>

On 10/17/2011 07:33 AM, David Gibson wrote:
> On Sun, Oct 16, 2011 at 05:29:07PM +0200, Avi Kivity wrote:
> > Now that the memory API supports full 64-bit buses, adjust the relevant
> > callers to take advantage of it.
>
> Note that this doesn't, strictly speaking doesn't give you full 64-bit
> coverage, since the range covered is 2^64-1 bytes rather than 2^64
> bytes.  Cases where that will matter would be very rare, of course.
>

An undocumented and indeed unmentioned feature of patch 2 is that
UINT64_MAX sizes are expanded to UINT64_MAX+1.  I did that to avoid
introducing memory_region_init_128() (or perhaps
memory_region_init_2_64() that doesn't take a size argument).  That
removes the ability to create container regions that span exactly
UINT64_MAX bytes.  It is strange in a patchset that tries to makes
things more regular, I admit.

-- 
error compiling committee.c: too many arguments to function

      reply	other threads:[~2011-10-17 10:38 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-16 11:24 [Qemu-devel] [RFC128 0/2] 128-bit arithmetic for the memory API Avi Kivity
2011-10-16 11:24 ` [Qemu-devel] [RFC128 1/2] Add support for 128-bit arithmetic Avi Kivity
2011-10-16 11:24 ` [Qemu-devel] [RFC128 2/2] memory: use 128-bit integers for sizes and intermediates Avi Kivity
2011-10-16 15:29 ` [Qemu-devel] [RFC128 3/2] Adjust system and pci address spaces to full 64-bit Avi Kivity
2011-10-17  5:33   ` David Gibson
2011-10-17 10:38     ` Avi Kivity [this message]

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=4E9C058C.6060307@redhat.com \
    --to=avi@redhat.com \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.