qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Juan Quintela <quintela@redhat.com>
To: qemu-devel@nongnu.org
Cc: "Bin Meng" <bin.meng@windriver.com>,
	"Laurent Vivier" <lvivier@redhat.com>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Alexander Bulekov" <alxndr@bu.edu>,
	"Darren Kenny" <darren.kenny@oracle.com>,
	"Stefan Hajnoczi" <stefanha@redhat.com>,
	"Ani Sinha" <ani@anisinha.ca>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	"Fam Zheng" <fam@euphon.net>, "Gerd Hoffmann" <kraxel@redhat.com>,
	"Bandan Das" <bsd@redhat.com>, "Thomas Huth" <thuth@redhat.com>,
	"Hannes Reinecke" <hare@suse.com>,
	"Qiuhao Li" <Qiuhao.Li@outlook.com>,
	"Igor Mammedov" <imammedo@redhat.com>,
	"Marc-André Lureau" <marcandre.lureau@redhat.com>,
	"Philippe Mathieu-Daudé" <f4bug@amsat.org>,
	qemu-block@nongnu.org, "Juan Quintela" <quintela@redhat.com>
Subject: [PATCH 0/8] tests: Make expliction defaults for tests
Date: Fri,  2 Sep 2022 18:51:18 +0200	[thread overview]
Message-ID: <20220902165126.1482-1-quintela@redhat.com> (raw)

Hi

For a long, long time I have had local hacks on my tree to be able to
run "make tests" when I have a minimal configure guest.  This is a
first try to upstream some of it.

- by default we always setup -display none (it already was the
  default, but some places added it anyways)

- by default we always setup -net none.  Not clear what was the
  default, but no tests use the default net, so it is safe change and
  now it is explicit.

- by default we always setup -vga none.  This is a complete difference
  can of worms.  Every tests that use vga already set vga correctly,
  so this is quite obvious, right?  Now they are acpi tables.  They
  are a mess.  And basically this means remove a device for each one
  of them.  Why going through all the trouble?  Because while I am
  develping, I normall compile out vga.

- Fix several error strings that were set with copy paste.

- replication test requires CONFIG_REPLICATION.
- test-crypto-secret requires CONFIG_SECRET_KEYRING.

Please review.  Except for the acpi changes (that I hope I have done
right following the instructions) the rest is quite obvious.

Later, Juan.

Juan Quintela (8):
  qtest: "-display none" is set in qtest_init()
  qtest: Set "-net none" in qtest_init()
  tests/acpi: The new default is -vga none
  tests/qtest: Add -vga none by default
  tests/acpi: Regenerate all needed tables
  tests: Fix error strings
  meson-build: Enable CONFIG_REPLICATION only when replication is set
  meson-build: test-crypto-secret depends on CONFIG_SECRET_KEYRING

 meson.build                           |   2 +-
 tests/qtest/bios-tables-test.c        |   2 +-
 tests/qtest/boot-serial-test.c        |   4 ++--
 tests/qtest/dbus-display-test.c       |   2 +-
 tests/qtest/display-vga-test.c        |  12 ++++++------
 tests/qtest/e1000-test.c              |   2 +-
 tests/qtest/es1370-test.c             |   2 +-
 tests/qtest/fuzz-lsi53c895a-test.c    |   2 +-
 tests/qtest/fuzz-megasas-test.c       |   2 +-
 tests/qtest/fuzz-sb16-test.c          |   6 +++---
 tests/qtest/fuzz-sdcard-test.c        |   6 +++---
 tests/qtest/fuzz-virtio-scsi-test.c   |   2 +-
 tests/qtest/fuzz-xlnx-dp-test.c       |   2 +-
 tests/qtest/fuzz/generic_fuzz.c       |   3 +--
 tests/qtest/fuzz/i440fx_fuzz.c        |   2 +-
 tests/qtest/fuzz/qos_fuzz.c           |   2 +-
 tests/qtest/libqtest.c                |   2 ++
 tests/data/acpi/pc/DSDT               | Bin 5987 -> 5992 bytes
 tests/data/acpi/pc/DSDT.acpierst      | Bin 5954 -> 5959 bytes
 tests/data/acpi/pc/DSDT.acpihmat      | Bin 7312 -> 7317 bytes
 tests/data/acpi/pc/DSDT.bridge        | Bin 8653 -> 8658 bytes
 tests/data/acpi/pc/DSDT.cphp          | Bin 6451 -> 6456 bytes
 tests/data/acpi/pc/DSDT.dimmpxm       | Bin 7641 -> 7646 bytes
 tests/data/acpi/pc/DSDT.hpbridge      | Bin 5954 -> 5959 bytes
 tests/data/acpi/pc/DSDT.hpbrroot      | Bin 3069 -> 3023 bytes
 tests/data/acpi/pc/DSDT.ipmikcs       | Bin 6059 -> 6064 bytes
 tests/data/acpi/pc/DSDT.memhp         | Bin 7346 -> 7351 bytes
 tests/data/acpi/pc/DSDT.nohpet        | Bin 5845 -> 5850 bytes
 tests/data/acpi/pc/DSDT.numamem       | Bin 5993 -> 5998 bytes
 tests/data/acpi/pc/DSDT.roothp        | Bin 6195 -> 6151 bytes
 tests/data/acpi/pc/ERST.acpierst      | Bin 912 -> 912 bytes
 tests/data/acpi/q35/DMAR.dmar         | Bin 120 -> 112 bytes
 tests/data/acpi/q35/DSDT              | Bin 8274 -> 8228 bytes
 tests/data/acpi/q35/DSDT.acpierst     | Bin 8291 -> 8245 bytes
 tests/data/acpi/q35/DSDT.acpihmat     | Bin 9599 -> 9553 bytes
 tests/data/acpi/q35/DSDT.applesmc     | Bin 8320 -> 8274 bytes
 tests/data/acpi/q35/DSDT.bridge       | Bin 10988 -> 10944 bytes
 tests/data/acpi/q35/DSDT.cphp         | Bin 8738 -> 8692 bytes
 tests/data/acpi/q35/DSDT.cxl          | Bin 9600 -> 9502 bytes
 tests/data/acpi/q35/DSDT.dimmpxm      | Bin 9928 -> 9882 bytes
 tests/data/acpi/q35/DSDT.ipmibt       | Bin 8349 -> 8303 bytes
 tests/data/acpi/q35/DSDT.ipmismbus    | Bin 8363 -> 8317 bytes
 tests/data/acpi/q35/DSDT.ivrs         | Bin 8291 -> 8245 bytes
 tests/data/acpi/q35/DSDT.memhp        | Bin 9633 -> 9587 bytes
 tests/data/acpi/q35/DSDT.mmio64       | Bin 9404 -> 9358 bytes
 tests/data/acpi/q35/DSDT.multi-bridge | Bin 8568 -> 8524 bytes
 tests/data/acpi/q35/DSDT.nohpet       | Bin 8132 -> 8086 bytes
 tests/data/acpi/q35/DSDT.numamem      | Bin 8280 -> 8234 bytes
 tests/data/acpi/q35/DSDT.pvpanic-isa  | Bin 8375 -> 8329 bytes
 tests/data/acpi/q35/DSDT.tis.tpm12    | Bin 8880 -> 8834 bytes
 tests/data/acpi/q35/DSDT.tis.tpm2     | Bin 8906 -> 8860 bytes
 tests/data/acpi/q35/DSDT.viot         | Bin 9383 -> 9339 bytes
 tests/data/acpi/q35/DSDT.xapic        | Bin 35637 -> 35591 bytes
 tests/data/acpi/q35/ERST.acpierst     | Bin 912 -> 912 bytes
 tests/data/acpi/q35/IVRS.ivrs         | Bin 104 -> 100 bytes
 tests/data/acpi/q35/VIOT.viot         | Bin 112 -> 112 bytes
 tests/unit/meson.build                |   4 +++-
 57 files changed, 31 insertions(+), 28 deletions(-)

-- 
2.37.2



             reply	other threads:[~2022-09-02 16:53 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-09-02 16:51 Juan Quintela [this message]
2022-09-02 16:51 ` [PATCH 1/8] qtest: "-display none" is set in qtest_init() Juan Quintela
2022-09-19 17:59   ` Thomas Huth
2022-09-20  9:20   ` Thomas Huth
2022-09-02 16:51 ` [PATCH 2/8] qtest: Set "-net none" " Juan Quintela
2022-09-04 13:41   ` Philippe Mathieu-Daudé via
2022-09-02 16:51 ` [PATCH 3/8] tests/acpi: The new default is -vga none Juan Quintela
2022-09-02 16:51 ` [PATCH 4/8] tests/qtest: Add -vga none by default Juan Quintela
2022-09-02 16:51 ` [PATCH 5/8] tests/acpi: Regenerate all needed tables Juan Quintela
2022-09-02 16:51 ` [PATCH 6/8] tests: Fix error strings Juan Quintela
2022-09-04 13:41   ` Philippe Mathieu-Daudé via
2022-09-02 16:51 ` [PATCH 7/8] meson-build: Enable CONFIG_REPLICATION only when replication is set Juan Quintela
2022-09-19 18:05   ` Thomas Huth
2022-09-02 16:51 ` [PATCH 8/8] meson-build: test-crypto-secret depends on CONFIG_SECRET_KEYRING Juan Quintela
2022-09-19 18:06   ` Thomas Huth
2022-09-20  8:06   ` Daniel P. Berrangé
2022-09-02 17:16 ` [PATCH 0/8] tests: Make expliction defaults for tests Alexander Bulekov
2022-09-19 18:09   ` Thomas Huth

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=20220902165126.1482-1-quintela@redhat.com \
    --to=quintela@redhat.com \
    --cc=Qiuhao.Li@outlook.com \
    --cc=alxndr@bu.edu \
    --cc=ani@anisinha.ca \
    --cc=bin.meng@windriver.com \
    --cc=bsd@redhat.com \
    --cc=darren.kenny@oracle.com \
    --cc=f4bug@amsat.org \
    --cc=fam@euphon.net \
    --cc=hare@suse.com \
    --cc=imammedo@redhat.com \
    --cc=kraxel@redhat.com \
    --cc=lvivier@redhat.com \
    --cc=marcandre.lureau@redhat.com \
    --cc=mst@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-block@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.com \
    --cc=thuth@redhat.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).