[PATCH 4.4 329/342] x86/irq: Copy vectormask instead of an AND operation
From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Date: 2016-03-01 23:58:45
Also in:
lkml
4.4-stable review patch. If anyone has any objections, please let me know. ------------------ From: Thomas Gleixner <redacted> commit 9ac15b7a8af4cf3337a101498c0ed690d23ade75 upstream. In the case that the new vector mask is a subset of the existing mask there is no point to do a AND operation of currentmask & newmask. The result is newmask. So we can simply copy the new mask to the current mask and be done with it. Preparatory patch for further consolidation. Signed-off-by: Thomas Gleixner <redacted> Tested-by: Borislav Petkov <bp@alien8.de> Tested-by: Joe Lawrence <redacted> Cc: Jiang Liu <redacted> Cc: Jeremiah Mahler <redacted> Cc: andy.shevchenko@gmail.com Cc: Guenter Roeck <linux@roeck-us.net> Link: http://lkml.kernel.org/r/20151231160106.640253454@linutronix.de Signed-off-by: Thomas Gleixner <redacted> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> --- arch/x86/kernel/apic/vector.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
--- a/arch/x86/kernel/apic/vector.c
+++ b/arch/x86/kernel/apic/vector.c@@ -154,7 +154,7 @@ static int __assign_irq_vector(int irq, vector_cpumask); d->move_in_progress = cpumask_intersects(d->old_domain, cpu_online_mask); - cpumask_and(d->domain, d->domain, vector_cpumask); + cpumask_copy(d->domain, vector_cpumask); goto success; }