From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julien Grall Subject: Re: [PATCH] xen/arm: Fix rtds scheduler for arm Date: Wed, 04 Feb 2015 16:45:46 +0000 Message-ID: <54D24CBA.50705@linaro.org> References: <54CF849D020000780005BEF2@mail.emea.novell.com> <1422900219-30012-1-git-send-email-denys.drozdov@globallogic.com> <1423062826.24924.17.camel@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1423062826.24924.17.camel@citrix.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: Ian Campbell , Denis Drozdov Cc: george.dunlap@eu.citrix.com, andrew.cooper3@citrix.com, JBeulich@suse.com, xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org Hello, On 04/02/2015 15:13, Ian Campbell wrote: > On Mon, 2015-02-02 at 20:03 +0200, Denis Drozdov wrote: >> From: denys drozdov >> >> Make Credit2 and RT schedulers to run on arm platform >> context_saved should be deferred on ARM after IRQs enabled > > A better subject+message would be: > > xen/arm: Call context_saved() with interrupts enabled during context switch > > This is a requirement of the scheduler interface, violating this > causes for example with the RT scheduler: > > >> Signed-off-by: denys drozdov > > Acked-by: Ian Campbell The commit message suggested by Ian looks more clear to me. With this change: Reviewed-by: Julien Grall Regards, -- Julien Grall