Re: [Qemu-devel] [PATCH 2/2] acpi-test: basic acpi unit-test

2013-10-19 Thread Michael S. Tsirkin
On Sat, Oct 19, 2013 at 02:13:44AM +0200, Andreas Färber wrote: > Am 17.10.2013 23:52, schrieb Michael S. Tsirkin: > > diff --git a/tests/acpi-test.c b/tests/acpi-test.c > > new file mode 100644 > > index 000..42de248 > > --- /dev/null > > +++ b/tests/acpi-test.c > [...] > > +static void test_a

Re: [Qemu-devel] [PATCH 2/2] acpi-test: basic acpi unit-test

2013-10-18 Thread Anthony Liguori
On Fri, Oct 18, 2013 at 5:13 PM, Andreas Färber wrote: > Am 17.10.2013 23:52, schrieb Michael S. Tsirkin: >> diff --git a/tests/acpi-test.c b/tests/acpi-test.c >> new file mode 100644 >> index 000..42de248 >> --- /dev/null >> +++ b/tests/acpi-test.c > [...] >> +static void test_acpi_one(const

Re: [Qemu-devel] [PATCH 2/2] acpi-test: basic acpi unit-test

2013-10-18 Thread Andreas Färber
Am 17.10.2013 23:52, schrieb Michael S. Tsirkin: > diff --git a/tests/acpi-test.c b/tests/acpi-test.c > new file mode 100644 > index 000..42de248 > --- /dev/null > +++ b/tests/acpi-test.c [...] > +static void test_acpi_one(const char *params) > +{ > +char *args; > +uint8_t signature_low

Re: [Qemu-devel] [PATCH 2/2] acpi-test: basic acpi unit-test

2013-10-18 Thread Markus Armbruster
Paolo Bonzini writes: > Il 18/10/2013 07:30, Markus Armbruster ha scritto: >> > +static void test_acpi_tcg(void) >> > +{ >> > +test_acpi_one("-machine accel=tcg"); >> > +} >> >> Since qtest_init() adds your parameters at the end, this should result >> in a command line ending with "-machine

Re: [Qemu-devel] [PATCH 2/2] acpi-test: basic acpi unit-test

2013-10-18 Thread Michael S. Tsirkin
On Fri, Oct 18, 2013 at 07:30:48AM +0200, Markus Armbruster wrote: > "Michael S. Tsirkin" writes: > > > We run bios, and boot a minimal boot sector that immediately halts. > > Then poke at memory to find ACPI tables. > > > > This only checks that RSDP is there. > > More will be added later. > > >

Re: [Qemu-devel] [PATCH 2/2] acpi-test: basic acpi unit-test

2013-10-17 Thread Paolo Bonzini
Il 18/10/2013 07:30, Markus Armbruster ha scritto: > > +static void test_acpi_tcg(void) > > +{ > > +test_acpi_one("-machine accel=tcg"); > > +} > > Since qtest_init() adds your parameters at the end, this should result > in a command line ending with "-machine accel=qtest -machine accel=tcg",

Re: [Qemu-devel] [PATCH 2/2] acpi-test: basic acpi unit-test

2013-10-17 Thread Markus Armbruster
"Michael S. Tsirkin" writes: > We run bios, and boot a minimal boot sector that immediately halts. > Then poke at memory to find ACPI tables. > > This only checks that RSDP is there. > More will be added later. > > Signed-off-by: Michael S. Tsirkin > --- > tests/acpi-test.c | 129 > +++

[Qemu-devel] [PATCH 2/2] acpi-test: basic acpi unit-test

2013-10-17 Thread Michael S. Tsirkin
We run bios, and boot a minimal boot sector that immediately halts. Then poke at memory to find ACPI tables. This only checks that RSDP is there. More will be added later. Signed-off-by: Michael S. Tsirkin --- tests/acpi-test.c | 129 ++ tests