[Xen-devel] [PATCH v1 2/5] xen: sched: null: don't assign down vcpus to pcpus

2018-08-24 Thread Dario Faggioli
If a pCPU has been/is being offlined, we don't want it to be neither assigned to any pCPU, nor in the wait list. So, avoid doing that while inserting or migrating vcpus. Signed-off-by: Dario Faggioli --- Cc: George Dunlap Cc: Stefano Stabellini Cc: Roger Pau Monne --- xen/common/sched_null.c

Re: [Xen-devel] [PATCH v1 2/5] xen: sched: null: don't assign down vcpus to pcpus

2019-07-15 Thread George Dunlap
On 8/25/18 1:21 AM, Dario Faggioli wrote: > If a pCPU has been/is being offlined, we don't want it to be neither > assigned to any pCPU, nor in the wait list. I take it the first `pCPU` should be `vCPU`? Also, English grammar agreement is funny: `neither` needs to agree with `nor`, but the two do

Re: [Xen-devel] [PATCH v1 2/5] xen: sched: null: don't assign down vcpus to pcpus

2019-07-16 Thread Dario Faggioli
On Mon, 2019-07-15 at 17:06 +0100, George Dunlap wrote: > On 8/25/18 1:21 AM, Dario Faggioli wrote: > > If a pCPU has been/is being offlined, we don't want it to be > > neither > > assigned to any pCPU, nor in the wait list. > > I take it the first `pCPU` should be `vCPU`? > Indeed. :-) > Also,

Re: [Xen-devel] [PATCH v1 2/5] xen: sched: null: don't assign down vcpus to pcpus

2019-07-16 Thread George Dunlap
> On Jul 16, 2019, at 11:50 AM, Dario Faggioli wrote: > > On Mon, 2019-07-15 at 17:06 +0100, George Dunlap wrote: >> On 8/25/18 1:21 AM, Dario Faggioli wrote: >>> If a pCPU has been/is being offlined, we don't want it to be >>> neither >>> assigned to any pCPU, nor in the wait list. >> >> I ta

Re: [Xen-devel] [PATCH v1 2/5] xen: sched: null: don't assign down vcpus to pcpus

2019-07-16 Thread Dario Faggioli
On Tue, 2019-07-16 at 12:02 +, George Dunlap wrote: > > On Jul 16, 2019, at 11:50 AM, Dario Faggioli > > On Mon, 2019-07-15 at 17:06 +0100, George Dunlap wrote: > > > On 8/25/18 1:21 AM, Dario Faggioli wrote: > > > > > > > The other thing is, from a "developmental purity" point of view, > > >