From: Frans Pop <elendil@planet.nl>
To: David Miller <davem@davemloft.net>
Cc: sparclinux@vger.kernel.org, linux-ide@vger.kernel.org,
bzolnier@gmail.com
Subject: Re: cmd64x: irq 14: nobody cared - system is dreadfully slow
Date: Sun, 21 Jun 2009 14:46:37 +0200 [thread overview]
Message-ID: <200906211446.38598.elendil@planet.nl> (raw)
In-Reply-To: <20090620.171945.140676687.davem@davemloft.net>
On Sunday 21 June 2009, David Miller wrote:
> From: Frans Pop <elendil@planet.nl>
> Date: Sat, 20 Jun 2009 23:52:33 +0200
>
> > ide0 at 0x1fe02c00000-0x1fe02c00007,0x1fe02c0000a on irq 14
> > (serialized) irq 14: nobody cared (try booting with the "irqpoll"
> > option) Call Trace:
[...]
> diff --git a/drivers/ide/cmd64x.c b/drivers/ide/cmd64x.c
> index 80b777e..f98ba24 100644
> --- a/drivers/ide/cmd64x.c
> +++ b/drivers/ide/cmd64x.c
> @@ -425,8 +425,7 @@ static const struct ide_port_info cmd64x_chipsets[]
> __devinitdata = { .enablebits = {{0x51,0x04,0x04}, {0x51,0x08,0x08}},
> .port_ops = &cmd64x_port_ops,
> .dma_ops = &cmd648_dma_ops,
> - .host_flags = IDE_HFLAG_SERIALIZE |
> - IDE_HFLAG_ABUSE_PREFETCH,
> + .host_flags = IDE_HFLAG_ABUSE_PREFETCH,
> .pio_mask = ATA_PIO5,
> .mwdma_mask = ATA_MWDMA2,
> .udma_mask = ATA_UDMA2,
This patch (tested on top of 2.6.30) gets rid of one of the two "nobody cared"
errors and also makes the system run at normal speed again. But I'm still
left with one error:
cmd64x 0000:01:03.0: IDE controller (0x1095:0x0646 rev 0x03)
cmd64x 0000:01:03.0: 100% native mode on irq 14
ide0: BM-DMA at 0x1fe02c00020-0x1fe02c00027
ide1: BM-DMA at 0x1fe02c00028-0x1fe02c0002f
hda: ST34342A, ATA DISK drive
hda: MWDMA2 mode selected
hdc: Maxtor 6E040L0, ATA DISK drive
hdd: CD-ROM 56X/AKH, ATAPI CD/DVD-ROM drive
hdc: MWDMA2 mode selected
hdd: bad DMA info in identify block
ide0 at 0x1fe02c00000-0x1fe02c00007,0x1fe02c0000a on irq 14
irq 14: nobody cared (try booting with the "irqpoll" option)
Call Trace:
[000000000048b594] note_interrupt+0x160/0x1e8
[000000000048bcc4] handle_fasteoi_irq+0x9c/0xc8
[0000000000429e60] handler_irq+0xb0/0xe8
[00000000004208b4] tl0_irq5+0x14/0x20
[0000000000455e80] __do_softirq+0x58/0x17c
[0000000000429d84] do_softirq+0x50/0x7c
[0000000000455bb0] irq_exit+0x38/0x88
[000000000042e710] timer_interrupt+0x70/0x88
[00000000004209d4] tl0_irq14+0x14/0x20
[000000000048ae04] request_threaded_irq+0x64/0x12c
[00000000005ff3c4] ide_host_register+0x4e4/0x6a8
[00000000006044bc] ide_pci_init_one+0xc8/0xf8
[000000001001271c] cmd64x_init_one+0x88/0x9c [cmd64x]
[0000000000588cb4] local_pci_probe+0x10/0x24
[0000000000588e38] pci_device_probe+0x4c/0x84
[00000000005ee2c8] driver_probe_device+0xa0/0x164
handlers:
[<00000000005fb72c>] (ide_intr+0x8/0x234)
Disabling IRQ #14
ide1 at 0x1fe02c00010-0x1fe02c00017,0x1fe02c0001a on irq 14
I tried the following additional patch, but unfortunately that did not help:
--- a/drivers/ide/cmd64x.c
+++ b/drivers/ide/cmd64x.c
@@ -426,6 +426,7 @@ static const struct ide_port_info cmd64x_chipsets[]
.port_ops = &cmd64x_port_ops,
.dma_ops = &cmd648_dma_ops,
.host_flags = IDE_HFLAG_ABUSE_PREFETCH,
+ .irq_flags = IRQF_SHARED,
.pio_mask = ATA_PIO5,
.mwdma_mask = ATA_MWDMA2,
.udma_mask = ATA_UDMA2,
I got the idea for that from 255115fb and had hoped it would compensate
for this change from Bart's commit:
--- a/drivers/ide/ide-probe.c
+++ b/drivers/ide/ide-probe.c
@@ -1125,7 +1125,7 @@ static int init_irq (ide_hwif_t *hwif)
sa = IRQF_SHARED;
#endif /* __mc68000__ */
- if (hwif->chipset == ide_pci || hwif->chipset == ide_cmd646)
+ if (hwif->chipset == ide_pci)
sa = IRQF_SHARED;
> Alternatively you can try using the pata_cmd64x.c driver instead :-)
That one works fine :-)
Cheers,
FJP
next prev parent reply other threads:[~2009-06-21 12:46 UTC|newest]
Thread overview: 57+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-06-20 21:52 cmd64x: irq 14: nobody cared - system is dreadfully slow Frans Pop
2009-06-21 0:19 ` David Miller
2009-06-21 4:47 ` Frans Pop
2009-06-21 12:46 ` Frans Pop [this message]
2009-06-21 13:33 ` Bartlomiej Zolnierkiewicz
2009-06-21 20:14 ` Frans Pop
2009-06-22 1:56 ` David Miller
2009-06-22 4:28 ` Frans Pop
2009-06-22 5:45 ` David Miller
2009-06-22 6:43 ` Frans Pop
2009-06-22 6:44 ` David Miller
2009-06-22 11:21 ` Bartlomiej Zolnierkiewicz
2009-06-22 14:04 ` Frans Pop
2009-06-22 14:39 ` Bartlomiej Zolnierkiewicz
2009-06-22 15:16 ` Frans Pop
2009-06-22 17:38 ` Bartlomiej Zolnierkiewicz
2009-06-22 19:01 ` Frans Pop
2009-06-22 21:35 ` Bartlomiej Zolnierkiewicz
2009-06-23 7:51 ` [PATCH] ide-cd: Improve "weird block size" error message Frans Pop
2009-06-23 7:57 ` Borislav Petkov
2009-06-23 8:02 ` Borislav Petkov
2009-06-23 23:03 ` David Miller
2009-06-23 8:20 ` Frans Pop
2009-06-23 10:59 ` David Miller
2009-06-23 11:13 ` Frans Pop
2009-06-23 11:18 ` David Miller
2009-06-23 21:30 ` Frans Pop
2009-06-23 23:01 ` David Miller
2009-06-29 11:19 ` Jan Engelhardt
2009-06-23 10:15 ` cmd64x: irq 14: nobody cared - system is dreadfully slow David Miller
2009-06-23 14:58 ` Frans Pop
2009-06-23 16:13 ` Bartlomiej Zolnierkiewicz
2009-06-23 23:04 ` David Miller
2009-06-23 10:47 ` David Miller
2009-06-23 10:43 ` David Miller
2009-07-31 14:08 ` Frans Pop
2009-08-01 5:46 ` David Miller
2009-08-05 20:43 ` [stable] " Greg KH
2009-06-21 13:15 ` Bartlomiej Zolnierkiewicz
2009-06-21 21:19 ` David Miller
2009-06-21 22:34 ` Bartlomiej Zolnierkiewicz
2009-06-21 22:57 ` David Miller
2009-06-21 23:13 ` New IDE maintainer (was Re: cmd64x: irq 14: nobody cared - system is dreadfully slow) David Miller
2009-06-21 23:45 ` Bartlomiej Zolnierkiewicz
2009-06-21 23:52 ` New IDE maintainer David Miller
2009-06-22 0:53 ` New IDE maintainer (was Re: cmd64x: irq 14: nobody cared - system is dreadfully slow) Matthew Wilcox
2009-06-22 0:00 ` Stephen Rothwell
2009-06-22 0:20 ` New IDE maintainer David Miller
2009-06-22 3:39 ` New IDE maintainer (was Re: cmd64x: irq 14: nobody cared - system is dreadfully slow) Greg Freemyer
2009-06-22 17:03 ` Jeff Garzik
2009-06-22 17:11 ` Alan Cox
2009-06-22 17:21 ` Arnd Bergmann
2009-06-22 17:32 ` Alan Cox
2009-06-22 3:47 ` cmd64x: irq 14: nobody cared - system is dreadfully slow Frans Pop
2009-06-21 15:43 ` Bartlomiej Zolnierkiewicz
2009-06-21 21:21 ` David Miller
-- strict thread matches above, loose matches on Subject: below --
2009-06-20 21:39 Frans Pop
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=200906211446.38598.elendil@planet.nl \
--to=elendil@planet.nl \
--cc=bzolnier@gmail.com \
--cc=davem@davemloft.net \
--cc=linux-ide@vger.kernel.org \
--cc=sparclinux@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 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).