From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH 12/23] memory: separate building the final memory map into two steps Date: Tue, 26 Jul 2011 13:43:31 +0300 Message-ID: <4E2E9A53.4060402@redhat.com> References: <1311602584-23409-1-git-send-email-avi@redhat.com> <1311602584-23409-13-git-send-email-avi@redhat.com> <4E2DC00D.5080608@codemonkey.ws> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: qemu-devel@nongnu.org, kvm@vger.kernel.org To: Anthony Liguori Return-path: In-Reply-To: <4E2DC00D.5080608@codemonkey.ws> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org Sender: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org List-Id: kvm.vger.kernel.org On 07/25/2011 10:12 PM, Anthony Liguori wrote: > On 07/25/2011 09:02 AM, Avi Kivity wrote: >> Instead of adding and deleting regions in one pass, do a delete >> pass followed by an add pass. This fixes the following case: >> >> from: >> 0x0000-0x0fff ram (a1) >> 0x1000-0x1fff mmio (a2) >> 0x2000-0x2fff ram (a3) >> >> to: >> 0x0000-0x2fff ram (b1) >> >> The single pass algorithm removed a1, added b2, then removed a2 and a2, > > You mean a2 and a3 I suppose? Yes; fixed. -- error compiling committee.c: too many arguments to function