LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Jeff Garzik <jeff@garzik.org>
To: Ingo Molnar <mingo@elte.hu>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
	Stephen Hemminger <shemminger@linux-foundation.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	Andrew Morton <akpm@osdl.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: Linux 2.6.20-rc6 - sky2 resume breakage
Date: Tue, 30 Jan 2007 03:02:58 -0500	[thread overview]
Message-ID: <45BEFBB2.8090402@garzik.org> (raw)
In-Reply-To: <20070130075325.GA591@elte.hu>

Ingo Molnar wrote:
> btw., it would be great if you could help us here: could you perhaps, 
> from a past example, outline a specific case of such an ATA/USB IRQ 
> storm and how it occured (precisely) - and what the fix was? I'd like to 
> analyze a specific case to make sure the genirq layer recovers from such 
> cases more gracefully. In general, i think the IRQ subsystem needs to 
> become more failure-resilient and needs to become more auto-learning 
> (and these two dont stand in the way of good performance). This problem 
> of shared IRQs will be with us for at least another 10 years, if not 
> more. (for example ISA is /still/ not dead everywhere and it was already 
> legacy technology 15 years ago when Linux was started.)


Easy to name an example, as they are pretty generic.  When sharing irqs 
-- usually ATA is configured to PCI native (IO-APIC-fasteoi) -- any 
interrupt storm causes the other devices sharing that irq to crap 
themselves (kernel turns off irq, suggests irqpoll, etc.)

ATA is unfortunately easier to cause interrupt storms than most because 
the standard PCI IDE definition has __no__ possible way to indicate 
certain interrupt conditions are pending.  You have to /know/ that you 
are expecting an interrupt, which causes problems if the hardware 
decides to send the interrupt early or late, rather than when its 
expected.  Most modern hardware has a read/write/clear interrupt status 
  register that gives you an immediate summary of the pending interrupt 
conditions, and an easy way to ack the pending events.  ATA does not 
have any such capability.

That said, stuff like AHCI or sata_sil or sata_sil24 do have modern 
designs with the expected interrupt status register(s), so they do not 
suffer from the problems suffered by the more legacy-like hardware 
(ata_piix, sata_via, pata_*)

	Jeff



  reply	other threads:[~2007-01-30  8:03 UTC|newest]

Thread overview: 84+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-01-25  2:58 Linux 2.6.20-rc6 Linus Torvalds
2007-01-25 10:09 ` Sunil Naidu
2007-01-25 11:10 ` Linux 2.6.20-rc6 - build failure Eyal Lebedinsky
2007-01-26  2:22   ` Eyal Lebedinsky
2007-01-26 18:49     ` [2.6 patch] fix OCFS2 compile error Adrian Bunk
2007-01-26 19:47       ` Mark Fasheh
2007-01-26 19:53         ` Adrian Bunk
2007-01-26 18:46   ` Linux 2.6.20-rc6 - build failure Mark Fasheh
2007-01-25 17:50 ` Linux 2.6.20-rc6 Arkadiusz Patyk
2007-01-25 21:05 ` Michal Piotrowski
2007-01-25 21:12   ` David Miller
2007-01-26 16:52     ` Venkat Yekkirala
2007-01-26 18:10 ` 2.6.20-rc6: known unfixed regressions (part 1) Adrian Bunk
2007-01-26 18:11 ` 2.6.20-rc6: known unfixed regressions (part 2) Adrian Bunk
2007-01-26 18:16   ` Malte Schröder
2007-01-27 17:28     ` Adrian Bunk
2007-01-27 17:39       ` Adrian Bunk
2007-01-27 17:58       ` Linus Torvalds
2007-01-26 19:04   ` Michal Piotrowski
2007-01-26 19:08     ` Venkat Yekkirala
2007-01-26 18:18 ` 2.6.20-rc6: known regressions with patches Adrian Bunk
2007-01-29  8:45   ` Ingo Molnar
2007-01-29 12:58     ` Dave Jones
2007-01-27 17:32 ` 2.6.20-rc6: known unfixed regressions (v2) (part 1) Adrian Bunk
2007-01-27 17:42 ` 2.6.20-rc6: known unfixed regressions (v2) (part 2) Adrian Bunk
2007-01-28 13:33   ` Uwe Bugla
2007-01-29  6:26     ` Mike Galbraith
2007-01-29  6:48       ` Andrew Morton
2007-01-29  7:08         ` Mike Galbraith
2007-01-29  7:13           ` Linus Torvalds
2007-01-27 17:44 ` 2.6.20-rc6: known regressions with patches (v2) Adrian Bunk
2007-01-27 20:47 ` Linux 2.6.20-rc6 - supend lockdep warning Thomas Gleixner
2007-01-27 20:55 ` Linux 2.6.20-rc6 - sky2 resume breakage Thomas Gleixner
2007-01-29 19:31   ` Stephen Hemminger
2007-01-29 20:10     ` Thomas Gleixner
2007-01-29 21:38       ` Stephen Hemminger
2007-01-29 22:23         ` Thomas Gleixner
2007-01-29 22:23           ` Stephen Hemminger
2007-01-29 22:31             ` Thomas Gleixner
2007-01-29 22:37           ` Linus Torvalds
2007-01-29 22:40             ` Stephen Hemminger
2007-01-29 23:04               ` Linus Torvalds
2007-01-29 23:45                 ` Stephen Hemminger
2007-01-30  0:12                   ` Linus Torvalds
2007-01-30  0:16                     ` Stephen Hemminger
2007-01-30  0:25                       ` Linus Torvalds
2007-01-30  0:26                         ` Stephen Hemminger
2007-01-30  6:54                         ` Ingo Molnar
2007-01-30  7:39                           ` Jeff Garzik
2007-01-30  7:53                             ` Ingo Molnar
2007-01-30  8:02                               ` Jeff Garzik [this message]
2007-01-30  8:08                                 ` Ingo Molnar
2007-01-30  8:13                                   ` Ingo Molnar
2007-01-31 15:27                                   ` Jeff Garzik
2007-01-31 17:38                                     ` Ingo Molnar
2007-01-31 17:52                                       ` Jeff Garzik
2007-01-31 20:13                                         ` Thomas Gleixner
2007-01-30  8:03                             ` Ingo Molnar
2007-02-01  6:15                             ` [LIBATA BUG] sr.c: TEST_UNIT_READY error Conke Hu
2007-02-07 12:40                               ` Jeff Garzik
2007-02-02  5:48                                 ` Conke Hu
2007-02-13  7:30                                   ` Conke Hu
2007-02-15  6:30                                     ` Conke Hu
2007-01-30  8:57                     ` Linux 2.6.20-rc6 - sky2 resume breakage Len Brown
2007-01-30 16:01                       ` Rafael J. Wysocki
2007-01-30 21:28                         ` Nigel Cunningham
2007-02-01 12:49                       ` Pavel Machek
2007-01-29 23:42             ` [PATCH] sky2: fix MSI related " Thomas Gleixner
2007-01-29 22:38           ` Linux 2.6.20-rc6 - sky2 " Frédéric Riss
2007-01-29 22:45             ` Thomas Gleixner
2007-01-29 22:50               ` Frédéric Riss
2007-01-29 22:57                 ` Thomas Gleixner
2007-01-29 23:26                   ` Frédéric Riss
2007-01-29 23:37                     ` Thomas Gleixner
2007-01-29 23:50                       ` [PATCH] block MSI on Sony Stephen Hemminger
2007-01-30  0:22                         ` Thomas Gleixner
2007-01-30  0:21                           ` Stephen Hemminger
2007-01-30  0:31                             ` Thomas Gleixner
2007-01-30  0:31                               ` Stephen Hemminger
2007-01-30  0:26                           ` Thomas Gleixner
2007-01-27 22:11 ` Linux 2.6.20-rc6 - suspend / resume ata_piix Thomas Gleixner
2007-01-27 22:40   ` Jeff Garzik
2007-01-27 22:44     ` Thomas Gleixner
2007-01-28 22:05       ` Thomas Gleixner

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=45BEFBB2.8090402@garzik.org \
    --to=jeff@garzik.org \
    --cc=akpm@osdl.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=shemminger@linux-foundation.org \
    --cc=tglx@linutronix.de \
    --cc=torvalds@linux-foundation.org \
    --subject='Re: Linux 2.6.20-rc6 - sky2 resume breakage' \
    /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).