Re: [Xen-devel] [PATCH 2/2] libacpi: Be specific about which DSDT files to build

2016-11-15 Thread Lars Kurth
On 15/11/2016 13:45, "Andrew Cooper" wrote: >On 15/11/16 13:35, Boris Ostrovsky wrote: >> On 11/15/2016 01:17 AM, Wei Liu wrote: >>> On Mon, Nov 14, 2016 at 11:52:27PM -0500, Boris Ostrovsky wrote: There is no reason to build, for example, dsdt_pvh.asl for hvmloader. We pass which

Re: [Xen-devel] [PATCH 2/2] libacpi: Be specific about which DSDT files to build

2016-11-15 Thread Andrew Cooper
On 15/11/16 13:35, Boris Ostrovsky wrote: > On 11/15/2016 01:17 AM, Wei Liu wrote: >> On Mon, Nov 14, 2016 at 11:52:27PM -0500, Boris Ostrovsky wrote: >>> There is no reason to build, for example, dsdt_pvh.asl for hvmloader. We >>> pass which DSDTs to build via DSDT_FILES parameter. >>> >>> If DSDT

Re: [Xen-devel] [PATCH 2/2] libacpi: Be specific about which DSDT files to build

2016-11-15 Thread Boris Ostrovsky
On 11/15/2016 01:17 AM, Wei Liu wrote: > On Mon, Nov 14, 2016 at 11:52:27PM -0500, Boris Ostrovsky wrote: >> There is no reason to build, for example, dsdt_pvh.asl for hvmloader. We >> pass which DSDTs to build via DSDT_FILES parameter. >> >> If DSDT_FILES is empty all DSDTs for a particular archit

Re: [Xen-devel] [PATCH 2/2] libacpi: Be specific about which DSDT files to build

2016-11-15 Thread Jan Beulich
> @@ -119,7 +119,7 @@ endif > clean: subdirs-clean > rm -f roms.inc roms.inc.new acpi.h > rm -f hvmloader hvmloader.tmp *.o $(DEPS) > - $(MAKE) -C $(ACPI_PATH) ACPI_BUILD_DIR=$(CURDIR) clean > + $(MAKE) -C $(ACPI_PATH) ACPI_BUILD_DIR=$(CURDIR) > DSDT_FILES="$(DSDT_FILES)"

Re: [Xen-devel] [PATCH 2/2] libacpi: Be specific about which DSDT files to build

2016-11-14 Thread Wei Liu
On Mon, Nov 14, 2016 at 11:52:27PM -0500, Boris Ostrovsky wrote: > There is no reason to build, for example, dsdt_pvh.asl for hvmloader. We > pass which DSDTs to build via DSDT_FILES parameter. > > If DSDT_FILES is empty all DSDTs for a particular architecture will be built. > > Signed-off-by: Bo

[Xen-devel] [PATCH 2/2] libacpi: Be specific about which DSDT files to build

2016-11-14 Thread Boris Ostrovsky
There is no reason to build, for example, dsdt_pvh.asl for hvmloader. We pass which DSDTs to build via DSDT_FILES parameter. If DSDT_FILES is empty all DSDTs for a particular architecture will be built. Signed-off-by: Boris Ostrovsky --- tools/firmware/hvmloader/Makefile | 8 tools/li