All of lore.kernel.org
 help / color / mirror / Atom feed
From: Lars Heete <hel@admin.de>
To: Arnd Hannemann <hannemann@i4.informatik.rwth-aachen.de>
Cc: Jordan Crouse <jordan.crouse@amd.com>,
	Andres Salomon <dilinger@queued.net>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: 2.6.24-rc8 hangs at mfgpt-timer
Date: Tue, 22 Jan 2008 11:11:20 +0100	[thread overview]
Message-ID: <200801221111.21240.hel@admin.de> (raw)
In-Reply-To: <4795B14C.7040502@i4.informatik.rwth-aachen.de>

Hello,

On Tuesday 22 January 2008 10:03:08 am Arnd Hannemann wrote:
> Jordan Crouse wrote:
> > Okay - I've been exploring a little bit more.  I talked to the TinyBIOS
> > developer, and he verified that TinyBIOS shouldn't use any MFGPT timers.
> > He also told me that the mysterious "MFGPT workaround" was in fact the
> > magic MFGPT erasing MSR that was in the old kernel driver.
> >
> > So with the "MFGPT workaround" turned off, TinyBIOS should be acting like
> > the OLPC firmware with regards to timers, yet it is not.  So that is
> > curious.  I think I might have identified a race condition in the code,
> > but I'm not 100% sure thats the same problem that the ALIX platform is
> > seeing.
> >
> > Anrd and others - will you please try the attached patch on your platform
> > with the "MFGPT workaround" turned off and mfgpts enabled, and send out
> > the dmesg?
>
> Of course, tinyBios version v0.99, "MFGPT workaround" turned off,
> CONFIG_GEODE_MFGPT_TIMER=n:
>
> [   67.369697] NET: Registered protocol family 16
> [   67.383059] geode-mfgpt:  IRQ MSR=0:0
> [   67.394058] geode-mfgpt:  NMI MSR=0:0
> [   67.405049] geode-mfgpt:  Unrestricted sources=0
> [   67.418909] geode:  8 MFGPT timers available.
> [   67.433211] PCI: PCI BIOS revision 2.10 entry at 0xfcd03, last bus=0
>
> same with CONFIG_GEODE_MFGPT_TIMER=y (sorry, without move printk patch):
>
> [   22.289349] NET: Registered protocol family 16
> [   22.302716] geode-mfgpt:  IRQ MSR=0:0
> [   22.313716] geode-mfgpt:  NMI MSR=0:0
> [   22.324704] geode-mfgpt:  Unrestricted sources=0
> [   22.338566] geode-mfgpt:  Registered timer 0
> [   22.351393] mfgpt-timer:  registering the MFGT timer as a clock event.
> ^^^^ Hangs here
I had the same problem with MFGPT Timers and alix (BIOS v0.99). I found that 
if you use a different interrupt than the default 7 for MFGPT (just append 
mfgpt_irq=8 to the kernel commandline), the timer seems to work.

CPU0
  0:         33    XT-PIC-XT        timer
  2:          0    XT-PIC-XT        cascade
  4:       1662    XT-PIC-XT        serial
  8:        766    XT-PIC-XT        mfgpt-timer
 14:        473    XT-PIC-XT        libata
NMI:          0
ERR:          0


> > This will give us some debug information that I can use to ensure that
> > the interrupts are set up correctly.  You can leave the timer tick
> > disabled if you want.
> >
> >
> > Thanks,
> > Jordan
>
> Arnd

Lars

  reply	other threads:[~2008-01-22 10:33 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-16 17:44 2.6.24-rc8 hangs at mfgpt-timer Arnd Hannemann
2008-01-16 21:19 ` Andres Salomon
2008-01-16 21:56   ` Andres Salomon
2008-01-17  9:54     ` Arnd Hannemann
2008-01-17 18:40       ` Andres Salomon
2008-01-17 19:53         ` Arnd Hannemann
2008-01-17 20:42           ` Andres Salomon
2008-01-17 21:19           ` Jordan Crouse
2008-01-17 21:50             ` Arnd Hannemann
2008-01-17 22:36               ` Jordan Crouse
2008-01-17 22:52                 ` Arnd Hannemann
2008-01-17 22:57                   ` Jordan Crouse
2008-01-17 23:39                     ` Arnd Hannemann
2008-01-18  0:40                       ` Jordan Crouse
2008-01-21 23:27                       ` Jordan Crouse
2008-01-21 23:32                         ` Willy Tarreau
2008-01-22 20:15                           ` Willy Tarreau
2008-01-22 21:08                             ` Jordan Crouse
2008-01-22 21:15                               ` Willy Tarreau
2008-01-23 16:36                                 ` Jordan Crouse
2008-01-23 16:10                                   ` Willy Tarreau
2008-01-22  9:03                         ` Arnd Hannemann
2008-01-22 10:11                           ` Lars Heete [this message]
2008-01-22 11:18                             ` Arnd Hannemann
2008-01-22 18:15                               ` Jordan Crouse
2008-01-22 19:27                               ` Jordan Crouse
2008-01-22 20:54                                 ` Arnd Hannemann
2008-01-22 21:10                                   ` Ingo Molnar
2008-01-22 21:20                                     ` Willy Tarreau
2008-01-22 21:53                                     ` [git pull] was: " Thomas Gleixner
2008-01-23 21:17                                     ` [PATCH 0/2] Was: " Willy Tarreau
2008-01-23 21:18                                       ` [PATCH 1/2] x86: GEODE fix MFGPT input clock value Willy Tarreau
2008-01-23 21:59                                         ` H. Peter Anvin
2008-01-23 22:11                                           ` Willy Tarreau
2008-01-23 22:22                                             ` H. Peter Anvin
2008-01-23 22:10                                               ` Willy Tarreau
2008-01-23 22:38                                             ` Jordan Crouse
2008-01-23 23:17                                               ` Arnd Hannemann
2008-01-23 21:19                                       ` [PATCH 2/2] x86: GEODE add the "mfgptfix" boot time option to fix MFGPT timers Willy Tarreau
2008-01-19  1:06                   ` [GEODE] Geode GX/LX watchdog timer (was 2.6.24-rc8 hangs at mfgpt-timer) Jordan Crouse
2008-01-19  6:36                     ` Willy Tarreau
2008-01-20 13:22                     ` Arnd Hannemann
2008-01-20 16:34                       ` Jordan Crouse
2008-01-21 17:07                       ` Geode GX/LX watchdog timer (RESEND) Jordan Crouse
2008-01-21 18:37                         ` Arnd Hannemann
2008-02-17 14:14                           ` Iain Paton
2008-02-17 14:46                             ` Arnd Hannemann
2008-02-17 14:54                               ` Adrian Bunk
2008-02-17 16:10                                 ` Iain Paton
2008-02-17 17:32                                   ` Andres Salomon
2008-02-17 19:46                                   ` Arnd Hannemann
2008-01-20 20:16                     ` [GEODE] Geode GX/LX watchdog timer (was 2.6.24-rc8 hangs at mfgpt-timer) Lennart Sorensen

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=200801221111.21240.hel@admin.de \
    --to=hel@admin.de \
    --cc=dilinger@queued.net \
    --cc=hannemann@i4.informatik.rwth-aachen.de \
    --cc=jordan.crouse@amd.com \
    --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.