From: Thomas Richter <thor@math.tu-berlin.de>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: linux-kernel@vger.kernel.org, benh@kernel.crashing.org
Subject: Re: parport_pc: irq= parameter with limited usefulness, kernel hang on PPC/OldWorldMac
Date: Thu, 16 Sep 2010 20:41:33 +0200 [thread overview]
Message-ID: <4C9264DD.5050903@math.tu-berlin.de> (raw)
In-Reply-To: <3821_1284476021_4C8F8C75_3821_60120_1_20100914161433.55457b6f@lxorguk.ukuu.org.uk>
Thanks Alan, hi Ben,
>> *) Under at least 2.6.31 and up, the kernel assigns IRQ 25 to the card.
>> Under 2.6.28 and below, the card
>> was run in polling mode (i.e. irq was -1, not 25).
>>
>> *) Furthermore, the above report suggests that the system wants to
>> handle an edge-triggered IRQ, though
>> according to /proc/interrupts, IRQ 25 is level-triggered.
>>
>
> The parport_pc code just asks for the IRQ so any level/edge mismatch
> would appear to be coming from the firmware or architecture code. The
> older kernel didn't know to use the IRQ on PCI devices which stopped some
> things working and hurt performance. That would also have masked the bug
> you see - so the report makes total sense.
>
I see, thanks.
>> *) parport_pc should take irq=none seriously and should not silently
>> override it,
>>
>
> parport_pc could do with a way to specify whether to use the IRQ or DMA
> on PCI ports but really this shouldn't be needed if the rest of the
> kernel logic is right.
>
>
>> *) parport_pc should at least not crash the system completely in case
>> the IRQ is enabled.
>>
>> The latter *might* be an issue with the G3 powermac hardware, which is
>> of course weird and has a couple of hardware
>>
>
> The latter looks like a bug in the PPC side support, perhaps misreporting
> an IRQ, or mislabelling it in some way. I think the first step is to find
> a PPC hacker who can figure out why the apparently sane IRQ request from
> the parport layer is exploding.
>
> Even if there is some technical reason the IRQ can't be used then the arch
> code shouldn't be reporting an IRQ for it, and the parport_pc could ought
> to just work.
>
> Adding Ben to the Cc: so we it can get hunted down.
Ben, anything I can do to find this bug? The current state is
"workaround that works ok for me", but that's not very satisfying.
Greetings,
Thomas
prev parent reply other threads:[~2010-09-16 18:41 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-09-14 14:38 parport_pc: irq= parameter with limited usefulness, kernel hang on PPC/OldWorldMac Thomas Richter
2010-09-14 15:14 ` Alan Cox
[not found] ` <3821_1284476021_4C8F8C75_3821_60120_1_20100914161433.55457b6f@lxorguk.ukuu.org.uk>
2010-09-16 18:41 ` Thomas Richter [this message]
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=4C9264DD.5050903@math.tu-berlin.de \
--to=thor@math.tu-berlin.de \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=benh@kernel.crashing.org \
--cc=linux-kernel@vger.kernel.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.