Re: [PATCH] 8250: option 'force_polling' for buggy IRQs

From: Eric W. Biederman
Date: Thu Jul 28 2016 - 14:53:51 EST

Max Staudt <mstaudt@xxxxxxx> writes:

> On 07/27/2016 03:33 PM, Theodore Ts'o wrote:
>> I wonder if it would just be simpler to make it be a CONFIG option
>> which causes the irq value to zero in arch/x86/include/asm/serial.h?
> Alan hit the nail on the head in his reply to your email: I can't set the IRQ in
> the system configuration. The other option is to compile a custom kernel for this
> system, but that's exactly what I'd like to avoid.
> My motivation for this patch is to have a boot time option that allows *any*
> kernel to boot on this broken hardware. This allows for console boot on a broken
> machine even when the machine has never been used in serial console mode before,
> and thus the hardware bug has not been discovered before either.

*Scratches my head*

There is already generic handling for this (that triggers polling) in
the irq code if an irq is screaming. I am assuming the problem is that
your irq never fires?

Serial consoles are already polled for output. So nothing should
care until userspace starts, and the full serial driver initializes.


The irq code already has handling for some of these situations,
and if that handling is not sufficient I suspect we could pretty
easily extend it for this case.

For irqs that are screaming we disable them and then poll them.
There there are the "irqfixup" and "irqpoll" command line options
that when an irq is received the code tries all of the irq handlers.

So I suspect either "irqfixup" or "irqpoll" would handle this for you.
If not I am certain a small tweak to some of that code would work.

That should be enough to get a system booted for debugging purposes,
which is the point here. At which point you can dig in figure out what
is wrong and get the problem design fixed.