linux-kernel - Re: [PATCH 0/8] tip related: radix tree for spareseirq and logical flat clean up

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: <4B772969.303@kernel.org>
Date:	Sat, 13 Feb 2010 14:36:25 -0800
From:	Yinghai Lu <yinghai@...nel.org>
To:	"Eric W. Biederman" <ebiederm@...ssion.com>
CC:	Ingo Molnar <mingo@...e.hu>, Thomas Gleixner <tglx@...utronix.de>,
	"H. Peter Anvin" <hpa@...or.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Suresh Siddha <suresh.b.siddha@...el.com>,
	linux-kernel@...r.kernel.org, Jeremy Fitzhardinge <jeremy@...p.org>
Subject: Re: [PATCH  0/8] tip related: radix tree for spareseirq and logical
 flat clean up

On 02/13/2010 04:26 AM, Eric W. Biederman wrote:

>> Index: linux-2.6/arch/x86/kernel/irqinit.c
>> ===================================================================
>> --- linux-2.6.orig/arch/x86/kernel/irqinit.c
>> +++ linux-2.6/arch/x86/kernel/irqinit.c
>> @@ -83,16 +83,14 @@ static struct irqaction irq2 = {
>>  	.name = "cascade",
>>  };
>>  
>> -DEFINE_PER_CPU(vector_irq_t, vector_irq) = {
>> -	[0 ... NR_VECTORS - 1] = -1,
>> -};
>> +DEFINE_PER_CPU(vector_desc_t, vector_desc);
>>  
>>  int vector_used_by_percpu_irq(unsigned int vector)
>>  {
>>  	int cpu;
>>  
>>  	for_each_online_cpu(cpu) {
>> -		if (per_cpu(vector_irq, cpu)[vector] != -1)
>> +		if (per_cpu(vector_desc, cpu)[vector] != NULL)
>>  			return 1;
>>  	}
>>  
>> @@ -139,7 +137,7 @@ void __init init_IRQ(void)
>>  	 * irq's migrate etc.
>>  	 */
>>  	for (i = 0; i < nr_legacy_irqs; i++)
>> -		per_cpu(vector_irq, 0)[IRQ0_VECTOR + i] = i;
>> +		per_cpu(vector_desc, 0)[IRQ0_VECTOR + i] = irq_to_desc(i);
> 
> I am not familiar with this hunk (it must be in the x86 tree).
> Are you certain we have allocated the legacy irq desc here?

yes.

early_irq_init() is called before init_IRQ in start_kernel()

it will have irq_desc for legacy ready.

YH

--
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/