>>> Andrew Cooper <andrew.coop...@citrix.com> 03/09/18 6:06 PM >>>
>On 09/03/18 17:00, Jan Beulich wrote:
>>>>> On 09.03.18 at 14:18, <andrew.coop...@citrix.com> wrote:
>>> --- a/xen/arch/x86/domain.c
>>> +++ b/xen/arch/x86/domain.c
>>> @@ -426,8 +426,8 @@ static bool emulation_flags_ok(const struct domain *d,
>>> uint32_t emflags)
>>> return true;
>>> -int arch_domain_create(struct domain *d, unsigned int domcr_flags,
>>> - struct xen_arch_domainconfig *config)
>>> +int arch_domain_create(struct domain *d,
>>> + struct xen_domctl_createdomain *config)
>> Is there any reason for this to not be const? There's no write now
>> afaics, and I can't imagine you wanting to add one later on.
>I originally planned to make them const, but the ARM side passes data
>back to the toolstack, and the prototype is (rightfully) common.
Oh, I see - certainly a little odd, but that's the way it is then.
Xen-devel mailing list