From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [RFC PATCH v2 06/16] libxl: Load guest ACPI table from file Date: Wed, 4 Nov 2015 10:57:35 +0000 Message-ID: <1446634655.6461.48.camel@citrix.com> References: <1445875397-2846-1-git-send-email-anthony.perard@citrix.com> <1445875397-2846-7-git-send-email-anthony.perard@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1445875397-2846-7-git-send-email-anthony.perard@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Anthony PERARD , xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On Mon, 2015-10-26 at 16:03 +0000, Anthony PERARD wrote: > The path to the ACPI tables blob can be override by xl's option "overridden" > acpi_table_override or by acpi_tables_filename in the domain_build_info > struct for libxl user. This needs the same libxl.h #define and xl.cfg update I mentioned before. The code, docs and commit message all need further consideration of the interactions with the existing acpi_firmware option which exists in libxl and is exposed in xl. It allows you to specify some extra tables which are merged (by hvmloader) into the base ones. The naming is a bit unfortunate but we are now stuck with those semantics for the existing option I think. If we can distinguish partial from full tables in the tools reusing the name and doing so might be the best approach. If we can't tell the difference then the new option needs some suitable name such that it is clear it is the full or base table or something. Ian.