From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:38701) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Rcebt-0001Sl-9p for qemu-devel@nongnu.org; Mon, 19 Dec 2011 09:54:42 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Rcebr-0001YB-R8 for qemu-devel@nongnu.org; Mon, 19 Dec 2011 09:54:37 -0500 Received: from mail-iy0-f173.google.com ([209.85.210.173]:57055) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Rcebr-0001Xs-O0 for qemu-devel@nongnu.org; Mon, 19 Dec 2011 09:54:35 -0500 Received: by iagj37 with SMTP id j37so9811559iag.4 for ; Mon, 19 Dec 2011 06:54:34 -0800 (PST) Message-ID: <4EEF5024.4060409@codemonkey.ws> Date: Mon, 19 Dec 2011 08:54:28 -0600 From: Anthony Liguori MIME-Version: 1.0 References: <1324304024-11220-1-git-send-email-avi@redhat.com> In-Reply-To: <1324304024-11220-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] [PATCH 00/23] Remove cpu_get_physical_page_desc() List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Avi Kivity Cc: xen-devel@lists.xensource.com, "Michael S. Tsirkin" , qemu-devel@nongnu.org, kvm@vger.kernel.org, Stefano Stabellini On 12/19/2011 08:13 AM, Avi Kivity wrote: > cpu_get_physical_page_desc() exposes the internals of the memory core to > callers; as such it prevents the refactoring planned there. This patchset > converts all callers memory API equivalents and removes the function. > > The conversion leaves a lot of potential for further cleanups; the > MemoryListener API (which replaces CPUPhysMemoryClient) guarantees matched > range_add and range_del calls, so the need to handle splitting is removed. > This is left for later. > > Please review and test, especially the vhost and Xen parts, which I only > build tested. Other than the few style comments, the whole series looks reasonable to me. Regards, Anthony Liguori > > Also available from: > > git://git.kernel.org/pub/scm/virt/kvm/qemu-kvm.git memory/page_desc > > Avi Kivity (23): > memory: introduce memory_region_find() > sysbus: add sysbus_address_space() > memory: add memory_region_is_ram() > framebuffer: drop use of cpu_get_physical_page_desc() > memory: add memory_region_is_rom() > loader: remove calls to cpu_get_physical_page_desc() > framebuffer: drop use of cpu_physical_sync_dirty_bitmap() > memory: replace cpu_physical_sync_dirty_bitmap() with a memory API > memory: add API for observing updates to the physical memory map > memory: add memory_region_is_logging() > kvm: switch kvm slots to use host virtual address instead of > ram_addr_t > fixup: listener fixes > kvm: convert to MemoryListener API > vhost: convert to MemoryListener API > xen, vga: add API for registering the framebuffer > memory: temporarily add memory_region_get_ram_addr() > xen: convert to MemoryListener API > memory: remove CPUPhysMemoryClient > kvm: avoid cpu_get_physical_page_desc() > vhost: avoid cpu_get_physical_page_desc() > virtio-balloon: avoid cpu_get_physical_page_desc() > sparc: avoid cpu_get_physical_page_desc() > Remove cpu_get_physical_page_desc() > > arch_init.c | 6 +- > cpu-all.h | 9 -- > cpu-common.h | 24 ------ > exec.c | 175 +--------------------------------------- > hw/framebuffer.c | 32 +++---- > hw/framebuffer.h | 3 + > hw/loader.c | 9 +- > hw/milkymist-vgafb.c | 2 +- > hw/omap_lcdc.c | 4 +- > hw/pl110.c | 2 +- > hw/pxa2xx_lcd.c | 10 ++- > hw/sysbus.c | 5 + > hw/sysbus.h | 1 + > hw/vga.c | 2 + > hw/vhost.c | 167 ++++++++++++++++++++++++++++++--------- > hw/vhost.h | 5 +- > hw/virtio-balloon.c | 14 +++- > hw/xen.h | 3 + > kvm-all.c | 151 +++++++++++++++++++++-------------- > kvm.h | 4 +- > memory.c | 193 ++++++++++++++++++++++++++++++++++++++++++--- > memory.h | 127 +++++++++++++++++++++++++++++ > target-i386/kvm.c | 7 +- > target-sparc/mmu_helper.c | 5 +- > trace-events | 2 +- > xen-all.c | 143 ++++++++++++++++++++------------- > xen-stub.c | 4 + > 27 files changed, 695 insertions(+), 414 deletions(-) >