From mboxrd@z Thu Jan 1 00:00:00 1970 From: Julien Grall Subject: Re: [PATCH v2 13/15] xen/arm: Add support for GIC v3 Date: Thu, 10 Apr 2014 12:24:39 +0100 Message-ID: <53467F77.5040403@linaro.org> References: <1396612593-443-1-git-send-email-vijay.kilari@gmail.com> <1396612593-443-14-git-send-email-vijay.kilari@gmail.com> <1397124031.9862.88.camel@kazak.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1397124031.9862.88.camel@kazak.uk.xensource.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 Cc: vijay.kilari@gmail.com, stefano.stabellini@eu.citrix.com, Prasun.Kapoor@caviumnetworks.com, vijaya.kumar@caviumnetworks.com, xen-devel@lists.xen.org, stefano.stabellini@citrix.com List-Id: xen-devel@lists.xenproject.org On 04/10/2014 11:00 AM, Ian Campbell wrote: >> + >> +static u16 gic_compute_target_list(int *base_cpu, const struct cpumask *mask, >> + u64 cluster_id) >> +{ >> + int cpu = *base_cpu; >> + u64 mpidr = cpu_logical_map(cpu); >> + u16 tlist = 0; >> + >> + while ( cpu < nr_cpu_ids ) >> + { >> + /* >> + * If we ever get a cluster of more than 16 CPUs, just >> + * scream and skip that CPU. > > I don't see any screaming. We would want to know if this was happening, > wouldn't we? It seems the code of GICv3 has been half-copied for Linux code :). I've just found this function in Mark's series. -- Julien Grall