lkml.org 
[lkml]   [2012]   [Aug]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 14/24] xen/arm: initialize grant_table on ARM
    On Wed, 1 Aug 2012, Konrad Rzeszutek Wilk wrote:
    > On Thu, Jul 26, 2012 at 04:33:56PM +0100, Stefano Stabellini wrote:
    > > Initialize the grant table mapping at the address specified at index 0
    > > in the DT under the /xen node.
    >
    > Is it always index 0? If so, should it have a #define for the
    > other index values?

    There are no other values at the moment but I'll add an #define.


    > > After the grant table is initialized, call xenbus_probe (if not dom0).
    > >
    > > Signed-off-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
    > > ---
    > > arch/arm/xen/enlighten.c | 13 +++++++++++++
    > > drivers/xen/grant-table.c | 2 +-
    > > 2 files changed, 14 insertions(+), 1 deletions(-)
    > >
    > > diff --git a/arch/arm/xen/enlighten.c b/arch/arm/xen/enlighten.c
    > > index 2e013cf..854af1e 100644
    > > --- a/arch/arm/xen/enlighten.c
    > > +++ b/arch/arm/xen/enlighten.c
    > > @@ -1,8 +1,12 @@
    > > #include <xen/xen.h>
    > > #include <xen/interface/xen.h>
    > > #include <xen/interface/memory.h>
    > > +#include <xen/interface/hvm/params.h>
    > > #include <xen/platform_pci.h>
    > > #include <xen/features.h>
    > > +#include <xen/grant_table.h>
    > > +#include <xen/hvm.h>
    > > +#include <xen/xenbus.h>
    > > #include <asm/xen/hypervisor.h>
    > > #include <asm/xen/hypercall.h>
    > > #include <linux/module.h>
    > > @@ -51,12 +55,16 @@ int __init xen_guest_init(void)
    > > struct xen_add_to_physmap xatp;
    > > static struct shared_info *shared_info_page = 0;
    > > struct device_node *node;
    > > + struct resource res;
    > >
    > > node = of_find_compatible_node(NULL, NULL, "arm,xen");
    > > if (!node) {
    > > pr_info("No Xen support\n");
    > > return 0;
    > > }
    > > + if (of_address_to_resource(node, 0, &res))
    > > + return -EINVAL;
    > > + xen_hvm_resume_frames = res.start >> PAGE_SHIFT;
    > > xen_domain_type = XEN_HVM_DOMAIN;
    > >
    > > xen_setup_features();
    > > @@ -97,6 +105,11 @@ int __init xen_guest_init(void)
    > > per_cpu(xen_vcpu, cpu) =
    > > &HYPERVISOR_shared_info->vcpu_info[cpu];
    > > }
    > > +
    > > + gnttab_init();
    > > + if (!xen_initial_domain())
    > > + xenbus_probe(NULL);
    > > +
    > > return 0;
    > > }
    > > EXPORT_SYMBOL_GPL(xen_guest_init);
    > > diff --git a/drivers/xen/grant-table.c b/drivers/xen/grant-table.c
    > > index 1d0d95e..fd2137a 100644
    > > --- a/drivers/xen/grant-table.c
    > > +++ b/drivers/xen/grant-table.c
    > > @@ -62,7 +62,7 @@
    > >
    > > static grant_ref_t **gnttab_list;
    > > static unsigned int nr_grant_frames;
    > > -static unsigned int boot_max_nr_grant_frames;
    > > +static unsigned int boot_max_nr_grant_frames = 1;
    >
    > Is this going to impact x86 version?

    It is not needed so I'll remove this change


    \
     
     \ /
      Last update: 2012-08-01 20:01    [W:4.411 / U:0.364 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site