PATCH powerpc: Merge asm-ppc*/hardirq.h
Jon Loeliger
jdl at freescale.com
Wed Sep 14 23:41:34 EST 2005
On Wed, 2005-09-14 at 06:05, Christoph Hellwig wrote:
> On Tue, Sep 13, 2005 at 03:38:40PM -0500, Jon Loeliger wrote:
> > +#ifndef _ASM_POWERPC_HARDIRQ_H
> > +#define _ASM_POWERPC_HARDIRQ_H
> > +#ifdef __KERNEL__
>
> the __KERNEL__ ifdefs is not needed. <asm/hardirq.h> is only included
> from <linux/hardirq.h> which doesn't have anything user-visible.
Will do.
> > +
> > +#include <linux/config.h>
>
> not needed.
>
> > +#include <linux/cache.h>
> > +
> > +#ifdef __powerpc64__
> > +#include <linux/preempt.h>
> > +#else
> > +#include <linux/smp_lock.h>
> > +#include <asm/irq.h>
> > +#endif
>
> We shouldn't need either of these include blocks at all.
OK. I'll do some out-ripping and verify that it
all compiles still.
> > +
> > +/* The __last_jiffy_stamp field is needed to ensure that no decrementer
> > + * interrupt is lost on SMP machines. Since on most CPUs it is in the same
> > + * cache line as local_irq_count, it is cheap to access and is also used on UP
> > + * for uniformity.
> > + */
> > +typedef struct {
> > + unsigned int __softirq_pending; /* set_bit is used on this */
> > + unsigned int __last_jiffy_stamp;
> > +} ____cacheline_aligned irq_cpustat_t;
>
> I'd suggest just using a DECLARE_PER_CPU variable for last_jiffy_stamp.
> In facct I plan to get rid of irq_cpustat_t completely at some point.
Hmmm... So, I don't want to mess that up. How about I leave it
as I've suggested and leave that step up to you as a follow on?
Thanks,
jdl
More information about the Linuxppc-dev
mailing list