]> git.ipfire.org Git - thirdparty/qemu.git/commit - qemu-nbd.c
nbd: Prepare for NBD_CMD_FLAG_FAST_ZERO
authorEric Blake <eblake@redhat.com>
Fri, 23 Aug 2019 14:37:23 +0000 (09:37 -0500)
committerEric Blake <eblake@redhat.com>
Thu, 5 Sep 2019 21:03:13 +0000 (16:03 -0500)
commit0a4795455ceeebdca999a60583dd42434f2ec0d1
treea60ccde57a3f3718d802e73085d2ce707a114642
parentdbb38caac54af7cd2432bec715da6ad33b3bdc22
nbd: Prepare for NBD_CMD_FLAG_FAST_ZERO

Commit fe0480d6 and friends added BDRV_REQ_NO_FALLBACK as a way to
avoid wasting time on a preliminary write-zero request that will later
be rewritten by actual data, if it is known that the write-zero
request will use a slow fallback; but in doing so, could not optimize
for NBD.  The NBD specification is now considering an extension that
will allow passing on those semantics; this patch updates the new
protocol bits and 'qemu-nbd --list' output to recognize the bit, as
well as the new errno value possible when using the new flag; while
upcoming patches will improve the client to use the feature when
present, and the server to advertise support for it.

The NBD spec recommends (but not requires) that ENOTSUP be avoided for
all but failures of a fast zero (the only time it is mandatory to
avoid an ENOTSUP failure is when fast zero is supported but not
requested during write zeroes; the questionable use is for ENOTSUP to
other actions like a normal write request).  However, clients that get
an unexpected ENOTSUP will either already be treating it the same as
EINVAL, or may appreciate the extra bit of information.  We were
equally loose for returning EOVERFLOW in more situations than
recommended by the spec, so if it turns out to be a problem in
practice, a later patch can tighten handling for both error codes.

Signed-off-by: Eric Blake <eblake@redhat.com>
Message-Id: <20190823143726.27062-3-eblake@redhat.com>
Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
[eblake: tweak commit message, also handle EOPNOTSUPP]
docs/interop/nbd.txt
include/block/nbd.h
nbd/common.c
nbd/server.c
qemu-nbd.c