Thanx for review, John! Let's wait for finishing -blockdev works indeed.
Nikolay
On 10.01.2019 15:15, Nikolay Shirokovskiy wrote:
As this patch series only works with -blockdev configurations it is
better to
be pushed only after -blockdev is supported. Then 'news' and 'xml' patch
need
to be updated to have correct libvirt version.
Diff from v2 [1]:
=================
- move caps patch to top
- add news patch
- add _CAPPED to capabilitity name to distinguish from
correspondent qemu capabilitiy
- better option explanation in formatdomain.html
- misc tests/english fixes as suggested
We have come to agreement with John in mailing list to support QEMU versions
which do not let use INT64_MAX as "maximum" value. But this will require
changing qemuBlockStorageSourceGetFormatQcow2Props and a lot of other functions
signature (which was not noticed during conversation). Thus I don't add such
a functionality. Anyway the patch requires -blockdev from QEMU so we could only
support single QEMU 3.0 version.
[1]
https://www.redhat.com/archives/libvir-list/2018-November/msg00313.html
Nikolay Shirokovskiy (5):
qemu: caps: add QEMU_CAPS_QCOW2_L2_CACHE_SIZE_CAPPED
xml: add disk driver metadata_cache_size option
qemu: support metadata-cache-size for blockdev
news: add notice for new metadata cache size policy option
DO NOT APPLY: add xml2argv test for metadata_cache_size
docs/formatdomain.html.in | 11 ++++++
docs/news.xml | 11 ++++++
docs/schemas/domaincommon.rng | 11 ++++++
src/conf/domain_conf.c | 17 +++++++++
src/conf/domain_conf.h | 9 +++++
src/qemu/qemu_block.c | 5 ++-
src/qemu/qemu_capabilities.c | 5 +++
src/qemu/qemu_capabilities.h | 1 +
src/qemu/qemu_command.c | 23 ++++++++++++
src/qemu/qemu_domain.c | 2 ++
src/util/virstoragefile.c | 1 +
src/util/virstoragefile.h | 1 +
.../qemuxml2argvdata/disk-metadata_cache_size.args | 39 ++++++++++++++++++++
.../qemuxml2argvdata/disk-metadata_cache_size.xml | 35 ++++++++++++++++++
tests/qemuxml2argvtest.c | 2 ++
.../disk-metadata_cache_size.xml | 41 ++++++++++++++++++++++
tests/qemuxml2xmltest.c | 2 ++
17 files changed, 215 insertions(+), 1 deletion(-)
create mode 100644 tests/qemuxml2argvdata/disk-metadata_cache_size.args
create mode 100644 tests/qemuxml2argvdata/disk-metadata_cache_size.xml
create mode 100644 tests/qemuxml2xmloutdata/disk-metadata_cache_size.xml