summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlexander Gordeev <agordeev@redhat.com>2012-06-14 12:39:34 +0200
committerIngo Molnar <mingo@kernel.org>2012-06-14 12:53:12 +0200
commitcac4afbc3da58d9e5701b34bd4c1f11ea13328d4 (patch)
treeb10f225017a117e2ac8fccdb09eaa7e3255c657f
parented88bed881c9948c4035828c5d63f60c7b015f86 (diff)
downloadlinux-cac4afbc3da58d9e5701b34bd4c1f11ea13328d4.tar.gz
linux-cac4afbc3da58d9e5701b34bd4c1f11ea13328d4.tar.bz2
linux-cac4afbc3da58d9e5701b34bd4c1f11ea13328d4.zip
x86/x2apic/cluster: Vector_allocation_domain() should return a value
Since commit 8637e38 ("x86/apic: Avoid useless scanning thru a cpumask in assign_irq_vector()") vector_allocation_domain() operation indicates if a cpumask is dynamic or static. This update fixes the oversight and makes the operation to return a value. Signed-off-by: Alexander Gordeev <agordeev@redhat.com> Cc: Suresh Siddha <suresh.b.siddha@intel.com> Cc: Yinghai Lu <yinghai@kernel.org> Link: http://lkml.kernel.org/r/20120614103933.GJ3383@dhcp-26-207.brq.redhat.com Signed-off-by: Ingo Molnar <mingo@kernel.org>
-rw-r--r--arch/x86/kernel/apic/x2apic_cluster.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/arch/x86/kernel/apic/x2apic_cluster.c b/arch/x86/kernel/apic/x2apic_cluster.c
index 23a46cf5b6fd..1885a73b7f33 100644
--- a/arch/x86/kernel/apic/x2apic_cluster.c
+++ b/arch/x86/kernel/apic/x2apic_cluster.c
@@ -228,10 +228,11 @@ static int x2apic_cluster_probe(void)
/*
* Each x2apic cluster is an allocation domain.
*/
-static void cluster_vector_allocation_domain(int cpu, struct cpumask *retmask)
+static bool cluster_vector_allocation_domain(int cpu, struct cpumask *retmask)
{
cpumask_clear(retmask);
cpumask_copy(retmask, per_cpu(cpus_in_cluster, cpu));
+ return true;
}
static struct apic apic_x2apic_cluster = {