From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:51792) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RLIlA-0006bq-M8 for qemu-devel@nongnu.org; Tue, 01 Nov 2011 14:08:29 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1RLIl9-0003ZE-FA for qemu-devel@nongnu.org; Tue, 01 Nov 2011 14:08:28 -0400 Received: from mail-qy0-f180.google.com ([209.85.216.180]:45242) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1RLIl9-0003Z9-Cy for qemu-devel@nongnu.org; Tue, 01 Nov 2011 14:08:27 -0400 Received: by qyl38 with SMTP id 38so1037072qyl.4 for ; Tue, 01 Nov 2011 11:08:27 -0700 (PDT) Message-ID: <4EB03597.2040307@codemonkey.ws> Date: Tue, 01 Nov 2011 13:08:23 -0500 From: Anthony Liguori MIME-Version: 1.0 References: <1319983368-21801-1-git-send-email-avi@redhat.com> In-Reply-To: <1319983368-21801-1-git-send-email-avi@redhat.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PULL 0/3] 128-bit support for the memory API List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Avi Kivity Cc: Blue Swirl , qemu-devel@nongnu.org, David Gibson On 10/30/2011 09:02 AM, Avi Kivity wrote: > This somewhat controversial patchset converts internal arithmetic in the > memory API to 128 bits. > > It has been argued that with careful coding we can make 64-bit work as > well. I don't think this is true in general - a memory router can adjust > addresses either forwards or backwards, and some buses (PCIe) need the > full 64-bit space - though it's probably the case for all the configurations > we support today. Regardless, the need for careful coding means subtle bugs, > which I don't want in a core API that is driven by guest supplied values. > > Avi Kivity (3): > Add support for 128-bit arithmetic > memory: use 128-bit integers for sizes and intermediates > Adjust system and pci address spaces to full 64-bit I now notice that this is not a PULL request... Did you mess up the subject or the pull request? Regards, Anthony Liguori > > exec.c | 2 +- > hw/pc_piix.c | 2 +- > hw/pci_bridge.c | 2 +- > int128.h | 116 ++++++++++++++++++++++++++++++++ > memory.c | 196 ++++++++++++++++++++++++++++++++---------------------- > memory.h | 3 +- > 6 files changed, 237 insertions(+), 84 deletions(-) > create mode 100644 int128.h >