From: "Tomáš Golembiovský" <tgolembi@redhat.com>
To: qemu-devel@nongnu.org
Cc: "Tomáš Golembiovský" <tgolembi@redhat.com>,
"Kevin Wolf" <kwolf@redhat.com>, "Max Reitz" <mreitz@redhat.com>,
"Markus Armbruster" <armbru@redhat.com>,
"Eric Blake" <eblake@redhat.com>,
"Daniel P . Berrange" <berrange@redhat.com>,
qemu-block@nongnu.org
Subject: [Qemu-devel] [PATCH v8 0/2] Add 'offset' and 'size' options
Date: Mon, 31 Oct 2016 11:27:39 +0100 [thread overview]
Message-ID: <cover.1477909279.git.tgolembi@redhat.com> (raw)
v7 -> v8:
- handling offset in raw_co_pwrite_zeroes and raw_co_pdiscard (+tests)
- offset and size are not allowed for SCSI generic devices
- raw_aio_ioctl: not allowed when offset or size is set
- raw_probe_blocksizes: return -ENOTSUP if offset is not multiple of block size
- raw_probe_geometry: return -ENOTSUP if offset or size is set
v6 -> v7:
- added tests
v5 -> v6:
- fix alignment check condition
- when size is not specified and device size is being used take offset
into account
v4 -> v5:
- added two missing overflow checks
- comments from Eric Blake:
- renamed 'fail' label to 'end'
- fixed optional fields in JSON scheme
- no punctuation at the end of error_setg() message
- spaces around PRI* macros
- using QEMU_IS_ALIGNED
- typos
v3 -> v4:
- fix stupid compilation error and formatting issue
v2 -> v3:
- changed overflow check to make it clearer
- produce error instead of warning when size is not multiple of sector
size
v1 -> v2:
- options were moved from 'file' driver into 'raw' driver as suggested
- added support for writing, reopen and truncate when possible
Tomáš Golembiovský (2):
raw_bsd: add offset and size options
qemu-iotests: test 'offset' and 'size' options in raw driver
block/raw_bsd.c | 217 ++++++++++++++++++++++++++++++-
qapi/block-core.json | 16 ++-
tests/qemu-iotests/171 | 212 ++++++++++++++++++++++++++++++
tests/qemu-iotests/171.out | 313 +++++++++++++++++++++++++++++++++++++++++++++
tests/qemu-iotests/group | 1 +
5 files changed, 755 insertions(+), 4 deletions(-)
create mode 100755 tests/qemu-iotests/171
create mode 100644 tests/qemu-iotests/171.out
--
2.10.1
next reply other threads:[~2016-10-31 10:28 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-10-31 10:27 Tomáš Golembiovský [this message]
2016-10-31 10:27 ` [Qemu-devel] [PATCH v8 1/2] raw_bsd: add offset and size options Tomáš Golembiovský
2016-10-31 11:11 ` Kevin Wolf
2016-10-31 10:27 ` [Qemu-devel] [PATCH v8 2/2] qemu-iotests: test 'offset' and 'size' options in raw driver Tomáš Golembiovský
2016-10-31 11:12 ` [Qemu-devel] [PATCH v8 0/2] Add 'offset' and 'size' options Kevin Wolf
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=cover.1477909279.git.tgolembi@redhat.com \
--to=tgolembi@redhat.com \
--cc=armbru@redhat.com \
--cc=berrange@redhat.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.