[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 0/2] acpi unit-test: rebuild aml files functi
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH v2 0/2] acpi unit-test: rebuild aml files functionality |
Date: |
Thu, 26 Dec 2013 15:53:25 +0200 |
On Thu, Dec 26, 2013 at 03:24:28PM +0200, Marcel Apfelbaum wrote:
> On Thu, 2013-12-26 at 14:46 +0200, Michael S. Tsirkin wrote:
> > On Thu, Dec 26, 2013 at 02:19:48PM +0200, Marcel Apfelbaum wrote:
> > > Acpi unit-test will fail every time the acpi tables change.
> > >
> > > The series adds a script that rebuilds the expected aml files, so the test
> > > will pass. It also validates the modifications.
> > >
> > > The acpi unit test will rebuild the aml tables if TEST_ACPI_REBUILD_AML
> > > environment variable is set.
> >
> > OK I had to fix some trailing whitespace, otherwise
> > looks good.
> > I applied it all, pls check the pci branch.
> Thanks!
> I pulled the pci branch, but I did not see the patches yet.
> I used git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git,
> pci branch.
>
> Thanks,
> Marcel
Actually had to revert. Build fails with recent gcc:
ests/acpi-test.c: In function ‘dump_aml_files’:
tests/acpi-test.c:392:14: error: ignoring return value of ‘write’,
declared with attribute warn_unused_result [-Werror=unused-result]
write(fd, sdt, sizeof(AcpiTableHeader));
^
tests/acpi-test.c:393:14: error: ignoring return value of ‘write’,
declared with attribute warn_unused_result [-Werror=unused-result]
write(fd, sdt->aml, sdt->aml_len);
^
cc1: all warnings being treated as errors
make: *** [tests/acpi-test.o] Error 1
and it's true: you need to wrap write in a loop as it can
write less than requested number of bytes.
See qemu_write_full - can it be used it tests?
> > > v1 -> v2:
> > > Addressed Michael S. Tsirkin's comments:
> > > - added TEST_ prefix to the environment variable
> > > - some rephrases
> > >
> > > Marcel Apfelbaum (2):
> > > acpi unit-test: added script to rebuild the expected aml files
> > > acpi unit-test: hook to rebuild expected aml files
> > >
> > > tests/acpi-test-data/rebuild-expected-aml.sh | 36
> > > ++++++++++++++++++++++++++++
> > > tests/acpi-test.c | 30 +++++++++++++++++++----
> > > 2 files changed, 61 insertions(+), 5 deletions(-)
> > > create mode 100755 tests/acpi-test-data/rebuild-expected-aml.sh
> > >
> > > --
> > > 1.8.3.1
> >
>
>
- [Qemu-devel] [PATCH v2 0/2] acpi unit-test: rebuild aml files functionality, Marcel Apfelbaum, 2013/12/26
- [Qemu-devel] [PATCH v2 1/2] acpi unit-test: added script to rebuild the expected aml files, Marcel Apfelbaum, 2013/12/26
- [Qemu-devel] [PATCH v2 2/2] acpi unit-test: hook to rebuild expected aml files, Marcel Apfelbaum, 2013/12/26
- Re: [Qemu-devel] [PATCH v2 0/2] acpi unit-test: rebuild aml files functionality, Michael S. Tsirkin, 2013/12/26
- Re: [Qemu-devel] [PATCH v2 0/2] acpi unit-test: rebuild aml files functionality, Marcel Apfelbaum, 2013/12/26
- Re: [Qemu-devel] [PATCH v2 0/2] acpi unit-test: rebuild aml files functionality,
Michael S. Tsirkin <=
- Re: [Qemu-devel] [PATCH v2 0/2] acpi unit-test: rebuild aml files functionality, Marcel Apfelbaum, 2013/12/26
- Re: [Qemu-devel] [PATCH v2 0/2] acpi unit-test: rebuild aml files functionality, Michael S. Tsirkin, 2013/12/26
- Re: [Qemu-devel] [PATCH v2 0/2] acpi unit-test: rebuild aml files functionality, Marcel Apfelbaum, 2013/12/26
- Re: [Qemu-devel] [PATCH v2 0/2] acpi unit-test: rebuild aml files functionality, Michael S. Tsirkin, 2013/12/26
- Re: [Qemu-devel] [PATCH v2 0/2] acpi unit-test: rebuild aml files functionality, Michael S. Tsirkin, 2013/12/26
- Re: [Qemu-devel] [PATCH v2 0/2] acpi unit-test: rebuild aml files functionality, Marcel Apfelbaum, 2013/12/26
- Re: [Qemu-devel] [PATCH v2 0/2] acpi unit-test: rebuild aml files functionality, Michael S. Tsirkin, 2013/12/26
- Re: [Qemu-devel] [PATCH v2 0/2] acpi unit-test: rebuild aml files functionality, Marcel Apfelbaum, 2013/12/26
- Re: [Qemu-devel] [PATCH v2 0/2] acpi unit-test: rebuild aml files functionality, Marcel Apfelbaum, 2013/12/29