[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [RFC v3 4/6] xen/arm: Add save/restore support for guest core registers

On 05/14/2014 02:25 PM, Ian Campbell wrote:
> On Wed, 2014-05-14 at 13:23 +0100, Julien Grall wrote:
>> On 05/14/2014 12:16 PM, Ian Campbell wrote:
>>> On Sun, 2014-05-11 at 10:06 +0100, Julien Grall wrote:
>>>> Hi Wei,
>>>> Thank you for the patch.
>>>> On 08/05/14 22:18, Wei Huang wrote:
>>>>> This patch implements a save/resore support for ARM guest core
>>>>> registers.
>>>> The commit 893256f "xen/arm: Correctly save/restore CNTKCTL_EL1" 
>>>> save/restore a new register during the context switch.
>>>> I think you forgot to add it in this patch.
>>> I think it would belong in the previous patch with the timer state.
>> If Wei plans to use his new structure (see [1]), I'm fine with
>> save/restore on the previous patch.
>> Otherwise it seems stupid to save twice the same value.
> Nobody has suggested saving it twice.

When I commented the code, there was 2 timer structure (one for the phys
timer and the other for the virt timer).

Anyway, it seems the new approach will be with a single structure.


Julien Grall

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.