Re: [Qemu-devel] [PATCH] CMOS file support

[Mathias Krause <mathias.krause@secunet.com>]

Am 16.09.2010 19:20 schrieb Anthony Liguori:
> Instead of using FILE, I'd suggest using a BlockDriver to read and write
> the data.

I'll fix that as soon as I figured how to use this interface.

> I think it would be very nice to add write support too so that writes to
> CMOS were persisted across boots.

Indeed. Also I would like to have a command line interface like '-cmos
cmos.bin' instead of the ugly '-global mc146818rtc.file=cmos.bin'. But
I'm not aware how to create such an alias. Any pointers?

>> +static long get_file_size(FILE *f)
>> +{
>> +    long where, size;
>> +
>> +    /* XXX: on Unix systems, using fstat() probably makes more sense */
>> +
>> +    where = ftell(f);
>> +    fseek(f, 0, SEEK_END);
>> +    size = ftell(f);
>> +    fseek(f, where, SEEK_SET);
>> +
>> +    return size;
>> +}
>>    
> 
> BlockDrivers have a getlength() functions.

Would reduce the size of the patch which is always a good thing (tm).


Mathias