[Qemu-devel] [PATCH v4 0/2] vl: Deprecate auto-loading of qemu.conf

Eduardo Habkost posted 2 patches 6 years, 6 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20171004025043.3788-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 ++++++
qemu-doc.texi      |  8 ++++++++
4 files changed, 21 insertions(+), 9 deletions(-)
[Qemu-devel] [PATCH v4 0/2] vl: Deprecate auto-loading of qemu.conf
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.13 or
2.14.

Changes v3 -> v4:
* Use warn_report() instead of error_report("warning: ...")
  (Eric Blake)
* Document as a deprecated feature in qemu-doc.texi
* Updated Subject line
  (was "vl: Print warning if a non-empty default config file is found")

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: Deprecate auto-loading of qemu.conf

 block/blkdebug.c   |  1 -
 util/qemu-config.c | 15 +++++++--------
 vl.c               |  6 ++++++
 qemu-doc.texi      |  8 ++++++++
 4 files changed, 21 insertions(+), 9 deletions(-)

-- 
2.13.5