Re: [PATCH] sparseirq: make some func to be used with genirq

[Mike Travis <travis@sgi.com>]

Ingo Molnar wrote:
> * Mike Travis <travis@sgi.com> wrote:
> 
>> Ingo Molnar wrote:
>>> * Matt Mackall <mpm@selenic.com> wrote:
>>>
>>>> On Thu, 2009-01-08 at 13:06 +0100, Ingo Molnar wrote:
>>>>> (Matt Cc:-ed)
>>>>>
>>>>> Matt, does this look good to you? The point of this cleanup would be to 
>>>>> make desc->timer_rand_state depend on hardirq (the presence of irq_desc[]) 
>>>>> - not sparseirq (which does not make much sense).
>>>> Seems fine.
>>>>
>>>> Signed-off-by: Matt Mackall <mpm@selenic.com>
>>> applied it to tip/irq/urgent, thanks guys!
>>>
>>> 	Ingo
>>> --
>>> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
>>> the body of a message to majordomo@vger.kernel.org
>>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>> Please read the FAQ at  http://www.tux.org/lkml/
>> This will conflict with the change of kstat_irqs from unsigned int to 
>> unsigned long.
>>
>> If you do your magic on tip/cpus4096, I will git-remote update my tree 
>> and make the conflict go away...
> 
> you can defer the change to unsigned long to a later stage, when these 
> have hit upstream. (in a few days)
> 
> 	Ingo

Good idea since the change to long tripped a panic in the CPUMASK_OFFSTACK=n
case (which doesn't make a lot of sense at this moment.)

Will push the remainder soon.

Thanks,
Mike