linux-kernel - Re: [PATCH v2 13/26] x86, irq: Move down arch_early_irq

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: <20130308195342.GF6622@phenom.dumpdata.com>
Date:	Fri, 8 Mar 2013 14:53:42 -0500
From:	Konrad Rzeszutek Wilk <konrad.wilk@...cle.com>
To:	Yinghai Lu <yinghai@...nel.org>
Cc:	Thomas Gleixner <tglx@...utronix.de>, Ingo Molnar <mingo@...e.hu>,
	"H. Peter Anvin" <hpa@...or.com>,
	Bjorn Helgaas <bhelgaas@...gle.com>,
	"Rafael J. Wysocki" <rjw@...k.pl>, linux-pci@...r.kernel.org,
	linux-kernel@...r.kernel.org, Joerg Roedel <joro@...tes.org>,
	Sebastian Andrzej Siewior <sebastian@...akpoint.cc>
Subject: Re: [PATCH v2 13/26] x86, irq: Move down arch_early_irq_init()

On Fri, Feb 08, 2013 at 11:28:10AM -0800, Yinghai Lu wrote:
> Change position only.
> 
> Prepare to update arch_early_irq_init(), it needs call some static functions.
                                                   ^^-> to
> 
> Signed-off-by: Yinghai Lu <yinghai@...nel.org>
> Cc: Joerg Roedel <joro@...tes.org>
> Cc: Konrad Rzeszutek Wilk <konrad.wilk@...cle.com>
> Cc: Sebastian Andrzej Siewior <sebastian@...akpoint.cc>
> ---
>  arch/x86/kernel/apic/io_apic.c |   89 ++++++++++++++++++++--------------------
>  1 file changed, 44 insertions(+), 45 deletions(-)
> 
> diff --git a/arch/x86/kernel/apic/io_apic.c b/arch/x86/kernel/apic/io_apic.c
> index 72a1351..deda981 100644
> --- a/arch/x86/kernel/apic/io_apic.c
> +++ b/arch/x86/kernel/apic/io_apic.c
> @@ -185,51 +185,6 @@ static struct irq_pin_list *alloc_irq_pin_list(int node)
>  	return kzalloc_node(sizeof(struct irq_pin_list), GFP_KERNEL, node);
>  }
>  
> -
> -/* irq_cfg is indexed by the sum of all RTEs in all I/O APICs. */
> -static struct irq_cfg irq_cfgx[NR_IRQS_LEGACY];
> -
> -int __init arch_early_irq_init(void)
> -{
> -	struct irq_cfg *cfg;
> -	int count, node, i;
> -
> -	if (!legacy_pic->nr_legacy_irqs)
> -		io_apic_irqs = ~0UL;
> -
> -	for (i = 0; i < nr_ioapics; i++) {
> -		ioapics[i].saved_registers =
> -			kzalloc(sizeof(struct IO_APIC_route_entry) *
> -				ioapics[i].nr_registers, GFP_KERNEL);
> -		if (!ioapics[i].saved_registers)
> -			pr_err("IOAPIC %d: suspend/resume impossible!\n", i);
> -	}
> -
> -	cfg = irq_cfgx;
> -	count = ARRAY_SIZE(irq_cfgx);
> -	node = cpu_to_node(0);
> -
> -	/* Make sure the legacy interrupts are marked in the bitmap */
> -	irq_reserve_irqs(0, legacy_pic->nr_legacy_irqs);
> -
> -	for (i = 0; i < count; i++) {
> -		INIT_LIST_HEAD(&cfg[i].irq_2_pin);
> -		irq_set_chip_data(i, &cfg[i]);
> -		zalloc_cpumask_var_node(&cfg[i].domain, GFP_KERNEL, node);
> -		zalloc_cpumask_var_node(&cfg[i].old_domain, GFP_KERNEL, node);
> -		/*
> -		 * For legacy IRQ's, start with assigning irq0 to irq15 to
> -		 * IRQ0_VECTOR to IRQ15_VECTOR for all cpu's.
> -		 */
> -		if (i < legacy_pic->nr_legacy_irqs) {
> -			cfg[i].vector = IRQ0_VECTOR + i;
> -			cpumask_setall(cfg[i].domain);
> -		}
> -	}
> -
> -	return 0;
> -}
> -
>  static struct irq_cfg *irq_cfg(unsigned int irq)
>  {
>  	return irq_get_chip_data(irq);
> @@ -332,6 +287,50 @@ static struct irq_cfg *realloc_irq_and_cfg_at(unsigned int at, int node)
>  	return alloc_irq_and_cfg_at(at, node);
>  }
>  
> +/* irq_cfg is indexed by the sum of all RTEs in all I/O APICs. */
> +static struct irq_cfg irq_cfgx[NR_IRQS_LEGACY];
> +
> +int __init arch_early_irq_init(void)
> +{
> +	struct irq_cfg *cfg;
> +	int count, node, i;
> +
> +	if (!legacy_pic->nr_legacy_irqs)
> +		io_apic_irqs = ~0UL;
> +
> +	for (i = 0; i < nr_ioapics; i++) {
> +		ioapics[i].saved_registers =
> +			kzalloc(sizeof(struct IO_APIC_route_entry) *
> +				ioapics[i].nr_registers, GFP_KERNEL);
> +		if (!ioapics[i].saved_registers)
> +			pr_err("IOAPIC %d: suspend/resume impossible!\n", i);
> +	}
> +
> +	cfg = irq_cfgx;
> +	count = ARRAY_SIZE(irq_cfgx);
> +	node = cpu_to_node(0);
> +
> +	/* Make sure the legacy interrupts are marked in the bitmap */
> +	irq_reserve_irqs(0, legacy_pic->nr_legacy_irqs);
> +
> +	for (i = 0; i < count; i++) {
> +		INIT_LIST_HEAD(&cfg[i].irq_2_pin);
> +		irq_set_chip_data(i, &cfg[i]);
> +		zalloc_cpumask_var_node(&cfg[i].domain, GFP_KERNEL, node);
> +		zalloc_cpumask_var_node(&cfg[i].old_domain, GFP_KERNEL, node);
> +		/*
> +		 * For legacy IRQ's, start with assigning irq0 to irq15 to
> +		 * IRQ0_VECTOR to IRQ15_VECTOR for all cpu's.
> +		 */
> +		if (i < legacy_pic->nr_legacy_irqs) {
> +			cfg[i].vector = IRQ0_VECTOR + i;
> +			cpumask_setall(cfg[i].domain);
> +		}
> +	}
> +
> +	return 0;
> +}
> +
>  struct io_apic {
>  	unsigned int index;
>  	unsigned int unused[3];
> -- 
> 1.7.10.4
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/