From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: [PATCH v4 14/24] xen/arm: initialize grant_table on ARM Date: Fri, 14 Sep 2012 09:10:33 -0400 Message-ID: <20120914131033.GM25249@phenom.dumpdata.com> References: <1347621207-11294-14-git-send-email-stefano.stabellini@eu.citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <1347621207-11294-14-git-send-email-stefano.stabellini-mvvWK6WmYclDPfheJLI6IQ@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linaro-dev-bounces-cunTk1MwBs8s++Sfvej+rw@public.gmane.org Errors-To: linaro-dev-bounces-cunTk1MwBs8s++Sfvej+rw@public.gmane.org To: Stefano Stabellini Cc: xen-devel-GuqFBffKawuULHF6PoxzQEEOCMrvLtNR@public.gmane.org, linaro-dev-cunTk1MwBs8s++Sfvej+rw@public.gmane.org, Ian.Campbell-Sxgqhf6Nn4DQT0dZR+AlfA@public.gmane.org, arnd-r2nGTMty4D4@public.gmane.org, catalin.marinas-5wv7dgnIgG8@public.gmane.org, tim-LM2mM/qkH7s@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org List-Id: xen-devel@lists.xenproject.org On Fri, Sep 14, 2012 at 12:13:16PM +0100, Stefano Stabellini wrote: > Initialize the grant table mapping at the address specified at index 0 > in the DT under the /xen node. > After the grant table is initialized, call xenbus_probe (if not dom0). So we don't really care about the grant's size then? The DT xen.txt talks about it.. > > Changes in v2: > > - introduce GRANT_TABLE_PHYSADDR; > - remove unneeded initialization of boot_max_nr_grant_frames. > > Signed-off-by: Stefano Stabellini > --- > arch/arm/xen/enlighten.c | 14 ++++++++++++++ > 1 files changed, 14 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/xen/enlighten.c b/arch/arm/xen/enlighten.c > index c2a47a7..036a4d8 100644 > --- a/arch/arm/xen/enlighten.c > +++ b/arch/arm/xen/enlighten.c > @@ -1,8 +1,12 @@ > #include > +#include > +#include > #include > #include > +#include > #include > #include > +#include > #include > #include > #include > @@ -42,6 +46,7 @@ EXPORT_SYMBOL_GPL(xen_remap_domain_mfn_range); > * see Documentation/devicetree/bindings/arm/xen.txt for the > * documentation of the Xen Device Tree format. > */ > +#define GRANT_TABLE_PHYSADDR 0 > static int __init xen_guest_init(void) > { > struct xen_add_to_physmap xatp; > @@ -51,6 +56,7 @@ static int __init xen_guest_init(void) > const char *s = NULL; > const char *version = NULL; > const char *xen_prefix = "xen,xen-"; > + struct resource res; > > node = of_find_compatible_node(NULL, NULL, "xen,xen"); > if (!node) { > @@ -65,6 +71,9 @@ static int __init xen_guest_init(void) > pr_debug("Xen version not found\n"); > return 0; > } > + if (of_address_to_resource(node, GRANT_TABLE_PHYSADDR, &res)) > + return 0; > + xen_hvm_resume_frames = res.start >> PAGE_SHIFT; > xen_domain_type = XEN_HVM_DOMAIN; > > xen_setup_features(); > @@ -98,6 +107,11 @@ static int __init xen_guest_init(void) > * is required to use VCPUOP_register_vcpu_info to place vcpu info > * for secondary CPUs as they are brought up. */ > per_cpu(xen_vcpu, 0) = &HYPERVISOR_shared_info->vcpu_info[0]; > + > + gnttab_init(); > + if (!xen_initial_domain()) > + xenbus_probe(NULL); > + > return 0; > } > core_initcall(xen_guest_init); > -- > 1.7.2.5