public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Bernhard Walle <bwalle@suse.de>
To: Gabor Gombas <gombasg@sztaki.hu>,
	Andrew Morton <akpm@linux-foundation.org>,
	linux-kernel@vger.kernel.org, Ingo Molnar <mingo@elte.hu>,
	Thomas Gleixner <tglx@linutronix.de>
Subject: Re: Solid freezes with 2.6.25
Date: Wed, 30 Apr 2008 16:51:31 +0200	[thread overview]
Message-ID: <20080430165131.7426d5d3@strauss.suse.de> (raw)
In-Reply-To: <20080430133824.GC18711@boogie.lpds.sztaki.hu>

Hi,

* Gabor Gombas [2008-04-30 15:38]:
>
> On Tue, Apr 29, 2008 at 05:54:19PM +0200, Bernhard Walle wrote:
> 
> > Hm ..., can you mail /proc/interrupts of the affected system? According
> > to the HPET specification 
> 
> The Intel box:
> 
>            CPU0       CPU1       
>   0:      27346      29145   IO-APIC-edge      timer
>   1:        842        819   IO-APIC-edge      i8042
>   4:          2          2   IO-APIC-edge    
>   6:          0          3   IO-APIC-edge      floppy
>   8:          2          3   IO-APIC-edge      rtc0

Well, looks fine. It's edge triggered. My assumption that the
T1_INT_STS must be cleared is wrong since the complete HPET code
assumes edge-triggered interrupts. To use level-triggered interrupts,
the HPET must be programmed in level triggered mode, i.e. the
Tn_INT_TYPE_CNF bit must be set. Which is never done.

Back to the original problem. Can you please watch the interrupts from
the RTC until the lockup happens with something like

------------------------- 8< -----------------------------------
function get_total_irqs()
{
	sum=0
	for n in $(grep rtc /proc/interrupts | \
				 sed -e 's/.*://g' | \
				 sed -e 's/[a-zA-Z].*//g' ) ; 
		do sum=$[$sum+$n]
	done

	echo $sum
}

last=$(get_total_irqs)
while true ; do
	sum=$(get_total_irqs)
	echo $[$sum-$last]
	last=$sum

	sleep 1
done
------------------------- >8 -----------------------------------

Also, please send the full .config (maybe via private mail if that's
too large for LKML, or upload it).

> > Can you try that patch and see if the IRQ handler is really called
> > repeatedly in that case?
> 
> Unfortunately I won't have time today and will be traveling till the end
> of the week. I'll give it a shot next week.

Ok. Maybe don't use the patch I sent, instead use:



	Bernhard


--- a/arch/x86/kernel/hpet.c
+++ b/arch/x86/kernel/hpet.c
@@ -680,6 +680,12 @@ irqreturn_t hpet_rtc_interrupt(int irq,
        struct rtc_time curr_time;
        unsigned long rtc_int_flag = 0;
 
+       if (printk_ratelimit())
+               printk(KERN_DEBUG "hpet_rtc_interrupt %s%s%s\n",
+                               (hpet_rtc_flags & RTC_UIE) ? "RTC_UIE " : "",
+                               (hpet_rtc_flags & RTC_AIE) ? "RTC_AIE " : "",
+                               (hpet_rtc_flags & RTC_PIE) ? "RTC_PIE " : "");
+
        hpet_rtc_timer_reinit();
        memset(&curr_time, 0, sizeof(struct rtc_time));



  reply	other threads:[~2008-04-30 14:50 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-28 14:29 Solid freezes with 2.6.25 Gabor Gombas
2008-04-28 14:54 ` Oliver Pinter
2008-04-28 14:59   ` Gabor Gombas
2008-04-28 16:25 ` Andrew Morton
2008-04-29  9:50   ` Bernhard Walle
2008-04-29  9:53     ` Gabor Gombas
2008-04-29 15:54   ` Bernhard Walle
2008-04-30 13:38     ` Gabor Gombas
2008-04-30 14:51       ` Bernhard Walle [this message]
2008-05-07 11:43         ` Gabor Gombas
2008-05-07 12:45           ` Bernhard Walle
2008-05-07 13:42             ` Gabor Gombas
2008-05-12 12:21               ` Bernhard Walle
2008-05-13 14:39                 ` Gabor Gombas
2008-06-15 17:11                   ` Gabor Gombas
2008-07-25  8:52                     ` Gabor Gombas
2008-07-25  9:04                       ` Bernhard Walle
2008-08-25 12:27                         ` Gabor Gombas
2008-04-29  7:24 ` Gabor Gombas
2008-04-29  7:37   ` Andrew Morton
2008-04-29  9:43     ` Gabor Gombas

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=20080430165131.7426d5d3@strauss.suse.de \
    --to=bwalle@suse.de \
    --cc=akpm@linux-foundation.org \
    --cc=gombasg@sztaki.hu \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=tglx@linutronix.de \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox