From mboxrd@z Thu Jan 1 00:00:00 1970 From: George Dunlap Subject: Re: [PATCH 2/3] arinc: Add cpu-pool support to scheduler. Date: Tue, 19 Nov 2013 11:30:16 +0000 Message-ID: <528B4BC8.4000300@eu.citrix.com> References: <1384805814-3597-1-git-send-email-nate.studer@dornerworks.com> <1384805814-3597-3-git-send-email-nate.studer@dornerworks.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1384805814-3597-3-git-send-email-nate.studer@dornerworks.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Nathan Studer Cc: Simon Martin , Ian Campbell , Stefano Stabellini , Ian Jackson , Robert VanVossen , xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On 11/18/2013 08:16 PM, Nathan Studer wrote: > From: Nathan Studer > > 1. Remove the restriction that dom0 must be in the schedule, since dom-0 may > not belong to the scheduler's pool. > 2. Add a schedule entry for each of dom-0's vcpus as they are created. > 3. Add code to deal with empty schedules in the do_schedule function. > 4. Call the correct idle task for the pcpu on which the scheduling decision > is being made in do_schedule. > 5. Add code to prevent migration of a vcpu. > 6. Implement a proper cpu_pick function, which prefers the current processor. > > These changes do not implement arinc653 multicore. Since the schedule only > supports 1 vcpu entry per slot, even if the vcpus of a domain are run on > multiple pcpus, the scheduler will essentially serialize their execution. > > Signed-off-by: Nathan Studer If this were a change to one of the main schedulers I think I would say that it was too late for such an intrusive change. But at the moment, I don't think there are other users of this code, so I'm inclined to be more permissive. Unless someone wants to argue otherwise: Release-acked-by: George Dunlap