Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754698Ab1BGTSy (ORCPT ); Mon, 7 Feb 2011 14:18:54 -0500 Received: from mail-qy0-f174.google.com ([209.85.216.174]:63837 "EHLO mail-qy0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754663Ab1BGTSx (ORCPT ); Mon, 7 Feb 2011 14:18:53 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type; b=UJ3cUAMdD7SYmLWnt/fzhqpCFlCimUfcXCmpEDZk58c2hKJmEl+PAQkoRKdcmsaxut kzbIiWeXJrxfs211ealyBv/TbBTeO7sV1XFMueqpqbK0DV2xWZVmhmhIbtDkyVCvlnNU RpnpNoKyhzBKL65WfOWiUrezUv2RIOgmzn9FY= MIME-Version: 1.0 In-Reply-To: <4D504161.2060900@kernel.org> References: <4D4A3782.3050702@zytor.com> <4D4ADFAD.7060507@zytor.com> <4D4CA568.70907@goop.org> <4D4E4E0D.2080806@zytor.com> <4D4EF553.6000000@kernel.org> <4D50343E.1020906@kernel.org> <4D504161.2060900@kernel.org> Date: Mon, 7 Feb 2011 11:18:51 -0800 X-Google-Sender-Auth: FDqB8buwfVjhdP9x9yCTU-RkZkY Message-ID: Subject: Re: [PATCH] x86/mm/init: respect memblock reserved regions when destroying mappings From: Yinghai Lu To: Stefano Stabellini Cc: "H. Peter Anvin" , Jeremy Fitzhardinge , "linux-kernel@vger.kernel.org" , "tglx@linutronix.de" , "x86@kernel.org" , Konrad Rzeszutek Wilk , Jan Beulich Content-Type: text/plain; charset=ISO-8859-1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1976 Lines: 54 On Mon, Feb 7, 2011 at 11:00 AM, Yinghai Lu wrote: > On 02/07/2011 10:58 AM, Stefano Stabellini wrote: >> On Mon, 7 Feb 2011, Yinghai Lu wrote: >>> On 02/07/2011 08:50 AM, Stefano Stabellini wrote: >>>> On Sun, 6 Feb 2011, Yinghai Lu wrote: >>>>> On 02/05/2011 11:30 PM, H. Peter Anvin wrote: >>>>>> On 02/05/2011 11:02 PM, Yinghai Lu wrote: >>>>>>> why not just move calling cleanup_highmap down? >>>>>>> >>>>>>> something like attached patch. >>>>>> >>>>>> This patch looks very clean and looks on the surface of it like it is >>>>>> removing some ugly ad hoc code, but (as always) it needs a description >>>>>> about the problem it solves and why it is correct. >>>>> >>>>> Sure. >>>>> >>>>> >>>>> Jeremy and xen guys, can you please check if it works well with xen ? >>>>> >>>> >>>> Actually this patch makes things worse on xen, because before >>>> cleanup_highmap() wasn't called at all on xen (on purpose) and now it >>>> is, fully destroying all the mappings we have at _end. >>>> >>>> Can we add a check on memblock reserved regions in cleanup_highmap()? >>>> Otherwise could we avoid calling cleanup_highmap() at all on xen? >>> >>> why DO xen need over-mapped kernel initial mapping? >>> >>> what is in that range after _end to 512M? >> >> The mfn list that is the list of machine frame numbers assigned to this >> domain; it is used across the kernel to convert pfns into mfns. >> It passed to us at that address by the domain builder. > > is it possible for you to pass physical address, and then map it in kernel? or wonder if you can use really kernel mapping for your mfn list accessing instead of initial mapping accessing? something like: mfn_list = __va(__pa(initial_mfn_...); Yinghai -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/