Re: [Qemu-block] [PATCH 17/18] nbd: BLOCK_STATUS for standard get_block_status function: server part

2017-02-15 Thread Paolo Bonzini
On 09/02/2017 16:38, Eric Blake wrote: > Umm, why are we sending only one status? If the client requests two ids > during NBD_OPT_SET_META_CONTEXT, we should be able to provide both > pieces of information at once. For a minimal implementation, it works > for proof of concept, but it is pretty r

Re: [Qemu-block] [PATCH 17/18] nbd: BLOCK_STATUS for standard get_block_status function: server part

2017-02-15 Thread Paolo Bonzini
On 09/02/2017 16:38, Eric Blake wrote: >> +static int blockstatus_to_extent(BlockDriverState *bs, uint64_t offset, >> + uint64_t length, NBDExtent *extent) >> +{ >> +BlockDriverState *file; >> +uint64_t start_sector = offset >> BDRV_SECTOR_BITS; >> +ui

Re: [Qemu-block] [PATCH 17/18] nbd: BLOCK_STATUS for standard get_block_status function: server part

2017-02-09 Thread Eric Blake
On 02/03/2017 09:47 AM, Vladimir Sementsov-Ogievskiy wrote: > Minimal realization: only one extent in server answer is supported. > > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > include/block/nbd.h | 3 ++ > nbd/nbd-internal.h | 1 + > nbd/server.c| 80 > +

[Qemu-block] [PATCH 17/18] nbd: BLOCK_STATUS for standard get_block_status function: server part

2017-02-03 Thread Vladimir Sementsov-Ogievskiy
Minimal realization: only one extent in server answer is supported. Signed-off-by: Vladimir Sementsov-Ogievskiy --- include/block/nbd.h | 3 ++ nbd/nbd-internal.h | 1 + nbd/server.c| 80 - 3 files changed, 77 insertions(+), 7 deleti