[libvirt] [PATCH v3 0/5] add disk driver metadata_cache_size option

Nikolay Shirokovskiy posted 5 patches 5 years, 2 months ago
Test syntax-check passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/1547122517-774781-1-git-send-email-nshirokovskiy@virtuozzo.com
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
[libvirt] [PATCH v3 0/5] add disk driver metadata_cache_size option
Posted by Nikolay Shirokovskiy 5 years, 2 months ago
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

-- 
1.8.3.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH v3 0/5] add disk driver metadata_cache_size option
Posted by Nikolay Shirokovskiy 5 years, 2 months ago
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
> 

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list