[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v6 16/16] libxl/arm: Add the size of ACPI tables to maxmem



Hi,

On 28/09/2016 06:17, Wei Liu wrote:
On Wed, Sep 28, 2016 at 06:11:53AM -0700, Shannon Zhao wrote:
  libxl__get_acpi_size(gc, info, gic_version /* not build_state anymore */)

  /* also fix up libxl__estimate_madt_size */


  /* this is the function called when constructing the domain etc, only
   * in libxl_arm.c */
  static acpi_extra_memory(gc, build_info, gic_version)
  {
       libxl__get_acpi_size...
  }

  libxl__arch_extra_memory(gc, d_config)
  {
       gic_version = d_config->..gic_version;

If user doesn't specify gic_version in xl config, the
d_config->b_info.arch_arm.gic_version will be LIBXL_GIC_VERSION_DEFAULT, so
we can't know the exact gic_version which will be constructed later.


First, can you confirm if it really can't be retrieved?

libxl__arch_domain_save_config updates that field after the domain is
constructed, so you might have a determined gic version to hand.

The target memory of the domain is created after the domain has been created. So we should have the correct GIC version is hand.

Regards,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.