Re: [Qemu-devel] [PATCH] block/raw: Add create_options for host_device

[Kevin Wolf]

Am 01.10.2009 19:13, schrieb Christoph Hellwig:
> On Thu, Oct 01, 2009 at 12:35:49PM +0200, Kevin Wolf wrote:
>> Today host_devices have a create function, so they also need a create_options
>> field to prevent qemu-img from complaining.
> 
> Yeah.  But looking deeper does the size argument actually make any
> sense for the host devices?  What we do right now is to seek to the
> total_size * 512 as some sort of tests, but it's not actually in any
> way encoded in the image, we'll always get the normal raw_getlength
> return value when quering for the size later.

Well, I do think that it makes sense to check if the device is at least
not too small for the data that we are going to write (we could also
check if it's an exact match, but I'm not sure if this is helpful). But
other than this there is probably little use for it, yes.

What would you suggest? Drop the check completely and change qemu-img to
deal with formats that don't support sizes?

Kevin