----- Original Message ----- > From: "Kevin Wolf" <[email protected]> > To: "Paolo Bonzini" <[email protected]> > Cc: [email protected], [email protected], [email protected], > [email protected] > Sent: Wednesday, July 31, 2013 11:12:27 AM > Subject: Re: [PATCH v3 18/19] block: add default get_block_status > implementation for protocols > > Am 25.07.2013 um 16:23 hat Paolo Bonzini geschrieben: > > Protocols return raw data, so you can assume the offsets to pass > > through unchanged. > > > > Reviewed-by: Eric Blake <[email protected]> > > Signed-off-by: Paolo Bonzini <[email protected]> > > Not really objecting, but is this useful? > > In formats, BDRV_BLOCK_OFFSET_VALID means that the given offset is valid > for bs->file, which is something that protocols simply don't have. You > can do it like you do here, but actually this information is redundant. > > Do you make use of it for anything?
It is used by format=raw. I could do it in raw.c, but some protocols are opened directly without the "raw" veneer on top. Paolo
