linux-kernel - Re: [patch 1/1] metag/irq: Use access helper irq_data_get_affinity

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <55A4C9AB.4080501@imgtec.com>
Date:	Tue, 14 Jul 2015 09:34:51 +0100
From:	James Hogan <james.hogan@...tec.com>
To:	Thomas Gleixner <tglx@...utronix.de>,
	LKML <linux-kernel@...r.kernel.org>
CC:	Jiang Liu <jiang.liu@...ux.intel.com>
Subject: Re: [patch 1/1] metag/irq: Use access helper irq_data_get_affinity_mask()

Hi,

On 13/07/15 21:43, Thomas Gleixner wrote:
> From: Jiang Liu <jiang.liu@...ux.intel.com>
> 
> This is a preparatory patch for moving irq_data struct members.
> 
> Signed-off-by: Jiang Liu <jiang.liu@...ux.intel.com>
> Cc: James Hogan <james.hogan@...tec.com>
> Signed-off-by: Thomas Gleixner <tglx@...utronix.de>

Looks reasonable. Do you want this to go through metag tree?

If not,
Acked-by: James Hogan <james.hogan@...tec.com>

Cheers
James

> ---
>  arch/metag/kernel/irq.c |   10 ++++++----
>  1 file changed, 6 insertions(+), 4 deletions(-)
> 
> Index: tip/arch/metag/kernel/irq.c
> ===================================================================
> --- tip.orig/arch/metag/kernel/irq.c
> +++ tip/arch/metag/kernel/irq.c
> @@ -270,23 +270,25 @@ void migrate_irqs(void)
>  
>  	for_each_active_irq(i) {
>  		struct irq_data *data = irq_get_irq_data(i);
> +		struct cpumask *mask;
>  		unsigned int newcpu;
>  
>  		if (irqd_is_per_cpu(data))
>  			continue;
>  
> -		if (!cpumask_test_cpu(cpu, data->affinity))
> +		mask = irq_data_get_affinity_mask(data);
> +		if (!cpumask_test_cpu(cpu, mask))
>  			continue;
>  
> -		newcpu = cpumask_any_and(data->affinity, cpu_online_mask);
> +		newcpu = cpumask_any_and(mask, cpu_online_mask);
>  
>  		if (newcpu >= nr_cpu_ids) {
>  			pr_info_ratelimited("IRQ%u no longer affine to CPU%u\n",
>  					    i, cpu);
>  
> -			cpumask_setall(data->affinity);
> +			cpumask_setall(mask);
>  		}
> -		irq_set_affinity(i, data->affinity);
> +		irq_set_affinity(i, mask);
>  	}
>  }
>  #endif /* CONFIG_HOTPLUG_CPU */
> 
> 


Download attachment "signature.asc" of type "application/pgp-signature" (820 bytes)