From: Leonid Bloch <lbloch@janustech.com>
To: qemu-devel@nongnu.org
Cc: qemu-block@nongnu.org, Kevin Wolf <kwolf@redhat.com>,
Max Reitz <mreitz@redhat.com>, Eric Blake <eblake@redhat.com>,
Leonid Bloch <lbloch@janustech.com>
Subject: [Qemu-devel] [PATCH v5 0/4] Introduction of l2-cache-full option for qcow2 images
Date: Wed, 25 Jul 2018 17:27:54 +0300 [thread overview]
Message-ID: <20180725142758.9980-1-lbloch@janustech.com> (raw)
This series introduces an option to calculate and allocate
automatically enough qcow2 L2 cache to cover the entire image.
Using cache that covers the entire image can benefit performance,
while having only a small memory overhead (just 1 MB for every 8 GB
of virtual image size with the default cluster size).
-------------------------
Differences from v1:
1) Documentation fixes in qapi/block-core.json and qemu-options.hx
2) Removal of the patch which was made to fix the default sizes in
docs/qcow2-cache.txt - it is not needed, as the default sizes imply
also default cluster sizes.
3) Documentation fixes in docs/qcow2-cache.txt, mentioning mutual
exclusivity of the options.
4) Squashing the iotests patch into the main feature addition patch
-------------------------
Differences from v2:
1) A separate patch for the grammar fix for 3.0
2) A separate patch for existing documentation fixes for 3.0
3) Separated back the iotests patch, because the grammar fix is separate now
-------------------------
Differences from v3:
1) Grammar change commit message fix
2) Rewording the documentation more concisely
3) Squashing the l2-cache-full docs commit to the one that introduces this
feature
-------------------------
Differences from v4:
1) l2-cache-full is now a proper boolean option.
2) tests/qemu-iotests/103.out fixed: the grammar fix is now applied to the
second relevant occurrence.
3) Error messages fixed in iotests for the l2-cache-full tests.
Leonid Bloch (4):
qcow2: A grammar fix in conflicting cache sizing error message
qcow2: Options' documentation fixes
qcow2: Introduce an option for sufficient L2 cache for the entire
image
iotests: Add tests for the new l2-cache-full option
block/qcow2.c | 37 +++++++++++++++++++++++++++++--------
block/qcow2.h | 1 +
docs/qcow2-cache.txt | 18 ++++++++++++++----
qapi/block-core.json | 8 +++++++-
qemu-options.hx | 14 ++++++++++----
tests/qemu-iotests/103 | 6 ++++++
tests/qemu-iotests/103.out | 6 ++++--
tests/qemu-iotests/137 | 2 ++
tests/qemu-iotests/137.out | 4 +++-
9 files changed, 76 insertions(+), 20 deletions(-)
--
2.14.1
next reply other threads:[~2018-07-25 14:28 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-07-25 14:27 Leonid Bloch [this message]
2018-07-25 14:27 ` [Qemu-devel] [PATCH v5 1/4 for-3.0] qcow2: A grammar fix in conflicting cache sizing error message Leonid Bloch
2018-07-25 22:19 ` [Qemu-devel] [Qemu-block] " John Snow
2018-07-26 9:57 ` Kevin Wolf
2018-07-25 14:27 ` [Qemu-devel] [PATCH v5 2/4 for-3.0] qcow2: Options' documentation fixes Leonid Bloch
2018-07-26 10:02 ` Kevin Wolf
2018-07-26 14:20 ` Eric Blake
2018-07-26 14:30 ` Leonid Bloch
2018-07-26 14:27 ` Leonid Bloch
2018-07-26 14:46 ` Kevin Wolf
2018-07-26 14:51 ` Leonid Bloch
2018-07-25 14:27 ` [Qemu-devel] [PATCH v5 3/4] qcow2: Introduce an option for sufficient L2 cache for the entire image Leonid Bloch
2018-07-25 14:27 ` [Qemu-devel] [PATCH v5 4/4] iotests: Add tests for the new l2-cache-full option Leonid Bloch
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=20180725142758.9980-1-lbloch@janustech.com \
--to=lbloch@janustech.com \
--cc=eblake@redhat.com \
--cc=kwolf@redhat.com \
--cc=mreitz@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
/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).