Re: [Xen-devel] [PATCH v4 10/21] acpi/hvmloader: Link ACPI object files directly

2016-09-21 Thread Boris Ostrovsky
On 09/21/2016 06:52 AM, Jan Beulich wrote: On 20.09.16 at 02:19, wrote: >> --- a/.gitignore >> +++ b/.gitignore >> @@ -127,13 +127,13 @@ tools/firmware/*bios/*bios*.txt >> tools/firmware/etherboot/gpxe/* >> tools/firmware/extboot/extboot.img >> tools/firmware/extboot/signrom >> -tools/firm

Re: [Xen-devel] [PATCH v4 10/21] acpi/hvmloader: Link ACPI object files directly

2016-09-21 Thread Ian Jackson
Boris Ostrovsky writes ("Re: [PATCH v4 10/21] acpi/hvmloader: Link ACPI object files directly"): > On 09/21/2016 11:05 AM, Ian Jackson wrote: > > Wait, the libxl makefiles are going to end up running iasl ? > > Not directly. There is a new target in libxl's Makefile: > > acpi: > $(MAKE)

Re: [Xen-devel] [PATCH v4 10/21] acpi/hvmloader: Link ACPI object files directly

2016-09-21 Thread Boris Ostrovsky
On 09/21/2016 11:05 AM, Ian Jackson wrote: > Boris Ostrovsky writes ("Re: [PATCH v4 10/21] acpi/hvmloader: Link ACPI > object files directly"): >> 2. ".tmp__" vs ".tmp": Because the temporary files are generated not in >> tools/libacpi but in the directory of the libacpi user (such as libxl) >> it

Re: [Xen-devel] [PATCH v4 10/21] acpi/hvmloader: Link ACPI object files directly

2016-09-21 Thread Ian Jackson
Boris Ostrovsky writes ("Re: [PATCH v4 10/21] acpi/hvmloader: Link ACPI object files directly"): > 2. ".tmp__" vs ".tmp": Because the temporary files are generated not in > tools/libacpi but in the directory of the libacpi user (such as libxl) > it is possible that a Makefile there might use ".tmp

Re: [Xen-devel] [PATCH v4 10/21] acpi/hvmloader: Link ACPI object files directly

2016-09-21 Thread Boris Ostrovsky
On 09/21/2016 07:40 AM, Jan Beulich wrote: On 21.09.16 at 13:38, wrote: >> Jan Beulich writes ("Re: [PATCH v4 10/21] acpi/hvmloader: Link ACPI object >> files directly"): >>> On 21.09.16 at 13:29, wrote: I think `.dummy' would be a better string, if indeed it's a string which we e

Re: [Xen-devel] [PATCH v4 10/21] acpi/hvmloader: Link ACPI object files directly

2016-09-21 Thread Jan Beulich
>>> On 21.09.16 at 13:38, wrote: > Jan Beulich writes ("Re: [PATCH v4 10/21] acpi/hvmloader: Link ACPI object > files directly"): >> On 21.09.16 at 13:29, wrote: >> > I think `.dummy' would be a better string, if indeed it's a string >> > which we expect always to be stripped, and not to appear

Re: [Xen-devel] [PATCH v4 10/21] acpi/hvmloader: Link ACPI object files directly

2016-09-21 Thread Ian Jackson
Jan Beulich writes ("Re: [PATCH v4 10/21] acpi/hvmloader: Link ACPI object files directly"): > On 21.09.16 at 13:29, wrote: > > I think `.dummy' would be a better string, if indeed it's a string > > which we expect always to be stripped, and not to appear in any > > filenames. > > Another (curre

Re: [Xen-devel] [PATCH v4 10/21] acpi/hvmloader: Link ACPI object files directly

2016-09-21 Thread Jan Beulich
>>> On 21.09.16 at 13:29, wrote: > Jan Beulich writes ("Re: [PATCH v4 10/21] acpi/hvmloader: Link ACPI object > files directly"): >> On 20.09.16 at 02:19, wrote: >> > --- a/tools/firmware/hvmloader/acpi/Makefile >> > +++ b/tools/firmware/hvmloader/acpi/Makefile >> > @@ -15,41 +15,45 @@ >> > XEN

Re: [Xen-devel] [PATCH v4 10/21] acpi/hvmloader: Link ACPI object files directly

2016-09-21 Thread Ian Jackson
Jan Beulich writes ("Re: [PATCH v4 10/21] acpi/hvmloader: Link ACPI object files directly"): > On 20.09.16 at 02:19, wrote: > > --- a/tools/firmware/hvmloader/acpi/Makefile > > +++ b/tools/firmware/hvmloader/acpi/Makefile > > @@ -15,41 +15,45 @@ > > XEN_ROOT = $(CURDIR)/../../../.. > > include

Re: [Xen-devel] [PATCH v4 10/21] acpi/hvmloader: Link ACPI object files directly

2016-09-21 Thread Jan Beulich
>>> On 20.09.16 at 02:19, wrote: > --- a/.gitignore > +++ b/.gitignore > @@ -127,13 +127,13 @@ tools/firmware/*bios/*bios*.txt > tools/firmware/etherboot/gpxe/* > tools/firmware/extboot/extboot.img > tools/firmware/extboot/signrom > -tools/firmware/hvmloader/acpi/mk_dsdt > -tools/firmware/hvmlo

[Xen-devel] [PATCH v4 10/21] acpi/hvmloader: Link ACPI object files directly

2016-09-19 Thread Boris Ostrovsky
ACPI sources will be available to various component which will build them according to their own rules. ACPI's Makefile will only generate necessary source files. Signed-off-by: Boris Ostrovsky --- Changes in v4: * Added a suffix to iasl's -p option to work around a bug in older iasl versions w