[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RFC 1/3] memory: Add dump-pc-mem command for che
From: |
Bohdan Trach |
Subject: |
Re: [Qemu-devel] [PATCH RFC 1/3] memory: Add dump-pc-mem command for checkpointing |
Date: |
Sat, 18 Apr 2015 09:40:31 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 |
Thank You for the review.
Please see comments inline.
On 04/17/2015 03:53 PM, Eric Blake wrote:
> On 04/17/2015 06:13 AM, Bohdan Trach wrote:
>> From: Bohdan Trach <address@hidden>
>>
>> dump-pc-mem command is added for checkpointing guest memory to
>> file. Only system RAM region is saved. This checkpoint is later used to
>> recover unchanged pages.
>>
>> Signed-off-by: Bohdan Trach <address@hidden>
>> ---
>
>>
>> +void qmp_dump_pc_ram(const char *file, Error **errp) {
>> +
>> + int rc;
>> + int fd;
>> + fd = open(file,
>
> Please use qemu_open() rather than raw open(), so that your command
> automatically supports /dev/fdset/nnn notation for reusing a file
> descriptor passed in via SCM_RIGHTS.
>
OK, this will be fixed.
>> +++ b/qapi-schema.json
>> @@ -3648,3 +3648,14 @@
>> # Since: 2.1
>> ##
>> { 'command': 'rtc-reset-reinjection' }
>> +
>> +##
>> +# @dump-pc-ram:
>> +#
>> +# Checkpoints guest.
>
> The whole guest, or just guest memory?
>
dump-pc-ram command currently writes "pc.ram" MemoryRegion to the
specified file.
>> +#
>> +# @file: the file to save the memory to as binary data
>> +#
>> +# Returns: Nothing on success
>
> Missing a 'Since: 2.4' designation.
>
OK, I'll add it.
--
With best regards,
Bohdan Trach