Re: [M68KNOMMU]: fix fec driver interrupt races

[Greg Ungerer]

Hi Jeff,

Jeff Garzik wrote:
> Greg Ungerer wrote:
> > The FEC driver has a common interrupt handler for all interrupt event
> > types. It is raised on a number of distinct interrupt vectors.
> > This handler can't be re-entered while processing an interrupt, so
> > make sure all requested vectors are flagged as IRQF_DISABLED.
> >
> > Signed-off-by: Greg Ungerer <gerg@xxxxxxxxxxx>
>
> NAK -- add spinlocks to your interrupt handling like you are supposed to

A better fix is to use the separate vectors, avoiding spinlocks
in most cases. But this is a simple immediate fix for 2.6.25.
The better solution is being worked on, but it needs some
reasonable testing before it is ready for inclusion, and that
won't be till post 2.6.25.

(This is a regression, fixes in the m68knommu interrupt subsystem
exposed this).

Regards
Greg



------------------------------------------------------------------------
Greg Ungerer  --  Chief Software Dude       EMAIL:     gerg@xxxxxxxxxxxx
Secure Computing Corporation                PHONE:       +61 7 3435 2888
825 Stanley St,                             FAX:         +61 7 3891 3630
Woolloongabba, QLD, 4102, Australia         WEB: http://www.SnapGear.com
--
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/