From: "Jan Beulich" <JBeulich@suse.com>
To: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Cc: wei.liu2@citrix.com, andrew.cooper3@citrix.com,
ian.jackson@eu.citrix.com, xen-devel@lists.xen.org,
julien.grall@arm.com, zhaoshenglong@huawei.com,
roger.pau@citrix.com
Subject: Re: [PATCH v4 20/21] libxl/acpi: Build ACPI tables for HVMlite guests
Date: Wed, 21 Sep 2016 09:16:51 -0600 [thread overview]
Message-ID: <57E2C08302000078001112E5@prv-mh.provo.novell.com> (raw)
In-Reply-To: <f0607931-bcb2-e224-18c7-650907ec6bf6@oracle.com>
>>> On 21.09.16 at 17:09, <boris.ostrovsky@oracle.com> wrote:
> On 09/21/2016 07:33 AM, Jan Beulich wrote:
>>>>> On 20.09.16 at 02:19, <boris.ostrovsky@oracle.com> wrote:
>>> --- a/tools/libacpi/build.c
>>> +++ b/tools/libacpi/build.c
>>> @@ -20,6 +20,7 @@
>>> #include "ssdt_s4.h"
>>> #include "ssdt_tpm.h"
>>> #include "ssdt_pm.h"
>>> +#include <xen/hvm/hvm_info_table.h>
>> ... I don't really see why this becomes necessary here. Please
>> clarify.
>
> xen/hvm/hvm_info_table.h in included by hvmloader/util.h so we haven't
> needed this include until now.
But you're not removing any inclusion here. Does that addition
perhaps belong elsewhere?
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
next prev parent reply other threads:[~2016-09-21 15:16 UTC|newest]
Thread overview: 59+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-20 0:19 [PATCH v4 00/21] Make ACPI builder available to components other than hvmloader Boris Ostrovsky
2016-09-20 0:19 ` [PATCH v4 01/21] acpi: Extract acpi info description into a separate ASL file Boris Ostrovsky
2016-09-21 10:29 ` Jan Beulich
2016-09-21 13:21 ` Boris Ostrovsky
2016-09-20 0:19 ` [PATCH v4 02/21] acpi: Prevent GPL-only code from seeping into non-GPL binaries Boris Ostrovsky
2016-09-20 10:14 ` Ian Jackson
2016-09-20 10:41 ` Lars Kurth
2016-09-20 14:07 ` Boris Ostrovsky
2016-09-20 14:19 ` Ian Jackson
2016-09-20 14:58 ` Boris Ostrovsky
2016-09-20 10:32 ` Lars Kurth
2016-09-21 10:39 ` Jan Beulich
2016-09-21 13:34 ` Boris Ostrovsky
2016-09-21 13:47 ` Jan Beulich
2016-09-20 0:19 ` [PATCH v4 03/21] acpi: Re-license ACPI builder files from GPLv2 to LGPLv2.1 Boris Ostrovsky
2016-09-20 10:17 ` Lars Kurth
2016-09-20 0:19 ` [PATCH v4 04/21] acpi/hvmloader: Collect processor and NUMA info in hvmloader Boris Ostrovsky
2016-09-21 10:42 ` Jan Beulich
2016-09-20 0:19 ` [PATCH v4 05/21] acpi/hvmloader: Set TIS header address " Boris Ostrovsky
2016-09-20 0:19 ` [PATCH v4 06/21] acpi/hvmloader: Make providing IOAPIC in MADT optional Boris Ostrovsky
2016-09-20 0:19 ` [PATCH v4 07/21] acpi/hvmloader: Build WAET optionally Boris Ostrovsky
2016-09-20 0:19 ` [PATCH v4 08/21] acpi/hvmloader: Replace mem_alloc() and virt_to_phys() with memory ops Boris Ostrovsky
2016-09-20 0:19 ` [PATCH v4 09/21] acpi/hvmloader: Translate all addresses when assigning addresses in ACPI tables Boris Ostrovsky
2016-09-20 0:19 ` [PATCH v4 10/21] acpi/hvmloader: Link ACPI object files directly Boris Ostrovsky
2016-09-21 10:52 ` Jan Beulich
2016-09-21 11:29 ` Ian Jackson
2016-09-21 11:36 ` Jan Beulich
2016-09-21 11:38 ` Ian Jackson
2016-09-21 11:40 ` Jan Beulich
2016-09-21 13:45 ` Boris Ostrovsky
2016-09-21 15:05 ` Ian Jackson
2016-09-21 15:21 ` Boris Ostrovsky
2016-09-21 15:25 ` Ian Jackson
2016-09-21 21:03 ` Boris Ostrovsky
2016-09-20 0:19 ` [PATCH v4 11/21] acpi/hvmloader: Include file/paths adjustments Boris Ostrovsky
2016-09-21 11:27 ` Jan Beulich
2016-09-20 0:19 ` [PATCH v4 12/21] acpi: Move ACPI code to tools/libacpi Boris Ostrovsky
2016-09-20 0:19 ` [PATCH v4 13/21] x86: Allow LAPIC-only emulation_flags for HVM guests Boris Ostrovsky
2016-09-20 0:19 ` [PATCH v4 14/21] libacpi: Build DSDT for PVH guests Boris Ostrovsky
2016-09-21 11:27 ` Jan Beulich
2016-09-20 0:19 ` [PATCH v4 15/21] acpi: Makefile should better tolerate interrupts Boris Ostrovsky
2016-09-20 0:19 ` [PATCH v4 16/21] libxc/libxl: Allow multiple ACPI modules Boris Ostrovsky
2016-09-20 0:19 ` [PATCH v4 17/21] libxl/acpi: Add ACPI e820 entry Boris Ostrovsky
2016-09-20 0:19 ` [PATCH v4 18/21] libxl/pvhv2: Include APIC page in MMIO hole for PVHv2 guests Boris Ostrovsky
2016-09-20 0:19 ` [PATCH v4 19/21] ilibxl: Initialize domain build info before calling libxl__domain_make Boris Ostrovsky
2016-09-21 11:29 ` Jan Beulich
2016-09-20 0:19 ` [PATCH v4 20/21] libxl/acpi: Build ACPI tables for HVMlite guests Boris Ostrovsky
2016-09-21 11:33 ` Jan Beulich
2016-09-21 15:09 ` Boris Ostrovsky
2016-09-21 15:16 ` Jan Beulich [this message]
2016-09-21 15:34 ` Boris Ostrovsky
2016-09-21 16:02 ` Jan Beulich
2016-09-21 16:38 ` Boris Ostrovsky
2016-09-22 8:39 ` Jan Beulich
2016-09-22 10:53 ` Wei Liu
2016-09-22 15:57 ` Boris Ostrovsky
2016-09-22 15:57 ` Wei Liu
2016-09-20 0:19 ` [PATCH v4 21/21] libxc/xc_dom_core: Copy ACPI tables to guest space Boris Ostrovsky
2016-09-22 10:53 ` Wei Liu
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=57E2C08302000078001112E5@prv-mh.provo.novell.com \
--to=jbeulich@suse.com \
--cc=andrew.cooper3@citrix.com \
--cc=boris.ostrovsky@oracle.com \
--cc=ian.jackson@eu.citrix.com \
--cc=julien.grall@arm.com \
--cc=roger.pau@citrix.com \
--cc=wei.liu2@citrix.com \
--cc=xen-devel@lists.xen.org \
--cc=zhaoshenglong@huawei.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.