From: "Denis V. Lunev" <den@openvz.org>
To: Alex Bligh <alex@alex.org.uk>
Cc: "nbd-general@lists.sourceforge.net"
<nbd-general@lists.sourceforge.net>,
"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] [RFC 1/1] nbd (specification): add NBD_CMD_WRITE_ZEROES command
Date: Thu, 18 Feb 2016 11:34:04 +0300 [thread overview]
Message-ID: <56C581FC.6020603@openvz.org> (raw)
In-Reply-To: <00466EEA-174D-40E5-B74F-974D11DBB97C@alex.org.uk>
On 02/18/2016 11:09 AM, Alex Bligh wrote:
> On 17 Feb 2016, at 18:10, Denis V. Lunev <den@openvz.org> wrote:
>
>> Currently available NBD_CMD_TRIM command can not be used as the
>> specification explicitely says that "a client MUST NOT make any
>> assumptions about the contents of the export affected by this
>> [NBD_CMD_TRIM] command, until overwriting it again with `NBD_CMD_WRITE`"
> Would a flag to NBD_CMD_TRIM that says "ensure the written
> data is zeroed" not be an easier solution than adding another
> very similar command?
>
> Or (cough) changing the spec?
>
from the point of the receiver the situation (from my POW) could
be different. Let us assume that we are writing to the plain
file.
There are 2 type of queries:
- pls make the target sparse, i.e. perform FALLOC_FL_PUNCH_HOLE
and there is no problem that the operation could not be performed,
this is a hint;
- pls write the following amount of zeroes in either way (even calling
write directly), i.e. ensure that the data is zeroed and the space on
the file system is allocated for that.
The difference comes from the situation when sparse files results
in a serious performance impact if written by small pieces. In this
case the data of one block of the virtual disk can be placed to
very different blocks of the physical disk seriously impacting
subsequent sequential reading.
Thus I propose to make the situation distinct for these commands.
Though you proposal to add NBD_FLAG_SEND_TRIM_ZEROED to export
flags is also great. Though this IMHO is a different story.
Den
next prev parent reply other threads:[~2016-02-18 8:34 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-17 18:10 [Qemu-devel] [RFC 1/1] nbd (specification): add NBD_CMD_WRITE_ZEROES command Denis V. Lunev
2016-02-17 20:58 ` Eric Blake
2016-02-18 4:46 ` Denis V. Lunev
2016-02-18 8:30 ` Denis V. Lunev
2016-02-18 9:18 ` Roman Kagan
2016-02-18 10:36 ` Denis V. Lunev
2016-02-18 16:35 ` Eric Blake
2016-02-18 17:23 ` [Qemu-devel] SUMMARY: " Denis V. Lunev
2016-02-18 17:55 ` Eric Blake
2016-02-18 19:29 ` [Qemu-devel] [Nbd] " Alex Bligh
2016-02-19 7:12 ` [Qemu-devel] " Denis V. Lunev
2016-02-19 8:56 ` Vladimir Sementsov-Ogievskiy
2016-02-19 9:11 ` Daniel P. Berrange
2016-02-18 12:14 ` [Qemu-devel] " Daniel P. Berrange
2016-02-18 14:05 ` Denis V. Lunev
2016-02-18 8:09 ` Alex Bligh
2016-02-18 8:34 ` Denis V. Lunev [this message]
2016-03-04 8:49 ` [Qemu-devel] [Nbd] " Wouter Verhelst
2016-03-04 9:54 ` Kevin Wolf
2016-03-04 14:03 ` Paolo Bonzini
2016-03-06 10:28 ` Wouter Verhelst
2016-03-06 18:54 ` Denis V. Lunev
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=56C581FC.6020603@openvz.org \
--to=den@openvz.org \
--cc=alex@alex.org.uk \
--cc=nbd-general@lists.sourceforge.net \
--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.