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

From: Ingo Molnar
Date: Sat Jan 10 2009 - 23:05:53 EST



* Mike Travis <travis@xxxxxxx> wrote:

> Ingo Molnar wrote:
> > * Matt Mackall <mpm@xxxxxxxxxxx> 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@xxxxxxxxxxx>
> >
> > 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@xxxxxxxxxxxxxxx
> > 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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/