vl: Print warning if a non-empty default config file is found

[Qemu-devel] [PATCH v3 0/2] vl: Print warning if a non-empty default config file is found

Eduardo Habkost posted 2 patches 6 years, 6 months ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20170927204148.2311-1-ehabkost@redhat.com

Test checkpatch passed

Test docker passed

Test s390x passed

block/blkdebug.c   |  1 -
util/qemu-config.c | 15 +++++++--------
vl.c               |  6 ++++++
3 files changed, 13 insertions(+), 9 deletions(-)

Expand all Fold all

[Qemu-devel] [PATCH v3 0/2] vl: Print warning if a non-empty default config file is found

Posted by Eduardo Habkost 6 years, 6 months ago

This missed v2.9 and v2.10.  Let's try again: we can include this
on v2.11, and remove the default config file in QEMU 2.12 or
2.13.

Changes v2 -> v3:
* Rebase to latest qemu.git master

Changes v1 -> v2:
* Remove unnecessary translation of qemu_config_parse()
  erros to -EINVAL at block/blkdebug.c:read_config()
  * Suggsted-by: Markus Armbruster <armbru@redhat.com>

We plan to remove support for /etc/qemu/qemu.conf in the near
future. Make QEMU print a warning in case there a non-empty
/etc/qemu/qemu.conf is loaded, so users have time to adapt.

Eduardo Habkost (2):
  config: qemu_config_parse() return number of config groups
  vl: Print warning when a default config file is loaded

 block/blkdebug.c   |  1 -
 util/qemu-config.c | 15 +++++++--------
 vl.c               |  6 ++++++
 3 files changed, 13 insertions(+), 9 deletions(-)

-- 
2.13.5