[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 08/18] migration-local: introduce qemu_fopen_loc
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 08/18] migration-local: introduce qemu_fopen_local() |
Date: |
Wed, 28 Aug 2013 08:37:32 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130805 Thunderbird/17.0.8 |
Il 28/08/2013 05:26, Lei Li ha scritto:
>>>
>>> RAMBlock structs are not visible outside of exec.c and arch_init.c,
>>> how did you do this?
>>
>> Hi Michael,
>>
>> Good catch!
>> Actually this is the 'Known issue' that I listed in the cover
>> letter, and I planed to ask suggestions on this.
>>
>> Currently the implementation of override of the RDMA hook like
>> save_page for localhost migration needs to have knowledge of
>> MemoryRegion and RAMBlock, say, when saving ram page, need to know
>> which RAMBlock the given ram address save_page hook passed is
>> belong to, or which is the last ram block has been sent. But seems
>> that such structs can not be exported to the private code in
>> migration-local.c.
>>
>> My guess for now is that there might be two possible ways to handle this:
>>
>> 1) Choose another way around, like representation of a RAMBlock and
>> MemoryRegion from localhost migration perspective. This might need
>> more work to handle the descriptions.
>>
>> 2) It could be exported directly into private code like migration-local.c
>> through some ways that I don't know yet. I remembered that the
>> implementation of postcopy migration did this.
>>
>> So I'd like to post and ask for your suggestions to make sure it's the
>> right and best way to handle.
>
> Hi Paolo,
>
> Any suggestions on this?
Unlike the RAM constants and migration states, I think exporting
MemoryRegion (not sure about RAMBlock) is fine. In fact, replacing the
save_page hook's block_offset argument with a MemoryRegion would be a
nice cleanup.
Paolo
- Re: [Qemu-devel] [PATCH 05/18] savevm: add comments for qemu_file_get_error(), (continued)
[Qemu-devel] [PATCH 08/18] migration-local: introduce qemu_fopen_local(), Lei Li, 2013/08/21
[Qemu-devel] [PATCH 07/18] arch_init: export RAM_SAVE_xxx flags, Lei Li, 2013/08/21
[Qemu-devel] [PATCH 09/18] exec: export qemu_get_ram_block(), Lei Li, 2013/08/21
[Qemu-devel] [PATCH 10/18] migration-local: implementation of outgoing part, Lei Li, 2013/08/21