Re: [PATCH v8 08/11] xen: arch-specific hooks for domain_soft_reset()

[Julien Grall <julien.grall@citrix.com>]

Hi Vitaly,

On 16/07/2015 13:36, Vitaly Kuznetsov wrote:
> Julien Grall <julien.grall@citrix.com> writes:
>> On 23/06/2015 18:11, Vitaly Kuznetsov wrote:
>>> diff --git a/xen/arch/arm/domain.c b/xen/arch/arm/domain.c
>>> index 24b8938..c112afa 100644
>>> --- a/xen/arch/arm/domain.c
>>> +++ b/xen/arch/arm/domain.c
>>> @@ -653,6 +653,11 @@ void arch_domain_unpause(struct domain *d)
>>>    {
>>>    }
>>>
>>> +int arch_domain_soft_reset(struct domain *d)
>>> +{
>>> +    return 0;
>>
>> I suspect that it would needs some code in this function in order to
>> support soft reset on ARM. If so, should not you return -ENOSYS until
>> someone take care of the ARM support?
>>
>
> I was hoping kexec for ARM will magically work without any arch-specific
> bits here :-)

I don't know how work kexec for Xen, but I think we will have at least 
to do the same as x86 for the shared info page.

> Sure, let's make it -ENOSYS.

Thank you!


-- 
Julien Grall