Re: [Xen-devel] [PATCH v3 01/15] xen: add xenstore domain flag to hypervisor

[Juergen Gross <jgross@xxxxxxxx>]

On 08/01/16 17:03, Jan Beulich wrote:
>>>> On 08.01.16 at 15:22, <JGross@xxxxxxxx> wrote:
>> On 08/01/16 15:07, Jan Beulich wrote:
>>>>>> On 08.01.16 at 14:08, <JGross@xxxxxxxx> wrote:
>>>> --- a/xen/common/domain.c
>>>> +++ b/xen/common/domain.c
>>>> @@ -318,6 +318,12 @@ struct domain *domain_create(domid_t domid, unsigned 
>> int domcr_flags,
>>>>          hardware_domain = d;
>>>>      }
>>>>  
>>>> +    if ( domcr_flags & DOMCRF_xs_domain )
>>>> +    {
>>>> +        d->is_xenstore = 1;
>>>> +        d->disable_migrate = 1;
>>>> +    }
>>>
>>> It only occurred to me now: Wouldn't it be worth denying the domain
>>> creation request if there already is a xenstore domain?
>>
>> Hmm, the only cases where this would make sense would be:
>>
>> - some bug in dom0 resulting in parallel calls of init-xenstore-domain
> 
> Or human (admin) error?

I don't think so. init-xenstore-domain checks whether xenstored or
another xenstore domain is already running and will fail in this case.
Multiple xenstore domains are possible only if init-xenstore-domain
is running multiple times in parallel and at least two instances
succeed in passing the check for another xenstored domain before
one instance did create the domain.

The only way to achieve this via admin error would be to duplicate
the start script/service and configure both to run in parallel.
I think this scenario is plausible only for an action done on
purpose.


Juergen


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel