Re: [Xen-devel] [PATCH 05/60] xen/sched: alloc struct sched_unit for each vcpu

2019-07-19 Thread Dario Faggioli
On Fri, 2019-07-19 at 06:56 +0200, Juergen Gross wrote: > On 18.07.19 19:57, Dario Faggioli wrote: > > On Tue, 2019-05-28 at 12:32 +0200, Juergen Gross wrote: > > > > > However, I don't see much value in not doing what's done here in > > patch > > 4 already (it's pretty much only line changed by

Re: [Xen-devel] [PATCH 05/60] xen/sched: alloc struct sched_unit for each vcpu

2019-07-18 Thread Juergen Gross
On 18.07.19 19:57, Dario Faggioli wrote: On Tue, 2019-05-28 at 12:32 +0200, Juergen Gross wrote: Allocate a struct sched_unit for each vcpu. This removes the need to have it locally on the stack in schedule.c. Signed-off-by: Juergen Gross And this patch looks ok as well. However, I don't

Re: [Xen-devel] [PATCH 05/60] xen/sched: alloc struct sched_unit for each vcpu

2019-07-18 Thread Dario Faggioli
On Tue, 2019-05-28 at 12:32 +0200, Juergen Gross wrote: > Allocate a struct sched_unit for each vcpu. This removes the need to > have it locally on the stack in schedule.c. > > Signed-off-by: Juergen Gross > And this patch looks ok as well. However, I don't see much value in not doing what's

[Xen-devel] [PATCH 05/60] xen/sched: alloc struct sched_unit for each vcpu

2019-05-28 Thread Juergen Gross
Allocate a struct sched_unit for each vcpu. This removes the need to have it locally on the stack in schedule.c. Signed-off-by: Juergen Gross --- xen/common/schedule.c | 67 +++-- xen/include/xen/sched.h | 2 ++ 2 files changed, 33 insertions(+),