From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Ingo Molnar <mingo@elte.hu>
Cc: Paul Mackerras <paulus@samba.org>, Andrew Morton <akpm@osdl.org>,
Linus Torvalds <torvalds@osdl.org>,
Anton Blanchard <anton@samba.org>,
Linux Kernel list <linux-kernel@vger.kernel.org>,
Christoph Hellwig <hch@infradead.org>,
Russell King <rmk+lkml@arm.linux.org.uk>
Subject: Re: New consolidate irqs vs . probe_irq_*()
Date: Wed, 20 Oct 2004 20:56:06 +1000 [thread overview]
Message-ID: <1098269765.20949.7.camel@gaston> (raw)
In-Reply-To: <20041020092017.GA28054@elte.hu>
On Wed, 2004-10-20 at 19:20, Ingo Molnar wrote:
>
> .../...
>
> this is not a concept that would be too useful for the server space, but
> it sure could be useful for the produce-and-forget PC mass-market. We
> are playing a constant and mostly losing catchup game with BIOS quirks.
>
> what can never work fully reliably is of course what the feature was
> used for primarily: ISA :-) One-time edge-triggered interrupts that get
> lost are nasty ...
>
> so i thought autodetect.c could survive in the generic code - maybe we
> can make something really nice out of it, based on Alan's patch.
Hrm... Okay, maybe, but I'd rather keep it disabled for now on ppc and
ppc64, there is simply no case there where the interrupt informations
are incorrect :) And the way PCMCIA uses it is a bit nasty imho, besides
the current interface passing in a mask isn't really useable when your
max interrupt number goes beyond the size of a long.
Note that the reason why it dies on ppc atm is that it calls the irq
desc functions like shutdown() without testing if they exist in the
first place...
Looking at the code, it seems that I need to add startup() & shutdown()
callbacks to all my irq controllers so they do the equivalent of enable
and disable at least ...
Ben.
next prev parent reply other threads:[~2004-10-20 11:07 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-10-20 7:08 [PATCH] Fix PREEMPT_ACTIVE definition Paul Mackerras
2004-10-20 8:33 ` Ingo Molnar
2004-10-20 8:42 ` New consolidate irqs vs . probe_irq_*() Benjamin Herrenschmidt
2004-10-20 8:48 ` Ingo Molnar
2004-10-20 8:53 ` Benjamin Herrenschmidt
2004-10-20 9:20 ` Ingo Molnar
2004-10-20 10:56 ` Benjamin Herrenschmidt [this message]
2004-10-20 9:31 ` Chris Wedgwood
2004-10-20 9:01 ` Russell King
2004-10-20 10:50 ` Benjamin Herrenschmidt
2004-10-20 11:01 ` Russell King
2004-10-20 11:06 ` Benjamin Herrenschmidt
2004-10-20 11:53 ` Paul Mackerras
2004-10-20 12:31 ` Russell King
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1098269765.20949.7.camel@gaston \
--to=benh@kernel.crashing.org \
--cc=akpm@osdl.org \
--cc=anton@samba.org \
--cc=hch@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=paulus@samba.org \
--cc=rmk+lkml@arm.linux.org.uk \
--cc=torvalds@osdl.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.