From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:58588) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aagqM-0002c4-Po for qemu-devel@nongnu.org; Tue, 01 Mar 2016 04:43:54 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aagqL-0006qM-Te for qemu-devel@nongnu.org; Tue, 01 Mar 2016 04:43:50 -0500 Received: from mail-wm0-x244.google.com ([2a00:1450:400c:c09::244]:36494) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aagqL-0006q8-Ik for qemu-devel@nongnu.org; Tue, 01 Mar 2016 04:43:49 -0500 Received: by mail-wm0-x244.google.com with SMTP id l68so3115853wml.3 for ; Tue, 01 Mar 2016 01:43:49 -0800 (PST) Sender: Paolo Bonzini References: <1456813104-25902-1-git-send-email-famz@redhat.com> From: Paolo Bonzini Message-ID: <56D56451.8060905@redhat.com> Date: Tue, 1 Mar 2016 10:43:45 +0100 MIME-Version: 1.0 In-Reply-To: <1456813104-25902-1-git-send-email-famz@redhat.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2 0/7] memory: Clean up MemoryRegion.ram_addr and optimize address_space_translate List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Fam Zheng , qemu-devel@nongnu.org Cc: arei.gonglei@huawei.com, peterx@redhat.com On 01/03/2016 07:18, Fam Zheng wrote: > v2: In the optimization patch, factor out section_covers_addr() and use it. > [Paolo, Peter] > Check "ram_block == NULL" in patch 3. [Gonglei] > Add Gonglei's rev-by in patches 1, 2, 4 and 5. > > The first four patches drop ram_addr from MemoryRegion on top of Gonglei's > optimization. > > The next patch simplifies qemu_ram_free a bit by passing the RAMBlock pointer. > > The last patch speeds up address_space_translate with a cache pointer inside > the AddressSpaceDispatch. > > Fam Zheng (7): > exec: Return RAMBlock pointer from allocating functions > memory: Move assignment to ram_block to memory_region_init_* > memory: Implement memory_region_get_ram_addr with mr->ram_block > memory: Drop MemoryRegion.ram_addr > exec: Pass RAMBlock pointer to qemu_ram_free > exec: Factor out section_covers_addr > exec: Introduce AddressSpaceDispatch.mru_section > > cputlb.c | 4 +- > exec.c | 106 +++++++++++++++++++++++++----------------------- > hw/misc/ivshmem.c | 9 ++-- > include/exec/memory.h | 9 +--- > include/exec/ram_addr.h | 24 +++++------ > kvm-all.c | 3 +- > memory.c | 56 ++++++++++++++----------- > 7 files changed, 111 insertions(+), 100 deletions(-) > Thanks, queued! Paolo