Re: [PATCH-for-4.17] xen/sched: fix restore_vcpu_affinity() by removing it

2022-10-21 Thread Marek Marczykowski-Górecki
On Fri, Oct 21, 2022 at 08:58:06AM +0200, Juergen Gross wrote: > When the system is coming up after having been suspended, > restore_vcpu_affinity() is called for each domain in order to adjust > the vcpu's affinity settings in case a cpu didn't come to live again. > > The way

Re: [PATCH-for-4.17] xen/sched: fix restore_vcpu_affinity() by removing it

2022-10-21 Thread Juergen Gross
On 21.10.22 12:37, Jan Beulich wrote: On 21.10.2022 08:58, Juergen Gross wrote: --- a/xen/common/sched/core.c +++ b/xen/common/sched/core.c @@ -1196,76 +1196,6 @@ static void sched_reset_affinity_broken(const struct sched_unit *unit) v->affinity_broken = false; } My pre-push

Re: [PATCH-for-4.17] xen/sched: fix restore_vcpu_affinity() by removing it

2022-10-21 Thread Jan Beulich
On 21.10.2022 08:58, Juergen Gross wrote: > --- a/xen/common/sched/core.c > +++ b/xen/common/sched/core.c > @@ -1196,76 +1196,6 @@ static void sched_reset_affinity_broken(const struct > sched_unit *unit) > v->affinity_broken = false; > } My pre-push build test failed because the

RE: [PATCH-for-4.17] xen/sched: fix restore_vcpu_affinity() by removing it

2022-10-21 Thread Henry Wang
Hi Juergen, > -Original Message- > From: Juergen Gross > Subject: [PATCH-for-4.17] xen/sched: fix restore_vcpu_affinity() by removing > it > > When the system is coming up after having been suspended, > restore_vcpu_affinity() is called for each domain in order to adjust > the vcpu's

Re: [PATCH-for-4.17] xen/sched: fix restore_vcpu_affinity() by removing it

2022-10-21 Thread Dario Faggioli
On Fri, 2022-10-21 at 09:06 +0200, Juergen Gross wrote: > On 21.10.22 08:58, Juergen Gross wrote: > > When the system is coming up after having been suspended, > > restore_vcpu_affinity() is called for each domain in order to > > adjust > > the vcpu's affinity settings in case a cpu didn't come to

Re: [PATCH-for-4.17] xen/sched: fix restore_vcpu_affinity() by removing it

2022-10-21 Thread Juergen Gross
On 21.10.22 08:58, Juergen Gross wrote: When the system is coming up after having been suspended, restore_vcpu_affinity() is called for each domain in order to adjust the vcpu's affinity settings in case a cpu didn't come to live again. The way restore_vcpu_affinity() is doing that is wrong,