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

On Thu, 01 Mar 2007 14:20:09 +0100 Gerd Hoffmann <kraxel@suse.de> wrote:

> Dave Jones wrote:
> > On Wed, Feb 28, 2007 at 11:23:46AM +0100, Gerd Hoffmann wrote:
> >  > The test which automatically enables the backup timer on some HP
> >  > machines doesn't trigger on other hardware which needs the backup
> >  > timer too.
> > 
> > Did you figure out *why* that test doesn't trigger?
> 
> I didn't, probably a slightly different hardware bug.
> 
> > Making that work seems a better solution to me than adding magic
> > options that users won't know they have to use.
> 
> Sure, that would be better.  I'll leave that to the ibm guys who own the
> hardware in question ;)
> 

Well, it doens't _have_ to be the IBM guys.  Anyone who can reproduce this
problem should be able to find the suitable magic to detect the broken
interrupt generation.

An automatic fix is much preferable to a module parameter which most people
won't even know exists.

Perhaps Alex can suggest some debugging steps we can take to work out
why the test isn't triggering?

  reply	other threads:[~2007-03-06  3:07 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 [this message]
2007-03-06  4:22       ` Alex Williamson

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=20070305190333.c01a7ffc.akpm@linux-foundation.org \
    --to=akpm@linux-foundation.org \
    --cc=alex.williamson@hp.com \
    --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).