From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Chen, Tiejun" Subject: Re: [v8][PATCH 08/17] hvmloader/mmio: reconcile guest mmio with reserved device memory Date: Mon, 08 Dec 2014 17:10:30 +0800 Message-ID: <54856B06.1020308@intel.com> References: <1417425875-9634-1-git-send-email-tiejun.chen@intel.com> <1417425875-9634-9-git-send-email-tiejun.chen@intel.com> <54809437020000780004CD43@mail.emea.novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <54809437020000780004CD43@mail.emea.novell.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Jan Beulich Cc: kevin.tian@intel.com, wei.liu2@citrix.com, ian.campbell@citrix.com, stefano.stabellini@eu.citrix.com, tim@xen.org, ian.jackson@eu.citrix.com, xen-devel@lists.xen.org, yang.z.zhang@intel.com List-Id: xen-devel@lists.xenproject.org On 2014/12/5 0:04, Jan Beulich wrote: >>>> On 01.12.14 at 10:24, wrote: >> We need to make sure all mmio allocation don't overlap >> any rdm, reserved device memory. Here we just skip >> all reserved device memory range in mmio space. > > I think someone else already suggested that this and patch 9 should Who? I just see Kevin commented something for this patch. > be swapped, and the BAR allocation be changed to use the E820 > map as input. That may end up being a bigger change, but will yield > ultimately better (and namely better maintainable) code. Maybe you're saying some comments in patch9? I need to take a look. Thanks Tiejun