Re: [PATCH] x86/Xen: streamline (and fix) PV CPU enumeration
From: Jan Beulich
Date: Fri Mar 29 2019 - 11:12:55 EST
>>> On 29.03.19 at 14:42, <boris.ostrovsky@xxxxxxxxxx> wrote:
> On 3/29/19 4:54 AM, Jan Beulich wrote:
>>>>> On 28.03.19 at 17:50, <boris.ostrovsky@xxxxxxxxxx> wrote:
>>>
>>> Given especially xen_pv_smp_prepare_cpus(), I think re-working proper
>>> setting of present/possible masks is well beyond the scope of your
>>> original patch.
>> Well, then the question is, what (if any) changes are you
>> expecting me to make for this change to be acceptable? Or do
>> you perhaps want me to add a 2nd patch on top addressing
>> the other outlined anomalies?
>
> If your goal is just to fix the dom0_max_vcpus issue then this patch is
> sufficient (but the commit message should say that this is what the
> patch is for).
>
> But if you are trying to make cpu masks management done properly then I
> think this patch alone does not address this fully.
I.e. folding the further items discussed into this patch would be
fine by you? (I'm just trying to avoid having to go through
several more rounds of patch submissions.)
Jan