From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH v3] minios: don't rely on specific page table allocation scheme Date: Mon, 23 Nov 2015 09:55:53 +0000 Message-ID: <1448272553.4973.17.camel@citrix.com> References: <1448044362-16104-1-git-send-email-jgross@suse.com> <20151120183929.GD3151@var.home> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20151120183929.GD3151@var.home> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Samuel Thibault , Juergen Gross Cc: minios-devel@lists.xenproject.org, wei.liu2@citrix.com, xen-devel@lists.xen.org, ian.jackson@eu.citrix.com, stefano.stabellini@eu.citrix.com List-Id: xen-devel@lists.xenproject.org On Fri, 2015-11-20 at 19:39 +0100, Samuel Thibault wrote: > Juergen Gross, on Fri 20 Nov 2015 19:32:42 +0100, wrote: This patch conflicted with 1855e8b2e884 "mini-os: enable compiler check for printk format types" which was committed in June. I have (hopefully correctly) resolved the conflict by simply removing the entirety of need_pt_frame(), but in the future please base your patches on something up to date. > > Today mini-os is making assumptions how the page tables it is started > > with are being allocated. Especially it is using the number of page > > table frames to calculate which is the first unmapped pfn. > > > > Instead of relying on page table number assumptions just look into the > > page tables to find the first pfn not already mapped. > > > > Signed-off-by: Juergen Gross > > Acked-by: Samuel Thibault Applied, along with a corresponding update to MINIOS_UPSTREAM_REVISION in xen.git:Config.mk. Ian.