Re: [PATCH 5/5] sched/arinc653: Implement CAST-32A multicore scheduling

2020-09-22 Thread Andrew Cooper
On 18/09/2020 21:03, Jeff Kubascik wrote: > On 9/17/2020 1:30 PM, Dario Faggioli wrote: >> On Thu, 2020-09-17 at 15:59 +, Stewart Hildebrand wrote: >>> On Thursday, September 17, 2020 11:20 AM, Dario Faggioli wrote: On Thu, 2020-09-17 at 15:10 +, Stewart Hildebrand wrote: >> It mig

Re: [PATCH 5/5] sched/arinc653: Implement CAST-32A multicore scheduling

2020-09-18 Thread Dario Faggioli
On Fri, 2020-09-18 at 16:03 -0400, Jeff Kubascik wrote: > On 9/17/2020 1:30 PM, Dario Faggioli wrote: > > On Thu, 2020-09-17 at 15:59 +, Stewart Hildebrand wrote: > > > > > And don't think only to the need of writing the code (as you kind > > of > > have it already), but also to testing. As in

Re: [PATCH 5/5] sched/arinc653: Implement CAST-32A multicore scheduling

2020-09-18 Thread Jeff Kubascik
On 9/17/2020 1:30 PM, Dario Faggioli wrote: >On Thu, 2020-09-17 at 15:59 +, Stewart Hildebrand wrote: >> On Thursday, September 17, 2020 11:20 AM, Dario Faggioli wrote: >>> On Thu, 2020-09-17 at 15:10 +, Stewart Hildebrand wrote: > It might be worth to consider using just the core sched

Re: [PATCH 5/5] sched/arinc653: Implement CAST-32A multicore scheduling

2020-09-18 Thread Jeff Kubascik
On 9/17/2020 1:57 PM, Stewart Hildebrand wrote: > On Thursday, September 17, 2020 12:19 PM, Andrew Cooper wrote: >> On 17/09/2020 15:57, Stewart Hildebrand wrote: >>> On Thursday, September 17, 2020 10:43 AM, Andrew Cooper wrote: On 16/09/2020 19:18, Jeff Kubascik wrote: > +/* > + * A

Re: [PATCH 5/5] sched/arinc653: Implement CAST-32A multicore scheduling

2020-09-17 Thread Andrew Cooper
On 17/09/2020 15:57, Stewart Hildebrand wrote: > On Thursday, September 17, 2020 10:43 AM, Andrew Cooper wrote: >> On 16/09/2020 19:18, Jeff Kubascik wrote: >>> +/* >>> + * A handle with all zeros represents domain 0 if present, otherwise idle >>> UNIT >>> + */ >>> +#define DOM0_HANDLE ((const xen

Re: [PATCH 5/5] sched/arinc653: Implement CAST-32A multicore scheduling

2020-09-17 Thread Andrew Cooper
On 16/09/2020 19:18, Jeff Kubascik wrote: > +/* > + * A handle with all zeros represents domain 0 if present, otherwise idle > UNIT > + */ > +#define DOM0_HANDLE ((const xen_domain_handle_t){0}) This isn't accurate. There are systems where dom0 doesn't have a zero UUID (XenServer for one), and i

RE: [PATCH 5/5] sched/arinc653: Implement CAST-32A multicore scheduling

2020-09-17 Thread Stewart Hildebrand
On Thursday, September 17, 2020 12:19 PM, Andrew Cooper wrote: >On 17/09/2020 15:57, Stewart Hildebrand wrote: >> On Thursday, September 17, 2020 10:43 AM, Andrew Cooper wrote: >>> On 16/09/2020 19:18, Jeff Kubascik wrote: +/* + * A handle with all zeros represents domain 0 if present, ot

Re: [PATCH 5/5] sched/arinc653: Implement CAST-32A multicore scheduling

2020-09-17 Thread Dario Faggioli
On Thu, 2020-09-17 at 15:59 +, Stewart Hildebrand wrote: > On Thursday, September 17, 2020 11:20 AM, Dario Faggioli wrote: > > On Thu, 2020-09-17 at 15:10 +, Stewart Hildebrand wrote: > > > > It might be worth to consider using just the core scheduling > > > > framework > > > > in order to

RE: [PATCH 5/5] sched/arinc653: Implement CAST-32A multicore scheduling

2020-09-17 Thread Stewart Hildebrand
On Thursday, September 17, 2020 11:20 AM, Dario Faggioli wrote: >On Thu, 2020-09-17 at 15:10 +, Stewart Hildebrand wrote: >> On Thursday, September 17, 2020 5:04 AM, Jürgen Groß wrote: >> > On 16.09.20 20:18, Jeff Kubascik wrote: >> > > This change is an overhaul of the ARINC653 scheduler to en

Re: [PATCH 5/5] sched/arinc653: Implement CAST-32A multicore scheduling

2020-09-17 Thread Dario Faggioli
On Thu, 2020-09-17 at 15:10 +, Stewart Hildebrand wrote: > On Thursday, September 17, 2020 5:04 AM, Jürgen Groß wrote: > > On 16.09.20 20:18, Jeff Kubascik wrote: > > > This change is an overhaul of the ARINC653 scheduler to enable > > > CAST-32A > > > multicore scheduling. CAST-32A specifies t

Re: [PATCH 5/5] sched/arinc653: Implement CAST-32A multicore scheduling

2020-09-17 Thread Jürgen Groß
On 17.09.20 17:10, Stewart Hildebrand wrote: On Thursday, September 17, 2020 5:04 AM, Jürgen Groß wrote: On 16.09.20 20:18, Jeff Kubascik wrote: This change is an overhaul of the ARINC653 scheduler to enable CAST-32A multicore scheduling. CAST-32A specifies that only one partition (domain) can

RE: [PATCH 5/5] sched/arinc653: Implement CAST-32A multicore scheduling

2020-09-17 Thread Stewart Hildebrand
On Thursday, September 17, 2020 5:04 AM, Jürgen Groß wrote: >On 16.09.20 20:18, Jeff Kubascik wrote: >> This change is an overhaul of the ARINC653 scheduler to enable CAST-32A >> multicore scheduling. CAST-32A specifies that only one partition >> (domain) can run during a minor frame, but that doma

RE: [PATCH 5/5] sched/arinc653: Implement CAST-32A multicore scheduling

2020-09-17 Thread Stewart Hildebrand
On Thursday, September 17, 2020 10:43 AM, Andrew Cooper wrote: >On 16/09/2020 19:18, Jeff Kubascik wrote: >> +/* >> + * A handle with all zeros represents domain 0 if present, otherwise idle >> UNIT >> + */ >> +#define DOM0_HANDLE ((const xen_domain_handle_t){0}) > >This isn't accurate. > >There a

Re: [PATCH 5/5] sched/arinc653: Implement CAST-32A multicore scheduling

2020-09-17 Thread Jürgen Groß
On 16.09.20 20:18, Jeff Kubascik wrote: This change is an overhaul of the ARINC653 scheduler to enable CAST-32A multicore scheduling. CAST-32A specifies that only one partition (domain) can run during a minor frame, but that domain is now allowed to have more than one vCPU. It might be worth to