Re: Patch(2.4.5): Fix PCMCIA ATA/IDE freeze (w/ PCI add-in cards)

Gunther Mayer (Gunther.Mayer@t-online.de)
Wed, 27 Jun 2001 18:23:33 +0200


Andre Hedrick wrote:
>
> I can not help if you have a device that not compliant to the rules.
> ATA-2 is OBSOLETED thus we forced (the NCITS Standards Body) the CFA
> people to move to ATA-4 or ATA-5.

See Alan's point about existing hardware.

>
> That device is enabling (with its ablity to assert) its device->host
> interrupt regardless of the HOST...that is a bad device.

Does this imply ATA-4 specifies the device should
come up with nIEN set (i.e. interrupts disabled)?

>
> Send me the manufacturer and I will tear them apart for making a
> non-compliant device.

see below.

>Then figure out a way to de-assert the like
> regardless if it exists without hang the rest of the driver.
>

My patch has figured this out !
It enables interrupts again (clear nIEN) shortly before it registers the irq handler.
At least boot+root on /dev/hda do not hang.

Regards, Gunther

1) Compact Flash 48MB, Silicon Tech/Hitachi
===========================================
dump_cis
Socket 0:
dev_info
fn_specific 400ns, 2kb
common_jedec 0xdf 0x01
manfid 0x014d, 0x0001
vers_1 4.1, "SiliconTech,Inc.", "48MB Compact PC Card", "Ver 3.0"
funcid fixed_disk [post]
disk_interface [ide]
disk_features [silicon] [unique] [single]
[sleep] [standby] [idle] [low power]
config base 0x0200 mask 0x000f last_index 0x03
cftable_entry 0x00 [default]
[rdybsy] [pwrdown]
Vcc Vnom 5V
memory 0x0000-0x07ff @ 0x0000
cftable_entry 0x00
Vcc Vnom 3300mV Ipeak 45mA
cftable_entry 0x01 [default]
[rdybsy] [pwrdown]
Vcc Vnom 5V
io 0x0000-0x000f [lines=4] [8bit] [16bit]
irq mask 0xffff [level] [pulse] [shared]
cftable_entry 0x01
Vcc Vnom 3300mV Ipeak 45mA
cftable_entry 0x02 [default]
[rdybsy] [pwrdown]
Vcc Vnom 5V
io 0x01f0-0x01f7, 0x03f6-0x03f7 [lines=10] [8bit] [16bit] [range]
irq 14 [level] [pulse] [shared]
cftable_entry 0x02
Vcc Vnom 3300mV Ipeak 45mA
cftable_entry 0x03 [default]
[rdybsy] [pwrdown]
Vcc Vnom 5V
io 0x0170-0x0177, 0x0376-0x0377 [lines=10] [8bit] [16bit] [range]
irq 14 [level] [pulse] [shared]
cftable_entry 0x03
Vcc Vnom 3300mV Ipeak 45mA

Jun 27 17:32:13 linux kernel: hde: Hitachi CV 7.1.1, ATA DISK drive

2) Iomega Clik40
================
dump_cis
Socket 0:
dev_info
NULL 0ns, 512b
attr_dev_info
EEPROM 250ns, 2kb
vers_1 4.1, "PC CARD MANUFACTURER", "PCMCIA ATA/ATAPI Adapter",
"Version 3.00"
manfid 0xffff, 0x0003
funcid fixed_disk
disk_interface [ide]
config base 0x0800 mask 0x0001 last_index 0x05
cftable_entry 0x25 [default]
io 0x0180-0x0187, 0x0386-0x0387 [lines=10] [16bit] [range]
irq mask 0xfe00 [level] [pulse]
cftable_entry 0x0d
io 0x0170-0x0177, 0x0376-0x0377 [lines=10] [16bit] [range]
cftable_entry 0x15
io 0x01e8-0x01ef, 0x03ee-0x03ef [lines=10] [16bit] [range]
cftable_entry 0x1d
io 0x0168-0x016f, 0x036e-0x036f [lines=10] [16bit] [range]
cftable_entry 0x05
io 0x01f0-0x01f7, 0x03f6-0x03f7 [lines=10] [16bit] [range]

Jun 27 17:36:39 linux kernel: hde: IOMEGA Clik! 40 CZ ATAPI, ATAPI cdrom or floppy?, assuming FLOPPY drive

>
> On Tue, 26 Jun 2001, Gunther Mayer wrote:
>
> > Hi,
> >
> > this patch fixes the hard hang (no SYSRQ) on inserting
> > any PCMCIA ATA/IDE card (e.g. CompactFlash, Clik40 etc)
> > to a PCI-Cardbus bridge add-in card.
> >
> > Thanks David for his valuable explanation about what happens:
> > ide-probe registers it's irq handler too late! After it
> > triggers the interrupt during the probe the (shared) irq
> > loops forever, effectively wedging the machine completely.
> >
> > Regards, Gunther
> >
> >
> >
> > --- linux245.orig/drivers/ide/ide-cs.c Fri Feb 9 20:40:02 2001
> > +++ linux/drivers/ide/ide-cs.c Tue Jun 26 21:22:19 2001
> > @@ -324,6 +324,9 @@
> > if (link->io.NumPorts2)
> > release_region(link->io.BasePort2, link->io.NumPorts2);
> >
> > + outb(0x02, ctl_base); // Set nIEN = disable device interrupts
> > + // else it hangs on PCI-Cardbus add-in cards, wedging irq
> > +
> > /* retry registration in case device is still spinning up */
> > for (i = 0; i < 10; i++) {
> > hd = ide_register(io_base, ctl_base, link->irq.AssignedIRQ);
> > --- linux245.orig/drivers/ide/ide-probe.c Sun Mar 18 18:25:02 2001
> > +++ linux/drivers/ide/ide-probe.c Tue Jun 26 21:25:07 2001
> > @@ -685,6 +685,8 @@
> > #else /* !CONFIG_IDEPCI_SHARE_IRQ */
> > int sa = (hwif->chipset == ide_pci) ? SA_INTERRUPT|SA_SHIRQ : SA_INTERRUPT;
> > #endif /* CONFIG_IDEPCI_SHARE_IRQ */
> > +
> > + outb(0x00, hwif->io_ports[IDE_CONTROL_OFFSET]); // clear nIEN == enable irqs
> > if (ide_request_irq(hwif->irq, &ide_intr, sa, hwif->name, hwgroup)) {
> > if (!match)
> > kfree(hwgroup);
> >
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/