summary refs log tree commit diff stats
path: root/include/hw/qdev-core.h
diff options
context:
space:
mode:
authorVladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>2018-01-10 17:08:21 -0600
committerEric Blake <eblake@redhat.com>2018-01-17 20:14:12 -0600
commit0cfae925d2e31a7d7613a6fb60aa58b38ec2b61e (patch)
treee387b106a4fd4d94397f4cec7017be81129e9486 /include/hw/qdev-core.h
parenta16a7907707e08c09843297a6f707f5bfb8beb22 (diff)
downloadfocaccia-qemu-0cfae925d2e31a7d7613a6fb60aa58b38ec2b61e.tar.gz
focaccia-qemu-0cfae925d2e31a7d7613a6fb60aa58b38ec2b61e.zip
nbd/server: refactor negotiation functions parameters
Instead of passing currently negotiating option and its length to
many of negotiation functions let's just store them on NBDClient
struct to be state-variables of negotiation phase.

This unifies semantics of negotiation functions and allows
tracking changes of remaining option length in future patches.

Asssert that optlen is back to 0 after negotiation (including
old-style connections which don't negotiate), although we need
more patches before we can assert optlen is 0 between options
during negotiation.

Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Message-Id: <20171122101958.17065-2-vsementsov@virtuozzo.com>
[eblake: rebase, commit message tweak, assert !optlen after
negotiation completes]
Signed-off-by: Eric Blake <eblake@redhat.com>
Diffstat (limited to 'include/hw/qdev-core.h')
0 files changed, 0 insertions, 0 deletions