From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:48234) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Rv9YJ-0000H2-HT for qemu-devel@nongnu.org; Wed, 08 Feb 2012 10:35:29 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Rv9YD-0001Zg-SJ for qemu-devel@nongnu.org; Wed, 08 Feb 2012 10:35:23 -0500 Received: from mx1.redhat.com ([209.132.183.28]:58436) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Rv9YD-0001Za-Ld for qemu-devel@nongnu.org; Wed, 08 Feb 2012 10:35:17 -0500 Received: from int-mx01.intmail.prod.int.phx2.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id q18FZGbO032728 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Wed, 8 Feb 2012 10:35:16 -0500 Message-ID: <4F329632.5020700@redhat.com> Date: Wed, 08 Feb 2012 17:35:14 +0200 From: Avi Kivity MIME-Version: 1.0 References: <1328714879-18906-1-git-send-email-avi@redhat.com> In-Reply-To: <1328714879-18906-1-git-send-email-avi@redhat.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 00/10] Remove AddressSpaceOps List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org On 02/08/2012 05:27 PM, Avi Kivity wrote: > This patchset makes the memory core (memory.c) talk to the backend (in exec.c) > via a MemoryListener instead of named functions. > > While the motivation for this is to simplify the memory core, it also enables > optimizing accelerators some more (by having a tcg MemoryListener to do tcg > specific core) and allows unit testing of memory.c (by adding a testing > MemoryListener and seeing what it outputs as various inputs are fed into the > core). > Something that is very visible here is that MemoryListeners switch quite a lot on the address space. This suggests that we need to allow observing a specific address space. I'll address that in a future patch. -- error compiling committee.c: too many arguments to function