[CRIU] [PATCH 05/22] protobuf: Update core.proto for new features

Pavel Emelyanov xemul at parallels.com
Wed Dec 12 09:10:24 EST 2012


On 12/12/2012 05:50 PM, Cyrill Gorcunov wrote:
> On Wed, Dec 12, 2012 at 05:40:53PM +0400, Pavel Emelyanov wrote:
>> On 11/22/2012 01:05 AM, Cyrill Gorcunov wrote:
>>>
>>> To restore FPU we will need
>>>
>>>  - the xsave data
>>>  - cpu parameters (features) data
>>>
>>> Note the user_x86_fpregs_entry::padding left for backward
>>> compatibility purpose since there might be images which
>>> were checkpointed where we had no FPU support.
>>>
>>> Signed-off-by: Cyrill Gorcunov <gorcunov at openvz.org>
>>> ---
>>>  protobuf/core.proto |   15 +++++++++++++++
>>>  1 files changed, 15 insertions(+), 0 deletions(-)
>>>
>>
>>
>>> @@ -87,4 +101,5 @@ message core_entry {
>>>  	optional task_core_entry	tc		= 3;
>>>  	optional task_kobj_ids_entry	ids		= 4;
>>>  	optional thread_core_entry	thread_core	= 5;
>>> +	optional cpu_params		cpu		= 6;
>>>  }
>>
>> All tasks will have the same "cpu" value. Why not carrying
>> the cpu_params stuff in separate image file?
> 
> It was _your_ (!) decision. I've been proposing to save it
> cpu.img, but you said to keep it inside core params.

I admit it was mine. But nowadays there appeared another argument which
in my eyes makes this thing better sit in separate image -- the information
about _host_ abilities will be required for live migration and it has
nothing to do with individual tasks.

> 	Cyrill
> .
> 




More information about the CRIU mailing list