From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jes Sorensen Date: Wed, 29 Apr 2009 15:07:23 +0000 Subject: Re: [PATCH 03/04] qemu-kvm: Remove the dependency for phys_ram_base Message-Id: <49F86D2B.8040308@sgi.com> List-Id: References: <706158FABBBA044BAD4FE898A02E4BC236A2BC04@pdsmsx503.ccr.corp.intel.com> In-Reply-To: <706158FABBBA044BAD4FE898A02E4BC236A2BC04@pdsmsx503.ccr.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: kvm-ia64@vger.kernel.org Zhang, Xiantao wrote: > Jes Sorensen wrote: >> The main difference is that my patch cleans up the interfaces and >> calls to the various functions, and removes a bunch of global >> variables as well. > > I still can't see the difference with the patch in Avi's tree except nvram stuff. And I believe the global variable you mentioned should be only used for nvram. So I propose an incremental patch for that. :) Let me try and take a look - last I pulled from Avi, those patches didn't show up in git. Been trying to get the mapping and alias stuff done, so probably won't get to this until tomorrow. Cheers, jes