From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH v2] xen/arm: Blacklist the memory mapped timer (armv7-timer-mem) Date: Thu, 15 Jan 2015 15:13:38 +0000 Message-ID: <1421334818.5437.20.camel@citrix.com> References: <1421258443-15333-1-git-send-email-julien.grall@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta14.messagelabs.com ([193.109.254.103]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1YBm86-0004yi-OL for xen-devel@lists.xenproject.org; Thu, 15 Jan 2015 15:14:38 +0000 In-Reply-To: <1421258443-15333-1-git-send-email-julien.grall@linaro.org> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Julien Grall Cc: xen-devel@lists.xenproject.org, tim@xen.org, stefano.stabellini@citrix.com List-Id: xen-devel@lists.xenproject.org On Wed, 2015-01-14 at 18:00 +0000, Julien Grall wrote: > Some platform (such as the VFP Base AEMv8 model) has a memory mapped > timer. We don't want DOM0 use this timer rather than the generic ARM > timer. So blacklist it for all platforms. > > Signed-off-by: Julien Grall Acked + applied. > > --- > This patch is candidate to backport for Xen 4.5 and Xen 4.4. > > It may not apply correctly for Xen 4.4. > > Changes in v2: > - Add a comment in the code to explain why we need to blacklist > memory mapped timer > --- > xen/arch/arm/domain_build.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/xen/arch/arm/domain_build.c b/xen/arch/arm/domain_build.c > index bf8dc78..d5959b5 100644 > --- a/xen/arch/arm/domain_build.c > +++ b/xen/arch/arm/domain_build.c > @@ -1047,6 +1047,8 @@ static int handle_node(struct domain *d, struct kernel_info *kinfo, > DT_MATCH_COMPATIBLE("arm,psci"), > DT_MATCH_PATH("/cpus"), > DT_MATCH_TYPE("memory"), > + /* The memory mapped timer is not supported by Xen. */ > + DT_MATCH_COMPATIBLE("arm,armv7-timer-mem"), > { /* sentinel */ }, > }; > static const struct dt_device_match gic_matches[] __initconst =