LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Alex Williamson <alex.williamson@hp.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: Gerd Hoffmann <kraxel@suse.de>, Dave Jones <davej@redhat.com>,
	linux-kernel@vger.kernel.org,
	Kevin Stansell <kstansel@us.ibm.com>
Subject: Re: [patch] Add insmod option to force the use of the backup timer.
Date: Mon, 05 Mar 2007 21:22:32 -0700	[thread overview]
Message-ID: <1173154952.5941.142.camel@bling> (raw)
In-Reply-To: <20070305190333.c01a7ffc.akpm@linux-foundation.org>

On Mon, 2007-03-05 at 19:03 -0800, Andrew Morton wrote:
> 
> Perhaps Alex can suggest some debugging steps we can take to work out
> why the test isn't triggering?

   I was lucky and had a hardware description of the bug I was trying to
work around with the 8250 backup timer patch.  If the UART on the system
in question is an integrated component, it might be worthwhile to start
with the errata documented in the hardware manual.

   The detection test is simply looking for UARTs that don't re-assert
the transmit holding register empty interrupt when it becomes
re-enabled.  Since the backup timer is successfully kicking the UART
back into action, it would be interesting to know whether this is
because the "Diva test" is detecting work to do or if it's just a matter
of reading the IIR bits (the interrupt is there, but not getting
delivered).  The state of the UART at that point may be a clue how to
detect the problem.

   UARTs often seem to be the most troubling part of a system, so I'm
not opposed to a boot/module option, but auto-detection provides a much
better user experience.  Thanks,

	Alex

-- 
Alex Williamson                             HP Open Source & Linux Org.


      reply	other threads:[~2007-03-06  4:22 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-02-28 10:23 Gerd Hoffmann
2007-02-28 18:55 ` Dave Jones
2007-03-01 13:20   ` Gerd Hoffmann
2007-03-06  3:03     ` Andrew Morton
2007-03-06  4:22       ` Alex Williamson [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=1173154952.5941.142.camel@bling \
    --to=alex.williamson@hp.com \
    --cc=akpm@linux-foundation.org \
    --cc=davej@redhat.com \
    --cc=kraxel@suse.de \
    --cc=kstansel@us.ibm.com \
    --cc=linux-kernel@vger.kernel.org \
    --subject='Re: [patch] Add insmod option to force the use of the backup timer.' \
    /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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).