From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alex Williamson Subject: Re: [PATCH 0/2] RAM cleanup Date: Fri, 11 Jun 2010 10:45:15 -0600 Message-ID: <1276274715.3100.48.camel@x201> References: <20100611141500.19378.43942.stgit@localhost.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: anthony@codemonkey.ws, kvm@vger.kernel.org, chrisw@redhat.com, Marcelo Tosatti To: qemu-devel@nongnu.org Return-path: Received: from mx1.redhat.com ([209.132.183.28]:35994 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752144Ab0FKRe7 (ORCPT ); Fri, 11 Jun 2010 13:34:59 -0400 In-Reply-To: <20100611141500.19378.43942.stgit@localhost.localdomain> Sender: kvm-owner@vger.kernel.org List-ID: On Fri, 2010-06-11 at 08:15 -0600, Alex Williamson wrote: > These are the first two patches from the RFC sent out a couple days > ago. There seemed to be concensus that these were headed in the > right direction, so I'll post them for commit while we decided how > to name RAMBlocks. > > Changes - rename 'ram' to 'ram_list' to avoid polluting the global > namespace quite so much, restore the error message and abort for > bad address in qemu_get_ram_ptr(). > > Alex > > --- > > Alex Williamson (2): > ram_blocks: Convert to a QLIST > qemu_ram_alloc: Remove duplicate code Marcelo pointed out that I forgot to rebase against qemu, where qemu_ram_map doesn't exist yet. So I'll drop the first patch, we can do that in kvm later, and I'll repost the qemu version of the second. Alex From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=50051 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1ON88C-0003Zm-4K for qemu-devel@nongnu.org; Fri, 11 Jun 2010 13:35:01 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1ON88A-0006cB-U2 for qemu-devel@nongnu.org; Fri, 11 Jun 2010 13:35:00 -0400 Received: from mx1.redhat.com ([209.132.183.28]:46318) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1ON88A-0006bs-K2 for qemu-devel@nongnu.org; Fri, 11 Jun 2010 13:34:58 -0400 From: Alex Williamson In-Reply-To: <20100611141500.19378.43942.stgit@localhost.localdomain> References: <20100611141500.19378.43942.stgit@localhost.localdomain> Content-Type: text/plain; charset="UTF-8" Date: Fri, 11 Jun 2010 10:45:15 -0600 Message-ID: <1276274715.3100.48.camel@x201> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] Re: [PATCH 0/2] RAM cleanup List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: chrisw@redhat.com, Marcelo Tosatti , kvm@vger.kernel.org On Fri, 2010-06-11 at 08:15 -0600, Alex Williamson wrote: > These are the first two patches from the RFC sent out a couple days > ago. There seemed to be concensus that these were headed in the > right direction, so I'll post them for commit while we decided how > to name RAMBlocks. > > Changes - rename 'ram' to 'ram_list' to avoid polluting the global > namespace quite so much, restore the error message and abort for > bad address in qemu_get_ram_ptr(). > > Alex > > --- > > Alex Williamson (2): > ram_blocks: Convert to a QLIST > qemu_ram_alloc: Remove duplicate code Marcelo pointed out that I forgot to rebase against qemu, where qemu_ram_map doesn't exist yet. So I'll drop the first patch, we can do that in kvm later, and I'll repost the qemu version of the second. Alex